Skip to content

Commit ab23133

Browse files
committed
fix linter issues
1 parent 1c2c42b commit ab23133

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

spec/rails_helper.rb

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,25 @@
5656
case driver
5757
when :poltergeist, :poltergeist_errors_ok, :poltergeist_no_animations
5858
require "capybara/poltergeist"
59-
opts = {
60-
# Leaving animations off, as a sleep was still needed.
61-
extensions: ["#{Rails.root}/spec/support/phantomjs-disable-animations.js"],
59+
window_opts = {
6260
window_size: [1280, 720],
6361
screen_size: [1600, 1200]
6462
}
6563

64+
Capybara.register_driver :poltergeist do |app|
65+
Capybara::Poltergeist::Driver.new(app, window_opts)
66+
end
67+
68+
no_animation_opts = window_opts.merge( # Leaving animations off, as a sleep was still needed.
69+
extensions: ["#{Rails.root}/spec/support/phantomjs-disable-animations.js"]
70+
)
71+
6672
Capybara.register_driver :poltergeist_no_animations do |app|
67-
Capybara::Poltergeist::Driver.new(app, opts)
73+
Capybara::Poltergeist::Driver.new(app, no_animation_opts)
6874
end
6975

7076
Capybara.register_driver :poltergeist_errors_ok do |app|
71-
Capybara::Poltergeist::Driver.new(app, opts.merge(js_errors: false))
77+
Capybara::Poltergeist::Driver.new(app, no_animation_opts.merge(js_errors: false))
7278
end
7379
when :selenium_chrome
7480
DriverRegistration.register_selenium_chrome

0 commit comments

Comments
 (0)