feat: include provider name in impersonate account error (#2507)

ApeWorX · Feb 15, 2025 · d927196 · d927196
1 parent f26546f
commit d927196
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/ape/managers/accounts.py b/src/ape/managers/accounts.py
@@ -164,7 +164,9 @@ def impersonate_account(self, address: AddressType) -> ImpersonatedAccount:
         try:
             result = self.provider.unlock_account(address)
         except NotImplementedError as err:
-            raise AccountsError("Your provider does not support impersonating accounts.") from err
+            raise AccountsError(
+                f"Provider '{self.provider.name}' does not support impersonating accounts."
+            ) from err
 
         if result:
             if address in self._impersonated_accounts:

diff --git a/tests/functional/test_accounts.py b/tests/functional/test_accounts.py
@@ -495,7 +495,7 @@ def test_autosign_transactions(runner, keyfile_account, receiver):
 
 def test_impersonate_not_implemented(accounts, address):
     expected_err_msg = (
-        r"Your provider does not support impersonating accounts:\\n"
+        r"Provider 'test' does not support impersonating accounts:\\n"
         rf"No account with address '{address}'\."
     )
     with pytest.raises(KeyError, match=expected_err_msg):
@@ -522,7 +522,7 @@ def test_impersonated_account_ignores_signature_check_on_txn(accounts, address):
 
 def test_contract_as_sender_non_fork_network(contract_instance):
     expected_err_msg = (
-        r"Your provider does not support impersonating accounts:\\n"
+        r"Provider 'test' does not support impersonating accounts:\\n"
         rf"No account with address '{contract_instance}'\."
     )
     with pytest.raises(KeyError, match=expected_err_msg):