[Fixes #227] Fix test

GeoNode · May 3, 2024 · f7e5d59 · f7e5d59
1 parent ae44f5c
commit f7e5d59
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 7 deletions.
diff --git a/importer/handlers/common/tests_vector.py b/importer/handlers/common/tests_vector.py
@@ -164,7 +164,9 @@ def test_import_resource_should_not_be_imported(self, celery_chord, ogr2ogr_driv
                     files=self.valid_files, execution_id=str(exec_id)
                 )
             self.assertIn(
-                "No valid layers found", exception.exception.args[0], 'No valid layers found.'
+                "No valid layers found",
+                exception.exception.args[0],
+                "No valid layers found.",
             )
 
             celery_chord.assert_not_called()

diff --git a/importer/orchestrator.py b/importer/orchestrator.py
@@ -1,5 +1,4 @@
 import logging
-import os
 from typing import Optional
 from uuid import UUID
 
@@ -9,7 +8,6 @@
 from django.utils import timezone
 from django.utils.module_loading import import_string
 from django_celery_results.models import TaskResult
-from geonode.base.enumerations import STATE_INVALID, STATE_PROCESSED, STATE_RUNNING
 from geonode.resource.models import ExecutionRequest
 from geonode.storage.manager import storage_manager
 from rest_framework import serializers

diff --git a/importer/tests/unit/test_orchestrator.py b/importer/tests/unit/test_orchestrator.py
@@ -12,7 +12,6 @@
 from django.utils import timezone
 from django_celery_results.models import TaskResult
 
-from geonode.base import enumerations as enum
 from geonode.resource.models import ExecutionRequest
 
 # Create your tests here.
@@ -95,7 +94,7 @@ def test_create_execution_request(self):
         exec_obj = ExecutionRequest.objects.filter(exec_id=exec_id).first()
         self.assertEqual(count + 1, ExecutionRequest.objects.count())
         self.assertDictEqual(input_files, exec_obj.input_params)
-        self.assertEqual(exec_obj.STATUS_READY, exec_obj.status))
+        self.assertEqual(exec_obj.STATUS_READY, exec_obj.status)
 
     @patch("importer.orchestrator.importer_app.tasks.get")
     def test_perform_next_step(self, mock_celery):

diff --git a/importer/tests/unit/test_task.py b/importer/tests/unit/test_task.py
@@ -184,7 +184,6 @@ def test_publish_resource_if_overwrite_should_call_the_publishing(
                 user=get_user_model().objects.get(username=self.user),
                 func_name="dummy_func",
                 step="dummy_step",
-                legacy_upload_name="dummy",
                 input_params={
                     "files": {"base_file": "/filepath"},
                     "overwrite_existing_layer": True,
@@ -239,7 +238,6 @@ def test_publish_resource_if_overwrite_should_not_call_the_publishing(
                     user=get_user_model().objects.get(username=self.user),
                     func_name="dummy_func",
                     step="dummy_step",
-                    legacy_upload_name="dummy",
                     input_params={
                         "files": {"base_file": "/filepath"},
                         "overwrite_existing_layer": True,