tabs_on_rails 1.1.0 → 1.2.0

data/CHANGELOG.rdoc

@@ -1,6 +1,15 @@
 = Changelog
 
 
+== Release 1.2.0
+
+* ADDED: Rails 3 compatibility.
+
+* CHANGED: Simplified tabs_tag helper by moving the rendering logic into TabsOnRails::Tabs#render.
+
+* CHANGED: TabsOnRails::Tabs::Builder (and all child classes) now has full access to @options hash.
+
+
 == Release 1.1.0
 
 * FIXED: Incompatibility with release < 1.0.0 caused by open_tabs and close_tabs methods changes compared to 0.8.0.

data/README.rdoc

@@ -9,19 +9,26 @@ It provides helpers for creating tabs with a flexible interface.
 * Ruby >= 1.8.6
 * Rails >= 2.2
 
-Note. TabsOnRails doesn't work with Rails 2.1 or previous versions (see {this comment}[http://www.simonecarletti.com/blog/2009/04/tabsonrails/#comment-2901] and {this commit}[http://github.com/weppos/tabs_on_rails/commit/d5ae9f401e3d0acc87251fa8957a8625e90ba4b3]).
+or
 
+* Rails 3
 
-== Rails Installation
+Warning: TabsOnRails doesn't work with Rails 2.1 or previous versions
+(see {this comment}[http://www.simonecarletti.com/blog/2009/04/tabsonrails/#comment-2901]
+and {this commit}[http://github.com/weppos/tabs_on_rails/commit/d5ae9f401e3d0acc87251fa8957a8625e90ba4b3]).
 
-=== As a Gem
 
-This is the preferred way to install TabsOnRails and the best way if you want install a stable version.
+== Installation
+
+=== As a Gem (Rails 2)
+
+RubyGems is the preferred way to install TabsOnRails and the best way if you want install a stable version.
 The GEM is hosted on {RubyGems}[http://rubygems.org/gems/tabs_on_rails].
 
   $ gem install tabs_on_rails
 
-With Rails >= 2.2, you can specify the GEM dependency in your environment.rb file so that Rails will automatically check the requirement on startup.
+With Rails >= 2.2, specify the Gem dependency in your environment.rb file so that Rails
+will automatically check the requirement on startup.
 
   Rails::Initializer.run do |config|
     
@@ -32,12 +39,19 @@ With Rails >= 2.2, you can specify the GEM dependency in your environment.rb fil
     
   end
 
-=== As a Plugin
+With Rails 3, specify the Gem dependency in the {Bundler}[http://gembundler.com/] Gemfile.
+
+  gem "tabs_on_rails"
 
-This is the preferred way if you want to live on the edge and install a development version.
+
+=== As a Plugin
 
   $ script/plugin install git://github.com/weppos/tabs_on_rails.git
 
+Warning: this method is deprecated and will be removed in a future version.
+Use Bundler and the {:git option}[http://gembundler.com/v1.0/git.html]
+if you want to grab the latest version from the Git repository.
+
 
 == Usage
 
@@ -67,7 +81,7 @@ when you want to mark a tab as the current tab.
     set_tab :dashboard
   end
 
-Now, if the action belongs to DashboardController, the template will automatically render the following HTML code.
+Now, if the action belongs to <tt>DashboardController</tt>, the template will automatically render the following HTML code.
 
   <ul>
     <li><a href="/">Homepage</a></li>
@@ -84,7 +98,7 @@ Use the <tt>current_tab</tt> helper method if you need to access the value of cu
   # In your view
   <p>The name of current tab is <%= current_tab %>.</p>
 
-The open_tag can be customized with the `:open_tabs` option. 
+The open_tag can be customized with the <tt>:open_tabs</tt> option.
 
   <% tabs_tag :open_tabs => { :id => "tabs", :class => "cool" } do |tab| %>
     <%= tab.home      'Homepage', root_path %>
@@ -98,7 +112,7 @@ The open_tag can be customized with the `:open_tabs` option.
     <li><a href="/account">Account</a></li>
   </ul>
 
-Further customizations require a custom Builder (see below).
+Further customizations require a custom <tt>Builder</tt> (see below).
 
 
 == Restricting set_tab scope

data/Rakefile

@@ -82,7 +82,7 @@ task :gemspec do
   File.open(file, "w") {|f| f << spec.to_ruby }
 end
 
-desc "Remove any temporary products, including gemspec."
+desc "Remove any temporary products, including gemspec"
 task :clean => [:clobber] do
   rm "#{spec.name}.gemspec"
 end
@@ -96,7 +96,7 @@ task :package => [:gemspec]
 begin
   require "rcov/rcovtask"
 
-  desc "Create a code coverage report."
+  desc "Create a code coverage report"
   Rcov::RcovTask.new do |t|
     t.test_files = FileList["test/**/*_test.rb"]
     t.ruby_opts << "-Itest -x mocha,rcov,Rakefile"
@@ -118,3 +118,9 @@ begin
 rescue LoadError
   puts "CodeStatistics (Rails) is not available"
 end
+
+desc "Publish documentation to the site"
+task :publish_rdoc => [:clobber_rdoc, :rdoc] do
+  ENV["username"] || raise(ArgumentError, "Missing ssh username")
+  sh "rsync -avz --delete rdoc/ #{ENV["username"]}@code:/var/www/apps/code/#{PKG_NAME}/api"
+end

data/init.rb

@@ -1 +1 @@
-require File.join(File.dirname(__FILE__), 'rails', 'init')
+require File.join(File.dirname(__FILE__), "rails", "init")

data/lib/tabs_on_rails/controller_mixin.rb

@@ -1,8 +1,8 @@
-# 
+#
 # = Tabs on Rails
 #
 # A simple Ruby on Rails plugin for creating and managing Tabs.
-# 
+#
 #
 # Category::    Rails
 # Package::     TabsOnRails
@@ -10,16 +10,16 @@
 # License::     MIT License
 #
 #--
-# 
+#
 #++
 
 
 module TabsOnRails
-  
+
   module ControllerMixin
-    
+
     def self.included(base)
-      base.extend         ClassMethods
+      base.extend     ClassMethods
       base.class_eval do
         include       InstanceMethods
         helper        HelperMethods
@@ -43,8 +43,8 @@ module TabsOnRails
       # For example, you can set the tab only for a restricted group of actions in the same controller
       # using the <tt>:only</tt> and <tt>:except</tt> options.
       #
-      # ==== Examples
-      # 
+      # Examples
+      #
       #   set_tab :foo
       #   set_tab :foo, :except => :new
       #   set_tab :foo, :only => [ :index, :show ]
@@ -55,7 +55,7 @@ module TabsOnRails
       def set_tab(*args)
         options = args.extract_options!
         name, namespace = args
-        
+
         before_filter(options) do |controller|
           controller.send(:set_tab, name, namespace)
         end
@@ -69,7 +69,7 @@ module TabsOnRails
         # Sets the value for current tab to given name.
         # If you need to manage multiple tabs, then you can pass an optional namespace.
         #
-        # ==== Examples
+        # Examples
         #
         #   set_tab :homepage
         #   set_tab :dashboard, :menu
@@ -82,7 +82,7 @@ module TabsOnRails
         # or nil if no tab has been set before.
         # You can pass <tt>namespace</tt> to get the value of current tab for a different namespace.
         #
-        # ==== Examples
+        # Examples
         #
         #   current_tab           # => nil
         #   current_tab :menu     # => nil
@@ -113,19 +113,71 @@ module TabsOnRails
 
     module HelperMethods
 
+      # In your template use the <tt>tabs_tag</tt> helper to create your tab.
+      #
+      #   <% tabs_tag do |tab| %>
+      #     <%= tab.home      'Homepage', root_path %>
+      #     <%= tab.dashboard 'Dashboard', dashboard_path %>
+      #     <%= tab.account   'Account', account_path %>
+      #   <% end %>
+      #
+      # The example above produces the following HTML output.
+      #
+      #   <ul>
+      #     <li><a href="/">Homepage</a></li>
+      #     <li><a href="/dashboard">Dashboard</a></li>
+      #     <li><a href="/account">Account</a></li>
+      #   </ul>
+      #
+      # The usage is similar to the Rails route file.
+      # You create named tabs with the syntax <tt>tab.name_of_tab</tt>.
+      #
+      # The name you use creating a tab is the same you're going to refer to in your controller
+      # when you want to mark a tab as the current tab.
+      #
+      #   class DashboardController < ApplicationController
+      #     set_tab :dashboard
+      #   end
+      #
+      # Now, if the action belongs to <tt>DashboardController</tt>,
+      # the template will automatically render the following HTML code.
+      # 
+      #   <ul>
+      #     <li><a href="/">Homepage</a></li>
+      #     <li><span>Dashboard</span></li>
+      #     <li><a href="/account">Account</a></li>
+      #   </ul>
+      #
+      # Use the <tt>current_tab</tt> helper method if you need to access
+      # the value of current tab in your controller or template.
+      #
+      #   class DashboardController < ApplicationController
+      #     set_tab :dashboard
+      #   end
+      #
+      #   # In your view
+      #   <p>The name of current tab is <%= current_tab %>.</p>
+      #
+      # The open_tag can be customized with the <tt>:open_tabs</tt> option.
+      #
+      #   <% tabs_tag :open_tabs => { :id => "tabs", :class => "cool" } do |tab| %>
+      #     <%= tab.home      'Homepage', root_path %>
+      #     <%= tab.dashboard 'Dashboard', dashboard_path %>
+      #     <%= tab.account   'Account', account_path %>
+      #   <% end %>
+      #
+      #   <ul id="tabs" class="cool">
+      #     <li><a href="/">Homepage</a></li>
+      #     <li><a href="/dashboard">Dashboard</a></li>
+      #     <li><a href="/account">Account</a></li>
+      #   </ul>
+      #
+      # Further customizations require a custom <tt>Builder</tt>.
+      #
       def tabs_tag(options = {}, &block)
-        raise LocalJumpError, "no block given" unless block_given?
-        
-        options = options.dup
-        open_tabs_options  = options.delete(:open_tabs)  || {}
-        close_tabs_options = options.delete(:close_tabs) || {}
-        tabs = Tabs.new(self, { :namespace => :default }.merge(options))
-
-        concat(tabs.open_tabs(open_tabs_options).to_s)
-        yield  tabs
-        concat(tabs.close_tabs(close_tabs_options).to_s)
+        Tabs.new(self, { :namespace => :default }.merge(options)).render(&block)
       end
 
     end
   end
-end
+end

data/lib/tabs_on_rails/railtie.rb

@@ -0,0 +1,44 @@
+#
+# = Tabs on Rails
+#
+# A simple Ruby on Rails plugin for creating and managing Tabs.
+#
+#
+# Category::    Rails
+# Package::     TabsOnRails
+# Author::      Simone Carletti <weppos@weppos.net>
+# License::     MIT License
+#
+#--
+#
+#++
+
+
+module TabsOnRails
+
+  if defined? Rails::Railtie
+    require "rails"
+
+    class Railtie < Rails::Railtie
+      initializer "tabs_on_rails.initialize" do
+        ActiveSupport.on_load :action_controller do
+          TabsOnRails::Railtie.init
+        end
+      end
+    end
+
+  end
+
+  class Railtie
+
+    def self.rails_version
+      @@rails_version ||= Rails::VERSION::STRING rescue "2.2"
+    end
+
+    def self.init
+      ActionController::Base.send :include, TabsOnRails::ControllerMixin
+    end
+
+  end
+
+end

data/lib/tabs_on_rails/tabs/builder.rb

@@ -22,24 +22,32 @@ module TabsOnRails
     #
     # The Builder class represents the interface for any custom Builder.
     # 
-    # To create a custom Builder extend this class 
+    # To create a custom Builder, extend this class
     # and implement the following abstract methods:
     # 
-    # * tab_for(args)
+    # * <tt>tab_for</tt>
+    #
+    # Optionally, you can override the following methods to customize
+    # the Builder behavior:
+    #
+    # * <tt>open_tabs</tt>
+    # * <tt>close_tabs</tt>
     #
     class Builder
 
-      # Initializes a new builder with +context+.
+      # Initializes a new builder with the given hash of <tt>options</tt>,
+      # providing the current Rails template as <tt>context</tt>.
       #
-      # Note. You should not overwrite this method to prevent incompatibility with future versions.
+      # Warning: You should not override this method to prevent incompatibility with future versions.
       def initialize(context, options = {})
         @context   = context
         @namespace = options.delete(:namespace) || :default
+        @options   = options
       end
 
       # Returns true if +tab+ is the +current_tab+.
       #
-      # ==== Examples
+      # Examples
       # 
       #   class MyController < ApplicationController
       #     tab :foo
@@ -56,24 +64,21 @@ module TabsOnRails
 
 
       # Creates and returns a tab with given +args+.
-      # 
-      # ==== Raises
-      # 
-      # NotImplemented:: you should implement this method in your custom Builder.
-      # 
+      #
+      # Raises NotImplemented: you should implement this method in your custom Builder.
       def tab_for(*args)
         raise NotImplementedError
       end
 
-      # Overwrite this method to use a custom open tag for your tabs.
+      # Override this method to use a custom open tag for your tabs.
       def open_tabs(*args)
       end
 
-      # Overwrite this method to use a custom close tag for your tabs.
+      # Override this method to use a custom close tag for your tabs.
       def close_tabs(*args)
       end
 
     end
 
   end
-end
+end

data/lib/tabs_on_rails/tabs/tabs_builder.rb

@@ -21,11 +21,9 @@ module TabsOnRails
     # = Tabs Builder
     # 
     # The TabsBuilder is and example of custom Builder.
-    # It creates a new tab
     #
     class TabsBuilder < Builder
 
-      # Implements Builder#tab_for.
       # Returns a link_to +tab+ with +name+ and +options+ if +tab+ is not the current tab,
       # a simple tab name wrapped by a span tag otherwise.
       # 
@@ -37,6 +35,8 @@ module TabsOnRails
       #   tab_for :bar, 'Bar', bar_path
       #   # => <li><a href="/link/to/bar">Bar</a></li>
       # 
+      # Implements Builder#tab_for.
+      #
       def tab_for(tab, name, options)
         content = @context.link_to_unless(current_tab?(tab), name, options) do
           @context.content_tag(:span, name)
@@ -44,10 +44,9 @@ module TabsOnRails
         @context.content_tag(:li, content)
       end
 
-      # Implements Builder#open_tabs.
-      # 
       # Returns an unordered list open tag.
-      # The <tt>options</tt> is used to customize the HTML attributes of the tag.
+      #
+      # The <tt>options</tt> hash is used to customize the HTML attributes of the tag.
       #
       #   open_tag
       #   # => "<ul>"
@@ -55,15 +54,16 @@ module TabsOnRails
       #   open_tag :class => "centered"
       #   # => "<ul class=\"centered\">"
       #
+      # Implements Builder#open_tabs.
+      #
       def open_tabs(options = {})
         @context.tag("ul", options, open = true)
       end
 
-      # Implements Builder#close_tabs.
-      # 
       # Returns an unordered list close tag.
-      # The <tt>options</tt> hash is ignored. It exists only for 
-      # coeherence with the parent Builder API.
+      #
+      # The <tt>options</tt> hash is ignored here.
+      # It exists only for coherence with the parent Builder API.
       #
       #   close_tag
       #   # => "</ul>"
@@ -71,11 +71,13 @@ module TabsOnRails
       #   close_tag :class => "centered"
       #   # => "</ul>"
       #
+      # Implements Builder#close_tabs.
+      #
       def close_tabs(options = {})
-        "</ul>"
+        "</ul>".try(:html_safe)
       end
 
     end
 
   end
-end
+end

data/lib/tabs_on_rails/tabs.rb

@@ -22,9 +22,10 @@ module TabsOnRails
 
   class Tabs
 
-    def initialize(context, options = {}, &block)
+    def initialize(context, options = {})
       @context = context
       @builder = (options.delete(:builder) || TabsBuilder).new(@context, options)
+      @options = options
     end
     
     %w(open_tabs close_tabs).each do |name|
@@ -45,7 +46,27 @@ module TabsOnRails
     def method_missing(*args)
       @builder.tab_for(*args)
     end
-    
+
+    def render(&block)
+      raise LocalJumpError, "no block given" unless block_given?
+
+      options = @options.dup
+      open_tabs_options  = options.delete(:open_tabs)  || {}
+      close_tabs_options = options.delete(:close_tabs) || {}
+
+      if Railtie.rails_version < "3"
+        @context.concat(open_tabs(open_tabs_options).to_s)
+        yield self
+        @context.concat(close_tabs(close_tabs_options).to_s)
+      else
+        "".tap do |html|
+          html << open_tabs(open_tabs_options).to_s
+          html << capture(self, &block)
+          html << close_tabs(close_tabs_options).to_s
+        end.html_safe
+      end
+    end
+
   end
 
-end
+end

data/lib/tabs_on_rails/version.rb

@@ -18,7 +18,7 @@ module TabsOnRails
 
   module Version
     MAJOR = 1
-    MINOR = 1
+    MINOR = 2
     PATCH = 0
     BUILD = nil
 

data/lib/tabs_on_rails.rb

@@ -16,13 +16,13 @@
 
 require 'tabs_on_rails/controller_mixin'
 require 'tabs_on_rails/tabs'
+require 'tabs_on_rails/railtie'
 require 'tabs_on_rails/version'
 
 
 module TabsOnRails
   
-  NAME            = 'Tabs on Rails'
-  GEM             = 'tabs_on_rails'
-  AUTHOR          = 'Simone Carletti <weppos@weppos.net>'
-  
-end
+  NAME            = "Tabs on Rails"
+  GEM             = "tabs_on_rails"
+
+end

data/rails/init.rb

@@ -1,3 +1,20 @@
-require 'tabs_on_rails'
+#
+# = Tabs on Rails
+#
+# A simple Ruby on Rails plugin for creating and managing Tabs.
+#
+#
+# Category::    Rails
+# Package::     TabsOnRails
+# Author::      Simone Carletti <weppos@weppos.net>
+# License::     MIT License
+#
+#--
+#
+#++
 
-ActionController::Base.send :include, TabsOnRails::ControllerMixin
+
+require "tabs_on_rails"
+
+
+TabsOnRails::Railtie.init

data/tabs_on_rails.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
 }
   s.email = %q{weppos@weppos.net}
   s.extra_rdoc_files = ["CHANGELOG.rdoc", "LICENSE.rdoc", "README.rdoc"]
-  s.files = ["Rakefile", "init.rb", "CHANGELOG.rdoc", "LICENSE.rdoc", "README.rdoc", "tabs_on_rails.gemspec", "test/controller_mixin_test.rb", "test/fixtures", "test/fixtures/mixin", "test/fixtures/mixin/default.html.erb", "test/fixtures/mixin/with_open_close_tabs.html.erb", "test/tabs", "test/tabs/builder_test.rb", "test/tabs/tabs_builder_test.rb", "test/tabs_test.rb", "test/test_helper.rb", "lib/tabs_on_rails", "lib/tabs_on_rails/controller_mixin.rb", "lib/tabs_on_rails/tabs", "lib/tabs_on_rails/tabs/builder.rb", "lib/tabs_on_rails/tabs/tabs_builder.rb", "lib/tabs_on_rails/tabs.rb", "lib/tabs_on_rails/version.rb", "lib/tabs_on_rails.rb", "rails/init.rb"]
+  s.files = ["Rakefile", "init.rb", "CHANGELOG.rdoc", "LICENSE.rdoc", "README.rdoc", "tabs_on_rails.gemspec", "test/controller_mixin_test.rb", "test/fixtures/mixin/default.html.erb", "test/fixtures/mixin/with_open_close_tabs.html.erb", "test/tabs/builder_test.rb", "test/tabs/tabs_builder_test.rb", "test/tabs_test.rb", "test/test_helper.rb", "lib/tabs_on_rails/controller_mixin.rb", "lib/tabs_on_rails/tabs/builder.rb", "lib/tabs_on_rails/tabs/tabs_builder.rb", "lib/tabs_on_rails/tabs.rb", "lib/tabs_on_rails/version.rb", "lib/tabs_on_rails.rb", "rails/init.rb"]
   s.homepage = %q{http://www.simonecarletti.com/code/tabs_on_rails}
   s.rdoc_options = ["--main", "README.rdoc"]
   s.require_paths = ["lib"]

data/test/tabs/builder_test.rb

@@ -24,42 +24,59 @@ class BuilderTest < ActiveSupport::TestCase
   end
 
 
-  def test_initialize_should_require_context
+  def test_initialize_without_context
     assert_raise(ArgumentError) { TabsOnRails::Tabs::Builder.new }
-    assert_nothing_raised { TabsOnRails::Tabs::Builder.new(@template) }
   end
   
-  def test_initialize_should_allow_options
-    assert_nothing_raised { TabsOnRails::Tabs::Builder.new(@template, { :namespace => 'foonamespace' }) }
+  def test_initialize_with_context
+    assert_nothing_raised { TabsOnRails::Tabs::Builder.new(@template) }
+  end
+
+  def test_initialize_with_options
+    assert_nothing_raised { TabsOnRails::Tabs::Builder.new(@template, { :namespace => "foonamespace" }) }
   end
 
   def test_initialize_should_set_context
-    assert_equal(@template, @builder.instance_variable_get(:'@context'))
+    assert_equal @template, @builder.instance_variable_get(:"@context")
   end
 
   def test_initialize_should_set_namespace
-    @builder  = TabsOnRails::Tabs::Builder.new(@template)
-    assert_equal(:default, @builder.instance_variable_get(:'@namespace'))
+    @builder = TabsOnRails::Tabs::Builder.new(@template)
+    assert_equal :default, @builder.instance_variable_get(:"@namespace")
 
-    @builder  = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foobar)
-    assert_equal(:foobar, @builder.instance_variable_get(:'@namespace'))
+    @builder = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foobar)
+    assert_equal :foobar, @builder.instance_variable_get(:"@namespace")
+  end
+
+  def test_initialize_should_set_options
+    @builder = TabsOnRails::Tabs::Builder.new(@template)
+    assert_equal({}, @builder.instance_variable_get(:"@options"))
+
+    @builder = TabsOnRails::Tabs::Builder.new(@template, :weather => :sunny)
+    assert_equal({ :weather => :sunny }, @builder.instance_variable_get(:"@options"))
+
+    @builder = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foobar, :weather => :sunny)
+    assert_equal({ :weather => :sunny }, @builder.instance_variable_get(:"@options"))
+
+    @builder = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foobar)
+    assert_equal({}, @builder.instance_variable_get(:"@options"))
   end
 
 
   def test_current_tab
-    @builder  = TabsOnRails::Tabs::Builder.new(@template)
-    assert( @builder.current_tab?(:dashboard))
-    assert( @builder.current_tab?('dashboard'))
-    assert(!@builder.current_tab?(:welcome))
-    assert(!@builder.current_tab?('welcome'))
+    @builder = TabsOnRails::Tabs::Builder.new(@template)
+    assert  @builder.current_tab?(:dashboard)
+    assert  @builder.current_tab?("dashboard")
+    assert !@builder.current_tab?(:welcome)
+    assert !@builder.current_tab?("welcome")
   end
 
   def test_current_tab_with_namespace
-    @builder  = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foospace)
-    assert( @builder.current_tab?(:footab))
-    assert( @builder.current_tab?('footab'))
-    assert(!@builder.current_tab?(:dashboard))
-    assert(!@builder.current_tab?('dashboard'))
+    @builder = TabsOnRails::Tabs::Builder.new(@template, :namespace => :foospace)
+    assert  @builder.current_tab?(:footab)
+    assert  @builder.current_tab?("footab")
+    assert !@builder.current_tab?(:dashboard)
+    assert !@builder.current_tab?("dashboard")
   end
 
 
@@ -82,8 +99,8 @@ class BuilderTest < ActiveSupport::TestCase
 
   def test_tab_for_should_raise_not_implemented_error
     assert_raise(NotImplementedError) { @builder.tab_for }
-    assert_raise(NotImplementedError) { @builder.tab_for('foo') }
-    assert_raise(NotImplementedError) { @builder.tab_for('foo', 'bar') }
+    assert_raise(NotImplementedError) { @builder.tab_for("foo") }
+    assert_raise(NotImplementedError) { @builder.tab_for("foo", "bar") }
   end
-  
-end
+
+end

data/test/tabs/tabs_builder_test.rb

@@ -20,39 +20,44 @@ class TabsBuilderTest < ActionView::TestCase
 
   def setup
     @template = Template.new
-    @builder  = TabsOnRails::Tabs::TabsBuilder.new(@template)
+    @klass    = TabsOnRails::Tabs::TabsBuilder
+    @builder  = @klass.new(@template)
+  end
+
+  def test_should_extend_builder
+    assert_equal TabsOnRails::Tabs::Builder, @klass.superclass
   end
 
 
   def test_open_tabs
-    assert_equal '<ul>', @builder.open_tabs
+    assert_dom_equal %Q{<ul>}, @builder.open_tabs
   end
 
   def test_open_tabs_with_options
-    assert_equal '<ul style="foo">', @builder.open_tabs(:style => "foo")
+    assert_dom_equal %Q{<ul style="foo">}, @builder.open_tabs(:style => "foo")
   end
 
   def test_close_tabs
-    assert_equal "</ul>", @builder.close_tabs
+    assert_dom_equal %Q{</ul>}, @builder.close_tabs
   end
 
   def test_close_tabs_with_options
-    assert_equal '</ul>', @builder.close_tabs(:foo => "bar")
+    assert_dom_equal %Q{</ul>}, @builder.close_tabs(:foo => "bar")
   end
 
 
-  def test_should_implement_builder
-    assert_equal(TabsOnRails::Tabs::Builder, TabsOnRails::Tabs::TabsBuilder.superclass)
-  end
-
   def test_tab_for_should_return_link_to_unless_current_tab
-    assert_dom_equal('<li><a href="#">Welcome</a></li>', @builder.tab_for(:welcome, 'Welcome', '#'))
-    assert_dom_equal('<li><a href="http://foobar.com/">Foo Bar</a></li>', @builder.tab_for(:welcome, 'Foo Bar', 'http://foobar.com/'))
+    assert_dom_equal %Q{<li><a href="#">Welcome</a></li>},
+                     @builder.tab_for(:welcome, 'Welcome', '#')
+    assert_dom_equal %Q{<li><a href="http://foobar.com/">Foo Bar</a></li>},
+                     @builder.tab_for(:welcome, 'Foo Bar', 'http://foobar.com/')
   end
 
   def test_tab_for_should_return_span_if_current_tab
-    assert_dom_equal('<li><span>Dashboard</span></li>', @builder.tab_for(:dashboard, 'Dashboard', '#'))
-    assert_dom_equal('<li><span>Foo Bar</span></li>', @builder.tab_for(:dashboard, 'Foo Bar', '#'))
+    assert_dom_equal %Q{<li><span>Dashboard</span></li>},
+                     @builder.tab_for(:dashboard, 'Dashboard', '#')
+    assert_dom_equal %Q{<li><span>Foo Bar</span></li>},
+                     @builder.tab_for(:dashboard, 'Foo Bar', '#')
   end
 
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: tabs_on_rails
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 31
   prerelease: false
   segments: 
   - 1
-  - 1
+  - 2
   - 0
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors: 
 - Simone Carletti
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-08-09 00:00:00 +02:00
+date: 2010-08-10 00:00:00 +02:00
 default_executable: 
 dependencies: []
 
@@ -44,6 +44,7 @@ files:
 - test/tabs_test.rb
 - test/test_helper.rb
 - lib/tabs_on_rails/controller_mixin.rb
+- lib/tabs_on_rails/railtie.rb
 - lib/tabs_on_rails/tabs/builder.rb
 - lib/tabs_on_rails/tabs/tabs_builder.rb
 - lib/tabs_on_rails/tabs.rb