chore: properly rename MustUnmarshalClusterWorkflowTemplate (argopr…

…oj#13431)

Signed-off-by: qingfeng777 <[email protected]>

pkg/apis/workflow/v1alpha1/marshall.go

@@ -49,7 +49,7 @@ func MustMarshallJSON(v interface{}) string {
 	return string(data)
 }
 
-func MustUnmarshalClusterWorkflow(text interface{}) *ClusterWorkflowTemplate {
+func MustUnmarshalClusterWorkflowTemplate(text interface{}) *ClusterWorkflowTemplate {
 	x := &ClusterWorkflowTemplate{}
 	MustUnmarshal(text, &x)
 	return x

pkg/apis/workflow/v1alpha1/marshall_test.go

@@ -6,14 +6,14 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-func TestMustUnmarshalClusterWorkflow(t *testing.T) {
+func TestMustUnmarshalClusterWorkflowTemplate(t *testing.T) {
 	defer func() {
 		if r := recover(); r == nil {
 			t.Fatalf("The code did not panic but should have")
 		} else {
 			assert.Equal(t, r.(string), "no text to unmarshal")
 		}
 	}()
-	_ = MustUnmarshalClusterWorkflow([]byte(""))
-	t.Fatalf("MustUnmarshalClusterWorkflow should have panicked and this part should not have been reached.")
+	_ = MustUnmarshalClusterWorkflowTemplate([]byte(""))
+	t.Fatalf("MustUnmarshalClusterWorkflowTemplate should have panicked and this part should not have been reached.")
 }

workflow/controller/operator_template_scope_test.go

@@ -473,7 +473,7 @@ spec:
 
 func TestTemplateClusterScope(t *testing.T) {
 	wf := wfv1.MustUnmarshalWorkflow(testTemplateClusterScopeWorkflowYaml)
-	cwftmpl := wfv1.MustUnmarshalClusterWorkflow(testTemplateClusterScopeWorkflowTemplateYaml1)
+	cwftmpl := wfv1.MustUnmarshalClusterWorkflowTemplate(testTemplateClusterScopeWorkflowTemplateYaml1)
 	wftmpl := wfv1.MustUnmarshalWorkflowTemplate(testTemplateScopeWorkflowTemplateYaml2)
 
 	cancel, controller := newController(wf, cwftmpl, wftmpl)