feat: uds config validation

src/cmd/uds.go

@@ -216,7 +216,7 @@ func loadViperConfig() error {
 
 	// read relevant config into DeployOpts.Variables
 	// need to use goyaml because Viper doesn't preserve case: https://github.com/spf13/viper/issues/1014
-	err = goyaml.Unmarshal(configFile, &bundleCfg.DeployOpts)
+	err = goyaml.UnmarshalWithOptions(configFile, &bundleCfg.DeployOpts, goyaml.Strict())
 	if err != nil {
 		return err
 	}

src/test/bundles/07-helm-overrides/uds-config-invalid.yaml

@@ -0,0 +1,6 @@
+options:
+  log_level: debug
+
+# helm-overrides is an invalid top level key. Missing `variables` parent key.
+helm-overrides:
+  ui_color: "orange"

src/test/e2e/bundle_test.go

@@ -601,3 +601,17 @@ func TestBundleTmpDir(t *testing.T) {
 	err = os.RemoveAll("./customtmp")
 	require.NoError(t, err)
 }
+
+func TestInvalidConfig(t *testing.T) {
+	os.Setenv("UDS_CONFIG", filepath.Join("src/test/bundles/07-helm-overrides", "uds-config-invalid.yaml"))
+	deployZarfInit(t)
+	zarfPkgPath := "src/test/packages/helm"
+	e2e.HelmDepUpdate(t, fmt.Sprintf("%s/unicorn-podinfo", zarfPkgPath))
+	e2e.CreateZarfPkg(t, zarfPkgPath, false)
+	bundleDir := "src/test/bundles/07-helm-overrides"
+	createLocal(t, bundleDir, e2e.Arch)
+	bundlePath := filepath.Join(bundleDir, fmt.Sprintf("uds-bundle-helm-overrides-%s-0.0.1.tar.zst", e2e.Arch))
+	_, stderr := deployWithError(t, bundlePath)
+	require.Contains(t, stderr, "unknown field")
+	os.Unsetenv("UDS_CONFIG")
+}

src/test/e2e/commands_test.go

@@ -112,6 +112,12 @@
 	return stdout, stderr
 }
 
+func deployWithError(t *testing.T, tarballPath string) (stdout string, stderr string) {