fix: re-add validations for policy groups (#1600)

Signed-off-by: Jose I. Paris <jiparis@chainloop.dev>

app/cli/internal/action/attestation_init.go

@@ -218,10 +218,7 @@ func enrichContractMaterials(ctx context.Context, schema *v1.CraftingSchema, cli
 			Logger: logger,
 		})
 		if err != nil {
-			// Temporarily skip if policy groups still use old schema
-			// TODO: remove this check in next release
-			logger.Warn().Msgf("policy group '%s' skipped since it's not found or it might use an old schema version", pgAtt.GetRef())
-			return nil
+			return fmt.Errorf("failed to load policy group: %w", err)
 		}
 		logger.Debug().Msgf("adding materials from policy group '%s'", group.GetMetadata().GetName())
 

app/cli/internal/action/attestation_init_test.go

@@ -67,8 +67,7 @@ func TestEnrichMaterials(t *testing.T) {
 			name:        "wrong policy group",
 			materials:   []*v1.CraftingSchema_Material{},
 			policyGroup: "file://testdata/idontexist.yaml",
-			// TODO: Fix this condition in next release
-			expectErr: false,
+			expectErr:   true,
 		},
 		{
 			name:        "name-less materials are not added",

app/controlplane/pkg/biz/workflowcontract.go

@@ -366,10 +366,7 @@ func (uc *WorkflowContractUseCase) findPolicyGroup(att *schemav1.PolicyGroupAtta
 		pr := loader.ProviderParts(att.GetRef())
 		remoteGroup, err := uc.GetPolicyGroup(pr.Provider, pr.Name, pr.OrgName, token)
 		if err != nil {
-			// Temporarily skip if policy groups still use old schema
-			// TODO: remove this check in next release
-			uc.logger.Warnf("policy group '%s' skipped since it's not found or it might use an old schema version", att.GetRef())
-			return nil, nil
+			return nil, NewErrValidation(fmt.Errorf("failed to get policy group: %w", err))
 		}
 		if remoteGroup.PolicyGroup != nil {
 			// validate group arguments

pkg/policies/policy_groups.go

@@ -58,10 +58,7 @@ func (pgv *PolicyGroupVerifier) VerifyMaterial(ctx context.Context, material *ap
 			Logger: pgv.logger,
 		})
 		if err != nil {
-			// Temporarily skip if policy groups still use old schema
-			// TODO: remove this check in next release
-			pgv.logger.Warn().Msgf("policy group '%s' skipped since it's not found or it might use an old schema version", groupAtt.GetRef())
-			return result, nil
+			return nil, NewPolicyError(err)
 		}
 
 		// matches group arguments against spec and apply defaults