Introduce raise_on_unhandled_modal browser configuration

README.md

@@ -61,6 +61,7 @@ end
 `Cuprite`-specific options are:
 
 * options `Hash`
+  * `:raise_on_unhandled_modal` (Boolean) - When set to `false`, output a warning. When set to `true`, raise an exception
   * `:url_blacklist` (Array) - array of regexes to match against requested URLs
   * `:url_whitelist` (Array) - array of regexes to match against requested URLs
 

lib/capybara/cuprite/browser.rb

@@ -14,6 +14,7 @@ class Browser < Ferrum::Browser
       def initialize(options = nil)
         super
 
+        @options.raise_on_unhandled_modal = options&.delete(:raise_on_unhandled_modal)
         @options.url_blacklist = prepare_wildcards(options&.dig(:url_blacklist))
         @options.url_whitelist = prepare_wildcards(options&.dig(:url_whitelist))
 
@@ -49,6 +50,14 @@ def resize(**options)
         super
       end
 
+      def raise_on_unhandled_modal
+        @options.raise_on_unhandled_modal
+      end
+
+      def raise_on_unhandled_modal=(value)
+        @options.raise_on_unhandled_modal = value
+      end
+
       def url_whitelist
         @options.url_whitelist
       end

lib/capybara/cuprite/driver.rb

@@ -137,6 +137,7 @@ def reset!
         @paper_size = nil
         browser.url_blacklist = @options[:url_blacklist]
         browser.url_whitelist = @options[:url_whitelist]
+        browser.raise_on_unhandled_modal = @options.fetch(:raise_on_unhandled_modal, false)
         browser.reset
         @started = false
       end

lib/capybara/cuprite/options.rb

@@ -4,7 +4,7 @@ module Ferrum
   class Browser
     class Options
       attr_writer :window_size
-      attr_accessor :url_blacklist, :url_whitelist
+      attr_accessor :url_blacklist, :url_whitelist, :raise_on_unhandled_modal
 
       def reset_window_size
         @window_size = @options[:window_size]

lib/capybara/cuprite/page.rb

@@ -162,10 +162,14 @@ def prepare_page
             response = @modal_response || params["defaultPrompt"]
           else
             with_text = params["message"] ? "with text `#{params['message']}` " : ""
-            warn "Modal window #{with_text}has been opened, but you didn't wrap " \
+            message =
+                 "Modal window #{with_text}has been opened, but you didn't wrap " \
                  "your code into (`accept_prompt` | `dismiss_prompt` | " \
                  "`accept_confirm` | `dismiss_confirm` | `accept_alert`), " \
                  "accepting by default"
+
+            @options.raise_on_unhandled_modal ? raise(message) : warn(message)
+
             options = { accept: true }
             response = params["defaultPrompt"]
           end

spec/features/session_spec.rb

@@ -1110,6 +1110,18 @@
         expect(@session).to have_xpath("//a[@id='open-match' and @confirmed='true']")
       end
 
+      it "configured to raise warning" do
+        @session.driver.browser.raise_on_unhandled_modal = true
+
+        @session.visit "/cuprite/with_js"
+
+        expect { @session.click_link("Open for match") }.to raise_error(
+          "Modal window with text `{T}ext \\w|th [reg.exp] (chara©+er$)?` has been opened, " \
+          "but you didn't wrap your code into (`accept_prompt` | `dismiss_prompt` | `accept_confirm` " \
+          "| `dismiss_confirm` | `accept_alert`), accepting by default"
+        )
+      end
+
       it "matches on partial strings" do
         @session.visit "/cuprite/with_js"
         expect do