Exception in question

mypy.ini

@@ -1,2 +1,13 @@
 [mypy]
+show_error_codes = True
+exclude = (?x)(
+          setup\.py
+          | tests/
+          | docs/
+          )
+
+[mypy-screenpy.*]
 disallow_untyped_defs = True
+
+[mypy-tests.*]
+ignore_missing_imports = True

screenpy_selenium/actions/chain.py

@@ -40,7 +40,7 @@ def perform_as(self, the_actor: Actor) -> None:
         the_chain = ActionChains(browser)
 
         for action in self.actions:
-            if "add_to_chain" not in dir(action):
+            if not isinstance(action, Chainable):
                 raise UnableToAct(
                     f"The {action.__class__.__name__} Action cannot be chained."
                 )

screenpy_selenium/actions/save_console_log.py

@@ -44,6 +44,10 @@ class SaveConsoleLog:
 
     attach_kwargs: Optional[dict]
 
+    def describe(self) -> str:
+        """Describe the Action in present tense."""
+        return f"Save browser console log as {self.filename}"
+
     @staticmethod
     def as_(path: str) -> "SaveConsoleLog":
         """Supply the name and/or filepath for the saved text file.

screenpy_selenium/actions/save_screenshot.py

@@ -40,6 +40,10 @@ class SaveScreenshot:
 
     attach_kwargs: Optional[dict]
 
+    def describe(self) -> str:
+        """Describe the Action in present tense."""
+        return f"Save screenshot as {self.filename}"
+
     @staticmethod
     def as_(path: str) -> "SaveScreenshot":
         """Supply the name and/or filepath for the screenshot.

screenpy_selenium/protocols.py

@@ -4,9 +4,10 @@
 
 from screenpy import Actor
 from selenium.webdriver.common.action_chains import ActionChains
-from typing_extensions import Protocol
+from typing_extensions import Protocol, runtime_checkable
 
 
+@runtime_checkable
 class Chainable(Protocol):
     """Actions that can be added to a chain are Chainable."""
 

screenpy_selenium/questions/element.py

@@ -23,6 +23,8 @@ class Element:
         the_actor.should(See.the(Element(WELCOME_BANNER), IsVisible()))
     """
 
+    caught_exception: Optional[TargetingError]
+
     def describe(self) -> str:
         """Describe the Question."""
         return f"The {self.target}."
@@ -32,8 +34,10 @@ def answered_by(self, the_actor: Actor) -> Optional[WebElement]:
         """Direct the Actor to find the element."""
         try:
             return self.target.found_by(the_actor)
-        except TargetingError:
+        except TargetingError as exc:
+            self.caught_exception = exc
             return None
 
     def __init__(self, target: Target) -> None:
         self.target = target
+        self.caught_exception = None

setup.py

@@ -3,7 +3,7 @@
 from setuptools import find_packages, setup
 
 requires = [
-    "screenpy>=4.0.0",
+    "screenpy>=4.0.2",
     "screenpy_pyotp>=4.0.0,<4.1",
     "typing-extensions>=4.1.1,<4.2",
     "selenium>=4.1.0,<4.2",

tests/conftest.py

@@ -5,15 +5,16 @@
 from screenpy_pyotp.abilities import AuthenticateWith2FA
 
 from screenpy_selenium.abilities import BrowseTheWeb
+from selenium.webdriver.remote.webdriver import WebDriver
 
 
 @pytest.fixture(scope="function")
 def Tester() -> AnActor:
     """Provide an Actor with mocked web browsing abilities."""
-    AuthenticateWith2FA_Mocked = mock.Mock(spec=AuthenticateWith2FA)
+    AuthenticateWith2FA_Mocked = mock.create_autospec(AuthenticateWith2FA, instance=True)
     AuthenticateWith2FA_Mocked.otp = mock.Mock()
-    BrowseTheWeb_Mocked = mock.Mock(spec=BrowseTheWeb)
-    BrowseTheWeb_Mocked.browser = mock.Mock()
+    BrowseTheWeb_Mocked = mock.create_autospec(BrowseTheWeb, instance=True)
+    BrowseTheWeb_Mocked.browser = mock.create_autospec(WebDriver, instance=True)
 
     return AnActor.named("Tester").who_can(
         AuthenticateWith2FA_Mocked, BrowseTheWeb_Mocked

tests/test_abilities.py

@@ -2,6 +2,7 @@
 from unittest import mock
 
 import pytest
+from screenpy.protocols import Forgettable
 
 from screenpy_selenium.abilities import BrowseTheWeb
 from screenpy_selenium.exceptions import BrowsingError
@@ -13,26 +14,30 @@ def test_can_be_instantiated(self):
 
         assert isinstance(b, BrowseTheWeb)
 
-    @mock.patch("screenpy_selenium.abilities.browse_the_web.Firefox")
+    def test_implements_protocol(self):
+        b = BrowseTheWeb(None)
+        assert isinstance(b, Forgettable)
+
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Firefox", autospec=True)
     def test_using_firefox(self, mocked_firefox):
         BrowseTheWeb.using_firefox()
 
         mocked_firefox.assert_called_once()
 
-    @mock.patch("screenpy_selenium.abilities.browse_the_web.Chrome")
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Chrome", autospec=True)
     def test_using_chrome(self, mocked_chrome):
         BrowseTheWeb.using_chrome()
 
         mocked_chrome.assert_called_once()
 
-    @mock.patch("screenpy_selenium.abilities.browse_the_web.Safari")
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Safari", autospec=True)
     def test_using_safari(self, mocked_safari):
         BrowseTheWeb.using_safari()
 
         mocked_safari.assert_called_once()
 
     @mock.patch.dict(os.environ, {"IOS_DEVICE_VERSION": "1"})
-    @mock.patch("screenpy_selenium.abilities.browse_the_web.Remote")
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Remote", autospec=True)
     def test_using_ios(self, mocked_remote):
         BrowseTheWeb.using_ios()
 
@@ -43,7 +48,7 @@ def test_using_ios_without_env_var(self):
             BrowseTheWeb.using_ios()
 
     @mock.patch.dict(os.environ, {"ANDROID_DEVICE_VERSION": "1"})
-    @mock.patch("screenpy_selenium.abilities.browse_the_web.Remote")
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Remote", autospec=True)
     def test_using_android(self, mocked_android):
         BrowseTheWeb.using_android()
 
@@ -52,3 +57,12 @@ def test_using_android(self, mocked_android):
     def test_using_android_without_env_var(self):
         with pytest.raises(BrowsingError):
             BrowseTheWeb.using_android()
+
+    @mock.patch("screenpy_selenium.abilities.browse_the_web.Chrome", autospec=True)
+    def test_forget_calls_quit(self, mocked_chrome):
+        b = BrowseTheWeb(mocked_chrome)
+        b.forget()
+        mocked_chrome.quit.assert_called_once()
+
+    def test_repr(self):
+        assert repr(BrowseTheWeb(None)) == "Browse the Web"