Merge pull request #865 from eth-brownie/fix-from-exception

Improved error message for ownerless contracts
eth-brownie · Nov 24, 2020 · 57cc1ac · 57cc1ac
2 parents e81cb67 + d99eb3e
commit 57cc1ac
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/brownie/network/contract.py b/brownie/network/contract.py
@@ -239,8 +239,8 @@ def __call__(self, *args: Tuple) -> Union["Contract", TransactionReceiptType]:
         args, tx = _get_tx(None, args)
         if not tx["from"]:
             raise AttributeError(
-                "No deployer address given. You must supply a tx dict"
-                " as the last argument with a 'from' field."
+                "Final argument must be a dict of transaction parameters that "
+                "includes a `from` field specifying the address to deploy from"
             )
         return tx["from"].deploy(
             self._parent,
@@ -293,8 +293,8 @@ def estimate_gas(self, *args: Tuple) -> int:
         args, tx = _get_tx(None, args)
         if not tx["from"]:
             raise AttributeError(
-                "Contract has no owner, you must supply a tx dict"
-                " as the last argument with a 'from' field."
+                "Final argument must be a dict of transaction parameters that "
+                "includes a `from` field specifying the sender of the transaction"
             )
 
         return tx["from"].estimate_gas(
@@ -1076,8 +1076,8 @@ def transact(self, *args: Tuple) -> TransactionReceiptType:
         args, tx = _get_tx(self._owner, args)
         if not tx["from"]:
             raise AttributeError(
-                "Contract has no owner, you must supply a tx dict"
-                " as the last argument with a 'from' field."
+                "Final argument must be a dict of transaction parameters that "
+                "includes a `from` field specifying the sender of the transaction"
             )
 
         return tx["from"].transfer(
@@ -1166,8 +1166,8 @@ def estimate_gas(self, *args: Tuple) -> int:
         args, tx = _get_tx(self._owner, args)
         if not tx["from"]:
             raise AttributeError(
-                "Contract has no owner, you must supply a tx dict"
-                " as the last argument with a 'from' field."
+                "Final argument must be a dict of transaction parameters that "
+                "includes a `from` field specifying the sender of the transaction"
             )
 
         return tx["from"].estimate_gas(