ehoch_simple_form 2.0.2.dev

data/test/form_builder/hint_test.rb

@@ -0,0 +1,139 @@
+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 not modify the options hash' do
+    options = { :hint => 'Use with care...' }
+    with_hint_for @user, :name, options
+    assert_select 'span.hint', 'Use with care...'
+    assert_equal({ :hint => 'Use with care...' }, options)
+  end
+
+  test 'hint should be generated cleanly with optional text' do
+    with_hint_for @user, :name, :hint => 'Use with care...', :hint_tag => :span
+    assert_no_select 'span.hint[hint]'
+    assert_no_select 'span.hint[hint_tag]'
+    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
+  
+  test 'hint should be output as html_safe' do
+    with_hint_for @user, :name, :hint => '<b>Bold</b> and not...'
+    assert_select 'span.hint', 'Bold and not...'  
+  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
+  
+  test 'hint should output translations as html_safe' do
+    store_translations(:en, :simple_form => { :hints => { :user => {
+      :edit => { :name => '<b>This is bold</b> and this is not...' }
+    } } }) do
+      with_hint_for @user, :name
+      assert_select 'span.hint', 'This is bold and this is not...'
+    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 'div.omg_hint', "can't be blank"
+    end
+  end
+end

data/test/form_builder/input_field_test.rb

@@ -0,0 +1,63 @@
+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 not modify the options hash' do
+    options = { :id => 'name_input', :class => 'name' }
+
+    with_concat_form_for(@user) do |f|
+      f.input_field :name, options
+    end
+
+    assert_select 'input.string.name#name_input'
+    assert_equal({ :id => 'name_input', :class => 'name' }, options)
+  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,71 @@
+# 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 for the boolean attrbiute' do
+    with_label_for @user, :name, :as => :boolean
+    assert_select 'label.boolean[for=user_name]', /Name/
+    assert_no_select 'label[as=boolean]'
+  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 label should generate label tag with clean HTML' do
+    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' }
+    with_label_for @user, :name, options
+    assert_select 'label.string#name_label', /My label/
+    assert_equal({ :label => 'My label', :id => 'name_label' }, options)
+  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,149 @@
+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
+
+  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
+      assert_select 'form div.wrapper'
+      assert_no_select 'div.required'
+      assert_no_select 'div.string'
+    end
+  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 > div.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