simple_form 3.0.4 → 3.1.0.rc2

data/test/form_builder/error_test.rb

@@ -14,69 +14,65 @@ class ErrorTest < ActionView::TestCase
     end
   end
 
-  def with_custom_error_for(object, *args)
-    with_form_for(object, *args)
-  end
-
-  test 'error should not generate content for attribute without errors' do
+  test 'error does not generate content for attribute without errors' do
     with_error_for @user, :active
     assert_no_select 'span.error'
   end
 
-  test 'error should not generate messages when object is not present' do
+  test 'error does not generate messages when object is not present' do
     with_error_for :project, :name
     assert_no_select 'span.error'
   end
 
-  test "error should not generate messages when object doesn't respond to errors method" do
+  test "error does not generate messages when object doesn't respond to errors method" do
     @user.instance_eval { undef errors }
     with_error_for @user, :name
     assert_no_select 'span.error'
   end
 
-  test 'error should generate messages for attribute with single error' do
+  test 'error generates messages for attribute with single error' do
     with_error_for @user, :name
-    assert_select 'span.error', "can&#39;t be blank"
+    assert_select 'span.error', "can't be blank"
   end
 
-  test 'error should generate messages for attribute with one error when using first' do
+  test 'error generates messages for attribute with one error when using first' do
     swap SimpleForm, error_method: :first do
       with_error_for @user, :age
       assert_select 'span.error', 'is not a number'
     end
   end
 
-  test 'error should generate messages for attribute with several errors when using to_sentence' do
+  test 'error generates messages for attribute with several errors when using to_sentence' do
     swap SimpleForm, error_method: :to_sentence do
       with_error_for @user, :age
       assert_select 'span.error', 'is not a number and must be greater than 18'
     end
   end
 
-  test 'error should be able to pass html options' do
+  test 'error is able to pass html options' do
     with_error_for @user, :name, id: 'error', class: 'yay'
     assert_select 'span#error.error.yay'
   end
 
-  test 'error should not modify the options hash' do
+  test 'error does not modify the options hash' do
     options = { id: 'error', class: 'yay' }
     with_error_for @user, :name, options
     assert_select 'span#error.error.yay'
     assert_equal({ id: 'error', class: 'yay' }, options)
   end
 
-  test 'error should find errors on attribute and association' do
+  test 'error finds errors on attribute and association' do
     with_error_for @user, :company_id, as: :select,
       error_method: :to_sentence, reflection: Association.new(Company, :company, {})
     assert_select 'span.error', 'must be valid and company must be present'
   end
 
-  test 'error should generate an error tag with a clean HTML' do
+  test 'error generates an error tag with a clean HTML' do
     with_error_for @user, :name
     assert_no_select 'span.error[error_html]'
   end
 
-  test 'error should generate an error tag with a clean HTML when errors options are present' do
+  test 'error generates an error tag with a clean HTML when errors options are present' do
     with_error_for @user, :name, error_tag: :p, error_prefix: 'Name', error_method: :first
     assert_no_select 'p.error[error_html]'
     assert_no_select 'p.error[error_tag]'
@@ -84,47 +80,38 @@ class ErrorTest < ActionView::TestCase
     assert_no_select 'p.error[error_method]'
   end
 
-  test 'error should escape error prefix text' do
+  test 'error escapes error prefix text' do
     with_error_for @user, :name, error_prefix: '<b>Name</b>'
-    assert_select 'span.error', "&lt;b&gt;Name&lt;/b&gt; can&#39;t be blank"
-  end
-
-  test 'error escapes error text' do
-    @user.errors.merge!(action: ['must not contain <b>markup</b>'])
-
-    with_error_for @user, :action
-
-    assert_select 'span.error'
-    assert_no_select 'span.error b', 'markup'
+    assert_select 'span.error', "&lt;b&gt;Name&lt;/b&gt; can't be blank"
   end
 
-  test 'error should generate an error message with raw HTML tags' do
+  test 'error generates an error message with raw HTML tags' do
     with_error_for @user, :name, error_prefix: '<b>Name</b>'.html_safe
-    assert_select 'span.error', "Name can&#39;t be blank"
+    assert_select 'span.error', "Name can't be blank"
     assert_select 'span.error b', "Name"
   end
 
   # FULL ERRORS
 
-  test 'full error should generate an full error tag for the attribute' do
+  test 'full error generates a full error tag for the attribute' do
     with_full_error_for @user, :name
-    assert_select 'span.error', "Super User Name! can&#39;t be blank"
+    assert_select 'span.error', "Super User Name! can't be blank"
   end
 
-  test 'full error should generate an full error tag with a clean HTML' do
+  test 'full error generates a full error tag with a clean HTML' do
     with_full_error_for @user, :name
     assert_no_select 'span.error[error_html]'
   end
 
-  test 'full error should allow passing options to full error tag' do
+  test 'full error allows passing options to full error tag' do
     with_full_error_for @user, :name, id: 'name_error', error_prefix: "Your name"
-    assert_select 'span.error#name_error', "Your name can&#39;t be blank"
+    assert_select 'span.error#name_error', "Your name can't be blank"
   end
 
-  test 'full error should not modify the options hash' do
+  test 'full error does not modify the options hash' do
     options = { id: 'name_error' }
     with_full_error_for @user, :name, options
-    assert_select 'span.error#name_error', "Super User Name! can&#39;t be blank"
+    assert_select 'span.error#name_error', "Super User Name! can't be blank"
     assert_equal({ id: 'name_error' }, options)
   end
 
@@ -133,36 +120,68 @@ class ErrorTest < ActionView::TestCase
   test 'error with custom wrappers works' do
     swap_wrapper do
       with_error_for @user, :name
-      assert_select 'span.omg_error', "can&#39;t be blank"
+      assert_select 'span.omg_error', "can't be blank"
+    end
+  end
+
+  # FULL_ERROR_WRAPPER
+
+  test 'full error finds errors on association' do
+    swap_wrapper :default, self.custom_wrapper_with_full_error do
+      with_form_for @user, :company_id, as: :select
+      assert_select 'span.error', 'Company must be valid'
+    end
+  end
+
+  test 'full error can be disabled' do
+    swap_wrapper :default, self.custom_wrapper_with_full_error do
+      with_form_for @user, :company_id, as: :select, full_error: false
+      assert_no_select 'span.error'
+    end
+  end
+
+  test 'full error can be disabled setting error to false' do
+    swap_wrapper :default, self.custom_wrapper_with_full_error do
+      with_form_for @user, :company_id, as: :select, error: false
+      assert_no_select 'span.error'
     end
   end
 
   # CUSTOM ERRORS
 
   test 'input with custom error works' do
-    with_custom_error_for(@user, :name, error: "Super User Name! can't be blank")
+    error_text = "Super User Name! can't be blank"
+    with_form_for @user, :name, error: error_text
 
-    assert_select 'span.error', "Super User Name! can&#39;t be blank"
+    assert_select 'span.error', error_text
+  end
+
+  test 'input with error option as true does not use custom error' do
+    with_form_for @user, :name, error: true
+
+    assert_select 'span.error', "can't be blank"
   end
 
   test 'input with custom error does not generate the error if there is no error on the attribute' do
-    error_text = "Super User Active! can't be blank"
-    with_form_for @user, :active, error: error_text
+    with_form_for @user, :active, error: "Super User Active! can't be blank"
 
     assert_no_select 'span.error'
   end
 
-  test 'input with custom error escapes the error text' do
-    with_form_for @user, :name, error: 'error must not contain <b>markup</b>'
+  test 'input with custom error works when using full_error component' do
+    swap_wrapper :default, self.custom_wrapper_with_full_error do
+      error_text = "Super User Name! can't be blank"
+      with_form_for @user, :name, error: error_text
 
-    assert_select 'span.error'
-    assert_no_select 'span.error b', 'markup'
+      assert_select 'span.error', error_text
+    end
   end
 
-  test 'input with custom error does not escape the error text if it is safe' do
-    with_form_for @user, :name, error: 'error must contain <b>markup</b>'.html_safe
+  test 'input with custom error when using full_error component does not generate the error if there is no error on the attribute' do
+    swap_wrapper :default, self.custom_wrapper_with_full_error do
+      with_form_for @user, :active, error: "Super User Active! can't be blank"
 
-    assert_select 'span.error'
-    assert_select 'span.error b', 'markup'
+      assert_no_select 'span.error'
+    end
   end
 end

data/test/form_builder/general_test.rb

@@ -8,7 +8,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'nested simple fields should yield an instance of FormBuilder' do
+  test 'nested simple fields yields an instance of FormBuilder' do
     simple_form_for :user do |f|
       f.simple_fields_for :posts do |posts_form|
         assert posts_form.instance_of?(SimpleForm::FormBuilder)
@@ -22,7 +22,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should work without controller' do
+  test 'builder works without controller' do
     stub_any_instance ActionView::TestCase, :controller, nil do
       simple_form_for @user do |f|
         assert f.input(:name)
@@ -30,7 +30,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder input should allow a block to configure input' do
+  test 'builder input allows a block to configure input' do
     with_form_for @user, :name do
       text_field_tag :foo, :bar, id: :cool
     end
@@ -38,7 +38,7 @@ class FormBuilderTest < ActionView::TestCase
     assert_select 'input#cool'
   end
 
-  test 'builder should allow adding custom input mappings for default input types' do
+  test 'builder allows adding custom input mappings for default input types' do
     swap SimpleForm, input_mappings: { /count$/ => :integer } do
       with_form_for @user, :post_count
       assert_no_select 'form input#user_post_count.string'
@@ -46,7 +46,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should allow to skip input_type class' do
+  test 'builder allows to skip input_type class' do
     swap SimpleForm, generate_additional_classes_for: [:label, :wrapper] do
       with_form_for @user, :post_count
       assert_no_select "form input#user_post_count.integer"
@@ -54,7 +54,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should allow to add additional classes only for wrapper' do
+  test 'builder allows to add additional classes only for wrapper' do
     swap SimpleForm, generate_additional_classes_for: [:wrapper] do
       with_form_for @user, :post_count
       assert_no_select "form input#user_post_count.string"
@@ -63,7 +63,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should allow adding custom input mappings for integer input types' do
+  test 'builder allows adding custom input mappings for integer input types' do
     swap SimpleForm, input_mappings: { /lock_version/ => :hidden } do
       with_form_for @user, :lock_version
       assert_no_select 'form input#user_lock_version.integer'
@@ -101,82 +101,87 @@ class FormBuilderTest < ActionView::TestCase
   end
 
   # INPUT TYPES
-  test 'builder should generate text fields for string columns' do
+  test 'builder generates text fields for string columns' do
     with_form_for @user, :name
     assert_select 'form input#user_name.string'
   end
 
-  test 'builder should generate text areas for text columns' do
+  test 'builder generates text areas for text columns' do
     with_form_for @user, :description
     assert_select 'form textarea#user_description.text'
   end
 
-  test 'builder should generate a checkbox for boolean columns' do
+  test 'builder generates a checkbox for boolean columns' do
     with_form_for @user, :active
     assert_select 'form input[type=checkbox]#user_active.boolean'
   end
 
-  test 'builder should use integer text field for integer columns' do
+  test 'builder uses integer text field for integer columns' do
     with_form_for @user, :age
     assert_select 'form input#user_age.numeric.integer'
   end
 
-  test 'builder should generate decimal text field for decimal columns' do
+  test 'builder generates decimal text field for decimal columns' do
     with_form_for @user, :credit_limit
     assert_select 'form input#user_credit_limit.numeric.decimal'
   end
 
-  test 'builder should generate password fields for columns that matches password' do
+  test 'builder generates uuid fields for uuid columns' do
+    with_form_for @user, :uuid
+    assert_select 'form input#user_uuid.string.uuid'
+  end
+
+  test 'builder generates password fields for columns that matches password' do
     with_form_for @user, :password
     assert_select 'form input#user_password.password'
   end
 
-  test 'builder should generate country fields for columns that matches country' do
+  test 'builder generates country fields for columns that matches country' do
     with_form_for @user, :residence_country
     assert_select 'form select#user_residence_country.country'
   end
 
-  test 'builder should generate time_zone fields for columns that matches time_zone' do
+  test 'builder generates time_zone fields for columns that matches time_zone' do
     with_form_for @user, :time_zone
     assert_select 'form select#user_time_zone.time_zone'
   end
 
-  test 'builder should generate email fields for columns that matches email' do
+  test 'builder generates email fields for columns that matches email' do
     with_form_for @user, :email
     assert_select 'form input#user_email.string.email'
   end
 
-  test 'builder should generate tel fields for columns that matches phone' do
+  test 'builder generates tel fields for columns that matches phone' do
     with_form_for @user, :phone_number
     assert_select 'form input#user_phone_number.string.tel'
   end
 
-  test 'builder should generate url fields for columns that matches url' do
+  test 'builder generates url fields for columns that matches url' do
     with_form_for @user, :url
     assert_select 'form input#user_url.string.url'
   end
 
-  test 'builder should generate date select for date columns' do
+  test 'builder generates date select for date columns' do
     with_form_for @user, :born_at
     assert_select 'form select#user_born_at_1i.date'
   end
 
-  test 'builder should generate time select for time columns' do
+  test 'builder generates time select for time columns' do
     with_form_for @user, :delivery_time
     assert_select 'form select#user_delivery_time_4i.time'
   end
 
-  test 'builder should generate datetime select for datetime columns' do
+  test 'builder generates datetime select for datetime columns' do
     with_form_for @user, :created_at
     assert_select 'form select#user_created_at_1i.datetime'
   end
 
-  test 'builder should generate datetime select for timestamp columns' do
+  test 'builder generates datetime select for timestamp columns' do
     with_form_for @user, :updated_at
     assert_select 'form select#user_updated_at_1i.datetime'
   end
 
-  test 'builder should generate file for file columns' do
+  test 'builder generates file for file columns' do
     @user.avatar = MiniTest::Mock.new
     @user.avatar.expect(:public_filename, true)
 
@@ -184,7 +189,7 @@ class FormBuilderTest < ActionView::TestCase
     assert_select 'form input#user_avatar.file'
   end
 
-  test 'builder should generate file for attributes that are real db columns but have file methods' do
+  test 'builder generates file for attributes that are real db columns but have file methods' do
     @user.home_picture = MiniTest::Mock.new
     @user.home_picture.expect(:mounted_as, true)
 
@@ -192,12 +197,12 @@ class FormBuilderTest < ActionView::TestCase
     assert_select 'form input#user_home_picture.file'
   end
 
-  test 'build should generate select if a collection is given' do
+  test 'build generates select if a collection is given' do
     with_form_for @user, :age, collection: 1..60
     assert_select 'form select#user_age.select'
   end
 
-  test 'builder should allow overriding default input type for text' do
+  test 'builder allows overriding default input type for text' do
     with_form_for @user, :name, as: :text
     assert_no_select 'form input#user_name'
     assert_select 'form textarea#user_name.text'
@@ -212,25 +217,25 @@ class FormBuilderTest < ActionView::TestCase
   end
 
   # COMMON OPTIONS
-  test 'builder should add chosen form class' do
+  test 'builder adds chosen form class' do
     swap SimpleForm, form_class: :my_custom_class do
       with_form_for @user, :name
       assert_select 'form.my_custom_class'
     end
   end
 
-  test 'builder should allow passing options to input' do
+  test 'builder allows passing options to input' do
     with_form_for @user, :name, input_html: { class: 'my_input', id: 'my_input' }
     assert_select 'form input#my_input.my_input.string'
   end
 
-  test 'builder should not propagate input options to wrapper' do
+  test 'builder does not propagate input options to wrapper' do
     with_form_for @user, :name, input_html: { class: 'my_input', id: 'my_input' }
     assert_no_select 'form div.input.my_input.string'
     assert_select 'form input#my_input.my_input.string'
   end
 
-  test 'builder should not propagate input options to wrapper with custom wrapper' do
+  test 'builder does not propagate input options to wrapper with custom wrapper' do
     swap_wrapper :default, self.custom_wrapper_with_wrapped_input do
       with_form_for @user, :name, input_html: { class: 'my_input' }
       assert_no_select 'form div.input.my_input'
@@ -238,7 +243,7 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should not propagate label options to wrapper with custom wrapper' do
+  test 'builder does not propagate label options to wrapper with custom wrapper' do
     swap_wrapper :default, self.custom_wrapper_with_wrapped_label do
       with_form_for @user, :name, label_html: { class: 'my_label' }
       assert_no_select 'form div.label.my_label'
@@ -246,42 +251,42 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should generate a input with label' do
+  test 'builder generates an input with label' do
     with_form_for @user, :name
     assert_select 'form label.string[for=user_name]', /Name/
   end
 
-  test 'builder should be able to disable the label for a input' do
+  test 'builder is able to disable the label for an input' do
     with_form_for @user, :name, label: false
     assert_no_select 'form label'
   end
 
-  test 'builder should be able to disable the label for an input and return a html safe string' do
+  test 'builder is able to disable the label for an input and return a html safe string' do
     with_form_for @user, :name, label: false, wrapper: custom_wrapper_with_wrapped_label_input
     assert_select 'form input#user_name'
   end
 
-  test 'builder should use custom label' do
+  test 'builder uses custom label' do
     with_form_for @user, :name, label: 'Yay!'
     assert_select 'form label', /Yay!/
   end
 
-  test 'builder should pass options to label' do
+  test 'builder passes options to label' do
     with_form_for @user, :name, label_html: { id: "cool" }
     assert_select 'form label#cool', /Name/
   end
 
-  test 'builder should not generate hints for a input' do
+  test 'builder does not generate hints for an input' do
     with_form_for @user, :name
     assert_no_select 'span.hint'
   end
 
-  test 'builder should be able to add a hint for a input' do
+  test 'builder is able to add a hint for an input' do
     with_form_for @user, :name, hint: 'test'
     assert_select 'span.hint', 'test'
   end
 
-  test 'builder should be able to disable a hint even if it exists in i18n' do
+  test 'builder is able to disable a hint even if it exists in i18n' do
     store_translations(:en, simple_form: { hints: { name: 'Hint test' } }) do
       stub_any_instance(SimpleForm::Inputs::Base, :hint, -> { raise 'Never' }) do
         with_form_for @user, :name, hint: false
@@ -290,32 +295,32 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test 'builder should pass options to hint' do
+  test 'builder passes options to hint' do
     with_form_for @user, :name, hint: 'test', hint_html: { id: "cool" }
     assert_select 'span.hint#cool', 'test'
   end
 
-  test 'builder should generate errors for attribute without errors' do
+  test 'builder generates errors for attribute without errors' do
     with_form_for @user, :credit_limit
     assert_no_select 'span.errors'
   end
 
-  test 'builder should generate errors for attribute with errors' do
+  test 'builder generates errors for attribute with errors' do
     with_form_for @user, :name
-    assert_select 'span.error', "can&#39;t be blank"
+    assert_select 'span.error', "can't be blank"
   end
 
-  test 'builder should be able to disable showing errors for a input' do
+  test 'builder is able to disable showing errors for an input' do
     with_form_for @user, :name, error: false
     assert_no_select 'span.error'
   end
 
-  test 'builder should pass options to errors' do
+  test 'builder passes options to errors' do
     with_form_for @user, :name, error_html: { id: "cool" }
-    assert_select 'span.error#cool', "can&#39;t be blank"
+    assert_select 'span.error#cool', "can't be blank"
   end
 
-  test 'placeholder should not be generated when set to false' do
+  test 'placeholder does not be generated when set to false' do
     store_translations(:en, simple_form: { placeholders: { user: {
       name: 'Name goes here'
     } } }) do
@@ -326,14 +331,14 @@ class FormBuilderTest < ActionView::TestCase
 
   # DEFAULT OPTIONS
   [:input, :input_field].each do |method|
-    test "builder should receive a default argument and pass it to the inputs when calling '#{method}'" do
+    test "builder receives a default argument and pass it to the inputs when calling '#{method}'" do
       with_concat_form_for @user, defaults: { input_html: { class: 'default_class' } } do |f|
         f.send(method, :name)
       end
       assert_select 'input.default_class'
     end
 
-    test "builder should receive a default argument and pass it to the inputs without changing the defaults when calling '#{method}'" do
+    test "builder receives a default argument and pass it to the inputs without changing the defaults when calling '#{method}'" do
       with_concat_form_for @user, defaults: { input_html: { class: 'default_class', id: 'default_id' } } do |f|
         concat(f.send(method, :name))
         concat(f.send(method, :credit_limit))
@@ -343,7 +348,7 @@ class FormBuilderTest < ActionView::TestCase
       assert_no_select "input.string[name='user[credit_limit]']"
     end
 
-    test "builder should receive a default argument and pass it to the inputs and nested form when calling '#{method}'" do
+    test "builder receives a default argument and pass it to the inputs and nested form when calling '#{method}'" do
       @user.company = Company.new(1, 'Empresa')
 
       with_concat_form_for @user, defaults: { input_html: { class: 'default_class' } } do |f|
@@ -358,28 +363,28 @@ class FormBuilderTest < ActionView::TestCase
     end
   end
 
-  test "builder should receive a default argument and pass it to the inputs when calling 'input', respecting the specific options" do
+  test "builder receives a default argument and pass it to the inputs when calling 'input', respecting the specific options" do
     with_concat_form_for @user, defaults: { input_html: { class: 'default_class' } } do |f|
       f.input :name, input_html: { id: 'specific_id' }
     end
     assert_select 'input.default_class#specific_id'
   end
 
-  test "builder should receive a default argument and pass it to the inputs when calling 'input_field', respecting the specific options" do
+  test "builder receives a default argument and pass it to the inputs when calling 'input_field', respecting the specific options" do
     with_concat_form_for @user, defaults: { input_html: { class: 'default_class' } } do |f|
       f.input_field :name, id: 'specific_id'
     end
     assert_select 'input.default_class#specific_id'
   end
 
-  test "builder should receive a default argument and pass it to the inputs when calling 'input', overwriting the defaults with specific options" do
+  test "builder receives a default argument and pass it to the inputs when calling 'input', overwriting the defaults with specific options" do
     with_concat_form_for @user, defaults: { input_html: { class: 'default_class', id: 'default_id' } } do |f|
       f.input :name, input_html: { id: 'specific_id' }
     end
     assert_select 'input.default_class#specific_id'
   end
 
-  test "builder should receive a default argument and pass it to the inputs when calling 'input_field', overwriting the defaults with specific options" do
+  test "builder receives a default argument and pass it to the inputs when calling 'input_field', overwriting the defaults with specific options" do
     with_concat_form_for @user, defaults: { input_html: { class: 'default_class', id: 'default_id' } } do |f|
       f.input_field :name, id: 'specific_id'
     end
@@ -387,24 +392,24 @@ class FormBuilderTest < ActionView::TestCase
   end
 
   # WITHOUT OBJECT
-  test 'builder should generate properly when object is not present' do
+  test 'builder generates properly when object is not present' do
     with_form_for :project, :name
     assert_select 'form input.string#project_name'
   end
 
-  test 'builder should generate password fields based on attribute name when object is not present' do
+  test 'builder generates password fields based on attribute name when object is not present' do
     with_form_for :project, :password_confirmation
     assert_select 'form input[type=password].password#project_password_confirmation'
   end
 
-  test 'builder should generate text fields by default for all attributes when object is not present' do
+  test 'builder generates text fields by default for all attributes when object is not present' do
     with_form_for :project, :created_at
     assert_select 'form input.string#project_created_at'
     with_form_for :project, :budget
     assert_select 'form input.string#project_budget'
   end
 
-  test 'builder should allow overriding input type when object is not present' do
+  test 'builder allows overriding input type when object is not present' do
     with_form_for :project, :created_at, as: :datetime
     assert_select 'form select.datetime#project_created_at_1i'
     with_form_for :project, :budget, as: :decimal
@@ -412,24 +417,24 @@ class FormBuilderTest < ActionView::TestCase
   end
 
   # CUSTOM FORM BUILDER
-  test 'custom builder should inherit mappings' do
+  test 'custom builder inherits mappings' do
     with_custom_form_for @user, :email
     assert_select 'form input[type=email]#user_email.custom'
   end
 
-  test 'form with CustomMapTypeFormBuilder should use custom map type builder' do
+  test 'form with CustomMapTypeFormBuilder uses custom map type builder' do
     with_concat_custom_mapping_form_for(:user) do |user|
       assert user.instance_of?(CustomMapTypeFormBuilder)
     end
   end
 
-  test 'form with CustomMapTypeFormBuilder should use custom mapping' do
+  test 'form with CustomMapTypeFormBuilder uses custom mapping' do
     with_concat_custom_mapping_form_for(:user) do |user|
       assert_equal SimpleForm::Inputs::StringInput, user.class.mappings[:custom_type]
     end
   end
 
-  test 'form without CustomMapTypeFormBuilder should not use custom mapping' do
+  test 'form without CustomMapTypeFormBuilder does not use custom mapping' do
     with_concat_form_for(:user) do |user|
       assert_nil user.class.mappings[:custom_type]
     end