Screenshot uploaded to S3 using cucumber reporting local file paths

[garytaylor]

I have looked into this and have made the following "Monkey Patch" in my code. It appears that the S3 saver does not have the output_screenshot_path method defined so it calls the method on the non S3 saver and gets it wrong.

Here is my code

module Capybara
  module Screenshot
    class S3Saver
      def output_screenshot_path
        output "HTML screenshot: #{html_path}" if html_saved?
        output "Image screenshot: #{screenshot_path}" if screenshot_saved?
      end
    end
  end
end

which works a treat :-) but unsure if it will have any impact on the rspec output or any other area where it shouldnt

[mattheworiordan]

Thanks for reporting this @garytaylor. Fancy rolling that patch up into a PR?