1
0
Fork 0
mirror of https://github.com/teamcapybara/capybara.git synced 2022-11-09 12:08:07 -05:00

update specs for #first

This commit is contained in:
Thomas Walpole 2018-02-16 15:02:01 -08:00
parent 5930e1f114
commit 4a19ed543a
2 changed files with 10 additions and 4 deletions

View file

@ -242,7 +242,7 @@ module Capybara
# @return [Capybara::Result] A collection of found elements
# @raise [Capybara::ExpectationNotMet] The number of elements found doesn't match the specified conditions
def all(*args, **options, &optional_filter_block)
minimum_specified = %i[count minimum between].any? { |k| options.key?(k) }
minimum_specified = options_include_minimum?(options)
options = { minimum: 1 }.merge(options) unless minimum_specified
options[:session_options] = session_options
query = Capybara::Queries::SelectorQuery.new(*args.push(options), &optional_filter_block)
@ -265,17 +265,17 @@ module Capybara
# Find the first element on the page matching the given selector
# and options. By default `first` will wait up to `Capybara.default_max_wait_time`
# seconds for matching elements to appear and then raise an error if no matching
# element is found
# element is found, or `nil` if the provided count options allow for empty results.
#
# @overload first([kind], locator, options)
# @param [:css, :xpath] kind The type of selector
# @param [String] locator The selector
# @param [Hash] options Additional options; see {#all}
# @return [Capybara::Node::Element] The found element or nil
# @raise [Capybara::ElementNotFound] If the element can't be found before time expires
# @raise [Capybara::ElementNotFound] If element(s) matching the provided options can't be found before time expires
#
def first(*args, **options, &optional_filter_block)
options = { minimum: 1 }.merge(options)
options = { minimum: 1 }.merge(options) unless options_include_minimum?(options)
all(*args, **options, &optional_filter_block).first
end
@ -304,6 +304,10 @@ module Capybara
def prefer_exact?(query)
query.match == :smart or query.match == :prefer_exact
end
def options_include_minimum?(opts)
%i[count minimum between].any? { |k| opts.key?(k) }
end
end
end
end

View file

@ -17,6 +17,8 @@ Capybara::SpecHelper.spec '#first' do
it "should return nil when nothing was found if count options allow no results" do
expect(@session.first('//div[@id="nosuchthing"]', minimum: 0)).to be_nil
expect(@session.first('//div[@id="nosuchthing"]', count: 0)).to be_nil
expect(@session.first('//div[@id="nosuchthing"]', between: (0..3))).to be_nil
end
it "should accept an XPath instance" do