site_prism 3.3 → 3.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 21261a1fb826c5500f7bc30e44729cb733b2f75b1ea7d3d5243400d9d29e8b2c
-  data.tar.gz: e50ab9e1b7d6c0e651d15aa6888828b789f38b7aed80511caa96d50dc767dcda
+  metadata.gz: 1ec5104247e3512bd7ebf0a48c579cc92041ef21b9c130531ef0cb4ff5a52f88
+  data.tar.gz: a44ff822766bb37ff023048349037c0467ae47333d4b421bc53d8a59cb2af087
 SHA512:
-  metadata.gz: 423f9ffb5d2b26f9619f81414074ab1006b72d7ac8516a355f6d4ffc3959aadb56bacda3a25f06c4bff8e4898609d6c5540e9e9856b3f53d613c5ec9e819125e
-  data.tar.gz: 1541d15de381f59d4b75c46ae322e26e3e89ccc80549f7b11d55fa05cd32ccd314ca41f67724d310d56ce48d87c6aec3cfe4a1275a1295279ec6de6ef1cb48b2
+  metadata.gz: 1a6991ecd0110bb53533fb848c40753a13067aed17913f2dd261a5d0b4c8c1747f9f54d1be83d954856037e6cbd24abd665d498b74ff0ad835ab1bea1e0a084a
+  data.tar.gz: bcfe1f63bcaf2b20c64241c03b1c9c718628609fa0bdaa67c01a78b02964b809135aa815e6b0073d4272235226eda5ad802b8b6852d8b118d0806d96e09e2068

data/README.md

@@ -1,6 +1,6 @@
 # SitePrism
 [![Gem Version](https://badge.fury.io/rb/site_prism.svg)](https://badge.fury.io/rb/site_prism)
-[![Build Status](https://travis-ci.org/natritmeyer/site_prism.png)](https://travis-ci.org/natritmeyer/site_prism)
+[![Build Status](https://travis-ci.com/site-prism/site_prism.png)](https://travis-ci.com/site-prism/site_prism)
 
 _A Page Object Model DSL for Capybara_
 
@@ -9,19 +9,19 @@ for use with Capybara in automated acceptance testing.
 
 Find the pretty documentation here: http://rdoc.info/gems/site_prism/frames
 
-Make sure to add your project/company to https://github.com/natritmeyer/site_prism/wiki/Who-is-using-SitePrism
+Make sure to add your project/company to https://github.com/site-prism/site_prism/wiki/Who-is-using-SitePrism
 
 ## Developing / Contributing to SitePrism
 
 We love it when people want to get involved with our Open Source Project.
 
-We have a brief set of setup docs [HERE](https://github.com/site-prism/site_prism/blob/master/development_setup.md)
+We have a brief set of setup docs [HERE](https://github.com/site-prism/site_prism/blob/master/HACKING.md)
 
 ## Supported Rubies / Browsers
 
 SitePrism is built and tested to work on Ruby 2.4 - 2.6. Ruby 2.3 (Now EOL), is supported but not tested against.
 If you are using SitePrism with Ruby 2.3 it is highly advisable to upgrade to a more modern Ruby
-such as 2.5 or 2.6, if for any other reason, to get a noticeable speed boost!
+such as 2.6 or 2.7, if for any other reason, to get a noticeable speed boost!
 
 SitePrism should run on all major browsers. The gem's integration tests are ran on Chrome and Firefox.
 
@@ -42,14 +42,14 @@ class Home < SitePrism::Page
   element :search_field, 'input[name="q"]'
   element :search_button, 'button[name="btnK"]'
   elements :footer_links, '#footer a'
-  section :menu, MenuSection, '#gbx3'
+  section :menu, Menu, '#gbx3'
 end
 
 class SearchResults < SitePrism::Page
   set_url_matcher(/google.com\/results\?.*/)
 
-  section :menu, MenuSection, '#gbx3'
-  sections :search_results, SearchResultSection, '#results li'
+  section :menu, Menu, '#gbx3'
+  sections :search_results, SearchResults, '#results li'
 
   def search_result_links
     search_results.map { |result| result.title['href'] }
@@ -58,13 +58,13 @@ end
 
 # define sections used on multiple pages or multiple times on one page
 
-class MenuSection < SitePrism::Section
+class Menu < SitePrism::Section
   element :search, 'a.search'
   element :images, 'a.image-search'
   element :maps, 'a.map-search'
 end
 
-class SearchResultSection < SitePrism::Section
+class SearchResults < SitePrism::Section
   element :title, 'a.title'
   element :blurb, 'span.result-description'
 end
@@ -124,6 +124,22 @@ require 'capybara'
 require 'capybara/cucumber'
 require 'selenium-webdriver'
 require 'site_prism'
+require 'site_prism/all_there' # Optional but needed to perform more complex matching
+```
+
+The driver creation is identical to how you would normally create a Capybara driver,
+a sample Selenium one could look something like this...
+
+```ruby
+Capybara.register_driver :site_prism do |app|
+  browser = ENV.fetch('browser', 'firefox').to_sym
+  Capybara::Selenium::Driver.new(app, browser: browser, desired_capabilities: capabilities)
+end
+
+# Then tell Capybara to use the Driver you've just defined as its default driver
+Capybara.configure do |config|
+  config.default_driver = :site_prism
+end
 ```
 
 ### Using SitePrism with RSpec
@@ -135,8 +151,11 @@ require 'capybara'
 require 'capybara/rspec'
 require 'selenium-webdriver'
 require 'site_prism'
+require 'site_prism/all_there' # Optional but needed to perform more complex matching
 ```
 
+And again, as above, a sample driver is no different to a normal driver instantiation in Capybara.
+
 ## Introduction to the Page Object Model
 
 The Page Object Model is a test automation pattern that aims to create
@@ -158,7 +177,7 @@ multiple pages, or many times on a page using the concept of sections.
 As you might be able to guess from the name, pages are fairly central to
 the Page Object Model. Here's how SitePrism models them:
 
-### Creating a Page Model
+### Creating your first Page using the Page Object Model
 
 The simplest page is one that has nothing defined in it. Here's an
 example of how to begin modelling a home page:
@@ -168,7 +187,7 @@ class Home < SitePrism::Page
 end
 ```
 
-The above has nothing useful defined, only the name.
+The above has nothing useful defined, so to start with lets give it some properties.
 
 ### Adding a URL
 
@@ -177,7 +196,7 @@ you'll need to set its URL. Here's how:
 
 ```ruby
 class Home < SitePrism::Page
-  set_url 'http://www.google.com'
+  set_url 'http://www.mysite.com/home.htm'
 end
 ```
 
@@ -332,7 +351,7 @@ end
 SitePrism's `#displayed?` predicate method allows for semantic code in your tests:
 
 ```ruby
-Then /^the account page is displayed$/ do
+Then(/^the account page is displayed$/) do
   expect(@account_page).to be_displayed
   expect(@some_other_page).not_to be_displayed
 end
@@ -431,7 +450,7 @@ end
 #### Testing for the existence of the element
 
 Another method added to the Page class by the `element` method is the
-`has_<element name>?` method. Using the same example as above:
+`has_<element_name>?` method. Using the same example as above:
 
 ```ruby
 class Home < SitePrism::Page
@@ -452,7 +471,7 @@ end
 ...which makes for nice test code:
 
 ```ruby
-Then /^the search field exists$/ do
+Then(/^the search field exists$/) do
   expect(@home).to have_search_field
 end
 ```
@@ -472,14 +491,14 @@ that should be used to test for non-existence. Using the above example:
 ...which makes for nice test code:
 
 ```ruby
-Then /^the search field exists$/ do
+Then(/^the search field exists$/)do
   expect(@home).to have_no_search_field #NB: NOT => expect(@home).not_to have_search_field
 end
 ```
 
 #### Waiting for an element to become visible
 
-A method that gets added by calling `element` is the 
+A method that gets added by calling `element` is the
 `wait_until_<element_name>_visible` method. Calling this method will
 cause the test to wait for Capybara's default wait time for the element
 to become visible. You can customise the wait time by supplying a number
@@ -619,7 +638,7 @@ Then the following method is available:
 This in turn allows the following nice test code
 
 ```ruby
-Then /^there should be some names listed on the page$/ do
+Then(/^there should be some names listed on the page$/) do
   expect(@friends_page).to have_names #=> This only passes if there is at least one `name`
 end
 ```
@@ -660,7 +679,7 @@ are present in the browser and `false` if they're not all there.
 
 # and...
 
-Then /^the friends page contains all the expected elements$/ do
+Then(/^the friends page contains all the expected elements$/) do
   expect(@friends_page).to be_all_there
 end
 ```
@@ -695,6 +714,50 @@ end
 @test_page.elements_present #=> [:address_field]
 ```
 
+If you are specifying a highly nested set of sections inside a Page and need to recurse
+through them to find out if all of your items are present then you can also do this.
+
+Simply pass a recursion parameter to the `#all_there?` check. Note that the only valid values
+for this at the moment are `:none` and `:one`
+
+Passing `:none` (default), will not change the functionality. However passing in `:one` will cause
+`site_prism` to recurse through all `section` / `sections` items defined in your present scope.
+
+Work alongside developing this functionality further is being continued in the
+[site_prism-all_there](http://www.github.com/site-prism/site_prism-all_there) repo. So head on over
+there if you're interested in how this feature will work going forwards
+
+NB: At the moment a "primitive" but working copy of this is hosted inside this gem. But if you wish to
+use the bleeding edge version of the logic. Then simply set the following configuration parameter
+
+```ruby
+`require 'site_prism/all_there'`
+
+SitePrism.use_all_there_gem = true
+```
+
+### Getting the list of missing elements
+
+If `#all_there?` returns false and you wish to get the list of missing elements for debugging purposes
+you may want to use `#elements_missing` method. It will return all missing elements from the expected_elements list
+
+If you do not provide a list of `expected_elements` this method will return all elements that are missing on the page;
+from those which are defined.
+
+```ruby
+class Home < SitePrism::Page
+  element :name, '#name'
+  element :address, '#address'
+  element :success_message, 'span.alert-success'
+
+  expected_elements :name, :address
+end
+
+# and... Only `address` is on the page
+
+@test_page.elements_missing #=> [:name]
+```
+
 ## Sections
 
 SitePrism allows you to model sections of a page that appear on multiple
@@ -750,7 +813,7 @@ class People < SitePrism::Section
   element :footer, 'h4'
 end
 
-class HomePage < SitePrism::Page
+class Home < SitePrism::Page
   # section people_with_block will have `headline` and
   # `footer` elements in it
   section :people_with_block, People do
@@ -870,7 +933,7 @@ end
 This then leads to some pretty test code ...
 
 ```ruby
-Then /^the home page menu contains a link to the various search functions$/ do
+Then(/^the home page menu contains a link to the various search functions$/) do
   expect(@home.menu).to have_search
   expect(@home.menu.search['href']).to include('google.com')
   expect(@home.menu).to have_images
@@ -880,14 +943,13 @@ end
 
 ##### Accessing section elements using a block
 
-A Section can be scoped so it is only accessible inside a block. This is
+You can execute a block within the context of a Section. This is
 similar to Capybara's `within` method and allows for shorter test code
-particularly with nested sections. Some of this test code can be
-made a little prettier by simply passing a block in.
+particularly with nested sections. Test code that might have to repeat the block name can be shortened up this way.
 
 ```ruby
-Then /^the home page menu contains a link to the various search functions$/ do
-  @home.menu do |menu|
+Then(/^the home page menu contains a link to the various search functions$/) do
+  @home.menu.within do |menu|
     expect(menu).to have_search
     expect(menu.search['href']).to include('google.com')
     expect(menu).to have_images
@@ -896,33 +958,43 @@ Then /^the home page menu contains a link to the various search functions$/ do
 end
 ```
 
+Note that on an individual section it's possible to pass a block directly to the section without using `within`.  Because the block is executed only during `Section` initialization this won't work when accessing a single Section from an array of Sections.  For that reason we recommend using `within` which works in either case.
+
+```ruby
+Then(/^the home page menu contains a link to the various search functions$/) do
+  @home.menu do |menu|  # possible, but prefer: `@home.menu.within`
+    expect(menu).to have_search
+  end
+end
+```
+
 #### Getting a section's parent
 
 It is possible to ask a section for its parent (page, or section if this
 section is a subsection). For example, given the following setup:
 
 ```ruby
-class MySubSection < SitePrism::Section
-  element :some_element, 'abc'
+class DestinationFilters < SitePrism::Section
+  element :morocco, 'abc'
 end
 
-class MySection < SitePrism::Section
-  section :my_subsection, MySubSection, 'def'
+class FilterPanel < SitePrism::Section
+  section :destination_filters, DestinationFilters, 'def'
 end
 
-class MyPage < SitePrism::Page
-  section :my_section, MySection, 'ghi'
+class Home < SitePrism::Page
+  section :filter_panel, FilterPanel, 'ghi'
 end
 ```
 
 Then calling `#parent` will return the following:
 
 ```ruby
-@my_page = MyPage.new
-@my_page.load
+@home = Home.new
+@home.load
 
-@my_page.my_section.parent #=> returns @my_page
-@my_page.my_section.my_subsection.parent #=> returns @my_page.my_section
+@home.filter_panel.parent #=> returns @home
+@home.filter_panel.destination_filters.parent #=> returns @home.filter_panel
 ```
 
 #### Getting a section's parent page
@@ -1043,7 +1115,7 @@ end
 
 # how to login (fatuous, but demonstrates the point):
 
-Then /^I sign in$/ do
+Then(/^I sign in$/) do
   @home = Home.new
   @home.load
   expect(@home).to have_login_and_registration
@@ -1055,7 +1127,7 @@ end
 
 # how to sign up:
 
-When /^I enter my name into the home page's registration form$/ do
+When(/^I enter my name into the home page's registration form$/) do
   @home = Home.new
   @home.load
   expect(@home.login_and_registration).to have_first_name
@@ -1132,9 +1204,9 @@ end
 This allows for pretty tests ...
 
 ```ruby
-Then /^there are lots of search_results$/ do
+Then(/^there are lots of search_results$/) do
   expect(@results_page.search_results.size).to eq(10)
-  
+
   @home.search_results.each do |result|
     expect(result).to have_title
     expect(result.blurb.text).not_to be_empty
@@ -1143,13 +1215,33 @@ end
 ```
 
 The css selector that is passed as the 3rd argument to the
-`sections` method ("#results li") is used to find a number of capybara
+`sections` method (`#results li`) is used to find a number of capybara
 elements. Each capybara element found using the css selector is used to
 create a new instance of `SearchResults` and becomes its root
 element. So if the css selector finds 3 `li` elements, calling
 `search_results` will return an array containing 3 instances of
 `SearchResults`, each with one of the `li` elements as it's root element.
 
+##### Accessing Within a Collection of Sections
+
+When using an iterator such as `each` to pass a block through to a collection of sections it is possible to skip using `within`.  However some caution is warranted when accessing the Sections directly from an array, as the block can only be executed when the section is being initialized.  The following does not work:
+
+```rb
+  @home.search_results.first do |result|
+    # This block is silently ignored.
+    expect(result).to have_title
+  end
+```
+Instead use `within` to access the inner-context of the Section.
+
+```rb
+  @home.search_results.first.within do |result|
+    # This block is run within the context of the Section.
+    expect(result).to have_title
+  end
+```
+
+
 #### Anonymous Section Collections
 
 You can define collections of anonymous sections the same way you would
@@ -1196,7 +1288,7 @@ Here's how to test for the existence of the section:
 This allows for some pretty tests ...
 
 ```ruby
-Then /^there are search results on the page$/ do
+Then(/^there are search results on the page$/) do
   expect(@home).to have_search_results
 end
 ```
@@ -1346,7 +1438,7 @@ instance of the class when created.
 
 ### Skipping load Validations
 
-Defined load validations can be skipped for one `load` call by 
+Defined load validations can be skipped for one `load` call by
 passing in `with_validations: false`.
 
 ```ruby
@@ -1397,7 +1489,7 @@ the validations will be performed in the following order:
 **NB:** `SitePrism::Page` **used to** include a default load validation on
 `page.displayed?` however for v3 this has been removed. It is therefore
 necessary to re-define this if you want to retain the behaviour
-from site_prism v2. See [UPGRADING.md](https://github.com/natritmeyer/site_prism/blob/master/UPGRADING.md#default-load-validations)
+from site_prism v2. See [UPGRADING.md](https://github.com/site-prism/site_prism/blob/master/UPGRADING.md#default-load-validations)
 for more info on this.
 
 ## Using Capybara Query Options
@@ -1447,7 +1539,7 @@ Now we can write pretty, non-failing tests without hard coding these options
 into our page and section classes:
 
 ```ruby
-Then /^there are search results on the page$/ do
+Then(/^there are search results on the page$/) do
   expect(@results_page).to have_search_results(count: 25)
 end
 ```
@@ -1580,7 +1672,7 @@ class Home < SitePrism::Page
 end
 
 # cucumber step that performs login
-When /^I log in$/ do
+When(/^I log in$/) do
   @home = Home.new
   @home.load
 
@@ -1611,14 +1703,14 @@ as per the code below
 
 ```ruby
 Capybara.configure do |config|
-  config.default_max_wait_time = 11 #=> Wait up to 11 seconds for all querys to fail
-  # or alternatively, if you don't want to ever wait
-  config.default_max_wait_time = 0 #=> Don't ever wait! 
+  config.default_max_wait_time = 11 #=> Wait up to 11 seconds for all queries to fail
+  # or if you don't want to ever wait
+  config.default_max_wait_time = 0 #=> Don't ever wait!
 end
 ```
 
 Note that even with implicit waits on you can dynamically modify the wait times
-in any SitePrism method to help work-around special circumstances.  
+in any SitePrism method to help work-around special circumstances.
 
 ```ruby
 # Option 1: using wait key assignment
@@ -1635,6 +1727,9 @@ end
 There's a SitePrism plugin called `site_prism.vcr` that lets you use
 SitePrism with the VCR gem. Check it out [HERE](https://github.com/dnesteryuk/site_prism.vcr)
 
+Note that as of 2016 this plugin doesn't appear to have been under active development. Also it is
+still pinned to the `2.x` series of site_prism so use it of your own accord.
+
 # Epilogue
 
 So, we've seen how to use SitePrism to put together page objects made up

data/lib/site_prism.rb

@@ -5,16 +5,21 @@ require 'addressable/template'
 
 module SitePrism
   autoload :AddressableUrlMatcher, 'site_prism/addressable_url_matcher'
-  autoload :Deprecator, 'site_prism/deprecator'
   autoload :DSL, 'site_prism/dsl'
+  autoload :Deprecator, 'site_prism/deprecator'
   autoload :ElementChecker, 'site_prism/element_checker'
-  autoload :RecursionChecker, 'site_prism/recursion_checker'
+  autoload :Loadable, 'site_prism/loadable'
   autoload :Logger, 'site_prism/logger'
   autoload :Page, 'site_prism/page'
+  autoload :RecursionChecker, 'site_prism/recursion_checker'
+  autoload :RspecMatchers, 'site_prism/rspec_matchers'
   autoload :Section, 'site_prism/section'
+  autoload :Timer, 'site_prism/timer'
   autoload :Waiter, 'site_prism/waiter'
 
   class << self
+    attr_reader :use_all_there_gem
+
     def configure
       yield self
     end
@@ -65,5 +70,13 @@ module SitePrism
     def log_level
       %i[DEBUG INFO WARN ERROR FATAL UNKNOWN][logger.level]
     end
+
+    # Whether you wish to use the new experimental all_there dependent gem
+    #   This will be enforced from site_prism v4 onwards as this is where
+    #   the development of this functionality will be focused
+    def use_all_there_gem=(value)
+      logger.debug("Setting use_all_there_gem to #{value}")
+      @use_all_there_gem = value
+    end
   end
 end

data/lib/site_prism/addressable_url_matcher.rb

@@ -98,7 +98,7 @@ module SitePrism
       begin
         Addressable::URI.parse(url)
       rescue Addressable::URI::InvalidURIError
-        SitePrism.logger.warn("Ensure you don't use templated port numbers.")
+        SitePrism.logger.fatal("Ensure you don't use templated port numbers.")
         raise SitePrism::InvalidUrlMatcherError
       end
     end

data/lib/site_prism/dsl.rb

@@ -8,6 +8,30 @@ module SitePrism
 
     private
 
+    # Call `find` inside context set on page/section
+    def _find(*find_args)
+      kwargs = find_args.pop
+      page.find(*find_args, **kwargs)
+    end
+
+    # Call `all` inside context set on page/section
+    def _all(*find_args)
+      kwargs = find_args.pop
+      page.all(*find_args, **kwargs)
+    end
+
+    # Call `has_selector?` inside context set on page/section
+    def element_exists?(*find_args)
+      kwargs = find_args.pop
+      page.has_selector?(*find_args, **kwargs)
+    end
+
+    # Call `has_no_selector?` inside context set on page/section
+    def element_does_not_exist?(*find_args)
+      kwargs = find_args.pop
+      page.has_no_selector?(*find_args, **kwargs)
+    end
+
     # The default waiting time set by Capybara's configuration settings.
     def wait_time
       Capybara.default_max_wait_time
@@ -36,6 +60,14 @@ module SitePrism
       raise SitePrism::UnsupportedBlockError
     end
 
+    # Warn users from naming the elements starting with no_
+    def warn_if_dsl_collision(obj, name)
+      return unless name.to_s.start_with?('no_')
+
+      SitePrism.logger.warn("#{obj.class}##{name} should not start with no_")
+      SitePrism::Deprecator.deprecate('Using no_ prefix in DSL definition')
+    end
+
     # Sanitize method called before calling any SitePrism DSL method or
     # meta-programmed method. This ensures that the Capybara query is correct.
     #
@@ -49,7 +81,7 @@ module SitePrism
 
       recombine_args(find_args, runtime_args, options)
 
-      return [*find_args, *runtime_args] if options.empty?
+      return [*find_args, *runtime_args, {}] if options.empty?
 
       [*find_args, *runtime_args, options]
     end
@@ -76,8 +108,10 @@ module SitePrism
       attr_reader :expected_items
 
       def element(name, *find_args)
+        SitePrism::Deprecator.deprecate('Passing a block to :element') if block_given?
         build(:element, name, *find_args) do
           define_method(name) do |*runtime_args, &element_block|
+            warn_if_dsl_collision(self, name)
             raise_if_block(self, name, !element_block.nil?, :element)
             _find(*merge_args(find_args, runtime_args))
           end
@@ -85,8 +119,10 @@ module SitePrism
       end
 
       def elements(name, *find_args)
+        SitePrism::Deprecator.deprecate('Passing a block to :elements') if block_given?
         build(:elements, name, *find_args) do
           define_method(name) do |*runtime_args, &element_block|
+            warn_if_dsl_collision(self, name)
             raise_if_block(self, name, !element_block.nil?, :elements)
             _all(*merge_args(find_args, runtime_args))
           end
@@ -101,6 +137,7 @@ module SitePrism
         section_class, find_args = extract_section_options(args, &block)
         build(:section, name, *find_args) do
           define_method(name) do |*runtime_args, &runtime_block|
+            warn_if_dsl_collision(self, name)
             section_element = _find(*merge_args(find_args, runtime_args))
             section_class.new(self, section_element, &runtime_block)
           end
@@ -120,6 +157,7 @@ module SitePrism
       end
 
       def iframe(name, klass, *args)
+        SitePrism.logger.debug('Block passed into iFrame construct at build time') if block_given?
         element_find_args = deduce_iframe_element_find_args(args)
         scope_find_args = deduce_iframe_scope_find_args(args)
         build(:iframe, name, *element_find_args) do
@@ -142,10 +180,11 @@ module SitePrism
       private
 
       def old_mapped_items
-        SitePrism.logger.debug("Calling .mapped_items on a class is changing!
-The structure will soon be changing format. This will allow #all_there? to
-recurse through section / sections going forwards. The old format will remain
-until a v5 has been released (And will be deprecated in v4 of site_prism")
+        SitePrism::Deprecator.soft_deprecate(
+          '.mapped_items on a class',
+          'To allow easier recursion through the items in conjunction with #all_there?',
+          '.mapped_items(legacy: false)'
+        )
         @old_mapped_items ||= []
       end
 
@@ -155,7 +194,7 @@ until a v5 has been released (And will be deprecated in v4 of site_prism")
           elements: [],
           section: [],
           sections: [],
-          iframe: [],
+          iframe: []
         }
       end
 
@@ -177,7 +216,7 @@ until a v5 has been released (And will be deprecated in v4 of site_prism")
       def add_helper_methods(name, *find_args)
         create_existence_checker(name, *find_args)
         create_nonexistence_checker(name, *find_args)
-        create_rspec_existence_matchers(name) if defined?(RSpec)
+        SitePrism::RspecMatchers.new(name)._create_rspec_existence_matchers if defined?(RSpec)
         create_visibility_waiter(name, *find_args)
         create_invisibility_waiter(name, *find_args)
       end
@@ -190,18 +229,6 @@ until a v5 has been released (And will be deprecated in v4 of site_prism")
         end
       end
 
-      def create_rspec_existence_matchers(element_name)
-        matcher = "has_#{element_name}?"
-        negated_matcher = "has_no_#{element_name}?"
-
-        RSpec::Matchers.define "have_#{element_name}" do |*args|
-          match { |actual| actual.public_send(matcher, *args) }
-          match_when_negated do |actual|
-            actual.public_send(negated_matcher, *args)
-          end
-        end
-      end
-
       def create_existence_checker(element_name, *find_args)
         method_name = "has_#{element_name}?"
         create_helper_method(method_name, *find_args) do
@@ -247,7 +274,7 @@ until a v5 has been released (And will be deprecated in v4 of site_prism")
       end
 
       def create_error_method(name)
-        SitePrism.logger.error("#{name} has come from an item with 0 locators.")
+        SitePrism.logger.error("#{name} has come from an item with no locators.")
         define_method(name) { raise SitePrism::InvalidElementError }
       end
 
@@ -273,7 +300,7 @@ until a v5 has been released (And will be deprecated in v4 of site_prism")
         return unless looks_like_xpath?(args[0])
 
         SitePrism.logger.warn('The arguments passed in look like xpath. Check your locators.')
-        SitePrism.logger.debug("Default locator: #{Capybara.default_selector}")
+        SitePrism.logger.debug("Default locator strategy: #{Capybara.default_selector}")
       end
 
       def looks_like_xpath?(arg)

data/lib/site_prism/element_checker.rb

@@ -24,7 +24,7 @@ module SitePrism
       if recursion == :none
         elements_to_check.all? { |name| there?(name) }
       elsif recursion == :one
-        RecursionChecker.new(self).all_there?
+        all_there_with_recursion
       else
         SitePrism.logger.debug("Input value '#{recursion}'. Valid values are :none or :one.")
         SitePrism.logger.error('Invalid recursion setting, Will not run #all_there?.')
@@ -35,8 +35,20 @@ module SitePrism
       _mapped_items.select { |name| there?(name) }
     end
 
+    def elements_missing
+      elements_to_check.reject { |name| there?(name) }
+    end
+
     private
 
+    def all_there_with_recursion
+      if SitePrism.use_all_there_gem
+        SitePrism::AllThere::RecursionChecker.new(self).all_there?
+      else
+        RecursionChecker.new(self).all_there?
+      end
+    end
+
     # If the page or section has expected_items set, return expected_items that are mapped
     # otherwise just return the list of all mapped_items
     def elements_to_check

data/lib/site_prism/error.rb

@@ -25,14 +25,6 @@ module SitePrism
   # Formerly known as `NoSelectorForElement`
   class InvalidElementError < SitePrismError; end
 
-  # A tool like Timecop is being used to "freeze time" by overriding Time.now
-  # and similar methods. In this case, our waiter functions won't work, because
-  # Time.now does not change.
-  # If you encounter this issue, check that you are not doing Timecop.freeze without
-  # an accompanying Timecop.return.
-  # Also check out Timecop.safe_mode https://github.com/travisjeffery/timecop#timecopsafe_mode
-  class FrozenInTimeError < SitePrismError; end
-
   # The condition that was being evaluated inside the block did not evaluate
   # to true within the time limit
   # Formerly known as `TimeoutException`

data/lib/site_prism/page.rb

@@ -1,9 +1,6 @@
 # frozen_string_literal: true
 
-require 'site_prism/loadable'
-
 module SitePrism
-  # rubocop:disable Metrics/ClassLength
   class Page
     include Capybara::DSL
     include ElementChecker
@@ -27,11 +24,7 @@ module SitePrism
     end
 
     def page
-      if defined?(@page)
-        @page
-      else
-        Capybara.current_session
-      end
+      (defined?(@page) && @page) || Capybara.current_session
     end
 
     # Loads the page.
@@ -79,18 +72,13 @@ module SitePrism
 
     def url_matches(seconds = wait_time)
       return unless displayed?(seconds)
+      return regexp_backed_matches if url_matcher.is_a?(Regexp)
 
-      if url_matcher.is_a?(Regexp)
-        regexp_backed_matches
-      else
-        template_backed_matches
-      end
+      template_backed_matches
     end
 
     def url(expansion = {})
-      return nil if self.class.url.nil?
-
-      Addressable::Template.new(self.class.url).expand(expansion).to_s
+      self.class.url && Addressable::Template.new(self.class.url).expand(expansion).to_s
     end
 
     def url_matcher
@@ -103,22 +91,6 @@ module SitePrism
 
     private
 
-    def _find(*find_args)
-      page.find(*find_args)
-    end
-
-    def _all(*find_args)
-      page.all(*find_args)
-    end
-
-    def element_exists?(*find_args)
-      page.has_selector?(*find_args)
-    end
-
-    def element_does_not_exist?(*find_args)
-      page.has_no_selector?(*find_args)
-    end
-
     def regexp_backed_matches
       url_matcher.match(page.current_url)
     end
@@ -167,5 +139,4 @@ module SitePrism
       end
     end
   end
-  # rubocop:enable Metrics/ClassLength
 end

data/lib/site_prism/recursion_checker.rb

@@ -28,7 +28,7 @@ module SitePrism
         expected(mapped_items, :elements),
         expected(mapped_items, :section),
         expected(mapped_items, :sections),
-        expected(mapped_items, :iframe),
+        expected(mapped_items, :iframe)
       ]
     end
 

data/lib/site_prism/rspec_matchers.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+module SitePrism
+  class RspecMatchers
+    attr_reader :element_name
+
+    def initialize(element_name)
+      @element_name = element_name
+    end
+
+    def _create_rspec_existence_matchers
+      SitePrism.logger.debug('Including all relevant matcher names / warnings in RSpec scope.')
+      create_rspec_existence_matchers(matcher, object_method, negated_object_method, warning)
+    end
+
+    private
+
+    def create_rspec_existence_matchers(matcher, object_method, negated_object_method, warning)
+      RSpec::Matchers.define(matcher) do |*args|
+        match { |actual| actual.public_send(object_method, *args) }
+        match_when_negated do |actual|
+          if actual.respond_to?(negated_object_method)
+            return actual.public_send(negated_object_method, *args)
+          end
+
+          SitePrism.logger.debug(warning)
+          !actual.public_send(object_method, *args)
+        end
+      end
+    end
+
+    def matcher
+      "have_#{element_name}"
+    end
+
+    def object_method
+      "has_#{element_name}?"
+    end
+
+    def negated_object_method
+      "has_no_#{element_name}?"
+    end
+
+    def warning
+      "The RSpec matcher '#{matcher}' was added by SitePrism, but the object under test "\
+        "does not respond to '#{negated_object_method}' and is probably not a SitePrism object. "\
+        'Falling back to the default RSpec matcher.'
+    end
+  end
+end

data/lib/site_prism/section.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'site_prism/loadable'
-
 module SitePrism
   class Section
     include Capybara::DSL
@@ -25,10 +23,14 @@ module SitePrism
         nil
     end
 
-    def initialize(parent, root_element)
+    def initialize(parent, root_element, &block)
       @parent = parent
       @root_element = root_element
-      Capybara.within(@root_element) { yield(self) } if block_given?
+      within(&block) if block_given?
+    end
+
+    def within
+      Capybara.within(@root_element) { yield(self) }
     end
 
     # Capybara::DSL module "delegates" Capybara methods to the "page" method
@@ -64,23 +66,5 @@ module SitePrism
     def native
       root_element.native
     end
-
-    private
-
-    def _find(*find_args)
-      page.find(*find_args)
-    end
-
-    def _all(*find_args)
-      page.all(*find_args)
-    end
-
-    def element_exists?(*find_args)
-      page.has_selector?(*find_args)
-    end
-
-    def element_does_not_exist?(*find_args)
-      page.has_no_selector?(*find_args)
-    end
   end
 end

data/lib/site_prism/timer.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module SitePrism
+  class Timer
+    attr_reader :wait_time
+
+    def self.run(wait_time, &block)
+      new(wait_time).run(&block)
+    end
+
+    def initialize(wait_time)
+      @wait_time = wait_time
+      @done = false
+    end
+
+    def done?
+      @done == true
+    end
+
+    def run
+      start
+      yield self
+    ensure
+      stop
+    end
+
+    def start
+      stop
+      return if wait_time.zero?
+
+      @done = false
+      @thread = Thread.start do
+        sleep wait_time
+        @done = true
+      end
+    end
+
+    def stop
+      if @thread
+        @thread.kill
+        @thread.join
+        @thread = nil
+      end
+      @done = true
+    end
+  end
+end

data/lib/site_prism/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SitePrism
-  VERSION = '3.3'
+  VERSION = '3.6'
 end

data/lib/site_prism/waiter.rb

@@ -2,33 +2,20 @@
 
 module SitePrism
   class Waiter
-    class << self
-      def wait_until_true(wait_time = Capybara.default_max_wait_time)
-        start_time = Time.now
+    def self.sleep_duration
+      0.05
+    end
 
+    def self.wait_until_true(wait_time = Capybara.default_max_wait_time)
+      Timer.run(wait_time) do |timer|
         loop do
           return true if yield
-          break if Time.now - start_time > wait_time
-
-          sleep(0.05)
+          break if timer.done?
 
-          check_for_time_stopped!(start_time)
+          sleep(sleep_duration)
         end
-
         raise SitePrism::TimeoutError, "Timed out after #{wait_time}s."
       end
-
-      private
-
-      def check_for_time_stopped!(start_time)
-        return unless start_time == Time.now
-
-        raise(
-          SitePrism::FrozenInTimeError,
-          'Time appears to be frozen. For more info, see ' \
-          'https://github.com/natritmeyer/site_prism/blob/master/lib/site_prism/error.rb'
-        )
-      end
     end
   end
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: site_prism
 version: !ruby/object:Gem::Version
-  version: '3.3'
+  version: '3.6'
 platform: ruby
 authors:
-- Nat Ritmeyer
 - Luke Hill
+- Nat Ritmeyer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-01 00:00:00.000000000 Z
+date: 2020-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -29,30 +29,42 @@ dependencies:
   name: capybara
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.8'
+    - - "<="
       - !ruby/object:Gem::Version
-        version: '3.3'
+        version: '3.29'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.8'
+    - - "<="
       - !ruby/object:Gem::Version
-        version: '3.3'
+        version: '3.29'
 - !ruby/object:Gem::Dependency
   name: site_prism-all_there
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
@@ -67,18 +79,32 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '12.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '12.3'
 - !ruby/object:Gem::Dependency
@@ -101,77 +127,95 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.67.0
+        version: 0.81.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.67.0
+        version: 0.81.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.5.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.5.1
 - !ruby/object:Gem::Dependency
-  name: selenium-webdriver
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: 1.33.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: 1.33.0
+- !ruby/object:Gem::Dependency
+  name: selenium-webdriver
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.9'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.9'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.1'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.16'
+        version: '0.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.16'
+        version: '0.17'
 - !ruby/object:Gem::Dependency
   name: webdrivers
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.9.1
+        version: '4.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.9.1
-description: |-
-  SitePrism gives you a simple,
-  clean and semantic DSL for describing your site.
-  SitePrism implements the Page Object Model pattern on top of Capybara.
+        version: '4.1'
+description: SitePrism gives you a simple, clean and semantic DSL for describing your
+  site. SitePrism implements the Page Object Model pattern on top of Capybara.
 email:
-- nat@natontesting.com
 - lukehill_uk@hotmail.com
+- nat@natontesting.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -188,20 +232,19 @@ files:
 - lib/site_prism/logger.rb
 - lib/site_prism/page.rb
 - lib/site_prism/recursion_checker.rb
+- lib/site_prism/rspec_matchers.rb
 - lib/site_prism/section.rb
+- lib/site_prism/timer.rb
 - lib/site_prism/version.rb
 - lib/site_prism/waiter.rb
-homepage: https://github.com/natritmeyer/site_prism
+homepage: https://github.com/site-prism/site_prism
 licenses:
 - BSD-3-Clause
 metadata:
-  bug_tracker_uri: https://github.com/natritmeyer/site_prism/issues
-  changelog_uri: https://github.com/natritmeyer/site_prism/blob/master/CHANGELOG.md
-  source_code_uri: https://github.com/natritmeyer/site_prism
-post_install_message: |-
-  site_prism has now moved to a new organisation to facilitate better
-  management of the codebase. The new organisation link is available at www.github.com/site-prism
-  and will house the site_prism gem plus new associated co-dependent gems.
+  bug_tracker_uri: https://github.com/site-prism/site_prism/issues
+  changelog_uri: https://github.com/site-prism/site_prism/blob/master/CHANGELOG.md
+  source_code_uri: https://github.com/site-prism/site_prism
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -209,15 +252,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.8
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: A Page Object Model DSL for Capybara