Move PyEthPMException -> EthPMException

Author: kclowes

ethpm/exceptions.py

@@ -1,12 +1,12 @@
-class PyEthPMException(Exception):
+class EthPMException(Exception):
     """
     Base class for all Py-EthPM errors.
     """
 
     pass
 
 
-class InsufficientAssetsError(PyEthPMException):
+class InsufficientAssetsError(EthPMException):
     """
     Raised when a Manifest or Package does not contain the required
     assets to do something.
@@ -15,47 +15,47 @@ class InsufficientAssetsError(PyEthPMException):
     pass
 
 
-class EthPMValidationError(PyEthPMException):
+class EthPMValidationError(EthPMException):
     """
     Raised when something does not pass a validation check.
     """
 
     pass
 
 
-class CannotHandleURI(PyEthPMException):
+class CannotHandleURI(EthPMException):
     """
     Raised when the given URI cannot be served by any of the available backends.
     """
 
     pass
 
 
-class FailureToFetchIPFSAssetsError(PyEthPMException):
+class FailureToFetchIPFSAssetsError(EthPMException):
     """
     Raised when an attempt to fetch a Package's assets via IPFS failed.
     """
 
     pass
 
 
-class BytecodeLinkingError(PyEthPMException):
+class BytecodeLinkingError(EthPMException):
     """
     Raised when an attempt to link a contract factory's bytecode failed.
     """
 
     pass
 
 
-class ManifestBuildingError(PyEthPMException):
+class ManifestBuildingError(EthPMException):
     """
     Raised when an attempt to build a manifest failed.
     """
 
     pass
 
 
-class ManifestValidationError(PyEthPMException):
+class ManifestValidationError(EthPMException):
     """
     Raised when a provided manifest cannot be published, since it's invalid.
     """

ethpm/package.py

@@ -53,10 +53,10 @@
 )
 from ethpm.exceptions import (
     BytecodeLinkingError,
+    EthPMException,
     EthPMValidationError,
     FailureToFetchIPFSAssetsError,
     InsufficientAssetsError,
-    PyEthPMException,
 )
 from ethpm.uri import (
     resolve_uri_contents,
@@ -338,7 +338,7 @@ def build_dependencies(self) -> "Dependencies":
             try:
                 validate_build_dependency(name, uri)
                 dependency_package = Package.from_uri(uri, self.w3)
-            except PyEthPMException as e:
+            except EthPMException as e:
                 raise FailureToFetchIPFSAssetsError(
                     f"Failed to retrieve build dependency: {name} from URI: {uri}.\n"
                     f"Got error: {e}."

newsfragments/1478.breaking.rst

@@ -1 +1 @@
-All exceptions inherit from a custom class. EthPM exceptions inherit from PyEthPMException, ENS exceptions inherit from ENSException, and all other web3.py exceptions inherit from Web3Exception
+All exceptions inherit from a custom class. EthPM exceptions inherit from EthPMException, ENS exceptions inherit from ENSException, and all other web3.py exceptions inherit from Web3Exception

tests/core/pm-module/test_pm_init.py

@@ -5,8 +5,8 @@
     Package,
 )
 from ethpm.exceptions import (
+    EthPMException,
     InsufficientAssetsError,
-    PyEthPMException,
 )
 from ethpm.tools import (
     get_ethpm_local_manifest,
@@ -121,10 +121,10 @@ def test_get_local_package(w3, tmp_ethpmdir):
 def test_get_local_package_with_invalid_ethpmdir(w3, tmp_path):
     invalid_ethpmdir = tmp_path / "invalid"
     invalid_ethpmdir.mkdir()
-    with pytest.raises(PyEthPMException, match="not a valid ethPM packages directory."):
+    with pytest.raises(EthPMException, match="not a valid ethPM packages directory."):
         w3.pm.get_local_package("owned", invalid_ethpmdir)
 
 
 def test_get_local_package_with_uninstalled_package(w3, tmp_ethpmdir):
-    with pytest.raises(PyEthPMException, match="Package: safe-math not found in "):
+    with pytest.raises(EthPMException, match="Package: safe-math not found in "):
         w3.pm.get_local_package("safe-math", tmp_ethpmdir)

tests/core/pm-module/test_registry_integration.py

@@ -12,7 +12,7 @@
     LinkableContract,
 )
 from ethpm.exceptions import (
-    PyEthPMException,
+    EthPMException,
 )
 from web3 import Web3
 from web3.pm import (
@@ -60,25 +60,25 @@ def test_pm_set_custom_registry(empty_sol_registry, fresh_w3):
 
 @pytest.mark.xfail(reason="Need to properly add authorization as of 8/10/2022")
 def test_pm_must_set_registry_before_all_registry_interaction_functions(fresh_w3):
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.release_package(
             "package", "1.0.0", "ipfs://QmcxvhkJJVpbxEAa6cgW3B6XwPJb79w9GpNUv2P2THUzZR"
         )
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_release_id_data(b"invalid_release_id")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_release_id("package", "1.0.0")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_release_data("package", "1.0.0")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_package("package", "1.0.0")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_all_package_names()
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_all_package_releases("package")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_release_count("package")
-    with pytest.raises(PyEthPMException):
+    with pytest.raises(EthPMException):
         fresh_w3.pm.get_package_count()
 
 

web3/pm.py

@@ -39,8 +39,8 @@
     Package,
 )
 from ethpm.exceptions import (
+    EthPMException,
     ManifestValidationError,
-    PyEthPMException,
 )
 from ethpm.uri import (
     is_supported_content_addressed_uri,
@@ -362,13 +362,13 @@ def get_local_package(self, package_name: str, ethpm_dir: Path = None) -> Packag
             ethpm_dir = Path.cwd() / "_ethpm_packages"
 
         if not ethpm_dir.name == "_ethpm_packages" or not ethpm_dir.is_dir():
-            raise PyEthPMException(
+            raise EthPMException(
                 f"{ethpm_dir} is not a valid ethPM packages directory."
             )
 
         local_packages = [pkg.name for pkg in ethpm_dir.iterdir() if pkg.is_dir()]
         if package_name not in local_packages:
-            raise PyEthPMException(
+            raise EthPMException(
                 f"Package: {package_name} not found in {ethpm_dir}. "
                 f"Available packages include: {local_packages}."
             )
@@ -406,7 +406,7 @@ def set_registry(self, address: Union[Address, ChecksumAddress, ENS]) -> None:
                 )
             self.registry = SimpleRegistry(addr_lookup, self.w3)
         else:
-            raise PyEthPMException(
+            raise EthPMException(
                 "Expected a canonical/checksummed address or ENS name for the address, "
                 f"instead received {type(address)}."
             )
@@ -559,14 +559,14 @@ def _validate_set_registry(self) -> None:
         try:
             self.registry
         except AttributeError:
-            raise PyEthPMException(
+            raise EthPMException(
                 "web3.pm does not have a set registry. "
                 "Please set registry with either: "
                 "web3.pm.set_registry(address) or "
                 "web3.pm.deploy_and_set_registry()"
             )
         if not isinstance(self.registry, ERC1319Registry):
-            raise PyEthPMException(
+            raise EthPMException(
                 "web3.pm requires an instance of a subclass of ERC1319Registry "
                 "to be set as the web3.pm.registry attribute. Instead found: "
                 f"{type(self.registry)}."