add missing test cases for matrix params array indexing validation

This commit adds missing test cases for matrix params array indexing validation. The array indexing for matrix params is supported in previous PR but test cases are missing. Closes tektoncd#6278 Signed-off-by: Yongxuan Zhang [email protected]
Yongxuanzhang · Mar 2, 2023 · 8886484 · 8886484
1 parent 67abc4a
commit 8886484
Show file tree

Hide file tree

Showing 2 changed files with 48 additions and 2 deletions.
diff --git a/pkg/apis/pipeline/v1/pipeline_validation_test.go b/pkg/apis/pipeline/v1/pipeline_validation_test.go
@@ -3589,10 +3589,33 @@ func TestValidateParamArrayIndex_invalid(t *testing.T) {
 					Operator: selection.In,
 					Values:   []string{"$(params.second-param[2])"},
 				}},
+				Matrix: &Matrix{
+					Params: []Param{
+						{Name: "final-task-first-param", Value: *NewStructuredValues("$(params.first-param[4])")},
+						{Name: "final-task-second-param", Value: *NewStructuredValues("$(params.second-param[4])")},
+					}},
+			}},
+		},
+		params:   []Param{{Name: "second-param", Value: *NewStructuredValues("second-value", "second-value-again")}},
+		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2]) $(params.first-param[3]) $(params.first-param[4]) $(params.second-param[2]) $(params.second-param[3]) $(params.second-param[4])]"),
+	}, {
+		name: "parameter in matrix reference out of bound",
+		original: PipelineSpec{
+			Params: []ParamSpec{
+				{Name: "first-param", Type: ParamTypeArray, Default: NewStructuredValues("default-value", "default-value-again")},
+				{Name: "second-param", Type: ParamTypeArray},
+			},
+			Tasks: []PipelineTask{{
+				Matrix: &Matrix{
+					Params: []Param{
+						{Name: "first-task-first-param", Value: *NewStructuredValues("$(params.first-param[2])")},
+						{Name: "first-task-second-param", Value: *NewStructuredValues("static value")},
+					},
+				},
 			}},
 		},
 		params:   []Param{{Name: "second-param", Value: *NewStructuredValues("second-value", "second-value-again")}},
-		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2]) $(params.first-param[3]) $(params.second-param[2]) $(params.second-param[3])]"),
+		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2])]"),
 	}, {
 		name: "parameter references with bracket notation and special characters reference out of bound",
 		original: PipelineSpec{

diff --git a/pkg/apis/pipeline/v1beta1/pipeline_validation_test.go b/pkg/apis/pipeline/v1beta1/pipeline_validation_test.go
@@ -4049,10 +4049,33 @@ func TestValidateParamArrayIndex_invalid(t *testing.T) {
 					Operator: selection.In,
 					Values:   []string{"$(params.second-param[2])"},
 				}},
+				Matrix: &Matrix{
+					Params: []Param{
+						{Name: "final-task-first-param", Value: *NewStructuredValues("$(params.first-param[4])")},
+						{Name: "final-task-second-param", Value: *NewStructuredValues("$(params.second-param[4])")},
+					}},
+			}},
+		},
+		params:   []Param{{Name: "second-param", Value: *NewStructuredValues("second-value", "second-value-again")}},
+		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2]) $(params.first-param[3]) $(params.first-param[4]) $(params.second-param[2]) $(params.second-param[3]) $(params.second-param[4])]"),
+	}, {
+		name: "parameter in matrix reference out of bound",
+		original: PipelineSpec{
+			Params: []ParamSpec{
+				{Name: "first-param", Type: ParamTypeArray, Default: NewStructuredValues("default-value", "default-value-again")},
+				{Name: "second-param", Type: ParamTypeArray},
+			},
+			Tasks: []PipelineTask{{
+				Matrix: &Matrix{
+					Params: []Param{
+						{Name: "first-task-first-param", Value: *NewStructuredValues("$(params.first-param[2])")},
+						{Name: "first-task-second-param", Value: *NewStructuredValues("static value")},
+					},
+				},
 			}},
 		},
 		params:   []Param{{Name: "second-param", Value: *NewStructuredValues("second-value", "second-value-again")}},
-		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2]) $(params.first-param[3]) $(params.second-param[2]) $(params.second-param[3])]"),
+		expected: fmt.Errorf("non-existent param references:[$(params.first-param[2])]"),
 	}, {
 		name: "parameter references with bracket notation and special characters reference out of bound",
 		original: PipelineSpec{