Add better error-handling for Recorder Mode

Author: mdmintz

seleniumbase/core/browser_launcher.py

@@ -399,10 +399,11 @@ def _set_chrome_options(
             )
         chrome_options.add_experimental_option("useAutomationExtension", False)
     if (settings.DISABLE_CSP_ON_CHROME or disable_csp) and not headless:
-        # Headless Chrome doesn't support extensions, which are required
-        # for disabling the Content Security Policy on Chrome
+        # Headless Chrome does not support extensions, which are required
+        # for disabling the Content Security Policy on Chrome.
         chrome_options = _add_chrome_disable_csp_extension(chrome_options)
     if ad_block_on and not headless:
+        # Headless Chrome does not support extensions.
         chrome_options = _add_chrome_ad_block_extension(chrome_options)
     if recorder_ext and not headless:
         chrome_options = _add_chrome_recorder_extension(chrome_options)

seleniumbase/fixtures/base_case.py

@@ -9964,10 +9964,14 @@ def setUp(self, masterqa_mode=False):
             if self._reuse_session:
                 sb_config.shared_driver = self.driver
 
-        if self.browser in ["firefox", "ie", "safari"]:
-            # Only Chromium-based browsers have the mobile emulator.
+        if self.browser in ["firefox", "ie", "safari", "opera"]:
+            # Only Chrome and Edge browsers have the mobile emulator.
             # Some actions such as hover-clicking are different on mobile.
             self.mobile_emulator = False
+            # The Recorder Mode browser extension is only for Chrome/Edge.
+            if self.recorder_mode:
+                print('\n* The Recorder extension is for Chrome & Edge only!')
+                self.recorder_mode = False
 
         # Configure the test time limit (if used).
         self.set_time_limit(self.time_limit)

seleniumbase/plugins/pytest_plugin.py

@@ -899,6 +899,19 @@ def pytest_addoption(parser):
                 '\n  (DO NOT combine "--recorder" with "-n NUM_PROCESSES"!)\n'
             )
 
+    # Recorder Mode does not support headless browser runs.
+    # (Chromium does not allow extensions in Headless Mode)
+    if (
+        "--recorder" in sys_argv
+        or "--record" in sys_argv
+        or "--rec" in sys_argv
+    ):
+        if ("--headless" in sys_argv):
+            raise Exception(
+                "\n\n  Recorder Mode does NOT support Headless Mode!"
+                '\n  (DO NOT combine "--recorder" with "--headless"!)\n'
+            )
+
     # As a shortcut, you can use "--edge" instead of "--browser=edge", etc,
     # but you can only specify one default browser for tests. (Default: chrome)
     browser_changes = 0