Don't reset the session when the example failed, because we need capybara-screenshot to have access to it

This commit is contained in:
Douwe Maan 2017-06-21 21:38:16 -05:00
parent f85d0a0024
commit af5bf71365

View file

@ -36,12 +36,13 @@ RSpec.configure do |config|
$capybara_server_already_started = true
end
config.after(:each, :js) do
config.after(:each, :js) do |example|
# capybara/rspec already calls Capybara.reset_sessions! in an `after` hook,
# but `block_and_wait_for_requests_complete` is called before it so by
# calling it explicitely here, we prevent any new requests from being fired
# See https://github.com/teamcapybara/capybara/blob/ffb41cfad620de1961bb49b1562a9fa9b28c0903/lib/capybara/rspec.rb#L20-L25
Capybara.reset_sessions!
# We don't reset the session when the example failed, because we need capybara-screenshot to have access to it.
Capybara.reset_sessions! unless example.exception
block_and_wait_for_requests_complete
end
end