make kpt binary optional in test harness (#2758)

natasha41575 · web-flow · commit 5dad3b5a3cca · 2022-02-08T13:20:47.000-08:00
diff --git a/pkg/test/runner/runner.go b/pkg/test/runner/runner.go
@@ -73,9 +73,11 @@ func NewRunner(t *testing.T, testCase TestCase, c string) (*Runner, error) {
 	}
 	kptBin, err := getKptBin()
 	if err != nil {
-		return nil, fmt.Errorf("failed to find kpt binary: %w", err)
+		t.Logf("failed to find kpt binary: %v", err)
+	}
+	if kptBin != "" {
+		t.Logf("Using kpt binary: %s", kptBin)
 	}
-	t.Logf("Using kpt binary: %s", kptBin)
 	return &Runner{
 		pkgName:  filepath.Base(testCase.Path),
 		testCase: testCase,

Original file line number	Diff line number	Diff line change
`@@ -73,9 +73,11 @@ func NewRunner(t testing.T, testCase TestCase, c string) (Runner, error) {`
`73`	`73`	`}`
`74`	`74`	`kptBin, err := getKptBin()`
`75`	`75`	`if err != nil {`
`76`		`- return nil, fmt.Errorf("failed to find kpt binary: %w", err)`
	`76`	`+ t.Logf("failed to find kpt binary: %v", err)`
	`77`	`+ }`
	`78`	`+ if kptBin != "" {`
	`79`	`+ t.Logf("Using kpt binary: %s", kptBin)`
`77`	`80`	`}`
`78`		`- t.Logf("Using kpt binary: %s", kptBin)`
`79`	`81`	`return &Runner{`
`80`	`82`	`pkgName: filepath.Base(testCase.Path),`
`81`	`83`	`testCase: testCase,`