Merge pull request #369 from mabruzzo/address-flake8-other

adjust pygrackle/utilities/testing.py to satisfy flake8

Author: brittonsmith

src/python/pygrackle/utilities/testing.py

@@ -13,17 +13,12 @@
 
 import importlib
 import numpy as np
-from numpy.testing import assert_array_equal, assert_almost_equal, \
-    assert_approx_equal, assert_array_almost_equal, assert_equal, \
-    assert_array_less, assert_string_equal, assert_array_almost_equal_nulp,\
-    assert_allclose, assert_raises
-import os
-import shutil
+from numpy.testing import assert_almost_equal
 import subprocess
 
 def assert_rel_equal(a1, a2, decimals, err_msg='', verbose=True):
     if isinstance(a1, np.ndarray):
-        assert(a1.size == a2.size)
+        assert a1.size == a2.size
         # Mask out NaNs
         a1[np.isnan(a1)] = 1.0
         a2[np.isnan(a2)] = 1.0
@@ -49,6 +44,7 @@ def requires_module(module):
     """
     def ffalse(func):
         return lambda: None
+
     def ftrue(func):
         return func
     try:

src/python/setup.cfg

@@ -1,6 +1,5 @@
 [flake8]
 max-line-length = 999
-exclude = pygrackle/utilities/testing.py
 ignore = E111,E121,E122,E123,E124,E125,E127,E129,E131,E201,E202,E211,E221,E222,E227,E228,E241,E301,E203,E225,E226,E231,E251,E261,E262,E265,E266,E302,E303,E402,E502,E701,E731,W292,W293,W391,W503,W504
 # when we switch to ruff-format (in the newchem-cpp branch) we won't need to
 # ignore F401 (ruff-format provides a nice workaround)

src/python/tests/test_chemistry.py

@@ -14,15 +14,14 @@
 import numpy as np
 import os
 
+from numpy.testing import assert_array_less
+
 from pygrackle import \
     chemistry_data, \
     setup_fluid_container, \
     set_cosmology_units
 
-from pygrackle.utilities.testing import \
-    random_logscale, \
-    assert_rel_equal, \
-    assert_array_less
+from pygrackle.utilities.testing import random_logscale, assert_rel_equal
 
 from testing_common import grackle_data_dir
 

src/python/tests/test_initialisation.py

@@ -9,12 +9,12 @@
 #Standard modules
 import h5py
 import os
+from numpy.testing import assert_allclose
 
 #Chemistry_data struct from grackle
 from pygrackle import chemistry_data
 #Necessary constants from grackle
 from pygrackle.utilities.physical_constants import mass_hydrogen_cgs
-from pygrackle.utilities.testing import assert_allclose
 
 #* Function which returns chemistry_data instance with default initialisation settings.
 def get_defChem():