[fix] Fixed test_device_with_config_change_deactivate_deactivate

+ reformatted code

Author: nemesifier

openwisp_controller/config/tests/test_admin.py

@@ -2190,6 +2190,8 @@ def _get_delete_btn_html(self, device):
             'Delete</a></p>'
         )
 
+    _deactivated_device_expected_readonly_fields = 22
+
     def test_device_with_config_change_deactivate_deactivate(self):
         """
         This test checks the following things
@@ -2229,7 +2231,7 @@ def test_device_with_config_change_deactivate_deactivate(self):
         self.assertContains(
             response,
             '<div class="readonly">',
-            22,
+            self._deactivated_device_expected_readonly_fields,
         )
         # Save buttons are absent on deactivated device
         self.assertNotContains(response, self._save_btn_html)

tests/openwisp2/sample_config/tests.py

@@ -1,11 +1,13 @@
+from openwisp_controller.config.tests.test_admin import TestAdmin as BaseTestAdmin
 from openwisp_controller.config.tests.test_admin import (
-        TestAdmin as BaseTestAdmin,
-        TestTransactionAdmin as BaseTestTransactionAdmin,
+    TestDeviceGroupAdmin as BaseTestDeviceGroupAdmin,
 )
 from openwisp_controller.config.tests.test_admin import (
-    TestDeviceGroupAdmin as BaseTestDeviceGroupAdmin,
     TestDeviceGroupAdminTransaction as BaseTestDeviceGroupAdminTransaction,
 )
+from openwisp_controller.config.tests.test_admin import (
+    TestTransactionAdmin as BaseTestTransactionAdmin,
+)
 from openwisp_controller.config.tests.test_api import TestConfigApi as BaseTestConfigApi
 from openwisp_controller.config.tests.test_apps import TestApps as BaseTestApps
 from openwisp_controller.config.tests.test_config import TestConfig as BaseTestConfig
@@ -44,6 +46,7 @@ class TestAdmin(BaseTestAdmin):
 
 class TestTransactionAdmin(BaseTestTransactionAdmin):
     app_label = 'sample_config'
+    _deactivated_device_expected_readonly_fields = 23
 
 
 class TestDeviceGroupAdmin(BaseTestDeviceGroupAdmin):