formtastic 2.0.2 → 2.1.0.beta1

data/spec/helpers/inputs_helper_spec.rb

@@ -556,6 +556,20 @@ describe 'Formtastic::FormBuilder#inputs' do
         output_buffer.should have_tag('form > fieldset.inputs > ol > li > fieldset.inputs > ol > li > fieldset.inputs > ol')
       end
     end
+
+    context "when several are nested" do
+      it "should wrap each of the nested inputs in an li block to maintain HTML validity" do
+        concat(semantic_form_for(@new_post) do |builder|
+          concat(builder.inputs do
+            concat(builder.inputs do
+            end)
+            concat(builder.inputs do
+            end)
+          end)
+        end)
+        output_buffer.should have_tag('form > fieldset.inputs > ol > li > fieldset.inputs > ol', :count => 2)
+      end
+    end
     
   end
 

data/spec/i18n_spec.rb

@@ -160,6 +160,17 @@ describe 'Formtastic::I18n' do
         output_buffer.should have_tag("form label", /Hello author name!/)
       end
     end
+
+    it 'should be able to translate nested objects with nested object translations' do
+      with_config :i18n_lookups_by_default, true do
+        concat(semantic_form_for(@new_post) do |builder|
+          builder.semantic_fields_for(:project) do |f|
+            concat(f.input(:title))
+          end
+        end)
+        output_buffer.should have_tag("form label", /Hello project!/)
+      end
+    end
     
     it 'should be able to translate nested forms with top level translations' do
       with_config :i18n_lookups_by_default, true do

data/spec/inputs/boolean_input_spec.rb

@@ -21,8 +21,6 @@ describe 'boolean input' do
 
   it 'should generate a label containing the input' do
     output_buffer.should_not have_tag('label.label')
-    
-    
     output_buffer.should have_tag('form li label', :count => 1)
     output_buffer.should have_tag('form li label[@for="post_allow_comments"]')
     output_buffer.should have_tag('form li label', /Allow comments/)
@@ -186,5 +184,36 @@ describe 'boolean input' do
     it_should_have_an_inline_label_for("context2_post_allow_comments")
 
   end
+  
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:name, :as => :boolean))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_name_input")
+    end
+    
+    it 'should index the id of the input tag' do
+      output_buffer.should have_tag("input#post_author_attributes_3_name")
+    end
+    
+    it 'should index the name of the hidden input' do
+      output_buffer.should have_tag("input[@type='hidden'][@name='post[author_attributes][3][name]']")
+    end
+
+    it 'should index the name of the checkbox input' do
+      output_buffer.should have_tag("input[@type='checkbox'][@name='post[author_attributes][3][name]']")
+    end
+    
+  end
 
 end

data/spec/inputs/check_boxes_input_spec.rb

@@ -348,7 +348,7 @@ describe 'check_boxes input' do
         item.should_not_receive(:id)
         item.stub!(:custom_value).and_return('custom_value')
         item.should_receive(:custom_value).exactly(3).times
-        @new_post.author.should_receive(:custom_value).exactly(3).times
+        @new_post.author.should_receive(:custom_value).exactly(1).times
         concat(semantic_form_for(@new_post) do |builder|
           concat(builder.input(:author, :as => :check_boxes, :member_value => :custom_value, :collection => [item, item, item]))
         end)
@@ -392,6 +392,34 @@ describe 'check_boxes input' do
     it_should_have_input_with_id('context2_author_post_ids_19')
     it_should_have_input_wrapper_with_id("context2_author_posts_input")
   end
+  
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@fred) do |builder|
+        concat(builder.fields_for(@fred.posts.first, :index => 3) do |author|
+          concat(author.input(:authors, :as => :check_boxes))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#author_post_3_authors_input")
+    end
+    
+    it 'should index the id of the input tag' do
+      output_buffer.should have_tag("input#author_post_3_author_ids_42")
+    end
+
+    it 'should index the name of the checkbox input' do
+      output_buffer.should have_tag("input[@type='checkbox'][@name='author[post][3][author_ids][]']")
+    end
+    
+  end
+  
 
   describe "when collection is an array" do
     before do

data/spec/inputs/date_input_spec.rb

@@ -70,6 +70,37 @@ describe 'date input' do
     it_should_have_select_with_id("context2_post_publish_at_3i")
 
   end
+  
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:created_at, :as => :date))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_created_at_input")
+    end
+    
+    it 'should index the id of the select tag' do
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_1i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_2i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_3i")
+    end
+    
+    it 'should index the name of the select tag' do
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(1i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(2i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(3i)]']")
+    end
+    
+  end
 
   describe ':labels option' do
     fields = [:year, :month, :day]
@@ -129,4 +160,68 @@ describe 'date input' do
     end
   end
   
+  describe "when order does not include day" do
+    before do
+      output_buffer.replace ''
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.input(:publish_at, :as => :date, :order => [:year, :month]))
+      end)
+    end
+
+    it "should include a hidden input for day" do
+      output_buffer.should have_tag('input[@type="hidden"][@name="post[publish_at(3i)]"][@value="1"]')
+    end
+
+    it "should not include a select for day" do
+      output_buffer.should_not have_tag('select[@name="post[publish_at(3i)]"]')
+    end
+  end
+
+  describe "when order does not include month" do
+    before do
+      output_buffer.replace ''
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.input(:publish_at, :as => :date, :order => [:year, :day]))
+      end)
+    end
+
+    it "should include a hidden input for month" do
+      output_buffer.should have_tag('input[@type="hidden"][@name="post[publish_at(2i)]"][@value="1"]')
+    end
+
+    it "should not include a select for month" do
+      output_buffer.should_not have_tag('select[@name="post[publish_at(2i)]"]')
+    end
+  end
+
+  describe "when order does not include year" do
+    before do
+      output_buffer.replace ''
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.input(:publish_at, :as => :date, :order => [:month, :day]))
+      end)
+    end
+
+    it "should include a hidden input for month" do
+      output_buffer.should have_tag("input[@type=\"hidden\"][@name=\"post[publish_at(1i)]\"][@value=\"#{Time.now.year}\"]")
+    end
+
+    it "should not include a select for month" do
+      output_buffer.should_not have_tag('select[@name="post[publish_at(1i)]"]')
+    end
+  end
+
+  describe "when order does not have year first" do
+    before do
+      output_buffer.replace ''
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.input(:publish_at, :as => :date, :order => [:day, :month, :year]))
+      end)
+    end
+
+    it 'should associate the legend label with the new first select' do
+      output_buffer.should have_tag('form li.date fieldset legend.label label[@for="post_publish_at_3i"]')
+    end
+  end
+
 end

data/spec/inputs/datetime_input_spec.rb

@@ -76,6 +76,42 @@ describe 'datetime input' do
   
   end
   
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:created_at, :as => :datetime))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_created_at_input")
+    end
+    
+    it 'should index the id of the select tag' do
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_1i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_2i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_3i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_4i")
+      output_buffer.should have_tag("select#post_author_attributes_3_created_at_5i")
+    end
+    
+    it 'should index the name of the select tag' do
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(1i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(2i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(3i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(4i)]']")
+      output_buffer.should have_tag("select[@name='post[author_attributes][3][created_at(5i)]']")
+    end
+    
+  end
+  
+  
   describe ':labels option' do
     fields = [:year, :month, :day, :hour, :minute]
     fields.each do |field|
@@ -132,5 +168,18 @@ describe 'datetime input' do
       end
     end
   end
+
+  describe "when order does not have year first" do
+    before do
+      output_buffer.replace ''
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.input(:publish_at, :as => :datetime, :order => [:day, :month, :year]))
+      end)
+    end
+
+    it 'should associate the legend label with the new first select' do
+      output_buffer.should have_tag('form li.datetime fieldset legend.label label[@for="post_publish_at_3i"]')
+    end
+  end
   
 end

data/spec/inputs/email_input_spec.rb

@@ -42,6 +42,34 @@ describe 'email input' do
 
   end
   
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:name, :as => :email))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_name_input")
+    end
+    
+    it 'should index the id of the select tag' do
+      output_buffer.should have_tag("input#post_author_attributes_3_name")
+    end
+    
+    it 'should index the name of the select tag' do
+      output_buffer.should have_tag("input[@name='post[author_attributes][3][name]']")
+    end
+    
+  end
+  
+  
   describe "when required" do
     it "should add the required attribute to the input's html options" do
       with_config :use_required_attribute, true do 

data/spec/inputs/file_input_spec.rb

@@ -46,6 +46,34 @@ describe 'file input' do
 
   end
   
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:name, :as => :file))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_name_input")
+    end
+    
+    it 'should index the id of the select tag' do
+      output_buffer.should have_tag("input#post_author_attributes_3_name")
+    end
+    
+    it 'should index the name of the select tag' do
+      output_buffer.should have_tag("input[@name='post[author_attributes][3][name]']")
+    end
+    
+  end
+  
+  
   context "when required" do
     it "should add the required attribute to the input's html options" do
       with_config :use_required_attribute, true do

data/spec/inputs/hidden_input_spec.rb

@@ -98,6 +98,34 @@ describe 'hidden input' do
 
   end
   
+  describe "when index is provided" do
+
+    before do
+      @output_buffer = ''
+      mock_everything
+
+      concat(semantic_form_for(@new_post) do |builder|
+        concat(builder.fields_for(:author, :index => 3) do |author|
+          concat(author.input(:name, :as => :hidden))
+        end)
+      end)
+    end
+    
+    it 'should index the id of the wrapper' do
+      output_buffer.should have_tag("li#post_author_attributes_3_name_input")
+    end
+    
+    it 'should index the id of the select tag' do
+      output_buffer.should have_tag("input#post_author_attributes_3_name")
+    end
+    
+    it 'should index the name of the select tag' do
+      output_buffer.should have_tag("input[@name='post[author_attributes][3][name]']")
+    end
+    
+  end
+  
+  
   context "when required" do
     it "should not add the required attribute to the input's html options" do
       concat(semantic_form_for(@new_post) do |builder|

data/spec/inputs/include_blank_spec.rb

@@ -11,59 +11,67 @@ describe "*select: options[:include_blank]" do
 
     @new_post.stub!(:author_id).and_return(nil)
     @new_post.stub!(:publish_at).and_return(nil)
-
-    @select_input_types = {
-        :select => :author,
-        :datetime => :publish_at,
-        :date => :publish_at,
-        :time => :publish_at
-      }
   end
 
-  describe 'when :include_blank is not set' do
-    it 'blank value should be included if the default value specified in config is true' do
-      Formtastic::FormBuilder.include_blank_for_select_by_default = true
-      @select_input_types.each do |as, attribute|
-        concat(semantic_form_for(@new_post) do |builder|
-          concat(builder.input(attribute, :as => as))
-        end)
-        output_buffer.should have_tag("form li select option[@value='']", "")
-      end
-    end
+  SELECT_INPUT_TYPES = {
+      :select => :author,
+      :datetime => :publish_at,
+      :date => :publish_at,
+      :time => :publish_at
+    }
+
+  SELECT_INPUT_TYPES.each do |as, attribute|
+    describe "for #{as} input" do
+
+      describe 'when :include_blank is not set' do
+        it 'blank value should be included if the default value specified in config is true' do
+          Formtastic::FormBuilder.include_blank_for_select_by_default = true
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.input(attribute, :as => as))
+          end)
+          output_buffer.should have_tag("form li select option[@value='']", "")
+        end
+
+        it 'blank value should not be included if the default value specified in config is false' do
+          Formtastic::FormBuilder.include_blank_for_select_by_default = false
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.input(attribute, :as => as))
+          end)
+          output_buffer.should_not have_tag("form li select option[@value='']", "")
+        end
 
-    it 'blank value should not be included if the default value specified in config is false' do
-      Formtastic::FormBuilder.include_blank_for_select_by_default = false
-      @select_input_types.each do |as, attribute|
-        concat(semantic_form_for(@new_post) do |builder|
-          concat(builder.input(attribute, :as => as))
-        end)
-        output_buffer.should_not have_tag("form li select option[@value='']", "")
+        after do
+          Formtastic::FormBuilder.include_blank_for_select_by_default = true
+        end
       end
-    end
 
-    after do
-      Formtastic::FormBuilder.include_blank_for_select_by_default = true
-    end
-  end
+      describe 'when :include_blank is set to false' do
+        it 'should not have a blank option' do
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.input(attribute, :as => as, :include_blank => false))
+          end)
+          output_buffer.should_not have_tag("form li select option[@value='']", "")
+        end
+      end
 
-  describe 'when :include_blank is set to false' do
-    it 'should not have a blank option' do
-      @select_input_types.each do |as, attribute|
-        concat(semantic_form_for(@new_post) do |builder|
-          concat(builder.input(attribute, :as => as, :include_blank => false))
-        end)
-        output_buffer.should_not have_tag("form li select option[@value='']", "")
+      describe 'when :include_blank is set to true' do
+        it 'should have a blank select option' do
+          concat(semantic_form_for(@new_post) do |builder|
+            concat(builder.input(attribute, :as => as, :include_blank => true))
+          end)
+          output_buffer.should have_tag("form li select option[@value='']", "")
+        end
       end
-    end
-  end
 
-  describe 'when :include_blank => true is set' do
-    it 'should have a blank select option' do
-      @select_input_types.each do |as, attribute|
-        concat(semantic_form_for(@new_post) do |builder|
-          concat(builder.input(attribute, :as => as, :include_blank => true))
-        end)
-        output_buffer.should have_tag("form li select option[@value='']", "")
+      if as == :select
+        describe 'when :include_blank is set to a string' do
+          it 'should have a select option with blank value but that string as text' do
+            concat(semantic_form_for(@new_post) do |builder|
+              concat(builder.input(attribute, :as => as, :include_blank => 'string'))
+            end)
+            output_buffer.should have_tag("form li select option[@value='']", "string")
+          end
+        end
       end
     end
   end