diff --git a/pkg/kubectl/cmd/create_role_test.go b/pkg/kubectl/cmd/create_role_test.go
index 59e86b8cb34..1dcd14705ba 100644
--- a/pkg/kubectl/cmd/create_role_test.go
+++ b/pkg/kubectl/cmd/create_role_test.go
@@ -348,8 +348,8 @@ func TestValidate(t *testing.T) {
 	for name, test := range tests {
 		test.roleOptions.Mapper, _ = f.Object()
 		err := test.roleOptions.Validate()
-		if test.expectErr && err != nil {
-			continue
+		if test.expectErr && err == nil {
+			t.Errorf("%s: expect error happens but validate passes.", name)
 		}
 		if !test.expectErr && err != nil {
 			t.Errorf("%s: unexpected error: %v", name, err)
@@ -496,8 +496,13 @@ func TestComplete(t *testing.T) {
 		if !test.expectErr && err != nil {
 			t.Errorf("%s: unexpected error: %v", name, err)
 		}
-		if test.expectErr && err != nil {
-			continue
+
+		if test.expectErr {
+			if err != nil {
+				continue
+			} else {
+				t.Errorf("%s: expect error happens but test passes.", name)
+			}
 		}
 
 		if test.roleOptions.Name != test.expected.Name {