Make HfHubHTTPError inherit from OSError (#3387)

Author: Wauplin

src/huggingface_hub/errors.py

@@ -37,7 +37,7 @@ class OfflineModeIsEnabled(ConnectionError):
     """Raised when a request is made but `HF_HUB_OFFLINE=1` is set as environment variable."""
 
 
-class HfHubHTTPError(HTTPError):
+class HfHubHTTPError(HTTPError, OSError):
     """
     HTTPError to inherit from for any custom HTTP Error raised in HF Hub.
 

tests/test_utils_errors.py

@@ -1,4 +1,5 @@
 import unittest
+from unittest.mock import Mock
 
 import pytest
 from httpx import Request, Response
@@ -308,3 +309,9 @@ def test_repo_api_regex(url: str, should_match: bool) -> None:
         assert REPO_API_REGEX.match(url)
     else:
         assert REPO_API_REGEX.match(url) is None
+
+
+def test_hf_hub_http_error_inherits_from_os_error() -> None:
+    """Test HfHubHTTPError inherits from OSError."""
+    with pytest.raises(OSError):
+        raise HfHubHTTPError("this is a message", response=Mock())