Prepare for 3.5.0 release

History.md

@@ -1,5 +1,5 @@
 # Version 3.5.0
-Release date: unreleased
+Release date: 2018-08-01
 
 ### Added
 
@@ -7,7 +7,7 @@ Release date: unreleased
 
 ### Fixed
 
-* `attach_file` with Selenium and local Firefox 62 now correctly generates only one change event when attaching multiple files
+* `attach_file` with Selenium and local Firefox 62+ now correctly generates only one change event when attaching multiple files
 
 # Version 3.4.2
 Release date: 2018-07-24

lib/capybara/selenium/node.rb

@@ -177,13 +177,13 @@ class Capybara::Selenium::Node < Capybara::Driver::Node
       if parent
         siblings = parent.find_xpath(selector)
         selector += case siblings.size
-          when 0
-            '[ERROR]' # IE doesn't support full XPath (namespace-uri, etc)
-          when 1
-            '' # index not necessary when only one matching element
-          else
-            "[#{siblings.index(node) + 1}]"
-          end
+        when 0
+          '[ERROR]' # IE doesn't support full XPath (namespace-uri, etc)
+        when 1
+          '' # index not necessary when only one matching element
+        else
+          "[#{siblings.index(node) + 1}]"
+        end
       end
       result.push selector
     end

lib/capybara/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Capybara
-  VERSION = '3.5.0.dev'
+  VERSION = '3.5.0'
 end

spec/selenium_spec_ie.rb

@@ -33,7 +33,7 @@ Capybara::SpecHelper.run_specs TestSessions::SeleniumIE, 'selenium', capybara_sk
   when /#fill_in with Date /
     pending "IE 11 doesn't support date input types"
   when /#click_link_or_button with :disabled option happily clicks on links which incorrectly have the disabled attribute$/
-    pending "IE 11 obeys non-standard disabled attribute on anchor tag"
+    pending 'IE 11 obeys non-standard disabled attribute on anchor tag'
   end
 end
 

spec/shared_selenium_session.rb

@@ -39,14 +39,14 @@ RSpec.shared_examples 'Capybara::Session' do |session, mode|
       end
 
       it 'should have return code 1 when running selenium_driver_rspec_failure.rb' do
-        skip "only setup for local non-headless" if headless_or_remote?
+        skip 'only setup for local non-headless' if headless_or_remote?
 
         system(@env, 'rspec spec/fixtures/selenium_driver_rspec_failure.rb', out: File::NULL, err: File::NULL)
         expect($CHILD_STATUS.exitstatus).to eq(1)
       end
 
       it 'should have return code 0 when running selenium_driver_rspec_success.rb' do
-        skip "only setup for local non-headless" if headless_or_remote?
+        skip 'only setup for local non-headless' if headless_or_remote?
 
         system(@env, 'rspec spec/fixtures/selenium_driver_rspec_success.rb', out: File::NULL, err: File::NULL)
         expect($CHILD_STATUS.exitstatus).to eq(0)