style cleanup

This commit is contained in:
Thomas Walpole 2019-02-18 16:40:27 -08:00
parent 03ada281ac
commit 5e9f81181d
3 changed files with 7 additions and 5 deletions

View File

@ -104,6 +104,6 @@ class Capybara::RackTest::Driver < Capybara::Driver::Base
def header(key, value); browser.header(key, value); end
def invalid_element_errors
[ Capybara::RackTest::Errors::StaleElementReferenceError ]
[Capybara::RackTest::Errors::StaleElementReferenceError]
end
end

View File

@ -116,9 +116,10 @@ class Capybara::RackTest::Node < Capybara::Driver::Node
native.css(locator, Capybara::RackTest::CSSHandlers.new).map { |el| self.class.new(driver, el) }
end
self.public_instance_methods(false).each do |meth_name|
public_instance_methods(false).each do |meth_name|
alias_method "unchecked_#{meth_name}", meth_name
private "unchecked_#{meth_name}"
private "unchecked_#{meth_name}" # rubocop:disable Layout/AccessModifierIndentation,Style/AccessModifierDeclarations
define_method meth_name do |*args|
stale_check
send("unchecked_#{meth_name}", *args)
@ -128,7 +129,7 @@ class Capybara::RackTest::Node < Capybara::Driver::Node
def ==(other)
native == other.native
end
protected
# @api private

View File

@ -59,7 +59,8 @@ class Capybara::Selenium::FirefoxNode < Capybara::Selenium::Node
end
def hover
return super unless (browser_version >= 65.0)
return super unless browser_version >= 65.0
# work around issue 2156 - https://github.com/teamcapybara/capybara/issues/2156
scroll_if_needed { browser_action.move_to(native, 0, 0).move_to(native).perform }
end