simple_form 3.1.0.rc1 → 3.1.0.rc2

data/test/inputs/datetime_input_test.rb

@@ -3,49 +3,49 @@ require 'test_helper'
 
 # Tests for datetime, date and time inputs when HTML5 compatibility is enabled in the wrapper.
 class DateTimeInputWithHtml5Test < ActionView::TestCase
-  test 'input should generate a datetime input for datetime attributes if HTML5 compatibility is explicitly enbled' do
+  test 'input generates a datetime input for datetime attributes if HTML5 compatibility is explicitly enbled' do
     with_input_for @user, :created_at, :datetime, html5: true
 
     assert_select 'input[type="datetime"]'
   end
 
-  test 'input should generate a datetime select for datetime attributes' do
+  test 'input generates a datetime select for datetime attributes' do
     with_input_for @user, :created_at, :datetime
 
     assert_select 'select.datetime'
   end
 
-  test 'input should generate a date input for date attributes if HTML5 compatibility is explicitly enbled' do
+  test 'input generates a date input for date attributes if HTML5 compatibility is explicitly enbled' do
     with_input_for @user, :born_at, :date, html5: true
 
     assert_select 'input[type="date"]'
   end
 
-  test 'input should generate a date select for date attributes' do
+  test 'input generates a date select for date attributes' do
     with_input_for @user, :born_at, :date
 
     assert_select 'select.date'
   end
 
-  test 'input should generate a time input for time attributes if HTML5 compatibility is explicitly enbled' do
+  test 'input generates a time input for time attributes if HTML5 compatibility is explicitly enbled' do
     with_input_for @user, :delivery_time, :time, html5: true
 
     assert_select 'input[type="time"]'
   end
 
-  test 'input should generate a time select for time attributes' do
+  test 'input generates a time select for time attributes' do
     with_input_for @user, :delivery_time, :time
 
     assert_select 'select.time'
   end
 
-  test 'input should generate required html attribute' do
+  test 'input generates required html attribute' do
     with_input_for @user, :delivery_time, :time, required: true, html5: true
     assert_select 'input.required'
     assert_select 'input[required]'
   end
 
-  test 'input should have an aria-required html attribute' do
+  test 'input has an aria-required html attribute' do
     with_input_for @user, :delivery_time, :time, required: true, html5: true
     assert_select 'input[aria-required=true]'
   end
@@ -53,7 +53,7 @@ end
 
 # Tests for datetime, date and time inputs when HTML5 compatibility is enabled in the wrapper.
 class DateTimeInputWithoutHtml5Test < ActionView::TestCase
-  test 'input should generate a datetime select by default for datetime attributes' do
+  test 'input generates a datetime select by default for datetime attributes' do
     swap_wrapper do
       with_input_for @user, :created_at, :datetime
       1.upto(5) do |i|
@@ -62,7 +62,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'input should be able to pass options to datetime select' do
+  test 'input is able to pass options to datetime select' do
     with_input_for @user, :created_at, :datetime, html5: false,
       disabled: true, prompt: { year: 'ano', month: 'mês', day: 'dia' }
 
@@ -72,7 +72,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     assert_select 'select.datetime option', 'dia'
   end
 
-  test 'input should generate a datetime input for datetime attributes if HTML5 compatibility is explicitly enabled' do
+  test 'input generates a datetime input for datetime attributes if HTML5 compatibility is explicitly enabled' do
     swap_wrapper do
       with_input_for @user, :created_at, :datetime, html5: true
 
@@ -80,7 +80,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'input should generate a date select for date attributes' do
+  test 'input generates a date select for date attributes' do
     swap_wrapper do
       with_input_for @user, :born_at, :date
       assert_select 'select.date#user_born_at_1i'
@@ -90,7 +90,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'input should be able to pass options to date select' do
+  test 'input is able to pass options to date select' do
     with_input_for @user, :born_at, :date, as: :date, html5: false,
       disabled: true, prompt: { year: 'ano', month: 'mês', day: 'dia' }
 
@@ -100,12 +100,12 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     assert_select 'select.date option', 'dia'
   end
 
-  test 'input should be able to pass :default to date select' do
+  test 'input is able to pass :default to date select' do
     with_input_for @user, :born_at, :date, default: Date.today, html5: false
     assert_select "select.date option[value=#{Date.today.year}][selected=selected]"
   end
 
-  test 'input should generate a date input for date attributes if HTML5 compatibility is explicitly enabled' do
+  test 'input generates a date input for date attributes if HTML5 compatibility is explicitly enabled' do
     swap_wrapper do
       with_input_for @user, :born_at, :date, html5: true
 
@@ -113,7 +113,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'input should generate a time select for time attributes' do
+  test 'input generates a time select for time attributes' do
     swap_wrapper do
       with_input_for @user, :delivery_time, :time
       assert_select 'input[type=hidden]#user_delivery_time_1i'
@@ -124,7 +124,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'input should be able to pass options to time select' do
+  test 'input is able to pass options to time select' do
     with_input_for @user, :delivery_time, :time, required: true, html5: false,
       disabled: true, prompt: { hour: 'hora', minute: 'minuto' }
 
@@ -133,7 +133,7 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     assert_select 'select.time option', 'minuto'
   end
 
-  test 'input should generate a time input for time attributes if HTML5 compatibility is explicitly enabled' do
+  test 'input generates a time input for time attributes if HTML5 compatibility is explicitly enabled' do
     swap_wrapper do
       with_input_for @user, :delivery_time, :time, html5: true
 
@@ -141,31 +141,31 @@ class DateTimeInputWithoutHtml5Test < ActionView::TestCase
     end
   end
 
-  test 'label should use i18n to get target for date input type' do
+  test 'label uses i18n to get target for date input type' do
     store_translations(:en, date: { order: ['month', 'day', 'year'] }) do
       with_input_for :project, :created_at, :date, html5: false
       assert_select 'label[for=project_created_at_2i]'
     end
   end
 
-  test 'label should use i18n to get target for datetime input type' do
+  test 'label uses i18n to get target for datetime input type' do
     store_translations(:en, date: { order: ['month', 'day', 'year'] }) do
       with_input_for :project, :created_at, :datetime, html5: false
       assert_select 'label[for=project_created_at_2i]'
     end
   end
 
-  test 'label should use order to get target when date input type' do
+  test 'label uses order to get target when date input type' do
     with_input_for :project, :created_at, :date, order: ['month', 'year', 'day'], html5: false
     assert_select 'label[for=project_created_at_2i]'
   end
 
-  test 'label should use order to get target when datetime input type' do
+  test 'label uses order to get target when datetime input type' do
     with_input_for :project, :created_at, :datetime, order: ['month', 'year', 'day'], html5: false
     assert_select 'label[for=project_created_at_2i]'
   end
 
-  test 'label should point to first option when time input type' do
+  test 'label points to first option when time input type' do
     with_input_for :project, :created_at, :time, html5: false
     assert_select 'label[for=project_created_at_4i]'
   end

data/test/inputs/disabled_test.rb

@@ -1,77 +1,77 @@
 require 'test_helper'
 
 class DisabledTest < ActionView::TestCase
-  test 'string input should be disabled when disabled option is true' do
+  test 'string input is disabled when disabled option is true' do
     with_input_for @user, :name, :string, disabled: true
     assert_select 'input.string.disabled[disabled]'
   end
 
-  test 'text input should be disabled when disabled option is true' do
+  test 'text input is disabled when disabled option is true' do
     with_input_for @user, :description, :text, disabled: true
     assert_select 'textarea.text.disabled[disabled]'
   end
 
-  test 'numeric input should be disabled when disabled option is true' do
+  test 'numeric input is disabled when disabled option is true' do
     with_input_for @user, :age, :integer, disabled: true
     assert_select 'input.integer.disabled[disabled]'
   end
 
-  test 'date input should be disabled when disabled option is true' do
+  test 'date input is disabled when disabled option is true' do
     with_input_for @user, :born_at, :date, disabled: true
     assert_select 'select.date.disabled[disabled]'
   end
 
-  test 'datetime input should be disabled when disabled option is true' do
+  test 'datetime input is disabled when disabled option is true' do
     with_input_for @user, :created_at, :datetime, disabled: true
     assert_select 'select.datetime.disabled[disabled]'
   end
 
-  test 'string input should not be disabled when disabled option is false' do
+  test 'string input does not be disabled when disabled option is false' do
     with_input_for @user, :name, :string, disabled: false
     assert_no_select 'input.string.disabled[disabled]'
   end
 
-  test 'text input should not be disabled when disabled option is false' do
+  test 'text input does not be disabled when disabled option is false' do
     with_input_for @user, :description, :text, disabled: false
     assert_no_select 'textarea.text.disabled[disabled]'
   end
 
-  test 'numeric input should not be disabled when disabled option is false' do
+  test 'numeric input does not be disabled when disabled option is false' do
     with_input_for @user, :age, :integer, disabled: false
     assert_no_select 'input.integer.disabled[disabled]'
   end
 
-  test 'date input should not be disabled when disabled option is false' do
+  test 'date input does not be disabled when disabled option is false' do
     with_input_for @user, :born_at, :date, disabled: false
     assert_no_select 'select.date.disabled[disabled]'
   end
 
-  test 'datetime input should not be disabled when disabled option is false' do
+  test 'datetime input does not be disabled when disabled option is false' do
     with_input_for @user, :created_at, :datetime, disabled: false
     assert_no_select 'select.datetime.disabled[disabled]'
   end
 
-  test 'string input should not be disabled when disabled option is not present' do
+  test 'string input does not be disabled when disabled option is not present' do
     with_input_for @user, :name, :string
     assert_no_select 'input.string.disabled[disabled]'
   end
 
-  test 'text input should not be disabled when disabled option is not present' do
+  test 'text input does not be disabled when disabled option is not present' do
     with_input_for @user, :description, :text
     assert_no_select 'textarea.text.disabled[disabled]'
   end
 
-  test 'numeric input should not be disabled when disabled option is not present' do
+  test 'numeric input does not be disabled when disabled option is not present' do
     with_input_for @user, :age, :integer
     assert_no_select 'input.integer.disabled[disabled]'
   end
 
-  test 'date input should not be disabled when disabled option is not present' do
+  test 'date input does not be disabled when disabled option is not present' do
     with_input_for @user, :born_at, :date
     assert_no_select 'select.date.disabled[disabled]'
   end
 
-  test 'datetime input should not be disabled when disabled option is not present' do
+  test 'datetime input does not be disabled when disabled option is not present' do
     with_input_for @user, :created_at, :datetime
     assert_no_select 'select.datetime.disabled[disabled]'
   end

data/test/inputs/discovery_test.rb

@@ -14,11 +14,13 @@ class DiscoveryTest < ActionView::TestCase
         Object.send :remove_const, :CustomizedInput
         Object.send :remove_const, :DeprecatedInput
         Object.send :remove_const, :CollectionSelectInput
+        CustomInputs.send :remove_const, :PasswordInput
+        CustomInputs.send :remove_const, :NumericInput
       end
     end
   end
 
-  test 'builder should not discover new inputs if cached' do
+  test 'builder does not discover new inputs if cached' do
     with_form_for @user, :name
     assert_select 'form input#user_name.string'
 
@@ -28,14 +30,14 @@ class DiscoveryTest < ActionView::TestCase
     end
   end
 
-  test 'builder should discover new inputs' do
+  test 'builder discovers new inputs' do
     discovery do
       with_form_for @user, :name, as: :customized
       assert_select 'form section input#user_name.string'
     end
   end
 
-  test 'builder should not discover new inputs if discovery is off' do
+  test 'builder does not discover new inputs if discovery is off' do
     with_form_for @user, :name
     assert_select 'form input#user_name.string'
 
@@ -47,20 +49,57 @@ class DiscoveryTest < ActionView::TestCase
     end
   end
 
-  test 'builder should discover new inputs from mappings if not cached' do
+  test 'builder discovers new inputs from mappings if not cached' do
     discovery do
       with_form_for @user, :name
       assert_select 'form section input#user_name.string'
     end
   end
 
-  test 'builder should discover new inputs from internal fallbacks if not cached' do
+  test 'builder discovers new inputs from internal fallbacks if not cached' do
     discovery do
       with_form_for @user, :age
       assert_select 'form section input#user_age.numeric.integer'
     end
   end
 
+  test 'builder discovers new maped inputs from configured namespaces if not cached' do
+    discovery do
+      swap SimpleForm, custom_inputs_namespaces: ['CustomInputs'] do
+        with_form_for @user, :password
+        assert_select 'form input#user_password.password-custom-input'
+      end
+    end
+  end
+
+  test 'builder discovers new maped inputs from configured namespaces before the ones from top level namespace' do
+    discovery do
+      swap SimpleForm, custom_inputs_namespaces: ['CustomInputs'] do
+        with_form_for @user, :age
+        assert_select 'form input#user_age.numeric-custom-input'
+      end
+    end
+  end
+
+  test 'builder discovers new custom inputs from configured namespace before the ones from top level namespace' do
+    discovery do
+      swap SimpleForm, custom_inputs_namespaces: ['CustomInputs'] do
+        with_form_for @user, :name, as: 'customized'
+        assert_select 'form input#user_name.customized-namespace-custom-input'
+      end
+    end
+  end
+
+  test 'raises error when configured namespace does not exists' do
+    discovery do
+      swap SimpleForm, custom_inputs_namespaces: ['InvalidNamespace'] do
+        assert_raise NameError do
+          with_form_for @user, :age
+        end
+      end
+    end
+  end
+
   test 'new inputs can override the input_html_options' do
     discovery do
       with_form_for @user, :active, as: :select

data/test/inputs/file_input_test.rb

@@ -2,12 +2,12 @@
 require 'test_helper'
 
 class FileInputTest < ActionView::TestCase
-  test 'input should generate a file field' do
+  test 'input generates a file field' do
     with_input_for @user, :name, :file
     assert_select 'input#user_name[type=file]'
   end
 
-  test "input should generate a file field that doesn't accept placeholder" do
+  test "input generates a file field that doesn't accept placeholder" do
     store_translations(:en, simple_form: { placeholders: { user: { name: "text" } } }) do
       with_input_for @user, :name, :file
       assert_no_select 'input[placeholder]'

data/test/inputs/general_test.rb

@@ -2,7 +2,7 @@
 require 'test_helper'
 
 class InputTest < ActionView::TestCase
-  test 'input should generate css class based on default input type' do
+  test 'input generates css class based on default input type' do
     with_input_for @user, :name, :string
     assert_select 'input.string'
     with_input_for @user, :description, :text
@@ -15,7 +15,7 @@ class InputTest < ActionView::TestCase
     assert_select 'select.datetime'
   end
 
-  test 'string input should generate autofocus attribute when autofocus option is true' do
+  test 'string input generates autofocus attribute when autofocus option is true' do
     with_input_for @user, :name, :string, autofocus: true
     assert_select 'input.string[autofocus]'
   end
@@ -36,94 +36,94 @@ class InputTest < ActionView::TestCase
     end
   end
 
-  test 'text input should generate autofocus attribute when autofocus option is true' do
+  test 'text input generates autofocus attribute when autofocus option is true' do
     with_input_for @user, :description, :text, autofocus: true
     assert_select 'textarea.text[autofocus]'
   end
 
-  test 'numeric input should generate autofocus attribute when autofocus option is true' do
+  test 'numeric input generates autofocus attribute when autofocus option is true' do
     with_input_for @user, :age, :integer, autofocus: true
     assert_select 'input.integer[autofocus]'
   end
 
-  test 'date input should generate autofocus attribute when autofocus option is true' do
+  test 'date input generates autofocus attribute when autofocus option is true' do
     with_input_for @user, :born_at, :date, autofocus: true
     assert_select 'select.date[autofocus]'
   end
 
-  test 'datetime input should generate autofocus attribute when autofocus option is true' do
+  test 'datetime input generates autofocus attribute when autofocus option is true' do
     with_input_for @user, :created_at, :datetime, autofocus: true
     assert_select 'select.datetime[autofocus]'
   end
 
-  test 'string input should generate autofocus attribute when autofocus option is false' do
+  test 'string input generates autofocus attribute when autofocus option is false' do
     with_input_for @user, :name, :string, autofocus: false
     assert_no_select 'input.string[autofocus]'
   end
 
-  test 'text input should generate autofocus attribute when autofocus option is false' do
+  test 'text input generates autofocus attribute when autofocus option is false' do
     with_input_for @user, :description, :text, autofocus: false
     assert_no_select 'textarea.text[autofocus]'
   end
 
-  test 'numeric input should generate autofocus attribute when autofocus option is false' do
+  test 'numeric input generates autofocus attribute when autofocus option is false' do
     with_input_for @user, :age, :integer, autofocus: false
     assert_no_select 'input.integer[autofocus]'
   end
 
-  test 'date input should generate autofocus attribute when autofocus option is false' do
+  test 'date input generates autofocus attribute when autofocus option is false' do
     with_input_for @user, :born_at, :date, autofocus: false
     assert_no_select 'select.date[autofocus]'
   end
 
-  test 'datetime input should generate autofocus attribute when autofocus option is false' do
+  test 'datetime input generates autofocus attribute when autofocus option is false' do
     with_input_for @user, :created_at, :datetime, autofocus: false
     assert_no_select 'select.datetime[autofocus]'
   end
 
-  test 'string input should generate autofocus attribute when autofocus option is not present' do
+  test 'string input generates autofocus attribute when autofocus option is not present' do
     with_input_for @user, :name, :string
     assert_no_select 'input.string[autofocus]'
   end
 
-  test 'text input should generate autofocus attribute when autofocus option is not present' do
+  test 'text input generates autofocus attribute when autofocus option is not present' do
     with_input_for @user, :description, :text
     assert_no_select 'textarea.text[autofocus]'
   end
 
-  test 'numeric input should generate autofocus attribute when autofocus option is not present' do
+  test 'numeric input generates autofocus attribute when autofocus option is not present' do
     with_input_for @user, :age, :integer
     assert_no_select 'input.integer[autofocus]'
   end
 
-  test 'date input should generate autofocus attribute when autofocus option is not present' do
+  test 'date input generates autofocus attribute when autofocus option is not present' do
     with_input_for @user, :born_at, :date
     assert_no_select 'select.date[autofocus]'
   end
 
-  test 'datetime input should generate autofocus attribute when autofocus option is not present' do
+  test 'datetime input generates autofocus attribute when autofocus option is not present' do
     with_input_for @user, :created_at, :datetime
     assert_no_select 'select.datetime[autofocus]'
   end
 
   # With no object
-  test 'input should be generated properly when object is not present' do
+  test 'input is generated properly when object is not present' do
     with_input_for :project, :name, :string
     assert_select 'input.string.required#project_name'
   end
 
-  test 'input as radio should be generated properly when object is not present ' do
+  test 'input as radio is generated properly when object is not present ' do
     with_input_for :project, :name, :radio_buttons
     assert_select 'input.radio_buttons#project_name_true'
     assert_select 'input.radio_buttons#project_name_false'
   end
 
-  test 'input as select with collection should be generated properly when object is not present' do
+  test 'input as select with collection is generated properly when object is not present' do
     with_input_for :project, :name, :select, collection: ['Jose', 'Carlos']
     assert_select 'select.select#project_name'
   end
 
-  test 'input should not generate empty css class' do
+  test 'input does not generate empty css class' do
     swap SimpleForm, generate_additional_classes_for: [:wrapper, :label] do
       with_input_for :project, :name, :string
       assert_no_select 'input#project_name[class]'