simple_form 2.1.0 → 3.0.0.rc

data/test/form_builder/label_test.rb

@@ -14,7 +14,7 @@ class LabelTest < ActionView::TestCase
   end
 
   test 'builder should generate a label for the boolean attrbiute' do
-    with_label_for @user, :name, :as => :boolean
+    with_label_for @user, :name, as: :boolean
     assert_select 'label.boolean[for=user_name]', /Name/
     assert_no_select 'label[as=boolean]'
   end
@@ -30,22 +30,22 @@ class LabelTest < ActionView::TestCase
   end
 
   test 'builder should allow passing options to label tag' do
-    with_label_for @user, :name, :label => 'My label', :id => 'name_label'
+    with_label_for @user, :name, label: 'My label', id: 'name_label'
     assert_select 'label.string#name_label', /My label/
   end
 
   test 'builder label should generate label tag with clean HTML' do
-    with_label_for @user, :name, :label => 'My label', :required => true, :id => 'name_label'
+    with_label_for @user, :name, label: 'My label', required: true, id: 'name_label'
     assert_select 'label.string#name_label', /My label/
     assert_no_select 'label[label]'
     assert_no_select 'label[required]'
   end
 
   test 'builder should not modify the options hash' do
-    options = { :label => 'My label', :id => 'name_label' }
+    options = { label: 'My label', id: 'name_label' }
     with_label_for @user, :name, options
     assert_select 'label.string#name_label', /My label/
-    assert_equal({ :label => 'My label', :id => 'name_label' }, options)
+    assert_equal({ label: 'My label', id: 'name_label' }, options)
   end
 
   test 'builder should fallback to default label when string is given' do
@@ -63,7 +63,7 @@ class LabelTest < ActionView::TestCase
   end
 
   test 'builder allows label order to be changed' do
-    swap SimpleForm, :label_text => lambda { |l, r| "#{l}:" } do
+    swap SimpleForm, label_text: lambda { |l, r| "#{l}:" } do
       with_label_for @user, :age
       assert_select 'label.integer[for=user_age]', "Age:"
     end

data/test/form_builder/wrapper_test.rb

@@ -38,7 +38,7 @@ class WrapperTest < ActionView::TestCase
   end
 
   test 'wrapper should add hint class for attribute with a hint' do
-    with_form_for @user, :name, :hint => 'hint'
+    with_form_for @user, :name, hint: 'hint'
     assert_select 'div.field_with_hint'
   end
 
@@ -48,18 +48,18 @@ class WrapperTest < ActionView::TestCase
   end
 
   test 'wrapper should have disabled class when input is disabled' do
-    with_form_for @user, :active, :disabled => true
+    with_form_for @user, :active, disabled: true
     assert_select 'div.disabled'
   end
 
   test 'wrapper should support no wrapping when wrapper is false' do
-    with_form_for @user, :name, :wrapper => false
+    with_form_for @user, :name, wrapper: false
     assert_select 'form > label[for=user_name]'
     assert_select 'form > input#user_name.string'
   end
 
   test 'wrapper should support no wrapping when wrapper tag is false' do
-    with_form_for @user, :name, :wrapper => custom_wrapper_without_top_level
+    with_form_for @user, :name, wrapper: custom_wrapper_without_top_level
     assert_select 'form > label[for=user_name]'
     assert_select 'form > input#user_name.string'
   end
@@ -68,28 +68,28 @@ class WrapperTest < ActionView::TestCase
     with_form_for @user, :name
     assert_select 'form div.input.required.string'
 
-    with_form_for @user, :age, :required => false
+    with_form_for @user, :age, required: false
     assert_select 'form div.input.optional.integer'
   end
 
   test 'wrapper should allow custom options to be given' do
-    with_form_for @user, :name, :wrapper_html => { :id => "super_cool", :class => 'yay' }
+    with_form_for @user, :name, wrapper_html: { id: "super_cool", class: 'yay' }
     assert_select 'form #super_cool.required.string.yay'
   end
 
   test 'wrapper should allow tag to be given on demand' do
-    with_form_for @user, :name, :wrapper_tag => :b
+    with_form_for @user, :name, wrapper_tag: :b
     assert_select 'form b.required.string'
   end
 
   test 'wrapper should allow wrapper class to be given on demand' do
-    with_form_for @user, :name, :wrapper_class => :wrapper
+    with_form_for @user, :name, wrapper_class: :wrapper
     assert_select 'form div.wrapper.required.string'
   end
 
   test 'wrapper should skip additional classes when configured' do
-    swap SimpleForm, :generate_additional_classes_for => [:input, :label] do
-      with_form_for @user, :name, :wrapper_class => :wrapper
+    swap SimpleForm, generate_additional_classes_for: [:input, :label] do
+      with_form_for @user, :name, wrapper_class: :wrapper
       assert_select 'form div.wrapper'
       assert_no_select 'div.required'
       assert_no_select 'div.string'
@@ -98,7 +98,7 @@ class WrapperTest < ActionView::TestCase
   end
 
   test 'wrapper should not generate empty css class' do
-    swap SimpleForm, :generate_additional_classes_for => [:input, :label] do
+    swap SimpleForm, generate_additional_classes_for: [:input, :label] do
       swap_wrapper :default, custom_wrapper_without_class do
         with_form_for @user, :name
         assert_no_select 'div#custom_wrapper_without_class[class]'
@@ -110,7 +110,7 @@ class WrapperTest < ActionView::TestCase
 
   test 'custom wrappers works' do
     swap_wrapper do
-      with_form_for @user, :name, :hint => "cool"
+      with_form_for @user, :name, hint: "cool"
       assert_select "section.custom_wrapper div.another_wrapper label"
       assert_select "section.custom_wrapper div.another_wrapper input.string"
       assert_no_select "section.custom_wrapper div.another_wrapper span.omg_error"
@@ -121,7 +121,7 @@ class WrapperTest < ActionView::TestCase
 
   test 'custom wrappers can be turned off' do
     swap_wrapper do
-      with_form_for @user, :name, :another => false
+      with_form_for @user, :name, another: false
       assert_no_select "section.custom_wrapper div.another_wrapper label"
       assert_no_select "section.custom_wrapper div.another_wrapper input.string"
       assert_select "section.custom_wrapper div.error_wrapper span.omg_error"
@@ -137,7 +137,7 @@ class WrapperTest < ActionView::TestCase
       assert_no_select "section.custom_wrapper div.another_wrapper label"
       assert_no_select "section.custom_wrapper div.another_wrapper input.string"
 
-      with_concat_form_for(@user, :wrapper => :another) do |f|
+      with_concat_form_for(@user, wrapper: :another) do |f|
         f.input :name
       end
 
@@ -153,20 +153,20 @@ class WrapperTest < ActionView::TestCase
       assert_no_select "section.custom_wrapper div.another_wrapper input.string"
       output_buffer.replace ""
 
-      with_form_for @user, :name, :wrapper => :another
+      with_form_for @user, :name, wrapper: :another
       assert_select "section.custom_wrapper div.another_wrapper label"
       assert_select "section.custom_wrapper div.another_wrapper input.string"
       output_buffer.replace ""
     end
 
-    with_form_for @user, :name, :wrapper => custom_wrapper
+    with_form_for @user, :name, wrapper: custom_wrapper
     assert_select "section.custom_wrapper div.another_wrapper label"
     assert_select "section.custom_wrapper div.another_wrapper input.string"
   end
 
   test 'access wrappers with indifferent access' do
     swap_wrapper :another do
-      with_form_for @user, :name, :wrapper => "another"
+      with_form_for @user, :name, wrapper: "another"
       assert_select "section.custom_wrapper div.another_wrapper label"
       assert_select "section.custom_wrapper div.another_wrapper input.string"
     end
@@ -175,8 +175,8 @@ class WrapperTest < ActionView::TestCase
   test 'do not duplicate label classes for different inputs' do
     swap_wrapper :default, self.custom_wrapper_with_label_html_option do
       with_concat_form_for(@user) do |f|
-        concat f.input :name, :required => false
-        concat f.input :email, :as => :email, :required => true
+        concat f.input :name, required: false
+        concat f.input :email, as: :email, required: true
       end
 
       assert_select "label.string.optional.extra-label-class[for='user_name']"
@@ -187,13 +187,13 @@ class WrapperTest < ActionView::TestCase
 
   test 'raise error when wrapper not found' do
     assert_raise SimpleForm::WrapperNotFound do
-      with_form_for @user, :name, :wrapper => :not_found
+      with_form_for @user, :name, wrapper: :not_found
     end
   end
 
   test 'use wrapper for specified in config mapping' do
     swap_wrapper :another do
-      swap SimpleForm, :wrapper_mappings => { :string => :another } do
+      swap SimpleForm, wrapper_mappings: { string: :another } do
         with_form_for @user, :name
         assert_select "section.custom_wrapper div.another_wrapper label"
         assert_select "section.custom_wrapper div.another_wrapper input.string"

data/test/inputs/boolean_input_test.rb

@@ -9,26 +9,26 @@ class BooleanInputTest < ActionView::TestCase
   end
 
   test 'input does not generate the label with the checkbox when label option is false' do
-    with_input_for @user, :active, :boolean, :label => false
+    with_input_for @user, :active, :boolean, label: false
     assert_select 'input[type=checkbox].boolean#user_active'
     assert_no_select 'label'
   end
 
   test 'input uses custom checked value' do
     @user.action = 'on'
-    with_input_for @user, :action, :boolean, :checked_value => 'on', :unchecked_value => 'off'
+    with_input_for @user, :action, :boolean, checked_value: 'on', unchecked_value: 'off'
     assert_select 'input[type=checkbox][value=on][checked=checked]'
   end
 
   test 'input uses custom unchecked value' do
     @user.action = 'off'
-    with_input_for @user, :action, :boolean, :checked_value => 'on', :unchecked_value => 'off'
+    with_input_for @user, :action, :boolean, checked_value: 'on', unchecked_value: 'off'
     assert_select 'input[type=checkbox][value=on]'
     assert_no_select 'input[checked=checked][value=on]'
   end
 
   test 'input generates hidden input with custom unchecked value' do
-    with_input_for @user, :action, :boolean, :checked_value => 'on', :unchecked_value => 'off'
+    with_input_for @user, :action, :boolean, checked_value: 'on', unchecked_value: 'off'
     assert_select 'input[type=hidden][value=off]'
   end
 
@@ -39,7 +39,7 @@ class BooleanInputTest < ActionView::TestCase
   end
 
   test 'input allows changing default boolean style config to nested, generating a default label and a manual hidden field for checkbox' do
-    swap SimpleForm, :boolean_style => :nested do
+    swap SimpleForm, boolean_style: :nested do
       with_input_for @user, :active, :boolean
       assert_select 'label[for=user_active]', 'Active'
       assert_select 'label.boolean > input.boolean'
@@ -48,21 +48,21 @@ class BooleanInputTest < ActionView::TestCase
   end
 
   test 'input boolean with nested allows :inline_label' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :label => false, :inline_label => 'I am so inline.'
-      assert_select 'label.checkbox', :text => 'I am so inline.'
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, label: false, inline_label: 'I am so inline.'
+      assert_select 'label.checkbox', text: 'I am so inline.'
     end
   end
 
   test 'input boolean with nested style creates an inline label using the default label text when inline_label option set to true' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :label => false, :inline_label => true
-      assert_select 'label.checkbox', :text => 'Active'
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, label: false, inline_label: true
+      assert_select 'label.checkbox', text: 'Active'
     end
   end
 
   test 'input boolean with nested generates a manual hidden field for checkbox outside the label, to recreate Rails functionality with valid html5' do
-    swap SimpleForm, :boolean_style => :nested do
+    swap SimpleForm, boolean_style: :nested do
       with_input_for @user, :active, :boolean
 
       assert_select "input[type=hidden][name='user[active]'] + label.boolean > input.boolean"
@@ -71,23 +71,23 @@ class BooleanInputTest < ActionView::TestCase
   end
 
   test 'input boolean with nested generates a disabled hidden field for checkbox outside the label, if the field is disabled' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :disabled => true
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, disabled: true
 
       assert_select "input[type=hidden][name='user[active]'][disabled] + label.boolean > input.boolean[disabled]"
     end
   end
 
   test 'input accepts changing boolean style to nested through given options' do
-    with_input_for @user, :active, :boolean, :boolean_style => :nested
+    with_input_for @user, :active, :boolean, boolean_style: :nested
     assert_select 'label[for=user_active]', 'Active'
     assert_select 'label.boolean > input.boolean'
     assert_no_select 'input[type=checkbox] + label'
   end
 
   test 'input accepts changing boolean style to inline through given options, when default is nested' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :boolean_style => :inline
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, boolean_style: :inline
       assert_select 'label[for=user_active]', 'Active'
       assert_select 'input.boolean + label.boolean'
       assert_no_select 'label > input'
@@ -95,16 +95,16 @@ class BooleanInputTest < ActionView::TestCase
   end
 
   test 'input with nested style allows disabling label' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :label => false
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, label: false
       assert_select 'input.boolean'
       assert_no_select 'label.boolean'
     end
   end
 
   test 'input with nested style allows customizing input_html' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :boolean, :input_html => { :name => 'active_user' }
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :boolean, input_html: { name: 'active_user' }
       assert_select "input[type=hidden][name=active_user] + label.boolean > input.boolean[name=active_user]"
     end
   end
@@ -120,7 +120,7 @@ class BooleanInputTest < ActionView::TestCase
 
   test 'input boolean with nested style works using :input only in wrapper config (no label_input), adding the extra "checkbox" label wrapper' do
     swap_wrapper do
-      swap SimpleForm, :boolean_style => :nested do
+      swap SimpleForm, boolean_style: :nested do
         with_input_for @user, :active, :boolean
 
         assert_select 'label.boolean + input[type=hidden] + label.checkbox > input.boolean'
@@ -130,7 +130,7 @@ class BooleanInputTest < ActionView::TestCase
 
   test 'input boolean with nested style works using :label_input in wrapper config, adding "checkbox" class to label' do
     swap_wrapper :default, self.custom_wrapper_without_top_level do
-      swap SimpleForm, :boolean_style => :nested do
+      swap SimpleForm, boolean_style: :nested do
         with_input_for @user, :active, :boolean
 
         assert_select 'input[type=hidden] + label.boolean.checkbox > input.boolean'

data/test/inputs/collection_check_boxes_input_test.rb

@@ -7,27 +7,33 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   end
 
   test 'input check boxes should not include for attribute by default' do
-    with_input_for @user, :gender, :check_boxes, :collection => [:male, :female]
+    with_input_for @user, :gender, :check_boxes, collection: [:male, :female]
     assert_select 'label'
     assert_no_select 'label[for=user_gender]'
   end
 
   test 'input check boxes should include for attribute when giving as html option' do
-    with_input_for @user, :gender, :check_boxes, :collection => [:male, :female], :label_html => { :for => 'gender' }
+    with_input_for @user, :gender, :check_boxes, collection: [:male, :female], label_html: { for: 'gender' }
     assert_select 'label[for=gender]'
   end
 
   test 'collection input with check_boxes type should not generate required html attribute' do
-    with_input_for @user, :name, :check_boxes, :collection => ['Jose' , 'Carlos']
+    with_input_for @user, :name, :check_boxes, collection: ['Jose', 'Carlos']
     assert_select 'input.required'
     assert_no_select 'input[required]'
   end
 
+  test 'collection input with check_boxes type should not generate aria-required html attribute' do
+    with_input_for @user, :name, :check_boxes, collection: ['Jose', 'Carlos']
+    assert_select 'input.required'
+    assert_no_select 'input[aria-required]'
+  end
+
   test 'input should do automatic collection translation for check_box types using defaults key' do
-    store_translations(:en, :simple_form => { :options => { :defaults => {
-      :gender => { :male => 'Male', :female => 'Female'}
+    store_translations(:en, simple_form: { options: { defaults: {
+      gender: { male: 'Male', female: 'Female'}
     } } } ) do
-      with_input_for @user, :gender, :check_boxes, :collection => [:male, :female]
+      with_input_for @user, :gender, :check_boxes, collection: [:male, :female]
       assert_select 'input[type=checkbox][value=male]'
       assert_select 'input[type=checkbox][value=female]'
       assert_select 'label.collection_check_boxes', 'Male'
@@ -36,10 +42,10 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   end
 
   test 'input should do automatic collection translation for check_box types using specific object key' do
-    store_translations(:en, :simple_form => { :options => { :user => {
-      :gender => { :male => 'Male', :female => 'Female'}
+    store_translations(:en, simple_form: { options: { user: {
+      gender: { male: 'Male', female: 'Female'}
     } } } ) do
-      with_input_for @user, :gender, :check_boxes, :collection => [:male, :female]
+      with_input_for @user, :gender, :check_boxes, collection: [:male, :female]
       assert_select 'input[type=checkbox][value=male]'
       assert_select 'input[type=checkbox][value=female]'
       assert_select 'label.collection_check_boxes', 'Male'
@@ -50,100 +56,100 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   test 'input check boxes does not wrap the collection by default' do
     with_input_for @user, :active, :check_boxes
 
-    assert_select 'form input[type=checkbox]', :count => 2
+    assert_select 'form input[type=checkbox]', count: 2
     assert_no_select 'form ul'
   end
 
   test 'input check boxes wraps the collection in the configured collection wrapper tag' do
-    swap SimpleForm, :collection_wrapper_tag => :ul do
+    swap SimpleForm, collection_wrapper_tag: :ul do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form ul input[type=checkbox]', :count => 2
+      assert_select 'form ul input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes does not wrap the collection when configured with falsy values' do
-    swap SimpleForm, :collection_wrapper_tag => false do
+    swap SimpleForm, collection_wrapper_tag: false do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form input[type=checkbox]', :count => 2
+      assert_select 'form input[type=checkbox]', count: 2
       assert_no_select 'form ul'
     end
   end
 
   test 'input check boxes allows overriding the collection wrapper tag at input level' do
-    swap SimpleForm, :collection_wrapper_tag => :ul do
-      with_input_for @user, :active, :check_boxes, :collection_wrapper_tag => :section
+    swap SimpleForm, collection_wrapper_tag: :ul do
+      with_input_for @user, :active, :check_boxes, collection_wrapper_tag: :section
 
-      assert_select 'form section input[type=checkbox]', :count => 2
+      assert_select 'form section input[type=checkbox]', count: 2
       assert_no_select 'form ul'
     end
   end
 
   test 'input check boxes allows disabling the collection wrapper tag at input level' do
-    swap SimpleForm, :collection_wrapper_tag => :ul do
-      with_input_for @user, :active, :check_boxes, :collection_wrapper_tag => false
+    swap SimpleForm, collection_wrapper_tag: :ul do
+      with_input_for @user, :active, :check_boxes, collection_wrapper_tag: false
 
-      assert_select 'form input[type=checkbox]', :count => 2
+      assert_select 'form input[type=checkbox]', count: 2
       assert_no_select 'form ul'
     end
   end
 
   test 'input check boxes renders the wrapper tag with the configured wrapper class' do
-    swap SimpleForm, :collection_wrapper_tag => :ul, :collection_wrapper_class => 'inputs-list' do
+    swap SimpleForm, collection_wrapper_tag: :ul, collection_wrapper_class: 'inputs-list' do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form ul.inputs-list input[type=checkbox]', :count => 2
+      assert_select 'form ul.inputs-list input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes allows giving wrapper class at input level only' do
-    swap SimpleForm, :collection_wrapper_tag => :ul do
-      with_input_for @user, :active, :check_boxes, :collection_wrapper_class => 'items-list'
+    swap SimpleForm, collection_wrapper_tag: :ul do
+      with_input_for @user, :active, :check_boxes, collection_wrapper_class: 'items-list'
 
-      assert_select 'form ul.items-list input[type=checkbox]', :count => 2
+      assert_select 'form ul.items-list input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes uses both configured and given wrapper classes for wrapper tag' do
-    swap SimpleForm, :collection_wrapper_tag => :ul, :collection_wrapper_class => 'inputs-list' do
-      with_input_for @user, :active, :check_boxes, :collection_wrapper_class => 'items-list'
+    swap SimpleForm, collection_wrapper_tag: :ul, collection_wrapper_class: 'inputs-list' do
+      with_input_for @user, :active, :check_boxes, collection_wrapper_class: 'items-list'
 
-      assert_select 'form ul.inputs-list.items-list input[type=checkbox]', :count => 2
+      assert_select 'form ul.inputs-list.items-list input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes wraps each item in the configured item wrapper tag' do
-    swap SimpleForm, :item_wrapper_tag => :li do
+    swap SimpleForm, item_wrapper_tag: :li do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form li input[type=checkbox]', :count => 2
+      assert_select 'form li input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes does not wrap items when configured with falsy values' do
-    swap SimpleForm, :item_wrapper_tag => false do
+    swap SimpleForm, item_wrapper_tag: false do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form input[type=checkbox]', :count => 2
+      assert_select 'form input[type=checkbox]', count: 2
       assert_no_select 'form li'
     end
   end
 
   test 'input check boxes allows overriding the item wrapper tag at input level' do
-    swap SimpleForm, :item_wrapper_tag => :li do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_tag => :dl
+    swap SimpleForm, item_wrapper_tag: :li do
+      with_input_for @user, :active, :check_boxes, item_wrapper_tag: :dl
 
-      assert_select 'form dl input[type=checkbox]', :count => 2
+      assert_select 'form dl input[type=checkbox]', count: 2
       assert_no_select 'form li'
     end
   end
 
   test 'input check boxes allows disabling the item wrapper tag at input level' do
-    swap SimpleForm, :item_wrapper_tag => :ul do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_tag => false
+    swap SimpleForm, item_wrapper_tag: :ul do
+      with_input_for @user, :active, :check_boxes, item_wrapper_tag: false
 
-      assert_select 'form input[type=checkbox]', :count => 2
+      assert_select 'form input[type=checkbox]', count: 2
       assert_no_select 'form li'
     end
   end
@@ -151,41 +157,41 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   test 'input check boxes wraps items in a span tag by default' do
     with_input_for @user, :active, :check_boxes
 
-    assert_select 'form span input[type=checkbox]', :count => 2
+    assert_select 'form span input[type=checkbox]', count: 2
   end
 
   test 'input check boxes renders the item wrapper tag with a default class "checkbox"' do
-    with_input_for @user, :active, :check_boxes, :item_wrapper_tag => :li
+    with_input_for @user, :active, :check_boxes, item_wrapper_tag: :li
 
-    assert_select 'form li.checkbox input[type=checkbox]', :count => 2
+    assert_select 'form li.checkbox input[type=checkbox]', count: 2
   end
 
   test 'input check boxes renders the item wrapper tag with the configured item wrapper class' do
-    swap SimpleForm, :item_wrapper_tag => :li, :item_wrapper_class => 'item' do
+    swap SimpleForm, item_wrapper_tag: :li, item_wrapper_class: 'item' do
       with_input_for @user, :active, :check_boxes
 
-      assert_select 'form li.checkbox.item input[type=checkbox]', :count => 2
+      assert_select 'form li.checkbox.item input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes allows giving item wrapper class at input level only' do
-    swap SimpleForm, :item_wrapper_tag => :li do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_class => 'item'
+    swap SimpleForm, item_wrapper_tag: :li do
+      with_input_for @user, :active, :check_boxes, item_wrapper_class: 'item'
 
-      assert_select 'form li.checkbox.item input[type=checkbox]', :count => 2
+      assert_select 'form li.checkbox.item input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes uses both configured and given item wrapper classes for item wrapper tag' do
-    swap SimpleForm, :item_wrapper_tag => :li, :item_wrapper_class => 'item' do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_class => 'inline'
+    swap SimpleForm, item_wrapper_tag: :li, item_wrapper_class: 'item' do
+      with_input_for @user, :active, :check_boxes, item_wrapper_class: 'inline'
 
-      assert_select 'form li.checkbox.item.inline input[type=checkbox]', :count => 2
+      assert_select 'form li.checkbox.item.inline input[type=checkbox]', count: 2
     end
   end
 
   test 'input check boxes respects the nested boolean style config, generating nested label > input' do
-    swap SimpleForm, :boolean_style => :nested do
+    swap SimpleForm, boolean_style: :nested do
       with_input_for @user, :active, :check_boxes
 
       assert_select 'label.checkbox > input#user_active_true[type=checkbox]'
@@ -197,7 +203,7 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   end
 
   test 'input check boxes with nested style overrides configured item wrapper tag, forcing the :label' do
-    swap SimpleForm, :boolean_style => :nested, :item_wrapper_tag => :li do
+    swap SimpleForm, boolean_style: :nested, item_wrapper_tag: :li do
       with_input_for @user, :active, :check_boxes
 
       assert_select 'label.checkbox > input'
@@ -206,8 +212,8 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   end
 
   test 'input check boxes with nested style overrides given item wrapper tag, forcing the :label' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_tag => :li
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :check_boxes, item_wrapper_tag: :li
 
       assert_select 'label.checkbox > input'
       assert_no_select 'li'
@@ -215,8 +221,8 @@ class CollectionCheckBoxesInputTest < ActionView::TestCase
   end
 
   test 'input check boxes with nested style accepts giving extra wrapper classes' do
-    swap SimpleForm, :boolean_style => :nested do
-      with_input_for @user, :active, :check_boxes, :item_wrapper_class => "inline"
+    swap SimpleForm, boolean_style: :nested do
+      with_input_for @user, :active, :check_boxes, item_wrapper_class: "inline"
 
       assert_select 'label.checkbox.inline > input'
     end