spreewald 4.6.5 → 4.6.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 391dac8d28db5141537172b358a00f971c397a8129c9ef1fa268610496c7a5c8
-  data.tar.gz: f941f249802922938a04aab2af437b077a561abad49256c1e2c75095accb5cbd
+  metadata.gz: ac93f0fef6e2be9998a2b9f1514469dcf25213c0e342d20fafd38e8328b382b5
+  data.tar.gz: d1701b593bc278db9c823ad15b9bfe462c3be2baa6fbc976f61807679bfe3f05
 SHA512:
-  metadata.gz: 4aa8806d50b6dc5b7adbf0d4bc30e869e69f6637f4a05d29d60ea655dac82daadab41ed3444b66ddd403c5e6eeac24eb7e0b9095d04accb4f70b981b58596f98
-  data.tar.gz: b8695184f2377cb6605f4c40f109758c5b7fea175ce37200ed5971965ee0ffe4a7c6c60368259bcee3514325edbee4bb6d79f0bac3e315a77596b2f298fc57e7
+  metadata.gz: de4329e2f458176854e0750c8e33520369c55085456fdb388e2c8e35c1fca2e623bcdb87851ef61375664303860fb03eaef61576031aa4c878526d4c22014912
+  data.tar.gz: a522c0aee98335099fb5ec7fbe4c9837ad31d5cc345696b3e5fdcb581b569c01c94ba981a70fb595efa6b9f87c3ab03ea074d3c6e17735da12e6902833612feb

data/.github/workflows/test.yml

@@ -20,13 +20,19 @@ jobs:
           gemfile: tests/rails-7_capybara-3/Gemfile
         - ruby: 3.3.10
           gemfile: tests/rails-8_capybara-3/Gemfile
+        - ruby: 3.4.9
+          gemfile: tests/rails-8_capybara-3/Gemfile
+        - ruby: 4.0.3
+          gemfile: tests/rails-8_capybara-3/Gemfile
         # Used to run rspec and integration tests for spreewald binary:
         - ruby: 3.2.0
-          gemfile: Gemfile.ruby_3_2_0
+          gemfile: Gemfile.ruby_3_2
         - ruby: 3.3.10
-          gemfile: Gemfile.ruby_3_3_10
+          gemfile: Gemfile.ruby_3_3
         - ruby: 3.4.9
-          gemfile: Gemfile.ruby_3_4_9
+          gemfile: Gemfile.ruby_3_4
+        - ruby: 4.0.3
+          gemfile: Gemfile.ruby_4_0
     env:
       BUNDLE_GEMFILE: ${{ matrix.gemfile }}
 

data/.ruby-version

@@ -1 +1 @@
-3.2.0
+4.0.3

data/CHANGELOG.md

@@ -3,6 +3,13 @@ All notable changes to this project will be documented in this file.
 
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## 4.6.6
+- Add support for Ruby 4.0
+
+## 4.6.5
+- Drop support for Ruby < 3.2
+- Allow Capybara to click on labels instead of inputs for checkboxes
+
 ## 4.6.4
 - New step `I attach the files ["file1", "file2" and "file3"] to "My Uploads"`
 

data/Gemfile

@@ -1,4 +1,5 @@
 source 'https://rubygems.org'
 
 gem 'rake'
+gem 'ostruct'
 gem 'gemika'

data/Gemfile.lock

@@ -2,6 +2,7 @@ GEM
   remote: https://rubygems.org/
   specs:
     gemika (0.8.1)
+    ostruct (0.6.3)
     rake (13.0.3)
 
 PLATFORMS
@@ -9,7 +10,8 @@ PLATFORMS
 
 DEPENDENCIES
   gemika
+  ostruct
   rake
 
 BUNDLED WITH
-   4.0.8
+  4.0.8

data/{Gemfile.ruby_3_2_0 → Gemfile.ruby_3_2}

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-ruby '3.2.0'
+ruby '~> 3.2'
 
 gemspec
 

data/{Gemfile.ruby_3_2_0.lock → Gemfile.ruby_3_2.lock}

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spreewald (4.6.5)
+    spreewald (4.6.6)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/{Gemfile.ruby_3_4_9 → Gemfile.ruby_3_3}

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-ruby '3.4.9'
+ruby '~> 3.3'
 
 gemspec
 

data/{Gemfile.ruby_3_3_10.lock → Gemfile.ruby_3_3.lock}

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spreewald (4.6.5)
+    spreewald (4.6.6)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/{Gemfile.ruby_3_3_10 → Gemfile.ruby_3_4}

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-ruby '3.3.10'
+ruby '~> 3.4'
 
 gemspec
 

data/{Gemfile.ruby_3_4_9.lock → Gemfile.ruby_3_4.lock}

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spreewald (4.6.5)
+    spreewald (4.6.6)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/Gemfile.ruby_4_0

@@ -0,0 +1,15 @@
+source 'https://rubygems.org'
+
+ruby '~> 4.0'
+
+gemspec
+
+gem 'rake'
+gem 'ostruct'
+gem 'rspec'
+gem 'cucumber'
+gem 'aruba'
+
+gem 'gemika'
+gem 'mail', '~> 2.8.0'
+gem 'nokogiri'

data/Gemfile.ruby_4_0.lock

@@ -0,0 +1,157 @@
+PATH
+  remote: .
+  specs:
+    spreewald (4.6.6)
+      capybara
+      cucumber
+      cucumber_priority (>= 0.3.0)
+      rspec (>= 2.13.0)
+      xpath
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    addressable (2.9.0)
+      public_suffix (>= 2.0.2, < 8.0)
+    aruba (2.4.0)
+      bundler (>= 1.17)
+      contracts (>= 0.16.0, < 0.18.0)
+      cucumber (>= 8.0, < 12.0)
+      irb (~> 1.16)
+      rspec-expectations (>= 3.4, < 5.0)
+      thor (~> 1.0)
+    base64 (0.3.0)
+    bigdecimal (4.1.2)
+    builder (3.3.0)
+    capybara (3.40.0)
+      addressable
+      matrix
+      mini_mime (>= 0.1.3)
+      nokogiri (~> 1.11)
+      rack (>= 1.6.0)
+      rack-test (>= 0.6.3)
+      regexp_parser (>= 1.5, < 3.0)
+      xpath (~> 3.2)
+    contracts (0.17.3)
+    cucumber (11.0.0)
+      base64 (~> 0.2)
+      builder (~> 3.2)
+      cucumber-ci-environment (> 9, < 12)
+      cucumber-core (>= 16.2.0, < 17)
+      cucumber-cucumber-expressions (> 17, < 20)
+      cucumber-html-formatter (> 21, < 24)
+      diff-lcs (~> 1.5)
+      logger (~> 1.6)
+      mini_mime (~> 1.1)
+      multi_test (~> 1.1)
+      sys-uname (~> 1.5)
+    cucumber-ci-environment (11.0.0)
+    cucumber-core (16.2.0)
+      cucumber-gherkin (> 36, < 40)
+      cucumber-messages (> 31, < 33)
+      cucumber-tag-expressions (> 6, < 9)
+    cucumber-cucumber-expressions (19.0.0)
+      bigdecimal
+    cucumber-gherkin (39.0.0)
+      cucumber-messages (>= 31, < 33)
+    cucumber-html-formatter (23.1.0)
+      cucumber-messages (> 23, < 33)
+    cucumber-messages (32.3.1)
+    cucumber-tag-expressions (8.1.0)
+    cucumber_priority (1.1.0)
+      cucumber
+    date (3.5.1)
+    diff-lcs (1.6.2)
+    erb (6.0.4)
+    ffi (1.17.4-x86_64-linux-gnu)
+    gemika (2.0.0)
+    io-console (0.8.2)
+    irb (1.18.0)
+      pp (>= 0.6.0)
+      prism (>= 1.3.0)
+      rdoc (>= 4.0.0)
+      reline (>= 0.4.2)
+    logger (1.7.0)
+    mail (2.8.1)
+      mini_mime (>= 0.1.1)
+      net-imap
+      net-pop
+      net-smtp
+    matrix (0.4.3)
+    memoist3 (1.0.0)
+    mini_mime (1.1.5)
+    multi_test (1.1.0)
+    net-imap (0.6.4)
+      date
+      net-protocol
+    net-pop (0.1.2)
+      net-protocol
+    net-protocol (0.2.2)
+      timeout
+    net-smtp (0.5.1)
+      net-protocol
+    nokogiri (1.19.3-x86_64-linux-gnu)
+      racc (~> 1.4)
+    ostruct (0.6.3)
+    pp (0.6.3)
+      prettyprint
+    prettyprint (0.2.0)
+    prism (1.9.0)
+    psych (5.3.1)
+      date
+      stringio
+    public_suffix (7.0.5)
+    racc (1.8.1)
+    rack (3.2.6)
+    rack-test (2.2.0)
+      rack (>= 1.3)
+    rake (13.4.2)
+    rdoc (7.2.0)
+      erb
+      psych (>= 4.0.0)
+      tsort
+    regexp_parser (2.12.0)
+    reline (0.6.3)
+      io-console (~> 0.5)
+    rspec (3.13.2)
+      rspec-core (~> 3.13.0)
+      rspec-expectations (~> 3.13.0)
+      rspec-mocks (~> 3.13.0)
+    rspec-core (3.13.6)
+      rspec-support (~> 3.13.0)
+    rspec-expectations (3.13.5)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-mocks (3.13.8)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.13.0)
+    rspec-support (3.13.7)
+    stringio (3.2.0)
+    sys-uname (1.5.1)
+      ffi (~> 1.1)
+      memoist3 (~> 1.0.0)
+    thor (1.5.0)
+    timeout (0.6.1)
+    tsort (0.2.0)
+    xpath (3.2.0)
+      nokogiri (~> 1.8)
+
+PLATFORMS
+  x86_64-linux
+
+DEPENDENCIES
+  aruba
+  cucumber
+  gemika
+  mail (~> 2.8.0)
+  nokogiri
+  ostruct
+  rake
+  rspec
+  spreewald!
+
+RUBY VERSION
+  ruby 4.0.3
+
+BUNDLED WITH
+  4.0.11

data/lib/spreewald/browser_tab_steps.rb

@@ -14,7 +14,7 @@ After do
 end
 
 # Opens [the page](https://github.com/makandra/spreewald/blob/master/examples/paths.rb) in a new browser tab and switches to it.
-When /^I open (.+?) in a new browser tab$/ do |page_name|
+When(/^I open (.+?) in a new browser tab$/) do |page_name|
   require_selenium!
   previous_handles_count = browser.window_handles.size
   relative_target_path = path_to(page_name)
@@ -39,14 +39,14 @@ When 'I close the browser tab' do
 end.overridable
 
 # Waits for the new browser tab to appear, then switches to it.
-When /^I switch to the new(?:ly opened)? browser tab$/ do
+When(/^I switch to the new(?:ly opened)? browser tab$/) do
   require_selenium!
   step 'there should be at least 2 browser tabs'
   browser.switch_to.window(browser.window_handles.last)
 end.overridable
 
 # Changes the browser context to the second-last browser tab.
-When /^I switch(?: back)? to the previous browser tab$/ do
+When(/^I switch(?: back)? to the previous browser tab$/) do
   require_selenium!
   previous_handle = browser.window_handles[-2] # Second last should be the previous browser tab
   previous_handle ||= browser.window_handles.first # Fall back if only one tab is left
@@ -69,21 +69,21 @@ end.overridable
 #     When I may open a new browser tab
 #       And I click on "Open link in current browser tab"
 #     Then I should not have opened a new browser tab
-Then /^I should( not)? have opened a new browser tab$/ do |negate|
+Then(/^I should( not)? have opened a new browser tab$/) do |negate|
   raise "you need to use the 'I may open a new tab' step beforehand" unless @previous_browser_tabs_count
   expected_browser_tab_count = negate ? @previous_browser_tabs_count : (@previous_browser_tabs_count + 1)
 
   step "there should be #{expected_browser_tab_count} browser tabs"
 end.overridable
 
-Then /^there should be (\d+) browser tabs?$/ do |expected_browser_tabs|
+Then(/^there should be (\d+) browser tabs?$/) do |expected_browser_tabs|
   patiently do
     actual_browser_tabs = browser.window_handles.size
     expect(actual_browser_tabs).to eq(expected_browser_tabs.to_i), "Expected #{expected_browser_tabs} browser tab to be open, but found #{actual_browser_tabs}"
   end
 end.overridable
 
-Then /^there should be at least (\d+) browser tabs?$/ do |expected_browser_tabs|
+Then(/^there should be at least (\d+) browser tabs?$/) do |expected_browser_tabs|
   patiently do
     actual_browser_tabs = browser.window_handles.size
     expect(actual_browser_tabs).to be >= (expected_browser_tabs.to_i), "Expected at least #{expected_browser_tabs} browser tab to be open, but found #{actual_browser_tabs}"

data/lib/spreewald/development_steps.rb

@@ -1,7 +1,7 @@
 # coding: UTF-8
 
 # Marks scenario as pending, optionally explained with a reason.
-Then /^it should work(.+?)?$/ do |message|
+Then(/^it should work(.+?)?$/) do |message|
   pending(message)
 end.overridable
 

data/lib/spreewald/email_steps.rb

@@ -8,7 +8,7 @@ Before do
   ActionMailer::Base.deliveries.clear
 end
 
-When /^I clear my e?mails$/ do
+When(/^I clear my e?mails$/) do
   ActionMailer::Base.deliveries.clear
 end.overridable
 
@@ -35,7 +35,7 @@ end.overridable
 # You may skip lines in the header.
 # Please note: In older versions of Spreewald, unmentioned body lines were skipped.
 # Now you have to use the asterisk explicitly to omit lines in the body.
-Then /^(an?|no)( HTML| plain-text|) e?mail should have been sent with:$/ do |mode, type, raw_data|
+Then(/^(an?|no)( HTML| plain-text|) e?mail should have been sent with:$/) do |mode, type, raw_data|
   patiently do
     results = MailFinder.find(raw_data, type.strip)
 
@@ -73,28 +73,28 @@ end.overridable
 
 # Please note that this step will only follow HTTP and HTTPS links.
 # Other links (such as mailto: or ftp:// links) are ignored.
-When /^I follow the (first|second|third)? ?link in the e?mail$/ do |index_in_words|
+When(/^I follow the (first|second|third)? ?link in the e?mail$/) do |index_in_words|
   mail = @mail || ActionMailer::Base.deliveries.last
   Spreewald::Steps::FollowTheLink.new(mail, index_in_words).run
 end.overridable
 
-Then /^no e?mail should have been sent$/ do
+Then(/^no e?mail should have been sent$/) do
   expect(ActionMailer::Base.deliveries).to be_empty
 end.overridable
 
 # Checks that the last sent email includes some text
-Then /^I should see "([^\"]*)" in the( HTML| plain-text|) e?mail$/ do |text, type|
+Then(/^I should see "([^\"]*)" in the( HTML| plain-text|) e?mail$/) do |text, type|
   expect(MailFinder.email_text_body(ActionMailer::Base.deliveries.last, type.strip)).to include(text)
 end.overridable
 
 # Print all sent emails to STDOUT (optionally only the headers).
-Then /^show me the e?mail( header)?s$/ do |only_header|
+Then(/^show me the e?mail( header)?s$/) do |only_header|
   Spreewald::Steps::ShowMeTheMails.new(ActionMailer::Base.deliveries, only_header).run
 end.overridable
 
 # Print a subset of all sent emails to STDOUT
 # This uses the same syntax as `Then an email should have been sent with:`
-Then /^show me the e?mail( header)?s with:$/ do |only_header, raw_data|
+Then(/^show me the e?mail( header)?s with:$/) do |only_header, raw_data|
   results = MailFinder.find(raw_data)
   if results.empty?
     if results.matching_header.empty?

data/lib/spreewald/frame_steps.rb

@@ -6,7 +6,7 @@
 #     Then I should see "Cherry" inside the fruits iframe
 #     When I press "Save" inside the 2nd iframe
 #
-When /^(.*?) inside the (.*?) iframe$/ do |nested_step, frame_identifier|
+When(/^(.*?) inside the (.*?) iframe$/) do |nested_step, frame_identifier|
   patiently do
     frame = find_frame(frame_identifier)
     page.within_frame(frame) do
@@ -16,7 +16,7 @@ When /^(.*?) inside the (.*?) iframe$/ do |nested_step, frame_identifier|
 end.overridable
 
 # nodoc
-When /^(.*?) inside the (.*?) iframe:$/ do |nested_step, frame_identifier, table_or_string|
+When(/^(.*?) inside the (.*?) iframe:$/) do |nested_step, frame_identifier, table_or_string|
   patiently do
     frame = find_frame(frame_identifier)
     page.within_frame(frame) do

data/lib/spreewald/session_steps.rb

@@ -4,14 +4,14 @@
 # You may need to update other steps to allow multiple sessions (e.g. your
 # authentication steps have to support multiple logged in users).
 # More details [here](https://makandracards.com/makandra/474480-how-to-make-a-cucumber-test-work-with-multiple-browser-sessions).
-When /^(.*) in the browser session "([^"]+)"$/ do |nested_step, session_name|
+When(/^(.*) in the browser session "([^"]+)"$/) do |nested_step, session_name|
   Capybara.using_session(session_name) do
     step(nested_step)
   end
 end.overridable(priority: 5)
 
 # nodoc
-When /^(.*) in the browser session "([^"]+)":$/ do |nested_step, session_name, table_or_string|
+When(/^(.*) in the browser session "([^"]+)":$/) do |nested_step, session_name, table_or_string|
   Capybara.using_session(session_name) do
     step("#{nested_step}:", table_or_string)
   end

data/lib/spreewald/table_steps.rb

@@ -120,7 +120,7 @@ World(TableStepsHelper)
 # Check the content of tables in your HTML.
 #
 # See [this article](https://makandracards.com/makandra/763-cucumber-step-to-match-table-rows-with-capybara) for details.
-Then /^I should( not)? see a table with (exactly )?the following rows( in any order)?:?$/ do |negate, exactly, unordered, expected_table|
+Then(/^I should( not)? see a table with (exactly )?the following rows( in any order)?:?$/) do |negate, exactly, unordered, expected_table|
   patiently do
     document = Nokogiri::HTML(page.body)
     tables = document.xpath('//table').collect { |table| table.xpath('.//tr').collect { |row| row.xpath('.//th|td') } }

data/lib/spreewald/web_steps.rb

@@ -44,7 +44,7 @@ require 'cgi'
 # Example:
 #
 #     Then I should see "some text" within ".page_body"
-When /^(.*) within (.*[^:])$/ do |nested_step, parent|
+When(/^(.*) within (.*[^:])$/) do |nested_step, parent|
   selector = _selector_for(parent)
   if selector.is_a?(String) || selector.is_a?(Array) # could also be a Capybara::Node::Element
     patiently do
@@ -58,21 +58,21 @@ When /^(.*) within (.*[^:])$/ do |nested_step, parent|
 end.overridable(:priority => -1)
 
 # nodoc
-When /^(.*) within (.*[^:]):$/ do |nested_step, parent, table_or_string|
+When(/^(.*) within (.*[^:]):$/) do |nested_step, parent, table_or_string|
   patiently do
     with_scope(parent) { step("#{nested_step}:", table_or_string) }
   end
 end.overridable(:priority => -1)
 
-Given /^(?:|I )am on (.+)$/ do |page_name|
+Given(/^(?:|I )am on (.+)$/) do |page_name|
   visit _path_to(page_name)
 end.overridable
 
-When /^(?:|I )go to (.+)$/ do |page_name|
+When(/^(?:|I )go to (.+)$/) do |page_name|
   visit _path_to(page_name)
 end.overridable
 
-Then /^(?:|I )should be on (.+)$/ do |page_name|
+Then(/^(?:|I )should be on (.+)$/) do |page_name|
   patiently do
     fragment = URI.parse(current_url).fragment
     fragment.sub!(/[#?].*/, '') if fragment # most js frameworks will usually use ? and # for params, we dont care about those
@@ -88,20 +88,20 @@ Then /^(?:|I )should be on (.+)$/ do |page_name|
   end
 end.overridable
 
-When /^(?:|I )press "([^"]*)"$/ do |button|
+When(/^(?:|I )press "([^"]*)"$/) do |button|
   patiently do
     click_button(button)
   end
 end.overridable
 
-When /^(?:|I )follow "([^"]*)"$/ do |link|
+When(/^(?:|I )follow "([^"]*)"$/) do |link|
   patiently do
     click_link(link)
   end
 end.overridable
 
 # Fill in text field
-When /^(?:|I )fill in "([^"]*)" (?:with|for) "([^"]*)"$/ do |field, value|
+When(/^(?:|I )fill in "([^"]*)" (?:with|for) "([^"]*)"$/) do |field, value|
   patiently do
     fill_in(field, :with => value)
   end
@@ -118,49 +118,49 @@ end.overridable
 #     Banana
 #     Pear
 #     """
-When /^(?:|I )fill in "([^"]*)" (?:with|for):$/ do |field, value|
+When(/^(?:|I )fill in "([^"]*)" (?:with|for):$/) do |field, value|
   patiently do
     fill_in(field, :with => value)
   end
 end.overridable
 
 # Fill in text field
-When /^(?:|I )fill in "([^"]*)" (?:with|for) '(.*)'$/ do |field, value|
+When(/^(?:|I )fill in "([^"]*)" (?:with|for) '(.*)'$/) do |field, value|
   patiently do
     fill_in(field, :with => value)
   end
 end.overridable
 
 # Select from select box
-When /^(?:|I )select "([^"]*)" from "([^"]*)"$/ do |value, field|
+When(/^(?:|I )select "([^"]*)" from "([^"]*)"$/) do |value, field|
   patiently do
     select(value, :from => field)
   end
 end.overridable
 
 # Check a checkbox
-When /^(?:|I )check "([^"]*)"$/ do |field|
+When(/^(?:|I )check "([^"]*)"$/) do |field|
   patiently do
     check(field, allow_label_click: true)
   end
 end.overridable
 
 # Uncheck a checkbox
-When /^(?:|I )uncheck "([^"]*)"$/ do |field|
+When(/^(?:|I )uncheck "([^"]*)"$/) do |field|
   patiently do
     uncheck(field, allow_label_click: true)
   end
 end.overridable
 
 # Select a radio button
-When /^(?:|I )choose "([^"]*)"$/ do |field|
+When(/^(?:|I )choose "([^"]*)"$/) do |field|
   patiently do
     choose(field)
   end
 end.overridable
 
 # Attach a file to a file upload form field
-When /^(?:|I )attach the file "([^"]*)" to "([^"]*)"$/ do |path, field|
+When(/^(?:|I )attach the file "([^"]*)" to "([^"]*)"$/) do |path, field|
   options = {}
   options[:make_visible] = true if javascript_capable?
 
@@ -188,7 +188,7 @@ end.overridable
 # Checks that some text appears on the page
 #
 # Note that this does not detect if the text might be hidden via CSS
-Then /^(?:|I )should( not)? see "([^"]*)"$/ do |negate, text|
+Then(/^(?:|I )should( not)? see "([^"]*)"$/) do |negate, text|
   expectation = negate ? :not_to : :to
 
   patiently do
@@ -199,7 +199,7 @@ end.overridable
 # Checks that a regexp appears on the page
 #
 # Note that this does not detect if the text might be hidden via CSS
-Then /^(?:|I )should( not)? see \/(.*)\/$/ do |negate, regexp|
+Then(/^(?:|I )should( not)? see \/(.*)\/$/) do |negate, regexp|
   expectation = negate ? :not_to : :to
   regexp = Regexp.new(regexp)
 
@@ -209,7 +209,7 @@ Then /^(?:|I )should( not)? see \/(.*)\/$/ do |negate, regexp|
 end.overridable
 
 # Checks for the existance of an input field (given its id or label)
-Then /^I should( not)? see a field "([^"]*)"$/ do |negate, name|
+Then(/^I should( not)? see a field "([^"]*)"$/) do |negate, name|
   if negate
     expect(page).to have_no_field(name)
   else
@@ -224,7 +224,7 @@ end.overridable
 # Checks for an unexpected minus sign, correct decimal places etc.
 #
 # See [here](https://makandracards.com/makandra/1225-test-that-a-number-or-money-amount-is-shown-with-cucumber) for details
-Then /^I should( not)? see the (?:number|amount) ([\-\d,\.]+)(?: (.*?))?$/ do |negate, amount, unit|
+Then(/^I should( not)? see the (?:number|amount) ([\-\d,\.]+)(?: (.*?))?$/) do |negate, amount, unit|
   expect_to_match = negate ? :not_to : :to
   is_negative = amount.start_with?('-')
   absolute_amount = amount.gsub(/^-/, '')
@@ -246,7 +246,7 @@ end.overridable(priority: -5) # priority lower than within
 
 # Like `Then I should see`, but with single instead of double quotes. In case
 # the expected string contains quotes as well.
-Then /^(?:|I )should( not)? see '([^']*)'$/ do |negate, text|
+Then(/^(?:|I )should( not)? see '([^']*)'$/) do |negate, text|
   expectation = negate ? :not_to : :to
 
   patiently do
@@ -255,7 +255,7 @@ Then /^(?:|I )should( not)? see '([^']*)'$/ do |negate, text|
 end.overridable
 
 # Check that the raw HTML contains a string
-Then /^I should( not)? see "([^\"]*)" in the HTML$/ do |negate, text|
+Then(/^I should( not)? see "([^\"]*)" in the HTML$/) do |negate, text|
   expectation = negate ? :not_to : :to
 
   patiently do
@@ -264,7 +264,7 @@ Then /^I should( not)? see "([^\"]*)" in the HTML$/ do |negate, text|
 end.overridable
 
 # Checks that status code is 400..599
-Then /^I should see an error$/ do
+Then(/^I should see an error$/) do
   expect((400 .. 599)).to include(page.status_code)
 end.overridable
 
@@ -276,7 +276,7 @@ end.overridable
 #     Then I should see the element ".panel"
 #     Then I should not see an element ".sidebar"
 #     Then I should not see the element ".sidebar"
-Then /^I should( not)? see (?:an|the) element "([^"]+)"$/ do |negate, selector|
+Then(/^I should( not)? see (?:an|the) element "([^"]+)"$/) do |negate, selector|
   expectation = negate ? :not_to : :to
   patiently do
     expect(page).send(expectation, have_css(selector))
@@ -291,7 +291,7 @@ end.overridable
 #     Then I should see the element for the panel
 #     Then I should not see an element for the sidebar
 #     Then I should not see the element for the sidebar
-Then /^I should( not)? see (?:an|the) element for (.*?)$/ do |negate, locator|
+Then(/^I should( not)? see (?:an|the) element for (.*?)$/) do |negate, locator|
   expectation = negate ? :not_to : :to
   selector = _selector_for(locator)
   patiently do
@@ -309,7 +309,7 @@ end.overridable(:priority => -5) # priority must be lower than the "within" step
 #       | Augsburg    |
 #       | Berlin      |
 #       | Beta Group  |
-Then /^I should see in this order:?$/ do |text|
+Then(/^I should see in this order:?$/) do |text|
   if text.is_a?(String)
     lines = text.split(/\n/)
   else
@@ -324,14 +324,14 @@ Then /^I should see in this order:?$/ do |text|
 end.overridable
 
 # Checks that the page contains a link with a given text or title attribute.
-Then /^I should( not)? see a link labeled "([^"]*)"$/ do |negate, label|
+Then(/^I should( not)? see a link labeled "([^"]*)"$/) do |negate, label|
   expectation = negate ? :not_to : :to
   link = page.first('a', :text => label, minimum: 0) || page.first(%(a[title="#{label}"]), minimum: 0)
   expect(link).send(expectation, be_present)
 end.overridable
 
 # Checks that an input field contains some value (allowing * as wildcard character)
-Then /^the "([^"]*)" field should( not)? contain "([^"]*)"$/ do |label, negate, expected_string|
+Then(/^the "([^"]*)" field should( not)? contain "([^"]*)"$/) do |label, negate, expected_string|
   patiently do
     field = find_with_disabled(:field, label)
     field_value = case field.tag_name
@@ -365,7 +365,7 @@ end.overridable
 #     Then I should see a form with the following values:
 #       | E-mail | foo@bar.com   |
 #       | Role   | Administrator |
-Then /^I should see a form with the following values:$/ do |table|
+Then(/^I should see a form with the following values:$/) do |table|
   expectations = table.raw
   expectations.each do |label, expected_value|
     step %(the "#{label}" field should contain "#{expected_value}")
@@ -373,7 +373,7 @@ Then /^I should see a form with the following values:$/ do |table|
 end.overridable
 
 # Checks that an input field was wrapped with a validation error
-Then /^the "([^"]*)" field should have the error "([^"]*)"$/ do |field, error_message|
+Then(/^the "([^"]*)" field should have the error "([^"]*)"$/) do |field, error_message|
   patiently do
     element = find_with_disabled(:field, field)
     field_error_finder = Spreewald::FieldErrorFinder.new(page, element)
@@ -404,7 +404,7 @@ Then /^the "([^"]*)" field should have the error "([^"]*)"$/ do |field, error_me
   end
 end.overridable
 
-Then /^the "([^\"]*)" field should( not)? have an error$/ do |label, negate|
+Then(/^the "([^\"]*)" field should( not)? have an error$/) do |label, negate|
   patiently do
     expectation = negate ? :not_to : :to
     element = find_with_disabled(:field, label)
@@ -414,7 +414,7 @@ Then /^the "([^\"]*)" field should( not)? have an error$/ do |label, negate|
   end
 end.overridable
 
-Then /^the "([^"]*)" checkbox should( not)? be checked?$/ do |label, negate|
+Then(/^the "([^"]*)" checkbox should( not)? be checked?$/) do |label, negate|
   expectation = negate ? :not_to : :to
 
   patiently do
@@ -423,7 +423,7 @@ Then /^the "([^"]*)" checkbox should( not)? be checked?$/ do |label, negate|
   end
 end.overridable
 
-Then /^the radio button "([^"]*)" should( not)? be (?:checked|selected)$/ do |field, negate|
+Then(/^the radio button "([^"]*)" should( not)? be (?:checked|selected)$/) do |field, negate|
   patiently do
     expect(page.send((negate ? :has_no_checked_field? : :has_checked_field?), field)).to eq(true)
   end
@@ -436,7 +436,7 @@ end.overridable
 #       | currency_code | EUR |
 #
 # Succeeds when the URL contains the given `locale` and `currency_code` params
-Then /^(?:|I )should have the following query string:$/ do |expected_pairs|
+Then(/^(?:|I )should have the following query string:$/) do |expected_pairs|
   patiently do
     query = URI.parse(current_url).query
     actual_params = query ? CGI.parse(query) : {}
@@ -448,7 +448,7 @@ Then /^(?:|I )should have the following query string:$/ do |expected_pairs|
 end.overridable
 
 # Open the current Capybara page using the `launchy` or `capybara_screenshot` gem
-Then /^show me the page$/ do
+Then(/^show me the page$/) do
   if defined? Capybara::Screenshot
     screenshot_and_save_page
   elsif defined? Launchy
@@ -460,14 +460,14 @@ end.overridable
 
 
 # Checks `Content-Type` HTTP header
-Then /^I should get a response with content-type "([^\"]*)"$/ do |expected_content_type|
+Then(/^I should get a response with content-type "([^\"]*)"$/) do |expected_content_type|
   expect(page.response_headers['Content-Type']).to match /\A#{Regexp.quote(expected_content_type)}($|;)/
 end.overridable
 
 # Checks `Content-Disposition` HTTP header
 #
 # Attention: Doesn't work with Selenium, see https://github.com/jnicklas/capybara#gotchas
-Then /^I should get a download with filename "([^\"]*)"$/ do |filename|
+Then(/^I should get a download with filename "([^\"]*)"$/) do |filename|
   content_disposition = page.response_headers['Content-Disposition']
   expect(content_disposition).to be_present
 
@@ -483,11 +483,11 @@ Then /^I should get a download with filename "([^\"]*)"$/ do |filename|
 end.overridable
 
 # Checks that a certain option is selected for a text field
-Then /^"([^"]*)" should( not)? be selected for "([^"]*)"$/ do |value, invert, field|
+Then(/^"([^"]*)" should( not)? be selected for "([^"]*)"$/) do |value, invert, field|
   step %(the "#{field}" field should#{invert} contain "#{value}")
 end.overridable
 
-Then /^nothing should be selected for "([^"]*)"$/ do |field|
+Then(/^nothing should be selected for "([^"]*)"$/) do |field|
   patiently do
     select = find_with_disabled(:field, field)
     begin
@@ -500,7 +500,7 @@ Then /^nothing should be selected for "([^"]*)"$/ do |field|
 end.overridable
 
 # Checks for the presence of an option in a select
-Then /^"([^"]*)" should( not)? be an option for "([^"]*)"$/ do |value, negate, field|
+Then(/^"([^"]*)" should( not)? be an option for "([^"]*)"$/) do |value, negate, field|
   finder_arguments = if Spreewald::Comparison.compare_versions(Capybara::VERSION, :<, "2.12")
     { text: value }
   else
@@ -518,7 +518,7 @@ Then /^"([^"]*)" should( not)? be an option for "([^"]*)"$/ do |value, negate, f
   end
 end.overridable
 
-Then /^the window should be titled "([^"]*)"$/ do |title|
+Then(/^the window should be titled "([^"]*)"$/) do |title|
   patiently do
     if Spreewald::Comparison.compare_versions(Capybara::VERSION, :<, "2.1")
       expect(page).to have_selector("title", :content => title)
@@ -528,7 +528,7 @@ Then /^the window should be titled "([^"]*)"$/ do |title|
   end
 end.overridable
 
-When /^I reload the page$/ do
+When(/^I reload the page$/) do
   if javascript_capable?
     page.execute_script(<<-JAVASCRIPT)
         window.location.reload(true);
@@ -543,7 +543,7 @@ end.overridable
 # In a non-selenium test, we only check for `.hidden`, `.invisible` or `style: display:none`
 #
 # More details [here](https://makandracards.com/makandra/1049-capybara-check-that-a-page-element-is-hidden-via-css)
-Then /^(the tag )?"([^\"]+)" should be visible$/ do |tag, selector_or_text|
+Then(/^(the tag )?"([^\"]+)" should be visible$/) do |tag, selector_or_text|
   options = {}
   tag ? options.store(:selector, selector_or_text) : options.store(:text, selector_or_text)
 
@@ -553,7 +553,7 @@ end.overridable
 # Checks that an element is actually present and hidden, also considering styles.
 # Within a selenium test, the browser is asked whether the element is really hidden.
 # In a non-selenium test, we only check for `.hidden`, `.invisible` or `style: display:none`
-Then /^(the tag )?"([^\"]+)" should be hidden$/ do |tag, selector_or_text|
+Then(/^(the tag )?"([^\"]+)" should be hidden$/) do |tag, selector_or_text|
   options = {}
   tag ? options.store(:selector, selector_or_text) : options.store(:text, selector_or_text)
 
@@ -566,7 +566,7 @@ end.overridable
 #
 #     When I click on "Collapse"
 #
-When /^I click on "([^\"]+)"$/ do |text|
+When(/^I click on "([^\"]+)"$/) do |text|
   patiently do
     contains_text = %{contains(., \"#{text}\")}
     # find the innermost selector that matches
@@ -581,7 +581,7 @@ end.overridable
 #
 #     When I click on the element ".sidebar"
 #
-When /^I click on the element "([^"]+)"$/ do |selector|
+When(/^I click on the element "([^"]+)"$/) do |selector|
   patiently do
     page.find(selector).click
   end
@@ -592,7 +592,7 @@ end.overridable
 # Example:
 #
 #     When I click on the element for the sidebar
-When /^I click on the element for (.+?)$/ do |locator|
+When(/^I click on the element for (.+?)$/) do |locator|
   patiently do
     selector = _selector_for(locator)
     args, kwargs = deconstruct_selector(selector)
@@ -609,7 +609,7 @@ end.overridable(priority: -5) # priority lower than within
 # Don't forget the trailing slash. Otherwise you'll get the error
 #   expected: /http:\/\/makandra.com(\?[^\/]*)?$/
 #        got: "http://makandra.com/" (using =~)
-Then /^"([^"]*)" should link to "([^"]*)"$/ do |link_label, target|
+Then(/^"([^"]*)" should link to "([^"]*)"$/) do |link_label, target|
   patiently do
     link = find_link(link_label)
     expect(link[:href]).to match(/#{Regexp.escape target}(\?[^\/]*)?$/) # ignore trailing timestamps
@@ -622,17 +622,17 @@ end.overridable
 # Example:
 #
 #     When I follow "Read more" inside any ".text_snippet"
-When /^I follow "([^"]*)" inside any "([^"]*)"$/ do |label, selector|
+When(/^I follow "([^"]*)" inside any "([^"]*)"$/) do |label, selector|
   node = find("#{selector} a", :text => label)
   node.click
 end.overridable
 
-Then /^I should( not)? see "([^"]*)" inside any "([^"]*)"$/ do |negate, text, selector|
+Then(/^I should( not)? see "([^"]*)" inside any "([^"]*)"$/) do |negate, text, selector|
   expectation = negate ? :not_to : :to
   expect(page).send(expectation, have_css(selector, :text => text))
 end.overridable
 
-When /^I fill in "([^"]*)" with "([^"]*)" inside any "([^"]*)"$/ do |field, value, selector|
+When(/^I fill in "([^"]*)" with "([^"]*)" inside any "([^"]*)"$/) do |field, value, selector|
   containers = all(:css, selector)
   input = nil
   containers.detect do |container|
@@ -647,19 +647,19 @@ When /^I fill in "([^"]*)" with "([^"]*)" inside any "([^"]*)"$/ do |field, valu
   end
 end.overridable
 
-When /^I confirm the browser dialog$/ do
+When(/^I confirm the browser dialog$/) do
   patiently do
     browser.switch_to.alert.accept
   end
 end.overridable
 
-When /^I cancel the browser dialog$/ do
+When(/^I cancel the browser dialog$/) do
   patiently do
     browser.switch_to.alert.dismiss
   end
 end.overridable
 
-When /^I enter "([^"]*)" into the browser dialog$/ do |text|
+When(/^I enter "([^"]*)" into the browser dialog$/) do |text|
   patiently do
     alert = browser.switch_to.alert
     alert.send_keys(text)
@@ -668,7 +668,7 @@ When /^I enter "([^"]*)" into the browser dialog$/ do |text|
 end.overridable
 
 # Tests that an input, button, checkbox or radio button with the given label is disabled.
-Then /^the "([^\"]*)" (field|button|checkbox|radio button) should( not)? be disabled$/ do |label, kind, negate|
+Then(/^the "([^\"]*)" (field|button|checkbox|radio button) should( not)? be disabled$/) do |label, kind, negate|
   patiently do
     element = if kind == 'button'
       find_with_disabled(:button, label)
@@ -685,7 +685,7 @@ Then /^the "([^\"]*)" (field|button|checkbox|radio button) should( not)? be disa
 end.overridable
 
 # Tests that a field with the given label is visible.
-Then /^the "([^\"]*)" field should( not)? be visible$/ do |label, hidden|
+Then(/^the "([^\"]*)" field should( not)? be visible$/) do |label, hidden|
   if Spreewald::Comparison.compare_versions(Capybara::VERSION, :<, "2.1")
     field = find_with_disabled(:field, label)
   else
@@ -706,7 +706,7 @@ end.overridable
 # Performs HTTP basic authentication with the given credentials and visits the given path.
 #
 # More details [here](https://makandracards.com/makandra/971-perform-http-basic-authentication-in-cucumber).
-When /^I perform basic authentication as "([^\"]*)\/([^\"]*)" and go to (.*)$/ do |user, password, page_name|
+When(/^I perform basic authentication as "([^\"]*)\/([^\"]*)" and go to (.*)$/) do |user, password, page_name|
   path = _path_to(page_name)
   if javascript_capable?
     server = Capybara.current_session.server rescue Capybara.current_session.driver.rack_server
@@ -724,7 +724,7 @@ When /^I perform basic authentication as "([^\"]*)\/([^\"]*)" and go to (.*)$/ d
 end.overridable
 
 # Goes to the previously viewed page.
-When /^I go back$/ do
+When(/^I go back$/) do
   if javascript_capable?
     page.execute_script('window.history.back()')
   else
@@ -738,7 +738,7 @@ end.overridable
 
 # Tests whether a select field is sorted. Uses Array#natural_sort, if defined;
 # Array#sort else.
-Then /^the "(.*?)" select should( not)? be sorted$/ do |label, negate|
+Then(/^the "(.*?)" select should( not)? be sorted$/) do |label, negate|
   select = find_with_disabled(:field, label)
   options = select.all('option').reject { |o| o.value.blank? }
   option_texts = options.collect(&:text)

data/lib/spreewald_support/version.rb

@@ -1,3 +1,3 @@
 module Spreewald
-  VERSION = '4.6.5'
+  VERSION = '4.6.6'
 end

data/spec/steps/follow_the_link_spec.rb

@@ -12,8 +12,8 @@ describe Spreewald::Steps::FollowTheLink do
       end
     end
 
-    step = ->() { Spreewald::Steps::FollowTheLink.new(mail_without_links, "first").run }
-    expect(step).to raise_error Spreewald::Steps::FollowTheLink::NoVisitableLinkFound
+    step = Spreewald::Steps::FollowTheLink.new(mail_without_links, "first")
+    expect { step.run }.to raise_error Spreewald::Steps::FollowTheLink::NoVisitableLinkFound
   end
 
   it "finds links in multipart html email" do

data/tests/rails-7_capybara-3/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../..
   specs:
-    spreewald (4.6.5)
+    spreewald (4.6.6)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)

data/tests/rails-8_capybara-3/Gemfile

@@ -6,6 +6,7 @@ gem 'sqlite3'
 gem 'haml-rails'
 gem 'gemika'
 gem 'rexml'
+gem 'cgi'
 gem 'webrick'
 
 group :development, :test do
@@ -17,7 +18,7 @@ group :test do
   gem 'database_cleaner'
   gem 'capybara', "~>3"
   gem 'capybara-screenshot'
-  gem 'cucumber-rails', require: false
+  gem 'cucumber-rails', '>= 4', require: false
   gem 'cucumber'
   gem 'selenium-webdriver'
   gem 'rspec-rails'

data/tests/rails-8_capybara-3/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../..
   specs:
-    spreewald (4.6.5)
+    spreewald (4.6.6)
       capybara
       cucumber
       cucumber_priority (>= 0.3.0)
@@ -101,40 +101,41 @@ GEM
     capybara-screenshot (1.0.26)
       capybara (>= 1.0, < 4)
       launchy
+    cgi (0.5.1)
     childprocess (5.1.0)
       logger (~> 1.5)
     coderay (1.1.3)
     concurrent-ruby (1.3.4)
     connection_pool (2.4.1)
     crass (1.0.6)
-    cucumber (9.2.0)
+    cucumber (10.2.0)
+      base64 (~> 0.2)
       builder (~> 3.2)
-      cucumber-ci-environment (> 9, < 11)
-      cucumber-core (> 13, < 14)
-      cucumber-cucumber-expressions (~> 17.0)
-      cucumber-gherkin (> 24, < 28)
-      cucumber-html-formatter (> 20.3, < 22)
-      cucumber-messages (> 19, < 25)
+      cucumber-ci-environment (> 9, < 12)
+      cucumber-core (> 15, < 17)
+      cucumber-cucumber-expressions (> 17, < 20)
+      cucumber-html-formatter (> 21, < 23)
       diff-lcs (~> 1.5)
+      logger (~> 1.6)
       mini_mime (~> 1.1)
       multi_test (~> 1.1)
-      sys-uname (~> 1.2)
+      sys-uname (~> 1.3)
     cucumber-ci-environment (10.0.1)
-    cucumber-core (13.0.3)
-      cucumber-gherkin (>= 27, < 28)
-      cucumber-messages (>= 20, < 23)
-      cucumber-tag-expressions (> 5, < 7)
+    cucumber-core (16.2.0)
+      cucumber-gherkin (> 36, < 40)
+      cucumber-messages (> 31, < 33)
+      cucumber-tag-expressions (> 6, < 9)
     cucumber-cucumber-expressions (17.1.0)
       bigdecimal
-    cucumber-gherkin (27.0.0)
-      cucumber-messages (>= 19.1.4, < 23)
-    cucumber-html-formatter (21.7.0)
-      cucumber-messages (> 19, < 27)
-    cucumber-messages (22.0.0)
-    cucumber-rails (3.1.0)
-      capybara (>= 3.11, < 4)
-      cucumber (>= 5, < 10)
-      railties (>= 5.2, < 9)
+    cucumber-gherkin (39.0.0)
+      cucumber-messages (>= 31, < 33)
+    cucumber-html-formatter (22.3.0)
+      cucumber-messages (> 23, < 33)
+    cucumber-messages (32.3.1)
+    cucumber-rails (4.0.1)
+      capybara (>= 3.25, < 4)
+      cucumber (>= 7, < 11)
+      railties (>= 6.1, < 9)
     cucumber-tag-expressions (6.1.1)
     cucumber_priority (1.1.0)
       cucumber
@@ -182,10 +183,13 @@ GEM
       net-smtp
     marcel (1.0.4)
     matrix (0.4.2)
+    memoist3 (1.0.0)
     method_source (1.1.0)
     mini_mime (1.1.5)
     mini_portile2 (2.8.9)
-    minitest (5.25.4)
+    minitest (6.0.6)
+      drb (~> 2.0)
+      prism (~> 1.5)
     multi_test (1.1.0)
     net-imap (0.5.1)
       date
@@ -202,6 +206,7 @@ GEM
       racc (~> 1.4)
     nokogiri (1.17.2-x86_64-linux)
       racc (~> 1.4)
+    prism (1.9.0)
     pry (0.15.0)
       coderay (~> 1.1)
       method_source (~> 1.0)
@@ -286,8 +291,9 @@ GEM
       mini_portile2 (~> 2.8.0)
     sqlite3 (2.4.1-x86_64-linux-gnu)
     stringio (3.1.2)
-    sys-uname (1.3.1)
+    sys-uname (1.5.1)
       ffi (~> 1.1)
+      memoist3 (~> 1.0.0)
     temple (0.10.3)
     thor (1.3.2)
     tilt (2.4.0)
@@ -312,8 +318,9 @@ DEPENDENCIES
   byebug
   capybara (~> 3)
   capybara-screenshot
+  cgi
   cucumber
-  cucumber-rails
+  cucumber-rails (>= 4)
   database_cleaner
   gemika
   haml-rails

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spreewald
 version: !ruby/object:Gem::Version
-  version: 4.6.5
+  version: 4.6.6
 platform: ruby
 authors:
 - Tobias Kraze
@@ -95,12 +95,14 @@ files:
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
-- Gemfile.ruby_3_2_0
-- Gemfile.ruby_3_2_0.lock
-- Gemfile.ruby_3_3_10
-- Gemfile.ruby_3_3_10.lock
-- Gemfile.ruby_3_4_9
-- Gemfile.ruby_3_4_9.lock
+- Gemfile.ruby_3_2
+- Gemfile.ruby_3_2.lock
+- Gemfile.ruby_3_3
+- Gemfile.ruby_3_3.lock
+- Gemfile.ruby_3_4
+- Gemfile.ruby_3_4.lock
+- Gemfile.ruby_4_0
+- Gemfile.ruby_4_0.lock
 - LICENSE
 - README.md
 - Rakefile
@@ -381,7 +383,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.6
 specification_version: 4
 summary: Collection of useful cucumber steps.
 test_files: