changes

Author: jauy123

package/MDAnalysis/fetch/pdb.py

@@ -206,7 +206,7 @@ def from_DOI(doi, file_name, remove_prefix=True, cache_path=None):
         Directory where downloaded file(s) will be cached.
         The default ``None`` argument uses the :mod:`pooch` default cache with
         project name :data:`DEFAULT_CACHE_NAME_DOWNLOADER`.
- 
+
 
     Returns
     -------
@@ -222,7 +222,7 @@ def from_DOI(doi, file_name, remove_prefix=True, cache_path=None):
 
     :class:`requests.exceptions.HTTPError`
         If an invalid file_name is specified.
-    
+
     Notes
     -----
     The DOI link, as specified by `doi`, should be in the format of
@@ -232,7 +232,7 @@ def from_DOI(doi, file_name, remove_prefix=True, cache_path=None):
 
     The current backend for downloading files is :class:`pooch.DOIDownloader`
     which only handles downloading from Zenodo and Figshare repositories.
-    
+
 
     Examples
     --------
@@ -261,7 +261,7 @@ def from_DOI(doi, file_name, remove_prefix=True, cache_path=None):
     """
 
     # Supress warnings from pooch about not specifying
-    # which version of the repostory to download. 
+    # which version of the repostory to download.
     warnings.filterwarnings("ignore", category=UserWarning)
 
     if remove_prefix:

testsuite/MDAnalysisTests/fetch/test_from_DOI.py

@@ -108,20 +108,24 @@ def test_different_sources(tmp_path, doi, file_name, expected_md5):
     assert isinstance(p1, Path)
     assert hashlib.md5(p1.read_bytes()).hexdigest() == expected_md5
 
+
 @pytest.mark.skipif(not HAS_POOCH, reason="Pooch is not installed.")
 def test_invalid_doi_link(tmp_path):
-
     with pytest.raises(
-        HTTPError,
+        ValueError,
+        match=re.escape(
+            "DOI link invalid is invalid. Please check the DOI link."
+        ),
     ):
+
         from_DOI("invalid", file_name="missing.dat", cache_path=tmp_path)
 
- 
+
+@pytest.mark.skipif(not HAS_POOCH, reason="Pooch is not installed.")
 def test_invalid_file_name(tmp_path):
     with pytest.raises(
-            ValueError,
-            match=re.escape(
-                "DOI link doi:invalid is invalid. Please check the DOI link."
-            ),
-        ):
-            from_DOI("doi.org/invalid", file_name="missing.dat", cache_path=tmp_path)
+        HTTPError,
+    ):
+        from_DOI(
+            "doi.org/invalid", file_name="missing.dat", cache_path=tmp_path
+        )