skip screenshots in rspec

lib/cyperful.rb

@@ -65,5 +65,6 @@ def self.add_step_at_methods(*mods_or_methods)
 require "cyperful/test_parser"
 require "cyperful/ui_server"
 require "cyperful/driver"
+require "cyperful/framework_helper"
 require "cyperful/framework_injections"
 require "cyperful/railtie" if defined?(Rails::Railtie)

lib/cyperful/framework_helper.rb

@@ -0,0 +1,7 @@
+module Cyperful::FrameworkHelper
+  # Disable default screenshot on failure b/c we handle them ourselves.
+  # https://github.com/rails/rails/blob/v7.0.1/actionpack/lib/action_dispatch/system_testing/test_helpers/screenshot_helper.rb#L156
+  def take_failed_screenshot
+    nil
+  end
+end

lib/cyperful/framework_injections.rb

@@ -65,9 +65,6 @@ def initialize(...)
         # hide the "Chrome is being controlled by automated test software" infobar
         driver_opts.args.delete("--enable-automation")
         driver_opts.exclude_switches << "enable-automation"
-
-        pp args: driver_opts.args,
-           exclude_switches: driver_opts.exclude_switches
       end
   end
 end

lib/cyperful/minitest.rb

@@ -3,6 +3,8 @@
 
 module Cyperful::Minitest # rubocop:disable Style/ClassAndModuleChildren
   module SystemTestHelper
+    include Cyperful::FrameworkHelper
+
     def setup
       Cyperful.setup(self.class, self.method_name)
       super
@@ -16,12 +18,6 @@ def teardown
       Cyperful.teardown(error)
       super
     end
-
-    # Disable default screenshot on failure b/c we handle them ourselves.
-    # https://github.com/rails/rails/blob/main/actionpack/lib/action_dispatch/system_testing/test_helpers/screenshot_helper.rb#L156
-    def take_failed_screenshot
-      nil
-    end
   end
 end
 

lib/cyperful/rspec.rb

@@ -2,6 +2,8 @@
 
 module Cyperful::Rspec
   def self.configure(rspec_conf)
+    rspec_conf.include(Cyperful::FrameworkHelper)
+
     rspec_conf.before(:example, type: :system) do
       # e.g. class = `RSpec::ExampleGroups::MyTest`
       # e.g. full_description = "my_test can visit root"
@@ -11,7 +13,7 @@ def self.configure(rspec_conf)
 
     rspec_conf.after(:example, type: :system) do
       example = RSpec.current_example
-      error = example.exception
+      error = example.exception # may be nil
 
       # if error.is_a?(RSpec::Expectations::ExpectationNotMetError)
       #   error = error.error

test/dummy/Gemfile

@@ -44,8 +44,7 @@ group :test do
   gem "capybara"
   gem "selenium-webdriver"
 
-  # manually load test-frameworks since we support multiple
-  gem "rspec-rails"
+  gem "rspec-rails" # should we require=false this?
 
   gem "cyperful", path: "../..", require: false
 end