Add a test for an error case

Signed-off-by: David Gageot <david@gageot.net>
GoogleContainerTools · Sep 26, 2018 · 8f878fe · 8f878fe
1 parent 093a17e
commit 8f878fe
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/pkg/skaffold/deploy/kustomize_test.go b/pkg/skaffold/deploy/kustomize_test.go
@@ -58,6 +58,11 @@ func TestDependenciesForKustomization(t *testing.T) {
 - files: [app2.properties, app3.properties]`,
 			expected: []string{"kustomization.yaml", "app1.properties", "app2.properties", "app3.properties"},
 		},
+		{
+			description: "unknown base",
+			yaml:        `bases: [other]`,
+			shouldErr:   true,
+		},
 	}
 
 	for _, test := range tests {
@@ -69,7 +74,7 @@ func TestDependenciesForKustomization(t *testing.T) {
 
 			deps, err := dependenciesForKustomization(tmp.Root())
 
-			testutil.CheckErrorAndDeepEqual(t, false, err, joinPaths(tmp.Root(), test.expected), deps)
+			testutil.CheckErrorAndDeepEqual(t, test.shouldErr, err, joinPaths(tmp.Root(), test.expected), deps)
 		})
 	}
 }