Remove URLParsingError

URLParsingError is a specific download error and
is not clear what benefit it provides.
It's used only once in the new code and the
message says everything you need to know about
the exception.

Signed-off-by: Martin Vrachev <[email protected]>

Author: MVrachev

tests/test_fetcher_ng.py

@@ -110,7 +110,7 @@ def test_fetch_in_chunks(self) -> None:
 
     # Incorrect URL parsing
     def test_url_parsing(self) -> None:
-        with self.assertRaises(exceptions.URLParsingError):
+        with self.assertRaises(exceptions.DownloadError):
             self.fetcher.fetch(self.random_string())
 
     # File not found error

tuf/api/exceptions.py

@@ -19,11 +19,6 @@ class LengthOrHashMismatchError(Exception):
     """An error while checking the length and hash values of an object."""
 
 
-class URLParsingError(Exception):
-    """If we are unable to parse a URL -- for example, if a hostname element
-    cannot be isoalted."""
-
-
 #### Repository errors ####
 
 

tuf/ngclient/_internal/requests_fetcher.py

@@ -132,9 +132,7 @@ def _get_session(self, url: str) -> requests.Session:
         parsed_url = parse.urlparse(url)
 
         if not parsed_url.scheme or not parsed_url.hostname:
-            raise exceptions.URLParsingError(
-                "Could not get scheme and hostname from URL: " + url
-            )
+            raise exceptions.DownloadError("Failed to parse URL")
 
         session_index = parsed_url.scheme + "+" + parsed_url.hostname
         session = self._sessions.get(session_index)