Use and generate 1.9 hash syntax

CHANGELOG.md

@@ -163,9 +163,9 @@ with a Step Definition that perhaps looks like this:
 
     Given /^I have signed up as "([^"]*)"$/ do |email|
       visit(signup_path)
-      fill_in('Email', :with => email)
-      fill_in('Password', :with => 's3cr3t')
-      fill_in('Password Confirmation', :with => 's3cr3t')
+      fill_in('Email', with: email)
+      fill_in('Password', with: 's3cr3t')
+      fill_in('Password Confirmation', with: 's3cr3t')
       click_button('Sign up')
     end
 

README.md

@@ -13,7 +13,7 @@ Cucumber-Rails brings Cucumber to Rails 4.x and 5.x. For Rails 3.x support, use
 Before you can use the generator, add the gem to your project's Gemfile as follows:
 
     group :test do
-      gem 'cucumber-rails', :require => false
+      gem 'cucumber-rails', require: false
       # database_cleaner is not required, but highly recommended
       gem 'database_cleaner'
     end

Rakefile

@@ -14,9 +14,9 @@ Dir["#{File.dirname(__FILE__)}/dev_tasks/*.rake"].sort.each { |ext| load ext }
 
 CLEAN.include('doc', 'tmp')
 
-task :default => :test
+task default: :test
 
-task :test => [:spec, :cucumber]
+task test: [:spec, :cucumber]
 
 namespace :test do
   desc "Run tests against all gemfiles"
@@ -45,5 +45,5 @@ namespace :gemfiles do
   end
 
   desc "Rebuild generated gemfiles and install dependencies"
-  task :rebuild => [:clean, :install]
+  task rebuild: [:clean, :install]
 end

dev_tasks/yard.rake

@@ -20,7 +20,7 @@ namespace :api do
     mkdir_p dir
     yard.options = ["--out", dir]
   end
-  task :yard => :dir
+  task yard: :dir
 
   task :release do
     Dir.chdir(SITE_DIR) do
@@ -31,5 +31,5 @@ namespace :api do
   end
 
   desc "Generate YARD docs for Cucumber-Rails' API"
-  task :doc => [:yard, :release]
+  task doc: [:yard, :release]
 end

features/capybara_javascript_drivers.feature

@@ -8,15 +8,15 @@ Feature: Capybara Javascript Drivers
     And I write to "features/step_definitions/date_time_steps.rb" with:
       """
       When /^(?:|I )select "([^"]+)" as the "([^"]+)" time$/ do |time, selector|
-        select_time(time, :from => selector)
+        select_time(time, from: selector)
       end
 
       When /^(?:|I )select "([^"]+)" as the "([^"]+)" date$/ do |date, selector|
-        select_date(date, :from => selector)
+        select_date(date, from: selector)
       end
 
       When /^(?:|I )select "([^"]+)" as the "([^"]+)" date and time$/ do |datetime, selector|
-        select_datetime(datetime, :from => selector)
+        select_datetime(datetime, from: selector)
       end
       """
 
@@ -56,7 +56,7 @@ Feature: Capybara Javascript Drivers
     And I write to "features/step_definitions/custom_steps.rb" with:
       """
       Given /^a random appointment$/ do
-        @appointment = Appointment.create!(:name => 'Random appointment', :when => DateTime.now)
+        @appointment = Appointment.create!(name: 'Random appointment', when: DateTime.now)
       end
 
       Given /^I am viewing random appointment$/ do

features/database_cleaner.feature

@@ -21,13 +21,13 @@ Feature: DatabaseCleaner
       """
       Given /^I have (\d+) widgets$/ do |n|
         n.to_i.times do |i|
-          Widget.create! :name => "Widget #{Widget.count + i}"
+          Widget.create! name: "Widget #{Widget.count + i}"
         end
       end
 
       When /^I create (\d+) more widgets$/ do |n|
         n.to_i.times do |i|
-          Widget.create! :name => "Widget #{Widget.count + i}"
+          Widget.create! name: "Widget #{Widget.count + i}"
         end
       end
 

features/disable_automatic_database_cleaning.feature

@@ -32,13 +32,13 @@ Feature: Disable automatic database cleaning
       """
       Given /^I have (\d+) widgets$/ do |n|
         n.to_i.times do |i|
-          Widget.create! :name => "Widget #{Widget.count + i}"
+          Widget.create! name: "Widget #{Widget.count + i}"
         end
       end
 
       When /^I create (\d+) widgets$/ do |n|
         n.to_i.times do |i|
-          Widget.create! :name => "Widget #{Widget.count + i}"
+          Widget.create! name: "Widget #{Widget.count + i}"
         end
       end
 

features/emulate_javascript.feature

@@ -16,7 +16,7 @@ Feature: Emulate Javascript
     And I write to "features/step_definitions/s.rb" with:
       """
       Given /^there is a widget named "([^"]*)"$/ do |name|
-        FactoryGirl.create(:widget, :name => name)
+        FactoryGirl.create(:widget, name: name)
       end
       """
     And I write to "features/support/factories.rb" with:
@@ -52,8 +52,8 @@ Feature: Emulate Javascript
       """
       class ApplicationController < ActionController::Base
         protect_from_forgery
-        before_action :except => :establish do
-          render :text => "denied", :status => :forbidden and return false unless session[:verified]
+        before_action except: :establish do
+          render text: "denied", status: :forbidden and return false unless session[:verified]
         end
       end
       """
@@ -73,7 +73,7 @@ Feature: Emulate Javascript
     And I write to "features/step_definitions/s.rb" with:
       """
       Given /^there is a widget named "([^"]*)"$/ do |name|
-        FactoryGirl.create(:widget, :name => name)
+        FactoryGirl.create(:widget, name: name)
       end
       """
     And I write to "features/support/factories.rb" with:

features/no_database.feature

@@ -31,9 +31,9 @@ Feature: No Database
       """
       source 'http://rubygems.org'
       gem 'rails'
-      gem "cucumber-rails", :group => :test, :path => "../../.."
-      gem "capybara", :group => :test
-      gem "rspec-rails", :group => :test
+      gem "cucumber-rails", group: :test, path: "../../.."
+      gem "capybara", group: :test
+      gem "rspec-rails", group: :test
       if RUBY_VERSION >= '2.0.0'
         gem 'sass-rails'
         gem 'uglifier'

features/rest_api.feature

@@ -6,7 +6,7 @@ Feature: REST API
       """
       class PostsController < ApplicationController
         def index
-          render :json => {'hello' => 'world'}.to_json
+          render json: {'hello' => 'world'}.to_json
         end
       end
       """

features/step_definitions/cucumber_rails_steps.rb

@@ -101,7 +101,7 @@ def fixture(path)
       browser_options = ::Selenium::WebDriver::Firefox::Options.new()
       browser_options.args << '--headless'
 
-      Capybara::Selenium::Driver.new(app, :browser => :firefox, options: browser_options)
+      Capybara::Selenium::Driver.new(app, browser: :firefox, options: browser_options)
     end
   }
 

features/support/legacy_web_steps_support.rb

@@ -53,11 +53,11 @@ def with_scope(locator)
 end
 
 When /^(?:|I )fill in "([^"]*)" with "([^"]*)"$/ do |field, value|
-  fill_in(field, :with => value)
+  fill_in(field, with: value)
 end
 
 When /^(?:|I )fill in "([^"]*)" for "([^"]*)"$/ do |value, field|
-  fill_in(field, :with => value)
+  fill_in(field, with: value)
 end
 
 # Use this to fill in an entire form with data from a table. Example:
@@ -78,7 +78,7 @@ def with_scope(locator)
 end
 
 When /^(?:|I )select "([^"]*)" from "([^"]*)"$/ do |value, field|
-  select(value, :from => field)
+  select(value, from: field)
 end
 
 When /^(?:|I )check "([^"]*)"$/ do |field|
@@ -109,9 +109,9 @@ def with_scope(locator)
   regexp = Regexp.new(regexp)
 
   if page.respond_to? :should
-    page.should have_xpath('//*', :text => regexp)
+    page.should have_xpath('//*', text: regexp)
   else
-    assert page.has_xpath?('//*', :text => regexp)
+    assert page.has_xpath?('//*', text: regexp)
   end
 end
 
@@ -127,9 +127,9 @@ def with_scope(locator)
   regexp = Regexp.new(regexp)
 
   if page.respond_to? :should
-    page.should have_no_xpath('//*', :text => regexp)
+    page.should have_no_xpath('//*', text: regexp)
   else
-    assert page.has_no_xpath?('//*', :text => regexp)
+    assert page.has_no_xpath?('//*', text: regexp)
   end
 end
 

gemfiles/rails_5_0.gemfile

@@ -15,4 +15,4 @@ gem "uglifier", "~> 2.7.2"
 gem "sqlite3", "~> 1.3.7"
 gem "database_cleaner", ">= 1.1.0"
 
-gemspec :path => "../"
+gemspec path: "../"

lib/cucumber/rails.rb

@@ -26,6 +26,6 @@
   MultiTest.disable_autorun
 else
   warn "WARNING: Cucumber-rails required outside of env.rb.  The rest of loading is being deferred until env.rb is called.
-  To avoid this warning, move 'gem \'cucumber-rails\', :require => false' under only group :test in your Gemfile.
-  If already in the :test group, be sure you are specifying ':require => false'."
+  To avoid this warning, move 'gem \'cucumber-rails\', require: false' under only group :test in your Gemfile.
+  If already in the :test group, be sure you are specifying 'require: false'."
 end

lib/cucumber/rails/capybara/select_dates_and_times.rb

@@ -3,7 +3,7 @@ module Rails
     module Capybara
       # This module defines methods for selecting dates and times
       module SelectDatesAndTimes
-        # Select a Rails date. Options hash must include :from => +label+
+        # Select a Rails date. Options hash must include from: +label+
         def select_date(date, options)
           date        = Date.parse(date)
           base_dom_id = get_base_dom_id_from_label_tag(options[:from])
@@ -13,7 +13,7 @@ def select_date(date, options)
           find(:xpath, ".//select[@id='#{base_dom_id}_3i']").select(date.day.to_s)
         end
 
-        # Select a Rails time. Options hash must include :from => +label+
+        # Select a Rails time. Options hash must include from: +label+
         def select_time(time, options)
           time        = Time.zone.parse(time)
           base_dom_id = get_base_dom_id_from_label_tag(options[:from])
@@ -22,7 +22,7 @@ def select_time(time, options)
           find(:xpath, ".//select[@id='#{base_dom_id}_5i']").select(time.min.to_s.rjust(2,  '0'))
         end
 
-        # Select a Rails datetime. Options hash must include :from => +label+
+        # Select a Rails datetime. Options hash must include from: +label+
         def select_datetime(datetime, options)
           select_date(datetime, options)
           select_time(datetime, options)

lib/generators/cucumber/install/templates/support/_rails_each_run.rb.erb

@@ -27,7 +27,7 @@ end
 # See the DatabaseCleaner documentation for details. Example:
 #
 #   Before('@no-txn,@selenium,@culerity,@celerity,@javascript') do
-#     # { :except => [:widgets] } may not do what you expect here
+#     # { except: [:widgets] } may not do what you expect here
 #     # as Cucumber::Rails::Database.javascript_strategy overrides
 #     # this setting.
 #     DatabaseCleaner.strategy = :truncation

lib/generators/cucumber/install/templates/tasks/cucumber.rake.erb

@@ -9,26 +9,26 @@ begin
   require 'cucumber/rake/task'
 
   namespace :cucumber do
-    Cucumber::Rake::Task.new({:ok => 'test:prepare'}, 'Run features that should pass') do |t|
+    Cucumber::Rake::Task.new({ok: 'test:prepare'}, 'Run features that should pass') do |t|
       t.binary = vendored_cucumber_bin # If nil, the gem's binary is used.
       t.fork = true # You may get faster startup if you set this to false
       t.profile = 'default'
     end
 
-    Cucumber::Rake::Task.new({:wip => 'test:prepare'}, 'Run features that are being worked on') do |t|
+    Cucumber::Rake::Task.new({wip: 'test:prepare'}, 'Run features that are being worked on') do |t|
       t.binary = vendored_cucumber_bin
       t.fork = true # You may get faster startup if you set this to false
       t.profile = 'wip'
     end
 
-    Cucumber::Rake::Task.new({:rerun => 'test:prepare'}, 'Record failing features and run only them if any exist') do |t|
+    Cucumber::Rake::Task.new({rerun: 'test:prepare'}, 'Record failing features and run only them if any exist') do |t|
       t.binary = vendored_cucumber_bin
       t.fork = true # You may get faster startup if you set this to false
       t.profile = 'rerun'
     end
 
     desc 'Run all features'
-    task :all => [:ok, :wip]
+    task all: [:ok, :wip]
 
     task :statsetup do
       require 'rails/code_statistics'
@@ -46,21 +46,21 @@ begin
     end
   end
   desc 'Alias for cucumber:ok'
-  task :cucumber => 'cucumber:ok'
+  task cucumber: 'cucumber:ok'
 
-  task :default => :cucumber
+  task default: :cucumber
 
-  task :features => :cucumber do
+  task features: :cucumber do
     STDERR.puts "*** The 'features' task is deprecated. See rake -T cucumber ***"
   end
 
   # In case we don't have the generic Rails test:prepare hook, append a no-op task that we can depend upon.
   task 'test:prepare' do
   end
 
-  task :stats => 'cucumber:statsetup'
+  task stats: 'cucumber:statsetup'
 
-  task :notes => 'cucumber:annotations_setup'
+  task notes: 'cucumber:annotations_setup'
 rescue LoadError
   desc 'cucumber rake task not available (cucumber not installed)'
   task :cucumber do

spec/generators/cucumber/install/install_generator_spec.rb

@@ -34,7 +34,7 @@
       subject { file('lib/tasks/cucumber.rake') }
       it { is_expected.to exist }
       it { is_expected.to contain '# IMPORTANT: This file is generated by cucumber-rails - edit at your own peril.' }
-      it { is_expected.to contain "task :cucumber => 'cucumber:ok'" }
+      it { is_expected.to contain "task cucumber: 'cucumber:ok'" }
     end
 
     describe 'script/cucumber' do