[0.14.3] Support custom save_path option

[singhprd]

Allows paths other than Capybara.save_path to be specified

This PR allows a custom save_path option to be configured, instead of defaulting to the Capybara.save_path.

The motivation is wanting to specify an ephemeral directory for downloads (PDFs, CSVs etc) which is cleared between tests, but keep other items saved to the Capybara.save_path (like screenshots) as CI artifacts.

This is especially useful when running multiple specs on a single instance. Here is how we separate the downloads using this patch:

module Downloads
  def downloads_path
    Rails.root.join("log", "capybara", "downloads", ENV.fetch("TEST_ENV_NUMBER", "1"))
  end
end

...
...

cuprite_options = {
  process_timeout: 60,
  ...
  save_path: Downloads.downloads_path,
}

...

Capybara.register_driver :cuprite_chrome do |app|
  Capybara::Cuprite::Driver.new(app, options)
end

Original PR: #192 (this one has just rebased against 0.14.3)

[singhprd]

Hey @route! 👋

Hope you're doing well. Thanks for your fantastic work maintaining this Gem, it's super helpful and really appreciated! 🙇

If you get some time, would you be able to evaluate if this is able to be merged in please, hopefully it helps someone else out and I can retire our fork 🍴

Thanks!
Pete

[singhprd]

Related issue: #198

[route]

The tests are all failing

[singhprd]

Ah, thanks for approving them to run. I'll take a look!

[singhprd]

Thanks @route! - 🎉 😃

I was just about to ask you to re-run the tests as they were passing on my fork, but you've already done it 🚀 🪨

And have a great rest of your day! 🌤️

[route]

Thank you! You too!