chore: Tidy up test/verification

test/e2e/functional_test.go

@@ -792,18 +792,6 @@ spec:
 		})
 }
 
-func (s *FunctionalSuite) TestK8SJSONPatch() {
-	s.Given().
-		Workflow("@functional/k8s-patch.yaml").
-		When().
-		SubmitWorkflow().
-		WaitForWorkflow().
-		Then().
-		ExpectWorkflow(func(t *testing.T, metadata *metav1.ObjectMeta, status *wfv1.WorkflowStatus) {
-			assert.Equal(t, wfv1.WorkflowSucceeded, status.Phase)
-		})
-}
-
 func (s *FunctionalSuite) TestWorkflowPodSpecPatch() {
 	s.Given().
 		Workflow(`apiVersion: argoproj.io/v1alpha1

workflow/verify/verifier.go

@@ -1,17 +1,21 @@
 package verify
 
 import (
+	"fmt"
+
 	_ "github.com/go-python/gpython/builtin"
 
 	"github.com/argoproj/argo-workflows/v3/pkg/apis/workflow"
 	wfv1 "github.com/argoproj/argo-workflows/v3/pkg/apis/workflow/v1alpha1"
 	"github.com/argoproj/argo-workflows/v3/util/python"
 )
 
+const annotationName = workflow.WorkflowFullName + "/verify.py"
+
 func Workflow(wf *wfv1.Workflow) error {
-	verify, ok := wf.GetAnnotations()[workflow.WorkflowFullName+"/verify.py"]
+	verify, ok := wf.GetAnnotations()[annotationName]
 	if !ok {
-		return nil
+		return fmt.Errorf("connot verify workflow: annotation %s not found", annotationName)
 	}
 	nodes := wfv1.Nodes{}
 	for _, n := range wf.Status.Nodes {