Use the Selenium Chrome driver download_path=

danshep · Jun 26, 2018 · 121c391 · 121c391
1 parent 4da9195
commit 121c391
Showing 1 changed file with 2 additions and 15 deletions.
diff --git a/spec/selenium_spec_chrome.rb b/spec/selenium_spec_chrome.rb
@@ -13,13 +13,7 @@
 
 Capybara.register_driver :selenium_chrome do |app|
   driver = Capybara::Selenium::Driver.new(app, browser: :chrome)
-  # TODO: Fix this when selenium with `download_path =` support is released
-  bridge = driver.browser.send(:bridge)
-  params = { cmd: 'Page.setDownloadBehavior', params: { behavior: 'allow', downloadPath: Capybara.save_path } }
-  command = +'/session/:session_id/chromium/send_command'
-  command[':session_id'] = bridge.session_id
-  bridge.http.call(:post, command, params)
-
+  driver.browser.download_path = Capybara.save_path
   driver
 end
 
@@ -28,14 +22,7 @@
   browser_options.args << '--headless'
   browser_options.args << '--disable-gpu' if Gem.win_platform?
   driver = Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
-
-  # TODO: Fix this when selenium with `download_path =` support is released
-  bridge = driver.browser.send(:bridge)
-  params = { cmd: 'Page.setDownloadBehavior', params: { behavior: 'allow', downloadPath: Capybara.save_path } }
-  command = +'/session/:session_id/chromium/send_command'
-  command[':session_id'] = bridge.session_id
-  bridge.http.call(:post, command, params)
-
+  driver.browser.download_path = Capybara.save_path
   driver
 end