fix: remove code for unit test fix

These will be addressed in #7663

Author: melton-jason

specifyweb/backend/locality_update_tool/tests/test_localityupdate_status.py

@@ -36,7 +36,7 @@ def test_localityupdate_not_exist(self):
         self._assertStatusCodeEqual(response, http.HttpResponseNotFound.status_code)
         self.assertEqual(response.content.decode(), f"The localityupdate with task id '{task_id}' was not found")
 
-    @patch("specifyweb.backend.locality_update_tool.views.update_locality_task.AsyncResult")
+    @patch("specifyweb.specify.views.update_locality_task.AsyncResult")
     def test_failed(self, AsyncResult: Mock):
         mock_result = Mock()
         mock_result.state = CELERY_TASK_STATE.FAILURE
@@ -70,7 +70,7 @@ def test_failed(self, AsyncResult: Mock):
             }
         )
 
-    @patch("specifyweb.backend.locality_update_tool.views.update_locality_task.AsyncResult")
+    @patch("specifyweb.specify.views.update_locality_task.AsyncResult")
     def test_parse_failed(self, AsyncResult: Mock):
         mock_result = Mock()
         mock_result.state = CELERY_TASK_STATE.SUCCESS
@@ -98,7 +98,7 @@ def test_parse_failed(self, AsyncResult: Mock):
             }
         )
 
-    @patch("specifyweb.backend.locality_update_tool.views.update_locality_task.AsyncResult")
+    @patch("specifyweb.specify.views.update_locality_task.AsyncResult")
     def test_parsed(self, AsyncResult: Mock):
         mock_result = Mock()
         mock_result.state = CELERY_TASK_STATE.SUCCESS
@@ -149,7 +149,7 @@ def test_parsed(self, AsyncResult: Mock):
             }
         )
 
-    @patch("specifyweb.backend.locality_update_tool.views.update_locality_task.AsyncResult")
+    @patch("specifyweb.specify.views.update_locality_task.AsyncResult")
     def test_succeeded(self, AsyncResult: Mock):
         mock_result = Mock()
         mock_result.state = LocalityUpdateStatus.SUCCEEDED
@@ -181,7 +181,7 @@ def test_succeeded(self, AsyncResult: Mock):
             }
         )
 
-    @patch("specifyweb.backend.locality_update_tool.views.update_locality_task.AsyncResult")
+    @patch("specifyweb.specify.views.update_locality_task.AsyncResult")
     def test_succeeded_locality_rows(self, AsyncResult: Mock):
         mock_result = Mock()
         mock_result.state = LocalityUpdateStatus.SUCCEEDED

specifyweb/backend/locality_update_tool/tests/test_parse_field.py

@@ -33,7 +33,7 @@ def test_no_ui_formatter(self):
 
         self.assertEqual(parsed_with_value, parsed_with_value_result)
 
-    @patch("specifyweb.backend.locality_update_tool.update_locality.get_uiformatter")
+    @patch("specifyweb.specify.update_locality.get_uiformatter")
     def test_cnn_formatter(self, get_uiformatter: Mock):
 
         get_uiformatter.return_value = UIFormatter(

specifyweb/backend/locality_update_tool/update_locality.py

@@ -379,19 +379,11 @@ def parse_locality_set(collection, raw_headers: list[str], data: list[list[str]]
         locality_id: int | None = None if len(
             locality_query) != 1 else locality_query[0].id
 
-        parsed_locality_fields = [
-            parse_field(
-                collection, 'Locality', d['field'], d['value'], locality_id, row_number
-            )
-            for d in locality_values
-        ]
+        parsed_locality_fields = [parse_field(
+            collection, 'Locality', dict['field'], dict['value'], locality_id, row_number) for dict in locality_values if dict['value'].strip() != ""]
 
-        parsed_geocoorddetail_fields = [
-            parse_field(
-                collection, 'Geocoorddetail', d['field'], d['value'], locality_id, row_number
-            )
-            for d in geocoorddetail_values
-        ]
+        parsed_geocoorddetail_fields = [parse_field(
+            collection, 'Geocoorddetail', dict["field"], dict['value'], locality_id, row_number) for dict in geocoorddetail_values if dict['value'].strip() != ""]
 
         parsed_row, parsed_errors = merge_parse_results(
             [*parsed_locality_fields, *parsed_geocoorddetail_fields], locality_id, row_number)

specifyweb/backend/trees/extras.py

@@ -411,46 +411,18 @@ def synonymize(node, into, agent, user=None, collection=None):
 
     # This check can be disabled by a remote pref
     import specifyweb.backend.context.app_resource as app_resource
-
-    collection_prefs_dict = {}  # always defined
-
-    res = app_resource.get_app_resource(collection, user, 'CollectionPreferences')
-    force_checks = (collection is None or user is None)
-    if res is not None:
-        collection_prefs_json, _, __ = res
-        if collection_prefs_json:
-            try:
-                collection_prefs_dict = json.loads(collection_prefs_json) or {}
-            except Exception:
-                collection_prefs_dict = {}
+    collection_prefs_json, _, __ = app_resource.get_app_resource(collection, user, 'CollectionPreferences')
+    if collection_prefs_json is not None:
+            collection_prefs_dict = json.loads(collection_prefs_json)
 
     treeManagement_pref = collection_prefs_dict.get('treeManagement', {})
-    if force_checks and target.children.exists():
-        raise TreeBusinessRuleException(
-            f'Synonymizing "{node.fullname}" to "{into.fullname}" which has children',
-            {"tree": "Taxon",
-             "localizationKey": "nodeSynonimizeWithChildren",
-             "node": {
-                "id": node.id,
-                "rankid": node.rankid,
-                "fullName": node.fullname,
-                "children": list(node.children.values('id', 'fullname'))
-             },
-             "parent": {
-                "id": into.id,
-                "rankid": into.rankid,
-                "fullName": into.fullname,
-                "parentid": into.parent.id,
-                "children": list(into.children.values('id', 'fullname'))
-             }}
-        )
-    force_checks = (collection is None or user is None)
+
     synonymized = treeManagement_pref.get('synonymized', {}) \
         if isinstance(treeManagement_pref, dict) else {}
 
     add_synonym_enabled = synonymized.get(r'^sp7\.allow_adding_child_to_synonymized_parent\.' + node.specify_model.name + '=(.+)', False) if isinstance(synonymized, dict) else False
 
-    if node.children.count() > 0 and (force_checks or add_synonym_enabled is False):
+    if node.children.count() > 0 and (add_synonym_enabled is True):
         raise TreeBusinessRuleException(
             f'Synonymizing node "{node.fullname}" which has children',
             {"tree" : "Taxon",