formtastic 1.2.5 → 2.0.0.rc1

data/spec/builder/errors_spec.rb

@@ -0,0 +1,193 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe 'Formtastic::FormBuilder#errors_on' do
+
+  include FormtasticSpecHelper
+
+  before do
+    @output_buffer = ''
+    mock_everything
+    @title_errors = ['must not be blank', 'must be longer than 10 characters', 'must be awesome']
+    @errors = mock('errors')
+    @new_post.stub!(:errors).and_return(@errors)
+  end
+
+  describe 'when there are errors' do
+    before do
+      @errors.stub!(:[]).with(:title).and_return(@title_errors)
+    end
+
+    after do
+      Formtastic::FormBuilder.default_inline_error_class = 'inline-errors'
+      Formtastic::FormBuilder.default_error_list_class = 'errors'
+    end
+
+    it 'should render a paragraph with the errors joined into a sentence when inline_errors config is :sentence' do
+      Formtastic::FormBuilder.inline_errors = :sentence
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title
+      end)
+      output_buffer.should have_tag('p.inline-errors', @title_errors.to_sentence)
+    end
+
+    it 'should render a paragraph with a overridden default class' do
+      Formtastic::FormBuilder.inline_errors = :sentence
+      Formtastic::FormBuilder.default_inline_error_class = 'better-errors'
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input(:title)
+      end)
+      output_buffer.should have_tag('p.better-errors', @title_errors.to_sentence)
+    end
+
+    it 'should render a paragraph with the errors joined into a sentence when inline_errors config is :sentence with a customized error class' do
+      Formtastic::FormBuilder.inline_errors = :sentence
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input(:title, :error_class => 'better-errors')
+      end)
+      output_buffer.should have_tag('p.better-errors', @title_errors.to_sentence)
+    end
+
+    it 'should render an unordered list with the class errors when inline_errors config is :list' do
+      Formtastic::FormBuilder.inline_errors = :list
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input(:title)
+      end)
+      output_buffer.should have_tag('ul.errors')
+      @title_errors.each do |error|
+        output_buffer.should have_tag('ul.errors li', error)
+      end
+    end
+
+    it 'should render an unordered list with the class overridden default class' do
+      Formtastic::FormBuilder.inline_errors = :list
+      Formtastic::FormBuilder.default_error_list_class = "better-errors"
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title
+      end)
+      output_buffer.should have_tag('ul.better-errors')
+      @title_errors.each do |error|
+        output_buffer.should have_tag('ul.better-errors li', error)
+      end
+    end
+
+    it 'should render an unordered list with the class errors when inline_errors config is :list with a customized error class' do
+      Formtastic::FormBuilder.inline_errors = :list
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title, :error_class => "better-errors"
+      end)
+      output_buffer.should have_tag('ul.better-errors')
+      @title_errors.each do |error|
+        output_buffer.should have_tag('ul.better-errors li', error)
+      end
+    end
+
+    it 'should render a paragraph with the first error when inline_errors config is :first' do
+      Formtastic::FormBuilder.inline_errors = :first
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title
+      end)
+      output_buffer.should have_tag('p.inline-errors', @title_errors.first)
+      
+    end
+
+    it 'should render a paragraph with the first error when inline_errors config is :first with a customized error class' do
+      Formtastic::FormBuilder.inline_errors = :first
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title, :error_class => "better-errors"
+      end)
+      output_buffer.should have_tag('p.better-errors', @title_errors.first)
+    end
+
+    it 'should return nil when inline_errors config is :none' do
+      Formtastic::FormBuilder.inline_errors = :none
+      concat(semantic_form_for(@new_post) do |builder|
+        builder.input :title
+      end)
+      output_buffer.should_not have_tag('p.inine-errors')
+      output_buffer.should_not have_tag('ul.errors')
+    end
+
+  end
+
+  describe 'when there are no errors (nil)' do
+    before do
+      @errors.stub!(:[]).with(:title).and_return(nil)
+    end
+
+    it 'should return nil when inline_errors config is :sentence, :list or :none' do
+      with_deprecation_silenced do
+        [:sentence, :list, :none].each do |config|
+          with_config :inline_errors, config do
+            semantic_form_for(@new_post) do |builder|
+              builder.errors_on(:title).should be_nil
+            end
+          end
+        end
+      end
+    end
+  end
+
+  describe 'when there are no errors (empty array)' do
+    before do
+      @errors.stub!(:[]).with(:title).and_return([])
+    end
+
+    it 'should return nil when inline_errors config is :sentence, :list or :none' do
+      with_deprecation_silenced do
+        [:sentence, :list, :none].each do |config|
+          Formtastic::FormBuilder.inline_errors = config
+          semantic_form_for(@new_post) do |builder|
+            builder.errors_on(:title).should be_nil
+          end
+        end
+      end
+    end
+  end
+
+  describe 'when file type columns have errors' do
+    it "should list errors added on metadata fields" do
+      @errors.stub!(:[]).with(:document_file_name).and_return(['must be provided'])
+      @errors.stub!(:[]).with(:document_file_size).and_return(['must be less than 4mb'])
+      @errors.stub!(:[]).with(:document_content_type).and_return(['must be an image'])
+      @errors.stub!(:[]).with(:document).and_return(nil)
+
+      with_config :inline_errors, :sentence do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.input(:document))
+        end)
+      end 
+      output_buffer.should have_tag("li.error")
+      output_buffer.should have_tag('p.inline-errors', (['must be an image','must be provided', 'must be less than 4mb']).to_sentence)
+    end
+  end
+
+  describe 'when there are errors on the association and column' do
+
+    it "should list all unique errors" do
+      ::Post.stub!(:reflections).and_return({:author => mock('reflection', :options => {}, :macro => :belongs_to)})
+
+      @errors.stub!(:[]).with(:author).and_return(['must not be blank'])
+      @errors.stub!(:[]).with(:author_id).and_return(['is already taken', 'must not be blank']) # note the duplicate of association
+
+      with_config :inline_errors, :list do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.input(:author))
+        end)
+      end
+      output_buffer.should have_tag("ul.errors li", /must not be blank/, :count => 1)
+      output_buffer.should have_tag("ul.errors li", /is already taken/, :count => 1)
+    end
+
+  end
+
+  describe "when there are errors on a has_many association" do
+    it "should include the association ids error messages" do
+      semantic_form_for(@new_post) do |builder|
+        builder.send(:error_keys, :sub_posts, {}).should include(:sub_post_ids)
+      end
+    end
+  end
+
+end
+

data/spec/builder/semantic_fields_for_spec.rb

@@ -0,0 +1,88 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe 'Formtastic::FormBuilder#fields_for' do
+
+  include FormtasticSpecHelper
+
+  before do
+    @output_buffer = ''
+    mock_everything
+    @new_post.stub!(:author).and_return(::Author.new)
+  end
+
+  it 'yields an instance of FormHelper.builder' do
+    semantic_form_for(@new_post) do |builder|
+      builder.fields_for(:author) do |nested_builder|
+        nested_builder.class.should == Formtastic::Helpers::FormHelper.builder
+      end
+    end
+  end
+
+  it 'nests the object name' do
+    semantic_form_for(@new_post) do |builder|
+      builder.fields_for(@bob) do |nested_builder|
+        nested_builder.object_name.should == 'post[author]'
+      end
+    end
+  end
+  
+  it 'supports passing collection as second parameter' do
+    semantic_form_for(@new_post) do |builder|
+      builder.semantic_fields_for(:author, [@fred,@bob]) do |nested_builder|
+        nested_builder.object_name.should =~ /post\[author_attributes\]\[\d+\]/
+      end
+    end
+  end
+
+  it 'should sanitize html id for li tag' do
+    @bob.stub!(:column_for_attribute).and_return(mock('column', :type => :string, :limit => 255))
+    concat(semantic_form_for(@new_post) do |builder|
+      concat(builder.fields_for(@bob, :index => 1) do |nested_builder|
+        concat(nested_builder.inputs(:login))
+      end)
+    end)
+    output_buffer.should have_tag('form fieldset.inputs #post_author_1_login_input')
+    # Not valid selector, so using good ol' regex
+    output_buffer.should_not =~ /id="post\[author\]_1_login_input"/
+    # <=> output_buffer.should_not have_tag('form fieldset.inputs #post[author]_1_login_input')
+  end
+
+  it 'should use namespace provided in nested fields' do
+    @bob.stub!(:column_for_attribute).and_return(mock('column', :type => :string, :limit => 255))
+    concat(semantic_form_for(@new_post, :namespace => 'context2') do |builder|
+      concat(builder.fields_for(@bob, :index => 1) do |nested_builder|
+        concat(nested_builder.inputs(:login))
+      end)
+    end)
+    output_buffer.should have_tag('form fieldset.inputs #context2_post_author_1_login_input')
+  end
+  
+  context "when I rendered my own hidden id input" do 
+    
+    before do
+      output_buffer.replace ''
+      
+      @fred.posts.size.should == 1
+      @fred.posts.first.stub!(:persisted?).and_return(true)
+      @fred.stub!(:posts_attributes=)
+
+      concat(semantic_form_for(@fred) do |builder|
+        concat(builder.fields_for(:posts) do |nested_builder|
+          concat(nested_builder.input(:id, :as => :hidden))
+          concat(nested_builder.input(:title))
+        end)
+      end)
+    end
+  
+    it "should only render one hidden input (my one)" do
+      output_buffer.should have_tag 'input#author_posts_attributes_0_id', :count => 1
+    end
+    
+    it "should render the hidden input inside an li.hidden" do
+      output_buffer.should have_tag 'li.hidden input#author_posts_attributes_0_id'
+    end
+  end
+
+end
+

data/spec/helpers/buttons_helper_spec.rb

@@ -0,0 +1,150 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe 'Formtastic::FormBuilder#buttons' do
+
+  include FormtasticSpecHelper
+
+  before do
+    @output_buffer = ''
+    mock_everything
+  end
+
+  describe 'with a block' do
+    describe 'when no options are provided' do
+      before do
+        concat(semantic_form_for(@new_post) do |builder|
+          buttons = builder.buttons do
+            concat('hello')
+          end
+          concat(buttons)
+        end)
+      end
+
+      it 'should render a fieldset inside the form, with a class of "inputs"' do
+        output_buffer.should have_tag("form fieldset.buttons")
+      end
+
+      it 'should render an ol inside the fieldset' do
+        output_buffer.should have_tag("form fieldset.buttons ol")
+      end
+
+      it 'should render the contents of the block inside the ol' do
+        output_buffer.should have_tag("form fieldset.buttons ol", /hello/)
+      end
+
+      it 'should not render a legend inside the fieldset' do
+        output_buffer.should_not have_tag("form fieldset.buttons legend")
+      end
+    end
+
+    describe 'when a :name option is provided' do
+      before do
+        @legend_text = "Advanced options"
+
+        concat(semantic_form_for(@new_post) do |builder|
+          builder.buttons :name => @legend_text do
+          end
+        end)
+      end
+      it 'should render a fieldset inside the form' do
+        output_buffer.should have_tag("form fieldset legend", /#{@legend_text}/)
+      end
+    end
+
+    describe 'when other options are provided' do
+      before do
+        @id_option = 'advanced'
+        @class_option = 'wide'
+
+        concat(semantic_form_for(@new_post) do |builder|
+          builder.buttons :id => @id_option, :class => @class_option do
+          end
+        end)
+      end
+      it 'should pass the options into the fieldset tag as attributes' do
+        output_buffer.should have_tag("form fieldset##{@id_option}")
+        output_buffer.should have_tag("form fieldset.#{@class_option}")
+      end
+    end
+
+  end
+
+  describe 'without a block' do
+
+    describe 'with no args (default buttons)' do
+
+      before do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.buttons)
+        end)
+      end
+
+      it 'should render a form' do
+        output_buffer.should have_tag('form')
+      end
+
+      it 'should render a buttons fieldset inside the form' do
+        output_buffer.should have_tag('form fieldset.buttons')
+      end
+
+      it 'should not render a legend in the fieldset' do
+        output_buffer.should_not have_tag('form fieldset.buttons legend')
+      end
+
+      it 'should render an ol in the fieldset' do
+        output_buffer.should have_tag('form fieldset.buttons ol')
+      end
+
+      it 'should render a list item in the ol for each default button' do
+        output_buffer.should have_tag('form fieldset.buttons ol li', :count => 1)
+      end
+
+      it 'should render a commit list item for the commit button' do
+        output_buffer.should have_tag('form fieldset.buttons ol li.commit')
+      end
+
+    end
+
+    describe 'with button names as args' do
+
+      before do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.buttons(:commit))
+        end)
+      end
+
+      it 'should render a form with a fieldset containing a list item for each button arg' do
+        output_buffer.should have_tag('form > fieldset.buttons > ol > li', :count => 1)
+        output_buffer.should have_tag('form > fieldset.buttons > ol > li.commit')
+      end
+
+    end
+
+    describe 'with button names as args and an options hash' do
+
+      before do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.buttons(:commit, :name => "Now click a button", :id => "my-id"))
+        end)
+      end
+
+      it 'should render a form with a fieldset containing a list item for each button arg' do
+        output_buffer.should have_tag('form > fieldset.buttons > ol > li', :count => 1)
+        output_buffer.should have_tag('form > fieldset.buttons > ol > li.commit', :count => 1)
+      end
+
+      it 'should pass the options down to the fieldset' do
+        output_buffer.should have_tag('form > fieldset#my-id.buttons')
+      end
+
+      it 'should use the special :name option as a text for the legend tag' do
+        output_buffer.should have_tag('form > fieldset#my-id.buttons > legend', /Now click a button/)
+      end
+
+    end
+
+  end
+
+end
+

data/spec/helpers/commit_button_helper_spec.rb

@@ -0,0 +1,470 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe 'Formtastic::FormBuilder#commit_button' do
+
+  include FormtasticSpecHelper
+
+  before do
+    @output_buffer = ''
+    mock_everything
+  end
+
+  describe 'when the object responds to :persisted? (ActiveModel)' do
+
+    before do
+      @new_post.stub(:respond_to?).with(:to_model).and_return("X")
+      @new_post.stub(:respond_to?).with(:persisted?).and_return(true)
+      @new_post.stub(:respond_to?).with(:new_record?).and_return(false)
+    end
+
+    it 'should call :persisted?' do
+      with_config :i18n_lookups_by_default, false do
+        @new_post.should_receive(:persisted?)
+        @new_post.should_not_receive(:new_record?)
+        semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button)
+        end
+      end
+    end
+
+  end
+  
+  describe 'when not persisted' do
+
+    before do
+      @new_post.stub(:respond_to?).with(:to_model).and_return("X")
+      @new_post.stub(:respond_to?).with(:persisted?).and_return(false)
+      @new_post.stub(:respond_to?).with(:new_record?).and_return(false)
+    end
+
+    it 'should have a submit button label' do
+      with_config :i18n_lookups_by_default, false do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button)
+        end)
+      end
+      
+      output_buffer.should have_tag('.commit input[@value="Submit Post"]')
+    end
+  end
+  
+  
+  describe 'when used on any record' do
+
+    before do
+      @new_post.stub!(:new_record?).and_return(false)
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.commit_button)
+      end)
+    end
+
+    it 'should render a commit li' do
+      output_buffer.should have_tag('li.commit')
+    end
+    
+    it 'should render a button li' do
+      output_buffer.should have_tag('li.button')
+    end
+
+    it 'should render an input with a type attribute of "submit"' do
+      output_buffer.should have_tag('li.commit input[@type="submit"]')
+    end
+
+    it 'should render an input with a name attribute of "commit"' do
+      output_buffer.should have_tag('li.commit input[@name="commit"]')
+    end
+
+    it 'should pass options given in :button_html to the button' do
+      @new_post.stub!(:new_record?).and_return(false)
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.commit_button('text', :button_html => {:class => 'my_class', :id => 'my_id'}))
+      end)
+
+      output_buffer.should have_tag('li.commit input#my_id')
+      output_buffer.should have_tag('li.commit input.my_class')
+    end
+
+  end
+
+  describe "its accesskey" do
+
+    it 'should allow nil default' do
+      with_config :default_commit_button_accesskey, nil do
+        output_buffer.should_not have_tag('li.commit input[@accesskey]')
+      end
+    end
+
+    it 'should use the default if set' do
+      with_config :default_commit_button_accesskey, 's' do
+        @new_post.stub!(:new_record?).and_return(false)
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :button_html => {}))
+        end)
+        output_buffer.should have_tag('li.commit input[@accesskey="s"]')
+      end
+    end
+
+    it 'should use the value set in options over the default' do
+      with_config :default_commit_button_accesskey, 's' do
+        @new_post.stub!(:new_record?).and_return(false)
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :accesskey => 'o'))
+        end)
+        output_buffer.should_not have_tag('li.commit input[@accesskey="s"]')
+        output_buffer.should have_tag('li.commit input[@accesskey="o"]')
+      end
+    end
+
+    it 'should use the value set in button_html over options' do
+      with_config :default_commit_button_accesskey, 's' do
+        @new_post.stub!(:new_record?).and_return(false)
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :accesskey => 'o', :button_html => {:accesskey => 't'}))
+        end)
+        output_buffer.should_not have_tag('li.commit input[@accesskey="s"]')
+        output_buffer.should_not have_tag('li.commit input[@accesskey="o"]')
+        output_buffer.should have_tag('li.commit input[@accesskey="t"]')
+      end
+    end
+
+  end
+
+  describe 'when the first option is a string and the second is a hash' do
+
+    before do
+      @new_post.stub!(:new_record?).and_return(false)
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.commit_button("a string", :button_html => { :class => "pretty"}))
+      end)
+    end
+
+    it "should render the string as the value of the button" do
+      output_buffer.should have_tag('li input[@value="a string"]')
+    end
+
+    it "should deal with the options hash" do
+      output_buffer.should have_tag('li input.pretty')
+    end
+
+  end
+
+  describe 'when the first option is a hash' do
+
+    before do
+      @new_post.stub!(:new_record?).and_return(false)
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.commit_button(:button_html => { :class => "pretty"}))
+      end)
+    end
+
+    it "should deal with the options hash" do
+      output_buffer.should have_tag('li input.pretty')
+    end
+
+  end
+
+  describe 'label' do
+
+    # No object
+    describe 'when used without object' do
+      describe 'when explicit label is provided' do
+        it 'should render an input with the explicitly specified label' do
+          concat(semantic_form_for(:post, :url => 'http://example.com') do |builder|
+            concat(builder.commit_button("Click!"))
+          end)
+          output_buffer.should have_tag('li.commit input[@value="Click!"][@class~="submit"]')
+        end
+      end
+
+      describe 'when no explicit label is provided' do
+        describe 'when no I18n-localized label is provided' do
+          before do
+            ::I18n.backend.store_translations :en, :formtastic => {:submit => 'Submit %{model}'}
+          end
+
+          after do
+            ::I18n.backend.reload!
+          end
+
+          it 'should render an input with default I18n-localized label (fallback)' do
+            concat(semantic_form_for(:post, :url => 'http://example.com') do |builder|
+              concat(builder.commit_button)
+            end)
+            output_buffer.should have_tag('li.commit input[@value="Submit Post"][@class~="submit"]')
+          end
+        end
+
+       describe 'when I18n-localized label is provided' do
+         before do
+           ::I18n.backend.store_translations :en,
+             :formtastic => {
+                 :actions => {
+                   :submit => 'Custom Submit',
+                  }
+               }
+         end
+         
+         after do
+           ::I18n.backend.reload!
+         end
+         
+         it 'should render an input with localized label (I18n)' do
+           with_config :i18n_lookups_by_default, true do
+             ::I18n.backend.store_translations :en,
+               :formtastic => {
+                   :actions => {
+                     :post => {
+                       :submit => 'Custom Submit %{model}'
+                      }
+                    }
+                 }
+             concat(semantic_form_for(:post, :url => 'http://example.com') do |builder|
+               concat(builder.commit_button)
+             end)
+             output_buffer.should have_tag(%Q{li.commit input[@value="Custom Submit Post"][@class~="submit"]})
+           end
+         end
+         
+         it 'should render an input with anoptional localized label (I18n) - if first is not set' do
+           with_config :i18n_lookups_by_default, true do
+             concat(semantic_form_for(:post, :url => 'http://example.com') do |builder|
+               concat(builder.commit_button)
+             end)
+             output_buffer.should have_tag(%Q{li.commit input[@value="Custom Submit"][@class~="submit"]})
+           end
+         end
+
+       end
+      end
+    end
+
+    # New record
+    describe 'when used on a new record' do
+      before do
+        @new_post.stub!(:new_record?).and_return(true)
+      end
+
+      describe 'when explicit label is provided' do
+        it 'should render an input with the explicitly specified label' do
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.commit_button("Click!"))
+          end)
+          output_buffer.should have_tag('li.commit input[@value="Click!"][@class~="create"]')
+        end
+      end
+
+      describe 'when no explicit label is provided' do
+        describe 'when no I18n-localized label is provided' do
+          before do
+            ::I18n.backend.store_translations :en, :formtastic => {:create => 'Create %{model}'}
+          end
+
+          after do
+            ::I18n.backend.reload!
+          end
+
+          it 'should render an input with default I18n-localized label (fallback)' do
+            concat(semantic_form_for(@new_post) do |builder|
+              concat(builder.commit_button)
+            end)
+            output_buffer.should have_tag('li.commit input[@value="Create Post"][@class~="create"]')
+          end
+        end
+
+        describe 'when I18n-localized label is provided' do
+          before do
+            ::I18n.backend.store_translations :en,
+              :formtastic => {
+                  :actions => {
+                    :create => 'Custom Create',
+                   }
+                }
+          end
+
+          after do
+            ::I18n.backend.reload!
+          end
+
+          it 'should render an input with localized label (I18n)' do
+            with_config :i18n_lookups_by_default, true do
+              ::I18n.backend.store_translations :en,
+                :formtastic => {
+                    :actions => {
+                      :post => {
+                        :create => 'Custom Create %{model}'
+                       }
+                     }
+                  }
+              concat(semantic_form_for(@new_post) do |builder|
+                concat(builder.commit_button)
+              end)
+              output_buffer.should have_tag(%Q{li.commit input[@value="Custom Create Post"][@class~="create"]})
+            end
+          end
+
+          it 'should render an input with anoptional localized label (I18n) - if first is not set' do
+            with_config :i18n_lookups_by_default, true do
+              concat(semantic_form_for(@new_post) do |builder|
+                concat(builder.commit_button)
+              end)
+              output_buffer.should have_tag(%Q{li.commit input[@value="Custom Create"][@class~="create"]})
+            end
+          end
+
+        end
+      end
+    end
+
+    # Existing record
+    describe 'when used on an existing record' do
+      before do
+        @new_post.stub!(:persisted?).and_return(true)
+      end
+
+      describe 'when explicit label is provided' do
+        it 'should render an input with the explicitly specified label' do
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.commit_button("Click!"))
+          end)
+          output_buffer.should have_tag('li.commit input[@value="Click!"][@class~="update"]')
+        end
+      end
+
+      describe 'when no explicit label is provided' do
+        describe 'when no I18n-localized label is provided' do
+          before do
+            ::I18n.backend.store_translations :en, :formtastic => {:update => 'Save %{model}'}
+          end
+
+          after do
+            ::I18n.backend.reload!
+          end
+
+          it 'should render an input with default I18n-localized label (fallback)' do
+            concat(semantic_form_for(@new_post) do |builder|
+              concat(builder.commit_button)
+            end)
+            output_buffer.should have_tag('li.commit input[@value="Save Post"][@class~="update"]')
+          end
+        end
+
+        describe 'when I18n-localized label is provided' do
+          before do
+            ::I18n.backend.reload!
+            ::I18n.backend.store_translations :en,
+              :formtastic => {
+                  :actions => {
+                    :update => 'Custom Save',
+                   }
+                }
+          end
+
+          after do
+            ::I18n.backend.reload!
+          end
+
+          it 'should render an input with localized label (I18n)' do
+            with_config :i18n_lookups_by_default, true do
+              ::I18n.backend.store_translations :en,
+                :formtastic => {
+                    :actions => {
+                      :post => {
+                        :update => 'Custom Save %{model}'
+                       }
+                     }
+                  }
+              concat(semantic_form_for(@new_post) do |builder|
+                concat(builder.commit_button)
+              end)
+              output_buffer.should have_tag(%Q{li.commit input[@value="Custom Save Post"][@class~="update"]})
+            end
+          end
+
+          it 'should render an input with anoptional localized label (I18n) - if first is not set' do
+            with_config :i18n_lookups_by_default, true do
+              concat(semantic_form_for(@new_post) do |builder|
+                concat(builder.commit_button)
+              end)
+              output_buffer.should have_tag(%Q{li.commit input[@value="Custom Save"][@class~="update"]})
+              ::I18n.backend.store_translations :en, :formtastic => {}
+            end
+          end
+
+        end
+      end
+    end
+  end
+
+  describe 'when the model is two words' do
+    before do
+      output_buffer = ''
+      class ::UserPost
+        extend ActiveModel::Naming if defined?(ActiveModel::Naming)
+        include ActiveModel::Conversion if defined?(ActiveModel::Conversion)
+
+        def id
+        end
+
+        def persisted?
+        end
+
+        # Rails does crappy human_name
+        def self.human_name
+          "User post"
+        end
+      end
+      @new_user_post = ::UserPost.new
+
+      @new_user_post.stub!(:new_record?).and_return(true)
+      concat(semantic_form_for(@new_user_post, :url => '') do |builder|
+        concat(builder.commit_button())
+      end)
+    end
+
+    it "should render the string as the value of the button" do
+      output_buffer.should have_tag('li input[@value="Create User post"]')
+    end
+
+  end
+  
+  describe ':wrapper_html option' do
+
+    describe 'when provided' do
+      it 'should be passed down to the li tag' do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :wrapper_html => {:id => :another_id}))
+        end)
+        output_buffer.should have_tag("form li#another_id")
+      end
+
+      it 'should append given classes to li default classes' do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :wrapper_html => {:class => :another_class}))
+        end)
+        output_buffer.should have_tag("form li.commit")
+        output_buffer.should have_tag("form li.another_class")
+      end
+
+      it 'should allow classes to be an array' do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text', :wrapper_html => {:class => [ :my_class, :another_class ]}))
+        end)
+        output_buffer.should have_tag("form li.commit")
+        output_buffer.should have_tag("form li.my_class")
+        output_buffer.should have_tag("form li.another_class")
+      end
+    end
+
+    describe 'when not provided' do
+      it 'should use default id and class' do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.commit_button('text'))
+        end)
+        output_buffer.should have_tag("form li.commit")
+      end
+    end
+
+  end
+  
+end