Miscellaneous cleanups

arches/app/models/tile.py

@@ -807,7 +807,7 @@ def __preSave(self, request=None, context=None):
             )
             logger.warning(e)
 
-    def __preDelete(self, request):
+    def __preDelete(self, request=None):
         try:
             for function in self._getFunctionClassInstances():
                 try:

arches/app/models/utils.py

@@ -14,5 +14,5 @@ def add_to_update_fields(kwargs, field_name):
             kwargs["update_fields"] = new
 
 
-def field_names(instance):
-    return {f.name for f in instance._meta.fields}
+def field_names(instance_or_class):
+    return {f.name for f in instance_or_class._meta.fields}

tests/models/resource_test.py

@@ -352,7 +352,8 @@ def test_delete_acts_on_custom_indices(self, mock):
         other_resource = Resource(pk=uuid.uuid4())
         with sync_overridden_test_settings_to_arches():
             self.test_resource.delete_index(other_resource.pk)
-        self.assertIn(str(other_resource.pk), str(mock._mock_call_args))
+        # delete_resources() was called with the correct resource id.
+        self.assertEqual(other_resource.pk, mock._mock_call_args[1]["resources"].pk)
 
     def test_publication_restored_on_save(self):
         """