simple_form 1.5.2 → 2.0.0.rc

data/test/form_builder/hint_test.rb

@@ -0,0 +1,115 @@
+require 'test_helper'
+
+# Tests for f.hint
+class HintTest < ActionView::TestCase
+  def with_hint_for(object, *args)
+    with_concat_form_for(object) do |f|
+      f.hint(*args)
+    end
+  end
+
+  test 'hint should not be generated by default' do
+    with_hint_for @user, :name
+    assert_no_select 'span.hint'
+  end
+
+  test 'hint should be generated with optional text' do
+    with_hint_for @user, :name, :hint => 'Use with care...'
+    assert_select 'span.hint', 'Use with care...'
+  end
+
+  test 'hint should be generated cleanly with optional text' do
+    with_hint_for @user, :name, :hint => 'Use with care...'
+    assert_no_select 'span.hint[hint]'
+    assert_no_select 'span.hint[hint_html]'
+  end
+
+  test 'hint uses the current component tag set' do
+    with_hint_for @user, :name, :hint => 'Use with care...', :hint_tag => :p
+    assert_select 'p.hint', 'Use with care...'
+  end
+
+  test 'hint should be able to pass html options' do
+    with_hint_for @user, :name, :hint => 'Yay!', :id => 'hint', :class => 'yay'
+    assert_select 'span#hint.hint.yay'
+  end
+
+  # Without attribute name
+
+  test 'hint without attribute name' do
+    with_hint_for @validating_user, 'Hello World!'
+    assert_select 'span.hint', 'Hello World!'
+  end
+
+  test 'hint without attribute name should generate component tag with a clean HTML' do
+    with_hint_for @validating_user, 'Hello World!'
+    assert_no_select 'span.hint[hint]'
+    assert_no_select 'span.hint[hint_html]'
+  end
+
+  test 'hint without attribute name uses the current component tag set' do
+    with_hint_for @user, 'Hello World!', :hint_tag => :p
+    assert_no_select 'p.hint[hint]'
+    assert_no_select 'p.hint[hint_html]'
+    assert_no_select 'p.hint[hint_tag]'
+  end
+
+  test 'hint without attribute name should be able to pass html options' do
+    with_hint_for @user, 'Yay', :id => 'hint', :class => 'yay'
+    assert_select 'span#hint.hint.yay', 'Yay'
+  end
+
+  # I18n
+
+  test 'hint should use i18n based on model, action, and attribute to lookup translation' do
+    store_translations(:en, :simple_form => { :hints => { :user => {
+      :edit => { :name => 'Content of this input will be truncated...' }
+    } } }) do
+      with_hint_for @user, :name
+      assert_select 'span.hint', 'Content of this input will be truncated...'
+    end
+  end
+
+  test 'hint should use i18n with model and attribute to lookup translation' do
+    store_translations(:en, :simple_form => { :hints => { :user => {
+      :name => 'Content of this input will be capitalized...'
+    } } }) do
+      with_hint_for @user, :name
+      assert_select 'span.hint', 'Content of this input will be capitalized...'
+    end
+  end
+
+  test 'hint should use i18n under defaults namespace to lookup translation' do
+    store_translations(:en, :simple_form => {
+      :hints => {:defaults => {:name => 'Content of this input will be downcased...' } }
+    }) do
+      with_hint_for @user, :name
+      assert_select 'span.hint', 'Content of this input will be downcased...'
+    end
+  end
+
+  test 'hint should use i18n with lookup for association name' do
+    store_translations(:en, :simple_form => { :hints => {
+      :user => { :company => 'My company!' }
+    } } ) do
+      with_hint_for @user, :company_id, :as => :string, :reflection => Association.new(Company, :company, {})
+      assert_select 'span.hint', /My company!/
+    end
+  end
+
+  # No object
+
+  test 'hint should generate properly when object is not present' do
+    with_hint_for :project, :name, :hint => 'Test without object'
+    assert_select 'span.hint', 'Test without object'
+  end
+
+  # Custom wrappers
+
+  test 'hint with custom wrappers works' do
+    swap_wrapper do
+      with_hint_for @user, :name, :hint => "can't be blank"
+      assert_select 'span.omg_hint', "can't be blank"
+    end
+  end
+end

data/test/form_builder/input_field_test.rb

@@ -0,0 +1,51 @@
+require 'test_helper'
+
+# Tests for f.input_field
+class InputFieldTest < ActionView::TestCase
+  test "builder input_field should only render the input tag, nothing else" do
+    with_concat_form_for(@user) do |f|
+      f.input_field :name
+    end
+    assert_select 'form > input.required.string'
+    assert_no_select 'div.string'
+    assert_no_select 'label'
+    assert_no_select '.hint'
+  end
+
+  test 'builder input_field should allow overriding default input type' do
+    with_concat_form_for(@user) do |f|
+      f.input_field :name, :as => :text
+    end
+
+    assert_no_select 'input#user_name'
+    assert_select 'textarea#user_name.text'
+  end
+
+  test 'builder input_field should allow passing options to input tag' do
+    with_concat_form_for(@user) do |f|
+      f.input_field :name, :id => 'name_input', :class => 'name'
+    end
+
+    assert_select 'input.string.name#name_input'
+  end
+
+  test 'builder input_field should generate an input tag with a clean HTML' do
+    with_concat_form_for(@user) do |f|
+      f.input_field :name, :as => :integer, :class => 'name'
+    end
+
+    assert_no_select 'input.integer[input_html]'
+    assert_no_select 'input.integer[as]'
+  end
+
+  test 'builder collection input_field should generate input tag with a clean HTML' do
+    with_concat_form_for(@user) do |f|
+      f.input_field :status, :collection => ['Open', 'Closed'], :class => 'status', :label_method => :to_s, :value_method => :to_s
+    end
+
+    assert_no_select 'select.status[input_html]'
+    assert_no_select 'select.status[collection]'
+    assert_no_select 'select.status[label_method]'
+    assert_no_select 'select.status[value_method]'
+  end
+end

data/test/form_builder/label_test.rb

@@ -0,0 +1,51 @@
+# encoding: UTF-8
+require 'test_helper'
+
+class LabelTest < ActionView::TestCase
+  def with_label_for(object, *args, &block)
+    with_concat_form_for(object) do |f|
+      f.label(*args, &block)
+    end
+  end
+
+  test 'builder should generate a label for the attribute' do
+    with_label_for @user, :name
+    assert_select 'label.string[for=user_name]', /Name/
+  end
+
+  test 'builder should generate a label componet tag with a clean HTML' do
+    with_label_for @user, :name
+    assert_no_select 'label.string[label_html]'
+  end
+
+  test 'builder should add a required class to label if the attribute is required' do
+    with_label_for @validating_user, :name
+    assert_select 'label.string.required[for=validating_user_name]', /Name/
+  end
+
+  test 'builder should allow passing options to label tag' do
+    with_label_for @user, :name, :label => 'My label', :id => 'name_label'
+    assert_select 'label.string#name_label', /My label/
+  end
+
+  test 'builder should fallback to default label when string is given' do
+    with_label_for @user, :name, 'Nome do usuário'
+    assert_select 'label', 'Nome do usuário'
+    assert_no_select 'label.string'
+  end
+
+  test 'builder should fallback to default label when block is given' do
+    with_label_for @user, :name do
+      'Nome do usuário'
+    end
+    assert_select 'label', 'Nome do usuário'
+    assert_no_select 'label.string'
+  end
+
+  test 'builder allows label order to be changed' do
+    swap SimpleForm, :label_text => lambda { |l, r| "#{l}:" } do
+      with_label_for @user, :age
+      assert_select 'label.integer[for=user_age]', "Age:"
+    end
+  end
+end

data/test/form_builder/wrapper_test.rb

@@ -0,0 +1,140 @@
+require 'test_helper'
+
+class WrapperTest < ActionView::TestCase
+  test 'wrapper should not have error class for attribute without errors' do
+    with_form_for @user, :active
+    assert_no_select 'div.field_with_errors'
+  end
+
+  test 'wrapper should not have error class when object is not present' do
+    with_form_for :project, :name
+    assert_no_select 'div.field_with_errors'
+  end
+
+  test 'wrapper should add error class for attribute with errors' do
+    with_form_for @user, :name
+    assert_select 'div.field_with_errors'
+  end
+
+  test 'wrapper should add hint class for attribute with a hint' do
+    with_form_for @user, :name, :hint => 'hint'
+    assert_select 'div.field_with_hint'
+  end
+
+  test 'wrapper should not have disabled class by default' do
+    with_form_for @user, :active
+    assert_no_select 'div.disabled'
+  end
+
+  test 'wrapper should have disabled class when input is disabled' do
+    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
+    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
+    assert_select 'form > label[for=user_name]'
+    assert_select 'form > input#user_name.string'
+  end
+
+  test 'wrapper should wrapping tag adds required/optional css classes' do
+    with_form_for @user, :name
+    assert_select 'form div.input.required.string'
+
+    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' }
+    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
+    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
+    assert_select 'form div.wrapper.required.string'
+  end
+
+  # Custom wrapper test
+
+  test 'custom wrappers works' do
+    swap_wrapper do
+      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"
+      assert_select "section.custom_wrapper div.error_wrapper span.omg_error"
+      assert_select "section.custom_wrapper > span.omg_hint", "cool"
+    end
+  end
+
+  test 'custom wrappers can be turned off' do
+    swap_wrapper do
+      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"
+    end
+  end
+
+  test 'custom wrappers on a form basis' do
+    swap_wrapper :another do
+      concat simple_form_for(@user) { |f|
+        f.input :name
+      }
+
+      assert_no_select "section.custom_wrapper div.another_wrapper label"
+      assert_no_select "section.custom_wrapper div.another_wrapper input.string"
+
+      concat simple_form_for(@user, :wrapper => :another) { |f|
+        f.input :name
+      }
+
+      assert_select "section.custom_wrapper div.another_wrapper label"
+      assert_select "section.custom_wrapper div.another_wrapper input.string"
+    end
+  end
+
+  test 'custom wrappers on input basis' do
+    swap_wrapper :another do
+      with_form_for @user, :name
+      assert_no_select "section.custom_wrapper div.another_wrapper label"
+      assert_no_select "section.custom_wrapper div.another_wrapper input.string"
+      output_buffer.replace ""
+
+      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
+    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"
+      assert_select "section.custom_wrapper div.another_wrapper label"
+      assert_select "section.custom_wrapper div.another_wrapper input.string"
+    end
+  end
+
+  test 'raise error when wrapper not found' do
+    assert_raise SimpleForm::WrapperNotFound do
+      with_form_for @user, :name, :wrapper => :not_found
+    end
+  end
+end

data/test/generators/simple_form_generator_test.rb

@@ -0,0 +1,32 @@
+require 'test_helper'
+
+class SimpleFormGeneratorTest < Rails::Generators::TestCase
+  tests SimpleForm::Generators::InstallGenerator
+  destination File.expand_path('../../tmp', __FILE__)
+  setup :prepare_destination
+  teardown { rm_rf(destination_root) }
+
+  test 'generates example locale file' do
+    run_generator
+    assert_file 'config/locales/simple_form.en.yml'
+  end
+
+  test 'generates the simple_form initializer' do
+    run_generator
+    assert_file 'config/initializers/simple_form.rb',
+      /config\.default_wrapper = :default/, /config\.boolean_style = :nested/
+  end
+
+  test 'generates the simple_form initializer with the bootstrap wrappers' do
+    run_generator %w(--bootstrap)
+    assert_file 'config/initializers/simple_form.rb', /config\.wrappers :bootstrap/,
+      /config\.default_wrapper = :bootstrap/
+  end
+
+  %W(erb haml slim).each do |engine|
+    test "generates the scaffold template when using #{engine}" do
+      run_generator ['-e', engine]
+      assert_file "lib/templates/#{engine}/scaffold/_form.html.#{engine}"
+    end
+  end
+end

data/test/inputs/boolean_input_test.rb

@@ -0,0 +1,91 @@
+# encoding: UTF-8
+require 'test_helper'
+
+class BooleanInputTest < ActionView::TestCase
+  test 'input should generate a checkbox by default for boolean attributes' do
+    with_input_for @user, :active, :boolean
+    assert_select 'input[type=checkbox].boolean#user_active'
+    assert_select 'label.boolean.optional', 'Active'
+  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
+    assert_select 'input[type=checkbox].boolean#user_active'
+    assert_no_select 'label'
+  end
+
+  test 'input uses inline boolean style by default' do
+    with_input_for @user, :active, :boolean
+    assert_select 'input.boolean + label.boolean.optional'
+    assert_no_select 'label > input'
+  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
+      with_input_for @user, :active, :boolean
+      assert_select 'label[for=user_active]', 'Active'
+      assert_select 'label.boolean > input.boolean'
+      assert_no_select 'input[type=checkbox] + label'
+    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
+      with_input_for @user, :active, :boolean
+
+      assert_select "input[type=hidden][name='user[active]'] + label.boolean > input.boolean"
+      assert_no_select 'input[type=checkbox] + label'
+    end
+  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
+
+      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
+    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
+      assert_select 'label[for=user_active]', 'Active'
+      assert_select 'input.boolean + label.boolean'
+      assert_no_select 'label > input'
+    end
+  end
+
+  test 'input with nested style allows disabling label' do
+    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 boolean works using :input only in wrapper config (no label_input)' do
+    swap_wrapper do
+      with_input_for @user, :active, :boolean
+
+      assert_select 'label.boolean + input[type=hidden] + input.boolean'
+      assert_no_select 'label.checkbox'
+    end
+  end
+
+  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
+        with_input_for @user, :active, :boolean
+
+        assert_select 'label.boolean + input[type=hidden] + label.checkbox > input.boolean'
+      end
+    end
+  end
+end