fix: clean up linting errors and test warnings

pytest.ini

@@ -1,7 +1,6 @@
 [pytest]
 testpaths = tests
 norecursedirs = data maps
-log_print = False
 minversion = 3.5
 console_output_style = count
 addopts = --pylint --cov-branch --cov=forklift --instafail --isort

tests/test_change_detection.py

@@ -7,9 +7,9 @@
 import logging
 from pathlib import Path
 
+import arcpy
 from pytest import raises
 
-import arcpy
 from forklift import core
 from forklift.change_detection import (ChangeDetection, _get_hashes,
                                        hash_field, table_name_field)

tests/test_core.py

@@ -11,10 +11,10 @@
 from os import path
 from pathlib import Path
 
+import arcpy
 import pytest
 from mock import MagicMock, Mock, patch
 
-import arcpy
 from forklift import core, engine
 from forklift.change_detection import ChangeDetection
 from forklift.exceptions import ValidationException

tests/test_crate.py

@@ -9,10 +9,10 @@
 import unittest
 from os import path
 
+from arcpy import SpatialReference, env
 from mock import patch
 from xxhash import xxh64
 
-from arcpy import SpatialReference, env
 from forklift.models import Crate, names_cache
 
 current_folder = path.dirname(path.abspath(__file__))

tests/test_lift.py

@@ -9,9 +9,9 @@
 import unittest
 from os import path
 
+import arcpy
 from mock import Mock, patch
 
-import arcpy
 from forklift import core, engine, lift
 from forklift.models import Crate, Pallet
 
@@ -118,6 +118,7 @@ def test_process_pallets_resets_arcpy(self):
         import arcpy
 
         def modify_workspace(value):
+            # pylint: disable-next=assigning-non-slot
             arcpy.env.workspace = value
 
         # pylint: disable=assignment-from-no-return