[tests] Added tests for multi-tenant admin

Author: pandafy

openwisp_controller/config/migrations/__init__.py

@@ -25,10 +25,10 @@ def update_vpn_dhparam_length(apps, schema_editor):
 
 def assign_permissions_to_groups(apps, schema_editor):
     create_default_permissions(apps, schema_editor)
-    operators_and_admins_can_change = ["device", "config", "template"]
-    operators_read_only_admins_manage = ["vpn"]
-    manage_operations = ["add", "change", "delete"]
-    Group = get_swapped_model(apps, "openwisp_users", "Group")
+    operators_and_admins_can_change = ['device', 'config', 'template']
+    operators_read_only_admins_manage = ['vpn']
+    manage_operations = ['add', 'change', 'view', 'delete']
+    Group = get_swapped_model(apps, 'openwisp_users', 'Group')
 
     try:
         admin = Group.objects.get(name="Administrator")

openwisp_controller/config/tests/test_api.py

@@ -658,15 +658,6 @@ def test_operator_access_shared_template(self):
             update_payload={'name': 'updated-test'},
             expected_count=1,
             token=token,
-            expected_status_codes={
-                'create': 400,
-                'list': 200,
-                'retrieve': 403,
-                'update': 403,
-                'delete': 403,
-                'head': 403,
-                'option': 200,
-            },
         )
 
     def test_org_admin_create_template_with_shared_vpn(self):
@@ -689,16 +680,16 @@ def test_org_admin_create_template_with_shared_vpn(self):
             create_payload=create_payload,
             update_payload=update_payload,
             expected_count=1,
+            token=self._obtain_auth_token(test_user),
             expected_status_codes={
                 'create': 201,
                 'list': 200,
                 'retrieve': 200,
                 'update': 200,
                 'delete': 204,
-                'head': 403,
+                'head': 200,
                 'option': 200,
-            },
-            token=self._obtain_auth_token(test_user),
+            }
         )
 
     def test_template_create_with_empty_config(self):

openwisp_controller/connection/api/serializers.py

@@ -64,7 +64,6 @@ class Meta:
 
 class CredentialSerializer(BaseSerializer):
     params = serializers.JSONField()
-    include_shared = True
 
     class Meta:
         model = Credentials

openwisp_controller/geo/migrations/__init__.py

@@ -5,9 +5,9 @@
 
 def assign_permissions_to_groups(apps, schema_editor):
     create_default_permissions(apps, schema_editor)
-    operators_and_admins_can_change = ["location", "floorplan", "devicelocation"]
-    manage_operations = ["add", "change", "delete"]
-    Group = get_swapped_model(apps, "openwisp_users", "Group")
+    operators_and_admins_can_change = ['location', 'floorplan', 'devicelocation']
+    manage_operations = ['add', 'change', 'view', 'delete']
+    Group = get_swapped_model(apps, 'openwisp_users', 'Group')
 
     try:
         admin = Group.objects.get(name="Administrator")