tests for that

Author: valeriupredoi

tests/test_real_https.py

@@ -59,6 +59,34 @@ def test_https_implicit_storage():
     assert result == np.array([0.6909787], dtype="float32")
 
 
+def test_https_implicit_storage_file_not_found():
+    """
+    Run a true test with a https FILE that is not found.
+    Code raises a very descriptive exception via fsspec.
+    Keep test to capture any changes in behaviour.
+    """
+    test_file_uri = "https://esgf.ceda.ac.uk/thredds/fileServer/esg_cmip6/CMIP6/AerChemMIP/MOHC/UKESM1-0-LL/ssp370SST-lowNTCF/r1i1p1f2/Amon/cl/gn/latest/cl_Amon_UKESM1-0-LL_ssp370SST-lowNTCF_r1i1p1f2_gn_205001-209912.ncx"
+
+    with pytest.raises(FileNotFoundError):
+        active = Active(test_file_uri, "cl")
+        active._version = 1
+        active._method = "min"
+        result = active[0:3, 4:6, 7:9]
+
+
+def test_https_implicit_storage_wrong_url():
+    """
+    Run a true test with a bogus URL.
+    """
+    test_file_uri = "https://esgf.cedacow.ac.uk/thredds/fileServer/esg_cmip6/CMIP6/AerChemMIP/MOHC/UKESM1-0-LL/ssp370SST-lowNTCF/r1i1p1f2/Amon/cl/gn/latest/cl_Amon_UKESM1-0-LL_ssp370SST-lowNTCF_r1i1p1f2_gn_205001-209912.nc"
+
+    with pytest.raises(ValueError):
+        active = Active(test_file_uri, "cl")
+        active._version = 1
+        active._method = "min"
+        result = active[0:3, 4:6, 7:9]
+
+
 @pytest.mark.skip(reason="save time: test_https_dataset_implicit_storage is more general.")
 def test_https_dataset():
     """Run a true test with a https DATASET."""