fixup! Issue #460/#259/#453/#458 test requirement pyproj>=3.2.0

Open-EO · Aug 25, 2023 · 0f739f4 · 0f739f4
1 parent 492c637
commit 0f739f4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/rest/datacube/test_datacube100.py b/tests/rest/datacube/test_datacube100.py
@@ -192,8 +192,8 @@ def _get_normalizable_crs_inputs():
     yield "EPSG:32631"
     yield 32631
     if pyproj.__version__ >= ComparableVersion("3.3.1"):
+        # TODO drop this skip once support for python 3.7 is dropped (pyproj 3.3.0 requires at least python 3.8)
         # pyproj below 3.3.1 does not support int-like strings
-        # TODO #460 this skip is only necessary for python 3.6 and lower
         yield "32631"
     yield "+proj=utm +zone=31 +datum=WGS84 +units=m +no_defs"  # is also EPSG:32631, in proj format
     if pyproj.__version__ >= ComparableVersion("3.1.0"):

diff --git a/tests/test_util.py b/tests/test_util.py
@@ -898,7 +898,7 @@ class TestNormalizeCrs:
     def test_normalize_crs_succeeds_with_correct_crses(self, epsg_input, expected):
         """Happy path, values that are allowed"""
         if isinstance(epsg_input, str) and epsg_input.isnumeric() and pyproj.__version__ < ComparableVersion("3.3.1"):
-            # TODO #460 this skip is only necessary for python 3.6 and lower
+            # TODO drop this skip once support for python 3.7 is dropped (pyproj 3.3.0 requires at least python 3.8)
             pytest.skip("pyproj below 3.3.1 does not support int-like strings")
 
         assert normalize_crs(epsg_input) == expected
@@ -1017,7 +1017,7 @@ def test_normalize_crs_without_pyproj_succeeds_with_wkt2_input(self):
     }
 
     @pytest.mark.skipif(
-        # TODO #460 this skip is only necessary for python 3.6 and lower
+        # TODO drop this skip once support for python 3.7 is dropped (pyproj 3.3.0 requires at least python 3.8)
         pyproj.__version__ < ComparableVersion("3.3.0"),
         reason="PROJJSON format support requires pyproj 3.3.0 or higher",
     )