capybara-screenshot 1.0.20 → 1.0.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8b4528fe18542217f5e612e18079cf7064306e55
-  data.tar.gz: bb49b402ac3866551d08ff84ae679a5d013bad99
+SHA256:
+  metadata.gz: 8539dcf08bbfb122dfb190cd76f82361373b4016888d02e65daedf42998e7cd2
+  data.tar.gz: bb64e6eb25278f8564a694784b21f9f38d3c550cf6cd2949fe5b5dd2adf5b916
 SHA512:
-  metadata.gz: 82f8f3aa1001acf169c44f38920299737d88880eec9cf18406af1ae66d3c47fdbf22ac161f14ab77b3e2853bc0658888ea10e26d092ef22d097185af735a6b31
-  data.tar.gz: ed3f9b017ea19ecd45ea0ab168a68f05d709f636e9b11b4cc6272c895b28165d30044a122945665a5ddcd7671bff28a01c1acde706fdb291df10620be1aee3ba
+  metadata.gz: c1b62365f1e18a17bc4e0d0443bce7194dd7fe9fdbe846df3ea68b45583d452e6fe0047611f8639eb43bc015307de9f993a1485164ddf502879801fd213f9031
+  data.tar.gz: 5a599e20a7684bf4045c2cb318099af8a9385705e83afe342fe756bd592dfc762816dcd54a3946a370b84805dc3d65c8ce2e5fb927c3673256e4c362131611aa

data/.travis.yml

@@ -4,9 +4,10 @@ rvm:
   - 2.0.0
   - 2.1.10
   - 2.2.10
-  - 2.3.7
-  - 2.4.4
-  - 2.5.1
+  - 2.3.8
+  - 2.4.9
+  - 2.5.7
+  - 2.6.5
 gemfile:
   - gemfiles/cucumber.1.3.gemfile
   - gemfiles/cucumber.2.4.gemfile

data/CHANGELOG.md

@@ -1,3 +1,28 @@
+4 Oct 2020 - 1.0.24 -> 1.0.25
+
+* [Fix Cucumber 5 deprecation warnings](https://github.com/mattheworiordan/capybara-screenshot/pull/274)
+* [Add support for Cuprite driver](https://github.com/mattheworiordan/capybara-screenshot/pull/270)
+* [Rails system spec compatibility](https://github.com/mattheworiordan/capybara-screenshot/pull/264)
+
+24 Nov 2019 - 1.0.23 -> 1.0.24
+
+* [Support Apparition driver](https://github.com/mattheworiordan/capybara-screenshot/commit/273317b241d1d1d8600b32fad998742894424f3b)
+
+10 Jun 2019 - 1.0.22 -> 1.0.23
+
+* [Support for Capybara sessions implicitly](https://github.com/mattheworiordan/capybara-screenshot/pull/255). Thanks https://github.com/TylerRick
+* [Support for S3 bucket host configuration](https://github.com/mattheworiordan/capybara-screenshot/pull/252). Thanks https://github.com/machty
+
+21 Oct 2018 - 1.0.21 -> 1.0.22
+
+* [Support for S3 URL in HTML screenshots](https://github.com/mattheworiordan/capybara-screenshot/pull/239)
+* [Fix for partial Rails environments](https://github.com/mattheworiordan/capybara-screenshot/pull/238)
+* [Capybara 3 support in CI](https://github.com/mattheworiordan/capybara-screenshot/pull/236)
+
+03 May 2018 - 1.0.20 -> 1.0.21
+
+* [Bug fix: Fix Ruby version compares in Gemspec](https://github.com/mattheworiordan/capybara-screenshot/pull/231)
+
 03 May 2018 - 1.0.19 -> 1.0.20
 
 * [Bug fix: Only take screenshots for feature tests](https://github.com/mattheworiordan/capybara-screenshot/pull/223)

data/Gemfile

@@ -4,6 +4,6 @@ source "http://rubygems.org"
 gemspec
 
 gem 'rack', '~> 1.0'
-gem 'rake', '~> 10.0'
+gem 'rake', '~> 13.0'
 gem 'appraisal', '~> 2.0'
 gem 'aruba', '~> 0.14.0'

data/README.md

@@ -5,26 +5,27 @@ capybara-screenshot gem
 [![Code Climate](https://d3s6mut3hikguw.cloudfront.net/github/mattheworiordan/capybara-screenshot.svg)](https://codeclimate.com/github/mattheworiordan/capybara-screenshot)
 [![Gem Version](https://badge.fury.io/rb/capybara-screenshot.svg)](http://badge.fury.io/rb/capybara-screenshot)
 
-#### Capture a screen shot for every test failure automatically!
+#### Capture a screenshot for every test failure automatically!
 
-`capybara-screenshot` used with [Capybara](https://github.com/jnicklas/capybara) and [Cucumber](http://cukes.info/), [Rspec](https://www.relishapp.com/rspec) or [Minitest](https://github.com/seattlerb/minitest), will capture a screen shot for each failure in your test suite.  The HTML for the failed page, and a screenshot image (when using [capybara-webkit](https://github.com/thoughtbot/capybara-webkit), [Selenium](http://seleniumhq.org/) or [poltergeist](https://github.com/jonleighton/poltergeist)) is saved into `$APPLICATION_ROOT/tmp/capybara`.
+`capybara-screenshot` used with [Capybara](https://github.com/jnicklas/capybara) alongside [Cucumber](http://cukes.info/), [Rspec](https://www.relishapp.com/rspec) or [Minitest](https://github.com/seattlerb/minitest), will capture a screenshot for each failure in your test suite. Associated screenshot and HTML file
+of the failed page (when using [capybara-webkit](https://github.com/thoughtbot/capybara-webkit), [Selenium](http://seleniumhq.org/), [poltergeist](https://github.com/jonleighton/poltergeist) or [cuprite](https://github.com/machinio/cuprite)) is saved into `$APPLICATION_ROOT/tmp/capybara`.
 
-Having screenshots readily available for each test failure is incredibly helpful when trying to quickly diagnose a problem in your failing steps.  You can view the source code, and have a screen shot of the page (when applicable), at the time of each failure.
+Available screenshots for each test failure is incredibly helpful for diagnosing problems quickly in your failing steps. You have the ability to view screenshots (when applicable) and source code at the time of each failure.
 
-_Please note that Ruby 1.9+ is required to use this Gem.  For Ruby 1.8 support, please see the [capybara-screenshot Ruby 1.8 branch](https://github.com/mattheworiordan/capybara-screenshot/tree/ruby-1.8-support)_
+_Please note that Ruby 1.9+ is required to use this Gem. For Ruby 1.8 support, please see the [capybara-screenshot Ruby 1.8 branch](https://github.com/mattheworiordan/capybara-screenshot/tree/ruby-1.8-support)_
 
 Installation
 -----
 
 ### Step 1: install the gem
 
-Using Bundler, add the following to your Gemfile
+Using Bundler, add the following to your Gemfile:
 
 ```ruby
 gem 'capybara-screenshot', :group => :test
 ```
 
-or install manually using Ruby Gems:
+Alternatively, manually install using Ruby Gems:
 
 ```
 gem install capybara-screenshot
@@ -76,7 +77,7 @@ Typically in 'test/test_helper.rb', please add:
 require 'capybara-screenshot/testunit'
 ```
 
-By default, screenshots will be captured for `Test::Unit` tests in the path 'test/integration'.  You can add additional paths as:
+By default, screenshots will be captured for `Test::Unit` tests in the path 'test/integration'.  You can add additional paths:
 
 ```ruby
 Capybara::Screenshot.testunit_paths << 'test/feature'
@@ -86,27 +87,28 @@ Capybara::Screenshot.testunit_paths << 'test/feature'
 Manual screenshots
 ----
 
-If you require more control, you can generate the screenshot on demand rather than on failure. This is useful
-if the failure occurs at a point where the screen shot is not as useful for debugging a rendering problem. This
-can be more useful if you disable the auto-generate on failure feature with the following config
+If you require more control, screenshots can be generated on demand rather than on failure. This is useful
+when screenshots produced at the time of failure are not as useful for debugging a rendering problem.
+Differentiating between manual and failure screenshots can be improved by disabling the auto-generate on
+failure feature:
 
 ```ruby
 Capybara::Screenshot.autosave_on_failure = false
 ```
 
-Anywhere the Capybara DSL methods (visit, click etc.) are available so too are the screenshot methods.
+Anywhere the Capybara DSL methods (visit, click etc.) are available so too are the screenshot methods:
 
 ```ruby
 screenshot_and_save_page
 ```
 
-Or for screenshot only, which will automatically open the image.
+Or for screenshot only, which will automatically open the image:
 
 ```ruby
 screenshot_and_open_image
 ```
 
-These are just calls on the main library methods.
+These are just calls on the main library methods:
 
 ```ruby
 Capybara::Screenshot.screenshot_and_save_page
@@ -116,7 +118,7 @@ Capybara::Screenshot.screenshot_and_open_image
 Better looking HTML screenshots
 -------------------------------
 
-By the default, HTML screenshots will not look very good when opened in a browser.  This happens because the browser can't correctly resolve relative paths like `<link href="/assets/...." />`, which stops CSS, images, etc... from beind loaded.  To get a nicer looking page, configure Capybara with:
+By the default, HTML screenshots will not look very good when opened in a browser.  This happens because the browser can't correctly resolve relative paths like `<link href="/assets/...." />`, which stops CSS, images, etc... from being loaded.  To get a nicer looking page, configure Capybara with:
 
 ```ruby
 Capybara.asset_host = 'http://localhost:3000'
@@ -128,7 +130,7 @@ This will cause Capybara to add `<base>http://localhost:3000</base>` to the HTML
 rails s -p 3000
 ```
 
-Now when you open the page, you should have something that looks much better.  You can leave this setup in place and use the default HTML pages when you don't care about the presentation, or start the rails server when you need something better looking.  
+Now when you open the page, you should have something that looks much better.  You can leave this setup in place and use the default HTML pages when you don't care about the presentation, or start the rails server when you need something better looking.
 
 Driver configuration
 --------------------
@@ -165,7 +167,7 @@ Custom screenshot filename
 --------------------------
 
 If you want to control the screenshot filename for a specific test library, to inject the test name into it for example,
-you can override how the basename is generated for the file like so
+you can override how the basename is generated for the file:
 
 ```ruby
 Capybara::Screenshot.register_filename_prefix_formatter(:rspec) do |example|
@@ -173,7 +175,7 @@ Capybara::Screenshot.register_filename_prefix_formatter(:rspec) do |example|
 end
 ```
 
-By default capybara-screenshot will append a timestamp to the basename. If you want to disable this behavior set the following option:
+By default capybara-screenshot will append a timestamp to the basename. If you want to disable this behavior, set the following option:
 
 ```ruby
 Capybara::Screenshot.append_timestamp = false
@@ -183,36 +185,25 @@ Capybara::Screenshot.append_timestamp = false
 Custom screenshot directory
 --------------------------
 By default, when running under Rails, Sinatra, and Padrino, screenshots are saved into `$APPLICATION_ROOT/tmp/capybara`. Otherwise, they're saved under `Dir.pwd`.
-If you want to customize the location, override the file path as:
+If you want to customize the location, override the file path:
 
 ```ruby
 Capybara.save_path = "/file/path"
 ```
 
-Usage with multiple Capybara sessions
--------------------------------------
-
-To make screenshots work with multiple Capybara sessions, replace `Capybara.using_session` with `Capybara.using_session_with_screenshot`:
-
-```ruby
-Cabybara.using_session_with_screenshot('User 1') do
-  # screenshots will work and use the correct session
-end
-```
-
 
 Uploading screenshots to S3
 --------------------------
 You can configure capybara-screenshot to automatically save your screenshots to an AWS S3 bucket.
 
-First, install the `aws-sdk-s3` gem or add it to your Gemfile
+First, install the `aws-sdk-s3` gem or add it to your Gemfile:
 
 ```ruby
 gem 'aws-sdk-s3', group: :test
 gem 'capybara-screenshot', group: :test
 ```
 
-Next, configure capybara-screenshot with your S3 credentials, the bucket to save to, and an optional region (default: `us-east-1`).
+Next, configure capybara-screenshot with your S3 credentials, the bucket to save to, and an optional region (default: `us-east-1`):
 
 ```ruby
 Capybara::Screenshot.s3_configuration = {
@@ -221,10 +212,16 @@ Capybara::Screenshot.s3_configuration = {
     secret_access_key: "my_secret_access_key",
     region: "eu-central-1"
   },
-  bucket_name: "my_screenshots"
+  bucket_name: "my_screenshots",
+  # Optionally: Specify the host used to access the uploaded screenshots
+  bucket_host: "my_screenshots.s3-eu-central-1.amazonaws.com",
 }
 ```
 
+The access key used for S3 uploads need to have at least the `s3:PutObject` permission.
+
+**Note**: If you do not provide the `bucket_host` configuration option, additionally the `s3:GetBucketLocation` permission is required on the bucket for uploads to succeed.
+
 It is also possible to specify the object parameters such as acl.
 Configure the capybara-screenshot with these options in this way:
 
@@ -245,7 +242,7 @@ Capybara::Screenshot.s3_configuration = {
 
 Pruning old screenshots automatically
 --------------------------
-By default screenshots are saved indefinitely, if you want them to be automatically pruned on a new failure, then you can specify one of the following prune strategies as follows:
+By default, screenshots are saved indefinitely. If you want them to be automatically pruned on a new failure, then you can specify one of the following prune strategies as follows:
 
 ```ruby
 # Keep only the screenshots generated from the last failing test suite
@@ -258,7 +255,7 @@ Capybara::Screenshot.prune_strategy = { keep: 20 }
 Callbacks
 ---------
 
-You can hook your own logic into callbacks after the html/screenshot has been saved.
+You can hook your own logic into callbacks after the html/screenshot has been saved:
 
 ```ruby
 # after Saver#save_html
@@ -322,7 +319,7 @@ Please raise an issue at [https://github.com/mattheworiordan/capybara-screenshot
 
 #### Contributions
 
-Contributions are welcome.  Please fork this gem, and submit a pull request.  New features must include test coverage and must pass on all versions of the testing frameworks supported.  Run `appraisal` to set up the your Gems. then `appraisal "rake travis:ci"` locally to test your changes against all versions of testing framework gems supported.
+Contributions are welcome.  Please fork this gem and then submit a pull request.  New features must include test coverage and must pass on all versions of the testing frameworks supported.  Run `appraisal` to set up the your Gems. then `appraisal "rake travis:ci"` locally to test your changes against all versions of testing framework gems supported.
 
 #### Rubygems
 
@@ -333,11 +330,12 @@ About
 
 This gem was written by **Matthew O'Riordan**, with contributions from [many kind people](https://github.com/mattheworiordan/capybara-screenshot/network/members).
 
- - [http://mattheworiordan.com](http://mattheworiordan.com)
+ - [http://mattheworiordan.com](http://mattheworiordan.com), [Technical co-founder & CEO of Ably Realtime](https://ably.com/)
  - [@mattheworiordan](http://twitter.com/#!/mattheworiordan)
- - [Linked In](http://www.linkedin.com/in/lemon)
+ - [Linked In](http://www.linkedin.com/in/mattoriordan)
+
 
 License
 -------
 
-Copyright © 2016 Matthew O'Riordan, inc. It is free software, and may be redistributed under the terms specified in the LICENSE file.
+Copyright © 2020 Matthew O'Riordan, inc. It is free software, and may be redistributed under the terms specified in the LICENSE file.

data/capybara-screenshot.gemspec

@@ -12,14 +12,21 @@ Gem::Specification.new do |s|
   s.summary     = %q{Automatically create snapshots when Cucumber steps fail with Capybara and Rails}
   s.description = %q{When a Cucumber step fails, it is useful to create a screenshot image and HTML file of the current page}
   s.license     = 'MIT'
+  s.metadata    = {
+    "bug_tracker_uri"   => "https://github.com/mattheworiordan/capybara-screenshot/issues",
+    "changelog_uri"     => "https://github.com/mattheworiordan/capybara-screenshot/blob/master/CHANGELOG.md",
+    "documentation_uri" => "https://www.rubydoc.info/gems/capybara-screenshot/#{s.version}",
+    "source_code_uri"   => "https://github.com/mattheworiordan/capybara-screenshot/tree/v#{s.version}",
+  }
 
-  s.rubyforge_project = "capybara-screenshot"
-
-  if RUBY_VERSION < "1.9"
+  if Gem::Version.new(RUBY_VERSION) < Gem::Version.new("1.9")
     s.add_dependency 'capybara', ['>= 1.0', '< 2']
+  elsif Gem::Version.new(RUBY_VERSION) <= Gem::Version.new("2.2.2")
+    s.add_dependency 'capybara', ['>= 1.0', '< 3']
   else
     s.add_dependency 'capybara', ['>= 1.0', '< 4']
   end
+
   s.add_dependency 'launchy'
 
   s.add_development_dependency 'rspec'

data/lib/capybara-screenshot.rb

@@ -58,7 +58,7 @@ module Capybara
     end
 
     def self.capybara_root
-      @capybara_root ||= if defined?(::Rails) && ::Rails.root.present?
+      @capybara_root ||= if defined?(::Rails) && ::Rails.respond_to?(:root) && ::Rails.root.present?
         ::Rails.root.join capybara_tmp_path
       elsif defined?(Padrino)
         File.expand_path(capybara_tmp_path, Padrino.root)
@@ -106,11 +106,11 @@ module Capybara
     end
 
     def self.after_save_html &block
-      Saver.after_save_html &block
+      Saver.after_save_html(&block)
     end
 
     def self.after_save_screenshot &block
-      Saver.after_save_screenshot &block
+      Saver.after_save_screenshot(&block)
     end
 
     private
@@ -190,6 +190,14 @@ Capybara::Screenshot.class_eval do
       :not_supported
     end
   end
+
+  register_driver(:apparition) do |driver, path|
+    driver.save_screenshot(path)
+  end
+
+  register_driver(:cuprite) do |driver, path|
+    driver.render(path, :full => true)
+  end
 end
 
 # Register filename prefix formatters

data/lib/capybara-screenshot/callbacks.rb

@@ -4,7 +4,7 @@ module Capybara
       class CallbackSet < Array
         def call *args
           each do |callback|
-            callback.call *args
+            callback.call(*args)
           end
         end
       end
@@ -24,7 +24,7 @@ module Capybara
 
         def run_callbacks name, *args
           if cb_set = callbacks[name]
-            cb_set.call *args
+            cb_set.call(*args)
           end
         end
       end

data/lib/capybara-screenshot/capybara.rb

@@ -1,6 +1,20 @@
-require "capybara-screenshot"
-
 module Capybara
+  class << self
+    module ScreenshotOverrides
+      def using_session(name, &block)
+        original_session_name = Capybara.session_name
+        Capybara::Screenshot.final_session_name = name
+        super(name, &block)
+        Capybara::Screenshot.final_session_name = original_session_name
+      end
+
+      # No longer needed. Kept for backwards compatibility only.
+      alias_method :using_session_with_screenshot, :using_session
+    end
+
+    prepend ScreenshotOverrides
+  end
+
   module DSL
     # Adds class methods to Capybara module and gets mixed into
     # the current scope during Cucumber and RSpec tests
@@ -12,15 +26,5 @@ module Capybara
     def screenshot_and_open_image
       Capybara::Screenshot.screenshot_and_open_image
     end
-
-    def using_session_with_screenshot(name,&blk)
-      original_session_name = Capybara.session_name
-      Capybara::Screenshot.final_session_name = name
-      using_session_without_screenshot(name,&blk)
-      Capybara::Screenshot.final_session_name = original_session_name
-    end
-
-    alias_method :using_session_without_screenshot, :using_session
-    alias_method :using_session, :using_session_with_screenshot
   end
 end

data/lib/capybara-screenshot/cucumber.rb

@@ -21,7 +21,13 @@ After do |scenario|
         saver.display_image
         #this will embed the image in the HTML report, embed() is defined in cucumber
         encoded_img = Base64.encode64(image)
-        embed(encoded_img, 'image/png;base64', "Screenshot of the error")
+
+        # cucumber5 deprecates embed in favor of attach
+        if respond_to? :attach
+          attach(encoded_img, 'image/png')
+        else
+          embed(encoded_img, 'image/png;base64', "Screenshot of the error")
+        end
       end
     end
   end

data/lib/capybara-screenshot/minitest.rb

@@ -27,7 +27,7 @@ begin
   class Minitest::Test
     include Capybara::Screenshot::MiniTestPlugin
   end
-rescue NameError => e
+rescue NameError
   class MiniTest::Unit::TestCase
     include Capybara::Screenshot::MiniTestPlugin
   end

data/lib/capybara-screenshot/pruner.rb

@@ -26,7 +26,10 @@ module Capybara
       end
 
       private
-      attr_reader :strategy_proc
+      
+      def strategy_proc
+        @strategy_proc
+      end
 
       def wildcard_path
         File.expand_path('*.{html,png}', Screenshot.capybara_root)

data/lib/capybara-screenshot/rspec.rb

@@ -90,6 +90,14 @@ RSpec.configure do |config|
     Capybara::Screenshot.final_session_name = nil
   end
 
+  # TODO: DRY refactor into a method?
+  # Add support for Rails system specs (previously only worked with feature specs)
+  config.after(type: :system) do |example_from_block_arg|
+    # RSpec 3 no longer defines `example`, but passes the example as block argument instead
+    example = config.respond_to?(:expose_current_running_example_as) ? example_from_block_arg : self.example
+
+    Capybara::Screenshot::RSpec.after_failed_example(example)
+  end
   config.after(type: :feature) do |example_from_block_arg|
     # RSpec 3 no longer defines `example`, but passes the example as block argument instead
     example = config.respond_to?(:expose_current_running_example_as) ? example_from_block_arg : self.example

data/lib/capybara-screenshot/rspec/base_reporter.rb

@@ -3,7 +3,7 @@ module Capybara
     module RSpec
       module BaseReporter
 
-        # Automatically set up method aliases (very much like ActiveSupport's `alias_method_chain`) 
+        # Automatically set up method aliases (very much like ActiveSupport's `alias_method_chain`)
         # when the module gets included.
         def enhance_with_screenshot(method)
           with_method, without_method = "#{method}_with_screenshot", "#{method}_without_screenshot"
@@ -14,7 +14,6 @@ module Capybara
             end
           end
         end
-
       end
     end
   end

data/lib/capybara-screenshot/rspec/html_embed_reporter.rb

@@ -1,5 +1,6 @@
 require 'capybara-screenshot/rspec/base_reporter'
 require 'base64'
+require 'uri'
 
 module Capybara
   module Screenshot
@@ -13,12 +14,22 @@ module Capybara
           example = @failed_examples.last
           # Ignores saved html file, only saved image will be embedded (if present)
           if (screenshot = example.metadata[:screenshot]) && screenshot[:image]
-            image = File.binread(screenshot[:image])
-            encoded_img = Base64.encode64(image)
-            result += "<img src='data:image/png;base64,#{encoded_img}' style='display: block'>"
+            result += "<img src='#{image_tag_source(screenshot[:image])}' style='display: block'>"
           end
           result
         end
+
+        private
+
+        def image_tag_source(path)
+          if URI.regexp(%w[http https]) =~ path
+            path
+          else
+            image = File.binread(path)
+            encoded_img = Base64.encode64(image)
+            "data:image/png;base64,#{encoded_img}"
+          end
+        end
       end
     end
   end

data/lib/capybara-screenshot/rspec/text_reporter.rb

@@ -26,8 +26,8 @@ module Capybara
         private
         def output_screenshot_info(example)
           return unless (screenshot = example.metadata[:screenshot])
-          output.puts(long_padding + CapybaraScreenshot::Helpers.yellow("HTML screenshot: file://#{screenshot[:html]}")) if screenshot[:html]
-          output.puts(long_padding + CapybaraScreenshot::Helpers.yellow("Image screenshot: file://#{screenshot[:image]}")) if screenshot[:image]
+          output.puts(long_padding + CapybaraScreenshot::Helpers.yellow("HTML screenshot: #{screenshot[:html]}")) if screenshot[:html]
+          output.puts(long_padding + CapybaraScreenshot::Helpers.yellow("Image screenshot: #{screenshot[:image]}")) if screenshot[:image]
         end
 
         def long_padding

data/lib/capybara-screenshot/s3_saver.rb

@@ -5,10 +5,13 @@ module Capybara
     class S3Saver
       DEFAULT_REGION = 'us-east-1'
 
+      attr_accessor :html_path, :screenshot_path
+
       def initialize(saver, s3_client, bucket_name, object_configuration, options={})
         @saver = saver
         @s3_client = s3_client
         @bucket_name = bucket_name
+        @bucket_host = options[:bucket_host]
         @key_prefix = options[:key_prefix]
         @object_configuration = object_configuration
       end
@@ -31,11 +34,13 @@ module Capybara
       end
 
       def save_and_upload_screenshot
-        save_and do |local_file_path|
+        save_and do |type, local_file_path|
           File.open(local_file_path) do |file|
+            s3_upload_path = "#{@key_prefix}#{File.basename(local_file_path)}"
+
             object_payload = {
               bucket: bucket_name,
-              key: "#{@key_prefix}#{File.basename(local_file_path)}",
+              key: s3_upload_path,
               body: file
             }
 
@@ -44,6 +49,10 @@ module Capybara
             s3_client.put_object(
                 object_payload
             )
+
+            host = bucket_host || determine_bucket_host
+
+            send("#{type}_path=", "https://#{host}/#{s3_upload_path}")
           end
         end
       end
@@ -60,14 +69,23 @@ module Capybara
       attr_reader :saver,
                   :s3_client,
                   :bucket_name,
+                  :bucket_host,
                   :object_configuration
                   :key_prefix
 
+      ##
+      # Reads the bucket location using a S3 get_bucket_location request.
+      # Requires the +s3:GetBucketLocation+ policy.
+      def determine_bucket_host
+        s3_region = s3_client.get_bucket_location(bucket: bucket_name).location_constraint
+        "#{bucket_name}.s3-#{s3_region}.amazonaws.com"
+      end
+
       def save_and
         saver.save
 
-        yield(saver.html_path) if block_given? && saver.html_saved?
-        yield(saver.screenshot_path) if block_given? && saver.screenshot_saved?
+        yield(:html, saver.html_path) if block_given? && saver.html_saved?
+        yield(:screenshot, saver.screenshot_path) if block_given? && saver.screenshot_saved?
       end
     end
   end

data/lib/capybara-screenshot/saver.rb

@@ -27,7 +27,9 @@ module Capybara
 
       def save
         current_path do |path|
-          if !path.empty?
+          if path.empty?
+            warn 'WARN: Screenshot could not be saved. `page.current_path` is empty.'
+          else
             begin
               save_html if @html_save
             rescue StandardError => e
@@ -39,8 +41,6 @@ module Capybara
             rescue StandardError => e
               warn "WARN: Screenshot could not be saved. An exception is raised: #{e.inspect}."
             end
-          else
-            warn 'WARN: Screenshot could not be saved. `page.current_path` is empty.'
           end
         end
       end
@@ -49,9 +49,9 @@ module Capybara
         path = html_path
         clear_save_path do
           if Capybara::VERSION.match(/^\d+/)[0] == '1'
-            capybara.save_page(page.body, "#{path}")
+            capybara.save_page(page.body, path.to_s)
           else
-            capybara.save_page("#{path}")
+            capybara.save_page(path.to_s)
           end
         end
         @html_saved = true
@@ -132,13 +132,15 @@ module Capybara
       #   which('ruby') #=> /usr/bin/ruby
       def which(cmd)
         exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
+
         ENV['PATH'].split(File::PATH_SEPARATOR).each do |path|
           exts.each { |ext|
             exe = File.join(path, "#{cmd}#{ext}")
             return exe if File.executable?(exe) && !File.directory?(exe)
           }
         end
-        return nil
+
+        nil
       end
     end
   end

data/lib/capybara-screenshot/version.rb

@@ -1,5 +1,5 @@
 module Capybara
   module Screenshot
-    VERSION = '1.0.20'
+    VERSION = '1.0.25'
   end
 end

data/spec/cucumber/cucumber_spec.rb

@@ -26,13 +26,13 @@ describe "Using Capybara::Screenshot with Cucumber" do
 
     write_file('features/cucumber.feature', code)
 
-    run_simple_with_retry cmd, false
+    run_simple_with_retry cmd
 
     expect(last_command_started.output).to_not match(/failed|failure/i) if options[:assert_all_passed]
   end
 
   it 'saves a screenshot on failure' do
-    run_failing_case %q{Unable to find visible link or button "you'll never find me"}, <<-CUCUMBER
+    run_failing_case %q{Unable to find (visible )?link or button "you'll never find me"}, <<-CUCUMBER
       Feature: Failure
         Scenario: Failure
           Given I visit "/"
@@ -52,7 +52,7 @@ describe "Using Capybara::Screenshot with Cucumber" do
   end
 
   it 'saves a screenshot for the correct session for failures using_session' do
-    run_failing_case(%q{Unable to find visible link or button "you'll never find me"}, <<-CUCUMBER)
+    run_failing_case(%q{Unable to find (visible )?link or button "you'll never find me"}, <<-CUCUMBER)
       Feature: Failure
         Scenario: Failure in different session
           Given I visit "/"
@@ -68,7 +68,7 @@ describe "Using Capybara::Screenshot with Cucumber" do
     end
 
     it 'on failure it prunes previous screenshots when strategy is set' do
-      run_failing_case %q{Unable to find visible link or button "you'll never find me"}, <<-CUCUMBER
+      run_failing_case %q{Unable to find (visible )?link or button "you'll never find me"}, <<-CUCUMBER
         Feature: Prune
           Scenario: Screenshots are pruned if strategy is set
             Given I visit "/"

data/spec/feature/minitest_spec.rb

@@ -22,8 +22,8 @@ describe "Using Capybara::Screenshot with MiniTest" do
     RUBY
 
     cmd = 'bundle exec ruby test_failure.rb'
-    run_simple_with_retry cmd, false
-    expect(last_command_started.output).to include %q{Unable to find visible link or button "you'll never find me"}
+    run_simple_with_retry cmd
+    expect(last_command_started.output).to match %r{Unable to find (visible )?link or button "you'll never find me"}
   end
 
   it 'saves a screenshot on failure' do

data/spec/feature/testunit_spec.rb

@@ -14,7 +14,7 @@ describe "Using Capybara::Screenshot with Test::Unit" do
 
       #{setup_test_app}
       Capybara::Screenshot.register_filename_prefix_formatter(:testunit) do | fault |
-        raise "expected fault" unless fault.exception.message.include? %q{Unable to find visible link or button "you'll never find me"}
+        raise "expected fault" unless fault.exception.message.match %r{Unable to find (visible )?link or button "you'll never find me"}
         'my_screenshot'
       end
 
@@ -28,8 +28,8 @@ describe "Using Capybara::Screenshot with Test::Unit" do
     RUBY
 
     cmd = "bundle exec ruby #{integration_path}/test_failure.rb"
-    run_simple_with_retry cmd, false
-    expect(last_command_started.output).to include %q{Unable to find visible link or button "you'll never find me"}
+    run_simple_with_retry cmd
+    expect(last_command_started.output).to match %r{Unable to find (visible )?link or button "you'll never find me"}
   end
 
   it "saves a screenshot on failure for any test in path 'test/integration'" do

data/spec/rspec/rspec_spec.rb

@@ -31,7 +31,7 @@ describe Capybara::Screenshot::RSpec, :type => :aruba do
       RUBY
 
       cmd = cmd_with_format(options[:format])
-      run_simple_with_retry cmd, false
+      run_simple_with_retry cmd
 
       expect(last_command_started.output).to match('0 failures') if options[:assert_all_passed]
     end
@@ -41,7 +41,7 @@ describe Capybara::Screenshot::RSpec, :type => :aruba do
     end
 
     it 'saves a screenshot when browser action fails' do
-      run_failing_case <<-RUBY, %q{Unable to find visible link or button "you'll never find me"}
+      run_failing_case <<-RUBY, %r{Unable to find (visible )?link or button "you'll never find me"}
         feature 'screenshot with failure' do
           scenario 'click on a missing link' do
             visit '/'
@@ -103,7 +103,7 @@ describe Capybara::Screenshot::RSpec, :type => :aruba do
     end
 
     it 'saves a screenshot for the correct session for failures using_session' do
-      run_failing_case <<-RUBY, %q{Unable to find visible link or button "you'll never find me"}
+      run_failing_case <<-RUBY, %r{Unable to find (visible )?link or button "you'll never find me"}
         feature 'screenshot with failure' do
           scenario 'click on a missing link' do
             visit '/'
@@ -116,7 +116,24 @@ describe Capybara::Screenshot::RSpec, :type => :aruba do
           end
         end
       RUBY
-      expect('tmp/screenshot.html').to have_file_content(/is/)
+      expect('tmp/screenshot.html').to have_file_content(/This is a different page/)
+    end
+
+    it 'saves a screenshot for the correct session for failures Capybara.using_session' do
+      run_failing_case <<-RUBY, %r{Unable to find (visible )?link or button "you'll never find me"}
+        feature 'screenshot with failure' do
+          scenario 'click on a missing link' do
+            visit '/'
+            expect(page.body).to include('This is the root page')
+            Capybara.using_session :different_session do
+              visit '/different_page'
+              expect(page.body).to include('This is a different page')
+              click_on "you'll never find me"
+            end
+          end
+        end
+      RUBY
+      expect('tmp/screenshot.html').to have_file_content(/This is a different page/)
     end
 
     context 'pruning' do

data/spec/spinach/spinach_spec.rb

@@ -12,12 +12,12 @@ describe "Using Capybara::Screenshot with Spinach" do
 
     write_file('spinach.feature', code)
     cmd = 'bundle exec spinach -f .'
-    run_simple_with_retry cmd, false
+    run_simple_with_retry cmd
     expect(last_command_started.output).to match(failure_message)
   end
 
   it "saves a screenshot on failure" do
-    run_failing_case(%q{Unable to find visible link or button "you'll never find me"}, <<-GHERKIN)
+    run_failing_case(%q{Unable to find (visible )?link or button "you'll never find me"}, <<-GHERKIN)
       Feature: Failure
         Scenario: Failure
           Given I visit "/"
@@ -37,7 +37,7 @@ describe "Using Capybara::Screenshot with Spinach" do
   end
 
   it "saves a screenshot for the correct session for failures using_session" do
-    run_failing_case(%q{Unable to find visible link or button "you'll never find me"}, <<-GHERKIN)
+    run_failing_case(%q{Unable to find (visible )?link or button "you'll never find me"}, <<-GHERKIN)
       Feature: Failure
         Scenario: Failure in different session
           Given I visit "/"
@@ -49,7 +49,7 @@ describe "Using Capybara::Screenshot with Spinach" do
   it 'on failure it prunes previous screenshots when strategy is set' do
     create_screenshot_for_pruning
     configure_prune_strategy :last_run
-    run_failing_case(%q{Unable to find visible link or button "you'll never find me"}, <<-GHERKIN)
+    run_failing_case(%q{Unable to find (visible )?link or button "you'll never find me"}, <<-GHERKIN)
       Feature: Failure
         Scenario: Failure
           Given I visit "/"

data/spec/support/common_setup.rb

@@ -41,11 +41,11 @@ module CommonSetup
       end
     end
 
-    def run_simple_with_retry(*args)
-      run_simple(*args)
+    def run_simple_with_retry(cmd, fail_on_error: false)
+      run_command_and_stop(cmd, fail_on_error: fail_on_error)
     rescue ChildProcess::TimeoutError => e
-      puts "run_simple(#{args.join(', ')}) failed. Will retry once. `#{e.message}`"
-      run_simple(*args)
+      puts "run_command_and_stop(#{cmd}, fail_on_error: #{fail_on_error}) failed. Will retry once. `#{e.message}`"
+      run_command_and_stop(cmd, fail_on_error: fail_on_error)
     end
 
     def configure_prune_strategy(strategy)

data/spec/unit/capybara_spec.rb

@@ -26,6 +26,19 @@ describe Capybara do
       }.to raise_exception ::RSpec::Expectations::ExpectationNotMetError
     end
   end
+
+  describe 'Capybara.using_session' do
+    include Capybara::DSL
+
+    it 'saves the name of the final session' do
+      expect(Capybara::Screenshot).to receive(:final_session_name=).with(:different_session)
+      expect {
+        Capybara.using_session :different_session do
+          expect(0).to eq 1
+        end
+      }.to raise_exception ::RSpec::Expectations::ExpectationNotMetError
+    end
+  end
 end
 
 describe 'final_session_name' do

data/spec/unit/rspec_reporters/html_embed_reporter_spec.rb

@@ -15,4 +15,17 @@ describe Capybara::Screenshot::RSpec::HtmlEmbedReporter do
       expect(content_without_styles).to eql("original content<img src='data:image/png;base64,#{encoded_image_data}'>")
     end
   end
+
+  context 'when an image was saved to s3' do
+    let(:s3_image_url) { "http://s3.amazon.com/path/to/image" }
+
+    before do
+      set_example double("example", metadata: {screenshot: {image: s3_image_url}})
+    end
+
+    it 'embeds the image URL into the content' do
+      content_without_styles = @reporter.extra_failure_content(nil).gsub(/ ?style='.*?' ?/, "")
+      expect(content_without_styles).to eql("original content<img src='#{s3_image_url}'>")
+    end
+  end
 end

data/spec/unit/rspec_reporters/text_reporter_spec.rb

@@ -64,20 +64,20 @@ describe Capybara::Screenshot::RSpec::TextReporter do
   end
 
   context 'when a html file was saved' do
-    let(:example) { example_failed_method_argument_double(screenshot: { html: "path/to/html" }) }
+    let(:example) { example_failed_method_argument_double(screenshot: { html: "/path/to/html" }) }
 
     it 'appends the html file path to the original output' do
       @reporter.send(example_failed_method, example)
-      expect(@reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: file://path/to/html")}\n")
+      expect(@reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: /path/to/html")}\n")
     end
   end
 
   context 'when a html file and an image were saved' do
-    let(:example) { example_failed_method_argument_double(screenshot: { html: "path/to/html", image: "path/to/image" }) }
+    let(:example) { example_failed_method_argument_double(screenshot: { html: "/path/to/html", image: "/path/to/image" }) }
 
     it 'appends the image path to the original output' do
       @reporter.send(example_failed_method, example)
-      expect(@reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: file://path/to/html")}\n  #{CapybaraScreenshot::Helpers.yellow("Image screenshot: file://path/to/image")}\n")
+      expect(@reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: /path/to/html")}\n  #{CapybaraScreenshot::Helpers.yellow("Image screenshot: /path/to/image")}\n")
     end
   end
 
@@ -91,8 +91,8 @@ describe Capybara::Screenshot::RSpec::TextReporter do
     end
     old_reporter = old_reporter_class.new
     old_reporter.singleton_class.send :include, described_class
-    example = example_failed_method_argument_double(screenshot: { html: "path/to/html" })
+    example = example_failed_method_argument_double(screenshot: { html: "/path/to/html" })
     old_reporter.send(example_failed_method, example)
-    expect(old_reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: file://path/to/html")}\n")
+    expect(old_reporter.output.string).to eql("original failure info\n  #{CapybaraScreenshot::Helpers.yellow("HTML screenshot: /path/to/html")}\n")
   end
 end

data/spec/unit/s3_saver_spec.rb

@@ -5,12 +5,19 @@ describe Capybara::Screenshot::S3Saver do
   let(:saver) { double('saver') }
   let(:bucket_name) { double('bucket_name') }
   let(:s3_object_configuration) { {} }
+  let(:options) { {} }
   let(:s3_client) { double('s3_client') }
   let(:key_prefix){ "some/path/" }
 
-  let(:s3_saver) { described_class.new(saver, s3_client, bucket_name, s3_object_configuration) }
+  let(:s3_saver) { described_class.new(saver, s3_client, bucket_name, s3_object_configuration, options) }
   let(:s3_saver_with_key_prefix) { described_class.new(saver, s3_client, bucket_name, s3_object_configuration, key_prefix: key_prefix) }
 
+  let(:region) { double('region') }
+
+  before do
+    allow(s3_client).to receive(:get_bucket_location).and_return(double(:bucket_location_response, location_constraint: region))
+  end
+
   describe '.new_with_configuration' do
     let(:access_key_id) { double('access_key_id') }
     let(:secret_access_key) { double('secret_access_key') }
@@ -21,7 +28,6 @@ describe Capybara::Screenshot::S3Saver do
       }
     }
 
-    let(:region) { double('region') }
     let(:s3_client_credentials) {
       s3_client_credentials_using_defaults.merge(region: region)
     }
@@ -99,6 +105,20 @@ describe Capybara::Screenshot::S3Saver do
       allow(saver).to receive(:save)
     end
 
+    context 'providing a bucket_host' do
+      let(:options) { { bucket_host: 'some other location' } }
+  
+      it 'does not request the bucket location' do
+        screenshot_path = '/baz/bim.jpg'
+
+        screenshot_file = double('screenshot_file')
+
+        expect(s3_saver).not_to receive(:determine_bucket_host)
+
+        s3_saver.save
+      end
+    end
+
     it 'calls save on the underlying saver' do
       expect(saver).to receive(:save)
 
@@ -120,6 +140,8 @@ describe Capybara::Screenshot::S3Saver do
         body: html_file
       )
 
+      expect(s3_saver).to receive(:determine_bucket_host).and_call_original
+
       s3_saver.save
     end
 

data/spec/unit/saver_spec.rb

@@ -298,6 +298,19 @@ describe Capybara::Screenshot::Saver do
     end
   end
 
+  describe "with cuprite driver" do
+    before do
+      allow(capybara_mock).to receive(:current_driver).and_return(:cuprite)
+    end
+
+    it 'saves driver render with :full => true' do
+      expect(driver_mock).to receive(:render).with(screenshot_path, {:full => true})
+
+      saver.save
+      expect(saver).to be_screenshot_saved
+    end
+  end
+
   describe "with webkit driver" do
     before do
       allow(capybara_mock).to receive(:current_driver).and_return(:webkit)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: capybara-screenshot
 version: !ruby/object:Gem::Version
-  version: 1.0.20
+  version: 1.0.25
 platform: ruby
 authors:
 - Matthew O'Riordan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-04 00:00:00.000000000 Z
+date: 2020-10-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: capybara
@@ -240,7 +240,11 @@ files:
 homepage: http://github.com/mattheworiordan/capybara-screenshot
 licenses:
 - MIT
-metadata: {}
+metadata:
+  bug_tracker_uri: https://github.com/mattheworiordan/capybara-screenshot/issues
+  changelog_uri: https://github.com/mattheworiordan/capybara-screenshot/blob/master/CHANGELOG.md
+  documentation_uri: https://www.rubydoc.info/gems/capybara-screenshot/1.0.25
+  source_code_uri: https://github.com/mattheworiordan/capybara-screenshot/tree/v1.0.25
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -256,8 +260,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: capybara-screenshot
-rubygems_version: 2.4.8
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Automatically create snapshots when Cucumber steps fail with Capybara and