mounttablelib: developer brain in neutral - fixed.
When building the database from the config, I didn't copy permissions
because I was breaking out early when there was no ctx or call passed in.
Also fixed reading in the config. It is read into a map by json and when
going through the map, we screw up the ordering. We now sort the keys of the
config nodes lexographically so that permissions are correctly inherited by
child nodes.
Change-Id: I6c53a5aa19be6f40415867db540f95d6514e0eed
diff --git a/services/mounttable/mounttablelib/mounttable_test.go b/services/mounttable/mounttablelib/mounttable_test.go
index 6d27b94..1a07339 100644
--- a/services/mounttable/mounttablelib/mounttable_test.go
+++ b/services/mounttable/mounttablelib/mounttable_test.go
@@ -795,6 +795,26 @@
}
}
+func TestIntermediateNodesCreatedFromConfig(t *testing.T) {
+ rootCtx, _, _, shutdown := initTest()
+ defer shutdown()
+
+ server, estr := newMT(t, "testdata/intermediate.perms", "", "TestIntermediateNodesCreatedFromConfig", rootCtx)
+ defer server.Stop()
+
+ // x and x/y should have the same permissions at the root.
+ rootPerms, _ := doGetPermissions(t, rootCtx, estr, "", true)
+ if perms, _ := doGetPermissions(t, rootCtx, estr, "x", true); !reflect.DeepEqual(rootPerms, perms) {
+ boom(t, "for x got %v, want %v", perms, rootPerms)
+ }
+ if perms, _ := doGetPermissions(t, rootCtx, estr, "x/y", true); !reflect.DeepEqual(rootPerms, perms) {
+ boom(t, "for x/y got %v, want %v", perms, rootPerms)
+ }
+ if perms, _ := doGetPermissions(t, rootCtx, estr, "x/y/z", true); reflect.DeepEqual(rootPerms, perms) {
+ boom(t, "for x/y/z got %v, don't want %v", perms, rootPerms)
+ }
+}
+
func initTest() (rootCtx *context.T, aliceCtx *context.T, bobCtx *context.T, shutdown v23.Shutdown) {
test.Init()
ctx, shutdown := test.InitForTest()
