Revert "fix(controller): validation failed when dynamic templateRef is used in nested template (argoproj#14053)"

This reverts commit 7fe50f3.

Author: tczhao

Diff for: workflow/validate/validate.go

@@ -962,12 +962,7 @@ func (ctx *templateValidationCtx) validateSteps(scope map[string]interface{}, tm
 			if err != nil {
 				return err
 			}
-			var args wfv1.ArgumentsProvider
-			args = &FakeArguments{}
-			if step.TemplateRef != nil {
-				args = &step.Arguments
-			}
-			resolvedTmpl, err := ctx.validateTemplateHolder(&step, tmplCtx, args, workflowTemplateValidation)
+			resolvedTmpl, err := ctx.validateTemplateHolder(&step, tmplCtx, &FakeArguments{}, workflowTemplateValidation)
 			if err != nil {
 				return errors.Errorf(errors.CodeBadRequest, "templates.%s.steps[%d].%s %s", tmpl.Name, i, step.Name, err.Error())
 			}
@@ -1360,13 +1355,7 @@ func (ctx *templateValidationCtx) validateDAG(scope map[string]interface{}, tmpl
 			return errors.Errorf(errors.CodeBadRequest, "templates.%s cannot use 'continueOn' when using 'depends'. Instead use 'dep-task.Failed'/'dep-task.Errored'", tmpl.Name)
 		}
 
-		var args wfv1.ArgumentsProvider
-		args = &FakeArguments{}
-		if task.TemplateRef != nil {
-			args = &task.Arguments
-		}
-
-		resolvedTmpl, err := ctx.validateTemplateHolder(&task, tmplCtx, args, workflowTemplateValidation)
+		resolvedTmpl, err := ctx.validateTemplateHolder(&task, tmplCtx, &FakeArguments{}, workflowTemplateValidation)
 
 		if err != nil {
 			return errors.Errorf(errors.CodeBadRequest, "templates.%s.tasks.%s %s", tmpl.Name, task.Name, err.Error())

Diff for: workflow/validate/validate_test.go

@@ -7,7 +7,6 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
 	apierr "k8s.io/apimachinery/pkg/api/errors"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 
@@ -3344,72 +3343,3 @@ func TestShouldCheckValidationToSpacedParameters(t *testing.T) {
 		assert.Contains(t, err.Error(), "failed to resolve {{  workflow.thisdoesnotexist  }}")
 	}
 }
-
-var dynamicWorkflowTemplateARefB = `
-apiVersion: argoproj.io/v1alpha1
-kind: WorkflowTemplate
-metadata:
-  name: workflow-template-a
-spec:
-  templates:
-  - name: template-a
-    inputs:
-      parameters:
-        - name: message
-    steps:
-      - - name: step-a
-          templateRef:
-            name: workflow-template-b
-            template: "{{ inputs.parameters.message }}"
-`
-
-var dynamicWorkflowTemplateRefB = `
-apiVersion: argoproj.io/v1alpha1
-kind: WorkflowTemplate
-metadata:
-  name: workflow-template-b
-spec:
-  templates:
-  - name: template-b
-    container:
-      image: docker/whalesay
-      command: [cowsay]
-      args: ["hello from template"]
-`
-
-var dynamicTemplateRefWorkflow = `
-apiVersion: argoproj.io/v1alpha1
-kind: Workflow
-metadata:
-  generateName: dynamic-workflow-
-spec:
-  entrypoint: whalesay
-  templates:
-  - name: whalesay
-    steps:
-      - - name: whalesay
-          templateRef:
-            name: workflow-template-a
-            template: template-a
-          arguments:
-            parameters:
-              - name: message
-                value: "template-b"
-`
-
-func TestDynamicWorkflowTemplateRef(t *testing.T) {
-	wf := wfv1.MustUnmarshalWorkflow(dynamicTemplateRefWorkflow)
-	wftmplA := wfv1.MustUnmarshalWorkflowTemplate(dynamicWorkflowTemplateARefB)
-	wftmplB := wfv1.MustUnmarshalWorkflowTemplate(dynamicWorkflowTemplateRefB)
-
-	err := createWorkflowTemplate(wftmplA)
-	require.NoError(t, err)
-	err = createWorkflowTemplate(wftmplB)
-	require.NoError(t, err)
-
-	err = ValidateWorkflow(wftmplGetter, cwftmplGetter, wf, ValidateOpts{})
-	require.NoError(t, err)
-
-	_ = deleteWorkflowTemplate(wftmplA.Name)
-	_ = deleteWorkflowTemplate(wftmplB.Name)
-}