Fixing pylint errors

OpenConceptLab · Apr 1, 2022 · 8d72104 · 8d72104
1 parent fcca81f
commit 8d72104
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/core/common/tests.py b/core/common/tests.py
@@ -794,6 +794,6 @@ def test_app_name(self):
 
 class TaskTest(OCLTestCase):
     @patch('core.common.tasks.S3')
-    def test_delete_s3_objects(self, s3_mock):
+    def test_delete_s3_objects(self, s3_mock):  # pylint: disable=no-self-use
         delete_s3_objects('/some/path')
         s3_mock.delete_objects.assert_called_once_with('/some/path')
diff --git a/core/sources/tests/tests.py b/core/sources/tests/tests.py
@@ -806,8 +806,8 @@ def test_seed_children_task_with_export(self, export_source_task, index_children
 
     def test_update_source_active_mappings_count(self):
         source = OrganizationSourceFactory()
-        mapping1 = MappingFactory(parent=source)
-        mapping2 = MappingFactory(retired=True, parent=source)
+        MappingFactory(parent=source)
+        MappingFactory(retired=True, parent=source)
 
         self.assertEqual(source.active_mappings, None)
 
@@ -818,8 +818,8 @@ def test_update_source_active_mappings_count(self):
 
     def test_update_source_active_concepts_count(self):
         source = OrganizationSourceFactory()
-        concept1 = ConceptFactory(parent=source)
-        concept2 = ConceptFactory(retired=True, parent=source)
+        ConceptFactory(parent=source)
+        ConceptFactory(retired=True, parent=source)
 
         self.assertEqual(source.active_concepts, None)
 
@@ -830,14 +830,14 @@ def test_update_source_active_concepts_count(self):
 
     @patch('core.sources.models.Source.mappings')
     @patch('core.sources.models.Source.batch_index')
-    def test_index_source_mappings(self, batch_index_mock, source_mappings_mock):
+    def test_index_source_mappings(self, batch_index_mock, source_mappings_mock):  # pylint: disable=no-self-use
         source = OrganizationSourceFactory()
         index_source_mappings(source.id)
         batch_index_mock.assert_called_once_with(source_mappings_mock, MappingDocument)
 
     @patch('core.sources.models.Source.concepts')
     @patch('core.sources.models.Source.batch_index')
-    def test_index_source_mappings(self, batch_index_mock, source_concepts_mock):
+    def test_index_source_concepts(self, batch_index_mock, source_concepts_mock):  # pylint: disable=no-self-use
         source = OrganizationSourceFactory()
         index_source_concepts(source.id)
         batch_index_mock.assert_called_once_with(source_concepts_mock, ConceptDocument)