Add test to not use top-level wrapper. Closes #359

lib/simple_form.rb

@@ -142,7 +142,7 @@ def self.wrappers(*args, &block)
 
   # Builds a new wrapper using SimpleForm::Wrappers::Builder.
   def self.build(options={})
-    options[:tag] = :div if !options.empty? && options[:tag].nil?
+    options[:tag] = :div if options[:tag].nil?
     builder = SimpleForm::Wrappers::Builder.new
     yield builder
     SimpleForm::Wrappers::Root.new(builder.to_a, options)

test/form_builder/wrapper_test.rb

@@ -32,6 +32,12 @@ class WrapperTest < ActionView::TestCase
     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'

test/support/misc_helpers.rb

@@ -46,6 +46,14 @@ def custom_wrapper
     end
   end
 
+  def custom_wrapper_without_top_level
+    SimpleForm.build :tag => false, :class => 'custom_wrapper_without_top_level' do |b|
+      b.use :label_input
+      b.use :hint,  :tag => :span, :class => :hint
+      b.use :error, :tag => :span, :class => :error
+    end
+  end
+
   def custom_form_for(object, *args, &block)
     simple_form_for(object, *(args << { :builder => CustomFormBuilder }), &block)
   end