address comments on PR #2547

Author: fselmo

tests/ens/test_get_text.py

@@ -10,13 +10,6 @@
 )
 from web3 import Web3
 
-SET_TEXT_RESOLVER_NOT_FOUND_CASES = (
-    ("avatar", "tester.jpeg"),
-    ("email", "user@example.com"),
-    ("url", "http://example.com"),
-    ("description", "a test"),
-    ("notice", "this contract is a test contract"),
-)
 GET_TEXT_TEST_CASES = (
     ("avatar", "tester.jpeg"),
     ("email", "user@example.com"),
@@ -26,7 +19,7 @@
 )
 
 
-@pytest.mark.parametrize("key,expected", SET_TEXT_RESOLVER_NOT_FOUND_CASES)
+@pytest.mark.parametrize("key,expected", GET_TEXT_TEST_CASES)
 def test_set_text_resolver_not_found(ens, key, expected):
     with pytest.raises(ResolverNotFound):
         ens.set_text("tld", key, expected)
@@ -92,7 +85,7 @@ def test_get_text_for_resolver_with_unsupported_function(ens):
 
 
 @pytest.mark.asyncio
-@pytest.mark.parametrize("key,expected", SET_TEXT_RESOLVER_NOT_FOUND_CASES)
+@pytest.mark.parametrize("key,expected", GET_TEXT_TEST_CASES)
 async def test_async_set_text_resolver_not_found(async_ens, key, expected):
     with pytest.raises(ResolverNotFound):
         await async_ens.set_text("tld", key, expected)

tests/ens/test_wildcard_resolution.py

@@ -10,7 +10,7 @@ def test_wildcard_resolution_with_extended_resolver_for_subdomains(ens, subdomai
     assert resolved_child_address == "0x000000000000000000000000000000000000dEaD"
 
 
-def test_async_wildcard_resolution_with_extended_resolver_for_parent_ens_domain(ens):
+def test_wildcard_resolution_with_extended_resolver_for_parent_ens_domain(ens):
     # validate `extended-resolver.eth` by asserting it returns the specified hard-coded address from
     # `tests/test_contracts/ExtendedResolver.sol` which requires a specific condition to be
     # met for the parent domain `extended-resolver.eth`
@@ -36,7 +36,7 @@ async def test_async_wildcard_resolution_with_extended_resolver_for_subdomains(
 
 
 @pytest.mark.asyncio
-async def test_wildcard_resolution_with_extended_resolver_for_parent_ens_domain(
+async def test_async_wildcard_resolution_with_extended_resolver_for_parent_ens_domain(
     async_ens,
 ):
     # validate `extended-resolver.eth` by asserting it returns the specified hard-coded address from

web3/main.py

@@ -249,7 +249,7 @@ def __init__(
         external_modules: Optional[
             Dict[str, Union[Type[Module], Sequence[Any]]]
         ] = None,
-        ens: Optional[Union[ENS, AsyncENS]] = None,
+        ens: Union[ENS, AsyncENS] = cast(ENS, empty),
     ) -> None:
         self.manager = self.RequestManager(self, provider, middlewares)
         # this codec gets used in the module initialization,
@@ -352,14 +352,12 @@ def is_encodable(self, _type: TypeStr, value: Any) -> bool:
     @property
     def ens(self) -> Union[ENS, AsyncENS]:
         if self._ens is cast(ENS, empty):
-            return ENS.fromWeb3(self)
-        elif self._ens is cast(AsyncENS, empty) or self.eth.is_async:
-            return AsyncENS.fromWeb3(self)
+            return AsyncENS.fromWeb3(self) if self.eth.is_async else ENS.fromWeb3(self)
 
         return self._ens
 
     @ens.setter
-    def ens(self, new_ens: Optional[Union[ENS, AsyncENS]]) -> None:
+    def ens(self, new_ens: Union[ENS, AsyncENS]) -> None:
         self._ens = new_ens
 
     @property