diff --git a/cmd/cli/kubectl-kyverno/processor/policy_processor_test.go b/cmd/cli/kubectl-kyverno/processor/policy_processor_test.go index 08f91446af..c30e75e573 100644 --- a/cmd/cli/kubectl-kyverno/processor/policy_processor_test.go +++ b/cmd/cli/kubectl-kyverno/processor/policy_processor_test.go @@ -84,7 +84,7 @@ func Test_NamespaceSelector(t *testing.T) { fail: 1, warn: 0, err: 0, - skip: 2, + skip: 0, }, }, { @@ -100,7 +100,7 @@ func Test_NamespaceSelector(t *testing.T) { fail: 1, warn: 0, err: 0, - skip: 4, + skip: 0, }, }, } diff --git a/cmd/cli/kubectl-kyverno/processor/result.go b/cmd/cli/kubectl-kyverno/processor/result.go index c49d6c2667..06f2f5510b 100644 --- a/cmd/cli/kubectl-kyverno/processor/result.go +++ b/cmd/cli/kubectl-kyverno/processor/result.go @@ -38,10 +38,8 @@ func (rc *ResultCounts) addEngineResponse(auditWarn bool, response engineapi.Eng scored := annotations.Scored(policy.GetAnnotations()) for _, rule := range autogen.ComputeRules(policy) { if rule.HasValidate() || rule.HasVerifyImageChecks() || rule.HasVerifyImages() { - ruleFoundInEngineResponse := false for _, valResponseRule := range response.PolicyResponse.Rules { if rule.Name == valResponseRule.Name() { - ruleFoundInEngineResponse = true switch valResponseRule.Status() { case engineapi.RuleStatusPass: rc.pass++ @@ -64,9 +62,6 @@ func (rc *ResultCounts) addEngineResponse(auditWarn bool, response engineapi.Eng continue } } - if !ruleFoundInEngineResponse { - rc.skip++ - } } } } @@ -79,10 +74,8 @@ func (rc *ResultCounts) addGenerateResponse(auditWarn bool, resPath string, resp } policy := genericPolicy.GetPolicy().(kyvernov1.PolicyInterface) for _, policyRule := range autogen.ComputeRules(policy) { - ruleFoundInEngineResponse := false for _, ruleResponse := range response.PolicyResponse.Rules { if policyRule.Name == ruleResponse.Name() { - ruleFoundInEngineResponse = true if ruleResponse.Status() == engineapi.RuleStatusPass { rc.pass++ } else { @@ -95,9 +88,6 @@ func (rc *ResultCounts) addGenerateResponse(auditWarn bool, resPath string, resp continue } } - if !ruleFoundInEngineResponse { - rc.skip++ - } } } @@ -118,10 +108,8 @@ func (rc *ResultCounts) addMutateResponse(resourcePath string, response engineap } printMutatedRes := false for _, policyRule := range autogen.ComputeRules(policy) { - ruleFoundInEngineResponse := false for _, mutateResponseRule := range response.PolicyResponse.Rules { if policyRule.Name == mutateResponseRule.Name() { - ruleFoundInEngineResponse = true if mutateResponseRule.Status() == engineapi.RuleStatusPass { rc.pass++ printMutatedRes = true @@ -135,9 +123,6 @@ func (rc *ResultCounts) addMutateResponse(resourcePath string, response engineap continue } } - if !ruleFoundInEngineResponse { - rc.skip++ - } } return printMutatedRes }