Fix typos

netbox_sys_plugin/tests/vm_assigned_extra_config/test_vm_assigned_extra_config_api.py

@@ -172,7 +172,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
             },
         ]
 
-    def test_create_valid_extra_config(self):
+    def test_create_valid_assigned_extra_config(self):
         """Test creating a valid extra config"""
         obj_perm = ObjectPermission(
             name="Create Extra Config Permission",
@@ -189,7 +189,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertEqual(response.data["extra_config_values"], form_data["extra_config_values"])
         self.assertEqual(response.data["provider_type_extra_config_assignment_desc"], form_data["provider_type_extra_config_assignment_desc"])
 
-    def test_extra_config_unique_key(self):
+    def test_assigned_extra_config_unique_key(self):
         """Test extra config unique key"""
         obj_perm = ObjectPermission(
             name="Invalid Extra Config Permission",
@@ -204,7 +204,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertHttpStatus(response, status.HTTP_400_BAD_REQUEST)
         self.assertIn('The fields assigned_object_id must make a unique set.',str(response.data))
 
-    def test_create_invalid_extra_config(self):
+    def test_create_invalid_assigned_extra_config(self):
         """Test creating invalid extra config"""
         obj_perm = ObjectPermission(
             name="Invalid Extra Config Permission",
@@ -234,7 +234,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertHttpStatus(response, status.HTTP_400_BAD_REQUEST)
         self.assertIn('This field may not be null.',str(response.data))
 
-    def test_update_extra_config(self):
+    def test_update_assigned_extra_config(self):
         """Test updating an existing extra config"""
         extra_config = VmAssignedExtraConfig.objects.first()
         obj_perm = ObjectPermission(
@@ -250,7 +250,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertHttpStatus(response, status.HTTP_200_OK)
         self.assertEqual(response.data["provider_type_extra_config_assignment_desc"], update_data["provider_type_extra_config_assignment_desc"])
 
-    def test_get_all_extra_config(self):
+    def test_get_all_assigned_extra_config(self):
         """Test fetching all extra config"""
         obj_perm = ObjectPermission(
             name="Create Extra Config Permission",
@@ -264,7 +264,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertHttpStatus(response, status.HTTP_200_OK)
         self.assertGreaterEqual(len(response.data), 2)
 
-    def test_get_single_extra_config(self):
+    def test_get_single_assigned_extra_config(self):
         """Test fetching a single extra config"""
         obj_perm = ObjectPermission(
             name="View Extra Config Permission",
@@ -280,7 +280,7 @@ class VmAssignedExtraConfigApiTestCase(BaseAPITestCase):
         self.assertEqual(response.data["extra_config_values"], extra_config.extra_config_values)
         self.assertEqual(response.data["provider_type_extra_config_assignment_desc"], extra_config.provider_type_extra_config_assignment_desc)
 
-    def test_delete_extra_config(self):
+    def test_delete_assigned_extra_config(self):
         """Test deleting a extra config"""
         extra_config = VmAssignedExtraConfig.objects.first()
         obj_perm = ObjectPermission(

netbox_sys_plugin/tests/vm_assigned_extra_config/test_vm_assigned_extra_config_view.py

@@ -17,7 +17,7 @@ class VmAssignedExtraConfigFormTestCase(
     model = VmAssignedExtraConfig
     form = VmAssignedExtraConfigForm
 
-    def test_create_valid_extra_config_value(self):
+    def test_create_valid_assigned_extra_config(self):
         """Create a valid Extra Config Assignment"""
 
         clt_content_type = ContentType.objects.get_for_model(ClusterType)
@@ -52,7 +52,7 @@ class VmAssignedExtraConfigFormTestCase(
         obj_perm.users.add(self.user)  # pylint: disable=no-member
         obj_perm.object_types.add(ContentType.objects.get_for_model(self.model)) # pylint: disable=no-member
 
-    def test_create_VM_with_two_extra_config_value_assigment(self):
+    def test_create_vm_with_two_extra_config_value_assigment(self):
         """Test the assignment of 2 extra configs to the same Cluster Type """
 
         clt_content_type = ContentType.objects.get_for_model(ClusterType)
@@ -103,7 +103,7 @@ class VmAssignedExtraConfigFormTestCase(
             form.errors.get("__all__",[])
         )
 
-    def test_invalid_extra_config_format(self):
+    def test_invalid_assigned_extra_config_format(self):
         """
         Test invalid extra config invalid format.
         """
@@ -141,7 +141,7 @@ class VmAssignedExtraConfigFormTestCase(
             form.errors.get("extra_config_values", []),
         )
         #List Validation
-        invalid_form_data["extra_config_values"] = {"description":"description"}  # wrong fild
+        invalid_form_data["extra_config_values"] = {"description":"description"}  # Wrong field
         form = self.form(data=invalid_form_data)
         self.assertFalse(form.is_valid())
         self.assertIn(
@@ -149,7 +149,7 @@ class VmAssignedExtraConfigFormTestCase(
             form.errors.get("extra_config_values", []),
         )
         #Missing property Type
-        invalid_form_data["extra_config_values"] = {"id":True}  # wrong type
+        invalid_form_data["extra_config_values"] = {"id":True}  # Wrong type
         form = self.form(data=invalid_form_data)
         self.assertFalse(form.is_valid())
         self.assertIn(