Remove in-place operations on model in test_label unit test

awx/main/models/label.py

@@ -37,7 +37,7 @@ class Label(CommonModelNameNotUnique):
         return \
             Label.objects.filter(
                 organization=None,
-                jobtemplate_labels__isnull=True
+                unifiedjobtemplate_labels__isnull=True
             )
 
     def is_detached(self):

awx/main/tests/unit/models/test_label.py

@@ -1,21 +1,24 @@
 import pytest
+import mock
 
 from awx.main.models.label import Label
 from awx.main.models.unified_jobs import UnifiedJobTemplate, UnifiedJob
 
 
-def test_get_orphaned_labels(mocker):
-    mock_query_set = mocker.MagicMock()
-    Label.objects.filter = mocker.MagicMock(return_value=mock_query_set)
+mock_query_set = mock.MagicMock()
 
+mock_objects = mock.MagicMock(filter=mock.MagicMock(return_value=mock_query_set))
+
+@mock.patch('awx.main.models.label.Label.objects', mock_objects)
+class TestLabelFilterMocked:
+
+    def test_get_orphaned_labels(self, mocker):
         ret = Label.get_orphaned_labels()
 
         assert mock_query_set == ret
-    Label.objects.filter.assert_called_with(organization=None, jobtemplate_labels__isnull=True)
+        Label.objects.filter.assert_called_with(organization=None, unifiedjobtemplate_labels__isnull=True)
 
-def test_is_detached(mocker):
-    mock_query_set = mocker.MagicMock()
-    Label.objects.filter = mocker.MagicMock(return_value=mock_query_set)
+    def test_is_detached(self, mocker):
         mock_query_set.count.return_value = 1
 
         label = Label(id=37)
@@ -25,9 +28,7 @@ def test_is_detached(mocker):
         Label.objects.filter.assert_called_with(id=37, unifiedjob_labels__isnull=True, unifiedjobtemplate_labels__isnull=True)
         mock_query_set.count.assert_called_with()
 
-def test_is_detached_not(mocker):
-    mock_query_set = mocker.MagicMock()
-    Label.objects.filter = mocker.MagicMock(return_value=mock_query_set)
+    def test_is_detached_not(self, mocker):
         mock_query_set.count.return_value = 0
 
         label = Label(id=37)
@@ -42,7 +43,7 @@ def test_is_detached_not(mocker):
         (0, 1, True),
         (1, 1, False),
     ])
-def test_is_candidate_for_detach(mocker, jt_count, j_count, expected):
+    def test_is_candidate_for_detach(self, mocker, jt_count, j_count, expected):
         mock_job_qs = mocker.MagicMock()
         mock_job_qs.count = mocker.MagicMock(return_value=j_count)
         UnifiedJob.objects = mocker.MagicMock()