Add test for read_permissions for model and folder

unit/test_all.py

@@ -2658,7 +2658,6 @@ class TestModelverse(unittest.TestCase):
             # Might already be a member
             group_join("users", "group_user")
         except:
-            raise
             pass
 
         permission_group("users/user/test/a/b", "users")
@@ -2691,6 +2690,56 @@ class TestModelverse(unittest.TestCase):
         except ReadPermissionDenied:
             pass
 
+    def test_op_read_permissions_model(self):
+        model_add("users/user/test/a", "formalisms/SimpleClassDiagrams")
+
+        try:
+            # Might already exist
+            group_create("users")
+        except:
+            pass
+        self.do_create_user("group_user")
+        self.do_create_user("other_user")
+        try:
+            # Might already be a member
+            group_join("users", "group_user")
+        except:
+            pass
+
+        permission_group("users/user/test/a", "users")
+        permission_modify("users/user/test/a", "210")
+
+        # Test all four cases
+        assert self.do_as_user("user", read_permissions, ["users/user/test/a"]) == "RW"
+        assert self.do_as_user("group_user", read_permissions, ["users/user/test/a"]) == "R"
+        assert self.do_as_user("other_user", read_permissions, ["users/user/test/a"]) == ""
+        assert self.do_as_user("admin", read_permissions, ["users/user/test/a"]) == "RW"
+
+    def test_op_read_permissions_folder(self):
+        folder_create("users/user/test/a")
+
+        try:
+            # Might already exist
+            group_create("users")
+        except:
+            pass
+        self.do_create_user("group_user")
+        self.do_create_user("other_user")
+        try:
+            # Might already be a member
+            group_join("users", "group_user")
+        except:
+            pass
+
+        permission_group("users/user/test/a", "users")
+        permission_modify("users/user/test/a", "210")
+
+        # Test all four cases
+        assert self.do_as_user("user", read_permissions, ["users/user/test/a"]) == "RW"
+        assert self.do_as_user("group_user", read_permissions, ["users/user/test/a"]) == "R"
+        assert self.do_as_user("other_user", read_permissions, ["users/user/test/a"]) == ""
+        assert self.do_as_user("admin", read_permissions, ["users/user/test/a"]) == "RW"
+
     """
     def test_op_model_render(self):
     def test_op_transformation_between(self):
@@ -2707,7 +2756,6 @@ class TestModelverse(unittest.TestCase):
     def test_op_group_owner_add(self):
     def test_op_group_owner_delete(self):
     def test_op_group_kick(self):
-    def test_op_read_permissions(self):
     def test_op_conformance_delete(self):
     def test_op_conformance_add(self):
     """