Mark tests which depend on ECS as expected fail; separate integration and unit tests

.github/workflows/integration_test.yml

@@ -39,9 +39,7 @@ jobs:
           EARTHDATA_PASSWORD: "${{ secrets.EARTHDATA_PASSWORD }}"
           NSIDC_LOGIN: "${{ secrets.EARTHDATA_PASSWORD }}"
         run: |
-          pytest icepyx/ --verbose --cov app \
-            icepyx/tests/test_behind_NSIDC_API_login.py \
-            icepyx/tests/test_auth.py
+          pytest icepyx/tests/integration --verbose --cov app
 
       - name: "Upload coverage report"
         uses: "codecov/[email protected]"

.github/workflows/unit_test.yml

@@ -31,8 +31,7 @@ jobs:
       - name: "Run tests"
         run: |
           pytest icepyx/ --verbose --cov app \
-            --ignore=icepyx/tests/test_behind_NSIDC_API_login.py \
-            --ignore=icepyx/tests/test_auth.py
+            --ignore=icepyx/tests/integration
 
       - name: "Upload coverage report"
         uses: "codecov/[email protected]"

icepyx/tests/test_behind_NSIDC_API_login.py → icepyx/tests/integration/test_behind_NSIDC_API_login.py

@@ -11,6 +11,15 @@
 import icepyx as ipx
 import icepyx.core.is2ref as is2ref
 
+# Skip the whole module. See:
+# https://docs.pytest.org/en/stable/reference/reference.html#globalvar-pytestmark
+pytestmark = pytest.mark.xfail(
+    reason=(
+        "The back-end API on which these tests depend, ECS/EGI/ESI, is scheduled for"
+        " shutdown in late 2024. At that point, these tests will begin failing."
+    )
+)
+
 # Misc notes and needed tests
 # test avail data and subsetting success for each input type
 # (kml, shp, list of coords, bbox)
@@ -40,7 +49,7 @@ def session(reg):
 
 def test_get_custom_options_output(session):
     obs = is2ref._get_custom_options(session, "ATL06", "006")
-    with open("./icepyx/tests/ATL06v06_options.json") as exp_json:
+    with open("./icepyx/tests/integration/ATL06v06_options.json") as exp_json:
         exp = json.load(exp_json)
         assert all(keys in obs for keys in exp)
         assert all(obs[key] == exp[key] for key in exp)

icepyx/tests/test_read.py → icepyx/tests/unit/test_read.py

@@ -29,12 +29,12 @@ def test_parse_source_no_files():
         (  # check list input
             [
                 "./icepyx/core/is2ref.py",
-                "./icepyx/tests/test_is2class_query.py",
+                "./icepyx/tests/unit/test_is2class_query.py",
             ],
             sorted(
                 [
                     "./icepyx/core/is2ref.py",
-                    "./icepyx/tests/test_is2class_query.py",
+                    "./icepyx/tests/unit/test_is2class_query.py",
                 ]
             ),
         ),
@@ -49,8 +49,8 @@ def test_parse_source_no_files():
             sorted(
                 [
                     "./icepyx/core/is2ref.py",
-                    "./icepyx/tests/test_is2class_query.py",
-                    "./icepyx/tests/test_is2ref.py",
+                    "./icepyx/tests/unit/test_is2class_query.py",
+                    "./icepyx/tests/unit/test_is2ref.py",
                 ]
             ),
         ),

icepyx/tests/test_spatial.py → icepyx/tests/unit/test_spatial.py

@@ -378,7 +378,7 @@ def test_bad_poly_inputfile_name_throws_error():
 
 def test_bad_poly_inputfile_type_throws_error():
     with pytest.raises(TypeError):
-        spat.Spatial(str(Path("./icepyx/tests/test_read.py").resolve()))
+        spat.Spatial(str(Path("./icepyx/tests/unit/test_read.py").resolve()))
 
 
 ########## geodataframe ##########