Added unit tests

hashicorp · May 6, 2024 · 886714f · 886714f
1 parent 78eebe3
commit 886714f
Showing 1 changed file with 43 additions and 5 deletions.
diff --git a/dependency/vault_read_test.go b/dependency/vault_read_test.go
@@ -652,66 +652,104 @@ func TestVaultReadQuery_String(t *testing.T) {
 
 func TestShimKVv2Path(t *testing.T) {
 	cases := []struct {
-		name      string
-		path      string
-		mountPath string
-		expected  string
+		name            string
+		path            string
+		mountPath       string
+		expected        string
+		clientNamespace string
 	}{
 		{
 			"full path",
 			"secret/data/foo/bar",
 			"secret/",
 			"secret/data/foo/bar",
+			"",
 		}, {
 			"data prefix added",
 			"secret/foo/bar",
 			"secret/",
 			"secret/data/foo/bar",
+			"",
 		}, {
 			"full path with data* in subpath",
 			"secret/data/datafoo/bar",
 			"secret/",
 			"secret/data/datafoo/bar",
+			"",
 		}, {
 			"prefix added with data* in subpath",
 			"secret/datafoo/bar",
 			"secret/",
 			"secret/data/datafoo/bar",
+			"",
 		}, {
 			"prefix added with *data in subpath",
 			"secret/foodata/foo/bar",
 			"secret/",
 			"secret/data/foodata/foo/bar",
+			"",
 		}, {
 			"prefix not added to metadata",
 			"secret/metadata/foo/bar",
 			"secret/",
 			"secret/metadata/foo/bar",
+			"",
 		}, {
 			"prefix added with metadata* in subpath",
 			"secret/metadatafoo/foo/bar",
 			"secret/",
 			"secret/data/metadatafoo/foo/bar",
+			"",
 		}, {
 			"prefix added with *metadata in subpath",
 			"secret/foometadata/foo/bar",
 			"secret/",
 			"secret/data/foometadata/foo/bar",
+			"",
 		}, {
 			"prefix added to mount path",
 			"secret/",
 			"secret/",
 			"secret/data",
+			"",
 		}, {
 			"prefix added to mount path not exact match",
 			"secret",
 			"secret/",
 			"secret/data",
+			"",
+		},
+		{
+			"raw path contains partial namespace, not adjusted",
+			"a/b/c/secret",
+			"c/secret/",
+			"c/secret/data",
+			"a/b",
+		},
+		{
+			"raw path contains partial namespace, adjusted",
+			"a/b/c/secret",
+			"c/secret/data",
+			"c/secret/data",
+			"a/b",
+		},
+		{
+			"raw path contains partial namespace, and 'data' in secret path, not adjusted",
+			"a/b/c/secret",
+			"c/secret/random/data/here",
+			"c/secret/data/random/data/here",
+			"a/b",
+		}, {
+			"raw path contains partial namespace, and 'data' in secret path, adjusted",
+			"a/b/c/secret",
+			"c/secret/data/random/data/here",
+			"c/secret/data/random/data/here",
+			"a/b",
 		},
 	}
 	for _, tc := range cases {
 		t.Run(tc.name, func(t *testing.T) {
-			actual := shimKVv2Path(tc.path, tc.mountPath, "")
+			actual := shimKVv2Path(tc.path, tc.mountPath, tc.clientNamespace)
 			assert.Equal(t, tc.expected, actual)
 		})
 	}