From 529a3509d5d79a2a997344384d8a7c2468afe602 Mon Sep 17 00:00:00 2001
From: Jim Bugwadia <jim@nirmata.com>
Date: Sun, 3 Oct 2021 22:36:06 -0700
Subject: [PATCH] fix deployment-missing-labels test

Signed-off-by: Jim Bugwadia <jim@nirmata.com>
---
 pkg/kyverno/test/test_command.go | 11 ++++++++---
 test/cli/test/autogen/test.yaml  |  2 +-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/pkg/kyverno/test/test_command.go b/pkg/kyverno/test/test_command.go
index 352eaa5b46..91dd4dabe8 100644
--- a/pkg/kyverno/test/test_command.go
+++ b/pkg/kyverno/test/test_command.go
@@ -50,11 +50,13 @@ func Command() *cobra.Command {
 					}
 				}
 			}()
+
 			_, err = testCommandExecute(dirPath, valuesFile, fileName)
 			if err != nil {
 				log.Log.V(3).Info("a directory is required")
 				return err
 			}
+
 			return nil
 		},
 	}
@@ -192,14 +194,16 @@ func testCommandExecute(dirPath []string, valuesFile string, fileName string) (r
 	}
 
 	if len(errors) > 0 && log.Log.V(1).Enabled() {
-		fmt.Printf("ignoring errors: \n")
+		fmt.Printf("test errors: \n")
 		for _, e := range errors {
 			fmt.Printf("    %v \n", e.Error())
 		}
 	}
+
 	if rc.Fail > 0 {
 		os.Exit(1)
 	}
+
 	os.Exit(0)
 	return rc, nil
 }
@@ -430,12 +434,13 @@ func applyPoliciesFromPath(fs billy.Filesystem, policyBytes []byte, valuesFile s
 			pvInfos = append(pvInfos, info)
 		}
 	}
-	resultsMap, testResults := buildPolicyResults(validateEngineResponses, values.Results, pvInfos)
 
+	resultsMap, testResults := buildPolicyResults(validateEngineResponses, values.Results, pvInfos)
 	resultErr := printTestResult(resultsMap, testResults, rc)
 	if resultErr != nil {
-		return sanitizederror.NewWithError("Unable to genrate result. Error:", resultErr)
+		return sanitizederror.NewWithError("failed to print test result:", resultErr)
 	}
+
 	return
 }
 
diff --git a/test/cli/test/autogen/test.yaml b/test/cli/test/autogen/test.yaml
index d6bef7195f..b1dd406ce0 100644
--- a/test/cli/test/autogen/test.yaml
+++ b/test/cli/test/autogen/test.yaml
@@ -24,7 +24,7 @@ results:
 # TEST: Deployment with Labels Should Fail
 - policy: require-common-labels
   rule: check-for-labels
-  result: fail
+  result: skip
   resource: deployment-missing-labels
 
 # TEST: StatefulSet with Labels Should Pass