[test] Fixed failing tests due to openwisp-utils menu changes

Co-authored-by: Pedro Peña <pedro@makedit.com>
tknv · Aug 24, 2021 · 2a64e09 · 2a64e09
1 parent a44e2fd
commit 2a64e09
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/openwisp_controller/config/tests/test_admin.py b/openwisp_controller/config/tests/test_admin.py
@@ -1259,16 +1259,16 @@ def test_admin_menu_groups(self):
         self.client.force_login(self._get_admin())
         response = self.client.get(reverse('admin:index'))
         for model in models:
-            with self.subTest(f'test_menu_group_for_{model}_model'):
+            with self.subTest(f'test menu group link for {model} model'):
                 url = reverse(f'admin:{self.app_label}_{model}_changelist')
-                self.assertContains(response, f'<a class="mg-link" href="{url}">')
-        with self.subTest('test_configuration_group_is_registered'):
+                self.assertContains(response, f' class="mg-link" href="{url}"')
+        with self.subTest('test "Configurations" group is registered'):
             self.assertContains(
                 response,
                 '<div class="mg-dropdown-label">Configurations </div>',
                 html=True,
             )
         # Test Device is registered as menu item
-        with self.subTest('test_device_is_registered_as_menu_item'):
+        with self.subTest('test device menu item is registered'):
             url = reverse(f'admin:{self.app_label}_device_changelist')
-            self.assertContains(response, f'<a class="menu-item" href="{url}">')
+            self.assertContains(response, f' class="menu-item" href="{url}"')
diff --git a/openwisp_controller/connection/tests/test_admin.py b/openwisp_controller/connection/tests/test_admin.py
@@ -126,9 +126,9 @@ def test_admin_menu_groups(self):
         models = ['credentials']
         response = self.client.get(reverse('admin:index'))
         for model in models:
-            with self.subTest(f'test_menu_group_for_{model}_model'):
+            with self.subTest(f'test menu group link for {model} model'):
                 url = reverse(f'admin:{self.app_label}_{model}_changelist')
-                self.assertContains(response, f'<a class="mg-link" href="{url}">')
+                self.assertContains(response, f' class="mg-link" href="{url}"')
 
 
 class TestCommandInlines(TestAdminMixin, CreateConnectionsMixin, TestCase):

diff --git a/openwisp_controller/geo/tests/test_admin.py b/openwisp_controller/geo/tests/test_admin.py
@@ -132,10 +132,10 @@ def test_admin_menu_groups(self):
         models = ['location', 'floorplan']
         response = self.client.get(reverse('admin:index'))
         for model in models:
-            with self.subTest(f'test_admin_group_for_{model}_model'):
+            with self.subTest(f'test menu group link for {model} model'):
                 url = reverse(f'admin:{self.app_label}_{model}_changelist')
-                self.assertContains(response, f'<a class="mg-link" href="{url}">')
-        with self.subTest('test_geo_group_is_registered'):
+                self.assertContains(response, f' class="mg-link" href="{url}"')
+        with self.subTest('test "Geographic Info" group is registered'):
             self.assertContains(
                 response,
                 '<div class="mg-dropdown-label">Geographic Info </div>',

diff --git a/openwisp_controller/pki/tests/test_admin.py b/openwisp_controller/pki/tests/test_admin.py
@@ -136,10 +136,10 @@ def test_admin_menu_groups(self):
         models = ['ca', 'cert']
         response = self.client.get(reverse('admin:index'))
         for model in models:
-            with self.subTest(f'test_admin_group_for_{model}_model'):
+            with self.subTest(f'test menu group link for {model} model'):
                 url = reverse(f'admin:{self.app_label}_{model}_changelist')
-                self.assertContains(response, f'<a class="mg-link" href="{url}">')
-        with self.subTest('test_configuration_group_is_registered'):
+                self.assertContains(response, f' class="mg-link" href="{url}"')
+        with self.subTest('test "Cas & Certificates" group is registered'):
             self.assertContains(
                 response,
                 '<div class="mg-dropdown-label">Cas & Certificates </div>',