Refine Conformance test for Pipeline

test/conformance_test.go

@@ -38,6 +38,9 @@ import (
 
 type conditionFn func(name string) ConditionAccessorFn
 
+// TestTaskRun examines the conformance of Tekton pipeline @HEAD. It does not
+// searve as part of the OSS conformance test suite but aims to keep the
+// devel conformant and to prevent regressions.
 func TestTaskRun(t *testing.T) {
 	ctx := context.Background()
 	ctx, cancel := context.WithCancel(ctx)
@@ -76,6 +79,7 @@ spec:
 					Reason:   "Completed",
 				},
 			},
+			TerminationReason: "Completed",
 		}},
 	}, {
 		name: "failed-task-run",
@@ -105,20 +109,23 @@ spec:
 					Reason:   "Completed",
 				},
 			},
+			TerminationReason: "Completed",
 		}, {
 			ContainerState: corev1.ContainerState{
 				Terminated: &corev1.ContainerStateTerminated{
 					ExitCode: 1,
 					Reason:   "Error",
 				},
 			},
+			TerminationReason: "Error",
 		}, {
 			ContainerState: corev1.ContainerState{
 				Terminated: &corev1.ContainerStateTerminated{
 					ExitCode: 1,
 					Reason:   "Error",
 				},
 			},
+			TerminationReason: "Skipped",
 		}},
 	}} {
 		t.Run(tc.name, func(t *testing.T) {
@@ -182,7 +189,7 @@ spec:
 			}
 
 			ignoreTerminatedFields := cmpopts.IgnoreFields(corev1.ContainerStateTerminated{}, "StartedAt", "FinishedAt", "ContainerID")
-			ignoreStepFields := cmpopts.IgnoreFields(v1.StepState{}, "ImageID", "Name", "ContainerName")
+			ignoreStepFields := cmpopts.IgnoreFields(v1.StepState{}, "ImageID", "Name", "Container")
 			if d := cmp.Diff(tr.Status.Steps, tc.expectedStepState, ignoreTerminatedFields, ignoreStepFields); d != "" {
 				t.Fatalf("-got, +want: %v", d)
 			}