table_helper 0.0.5 → 0.1.0

data/lib/table_helper/html_element.rb

@@ -1,46 +1,44 @@
-module PluginAWeek #:nodoc:
-  module TableHelper
-    # Represents an HTML element
-    # 
-    # == Modifying HTML options
-    # 
-    # HTML options can normally be specified when creating the element.
-    # However, if they need to be modified after the element has been created,
-    # you can access the properties like so:
-    # 
-    #   r = Row.new
-    #   r[:style] = 'display: none;'
-    # 
-    # or for a cell:
-    # 
-    #   c = Cell.new
-    #   c[:style] = 'display: none;'
-    class HtmlElement
-      include ActionView::Helpers::TagHelper
-      
-      delegate  :[],
-                :[]=,
-                  :to => '@html_options'
-      
-      def initialize(html_options = {}) #:nodoc:
-        @html_options = html_options.symbolize_keys
+module TableHelper
+  # Represents an HTML element
+  # 
+  # == Modifying HTML options
+  # 
+  # HTML options can normally be specified when creating the element.
+  # However, if they need to be modified after the element has been created,
+  # you can access the properties like so:
+  # 
+  #   r = Row.new
+  #   r[:style] = 'display: none;'
+  # 
+  # or for a cell:
+  # 
+  #   c = Cell.new
+  #   c[:style] = 'display: none;'
+  class HtmlElement
+    include ActionView::Helpers::TagHelper
+    
+    delegate  :[],
+              :[]=,
+                :to => '@html_options'
+    
+    def initialize(html_options = {}) #:nodoc:
+      @html_options = html_options.symbolize_keys
+    end
+    
+    # Generates the html representing this element
+    def html
+      content_tag(tag_name, content, @html_options)
+    end
+    
+    private
+      # The name of the element tag to use (e.g. td, th, tr, etc.)
+      def tag_name
+        ''
       end
       
-      # Generates the html representing this element
-      def html
-        content_tag(tag_name, content, @html_options)
+      # The content that will be displayed inside of the tag
+      def content
+        ''
       end
-      
-      private
-        # The name of the element tag to use (e.g. td, th, tr, etc.)
-        def tag_name
-          ''
-        end
-        
-        # The content that will be displayed inside of the tag
-        def content
-          ''
-        end
-    end
   end
 end

data/lib/table_helper/row.rb

@@ -1,103 +1,101 @@
 require 'table_helper/cell'
 
-module PluginAWeek #:nodoc:
-  module TableHelper
-    # Provides a blank class that can be used to build the cells for a row
-    class RowBuilder < BlankSlate
-      reveal :respond_to?
-      
-      attr_reader :row
-      
-      # Creates a builder for the given row
-      def initialize(row)
-        @row = row
-      end
-      
-      # Proxies all missed methods to the row
-      def method_missing(*args)
-        row.send(*args)
-      end
+module TableHelper
+  # Provides a blank class that can be used to build the cells for a row
+  class RowBuilder < BlankSlate #:nodoc:
+    reveal :respond_to?
+    
+    attr_reader :row
+    
+    # Creates a builder for the given row
+    def initialize(row)
+      @row = row
+    end
+    
+    # Proxies all missed methods to the row
+    def method_missing(*args)
+      row.send(*args)
+    end
+    
+    # Defines the builder method for the given cell name.  For example, if
+    # a cell with the name :title was defined, then the cell would be able
+    # to be read and written like so:
+    # 
+    #   row.title               #=> Accesses the title
+    #   row.title "Page Title"  #=> Creates a new cell with "Page Title" as the content
+    def define_cell(name)
+      method_name = name.gsub('-', '_')
       
-      # Defines the builder method for the given cell name.  For example, if
-      # a cell with the name :title was defined, then the cell would be able
-      # to be read and written like so:
-      # 
-      #   row.title               #=> Accesses the title
-      #   row.title "Page Title"  #=> Creates a new cell with "Page Title" as the content
-      def define_cell(name)
-        method_name = name.gsub('-', '_')
-        
-        klass = class << self; self; end
-        klass.class_eval do
-          define_method(method_name) do |*args|
-            if args.empty?
-              row.cells[name]
-            else
-              row.cell(name, *args)
-            end
+      klass = class << self; self; end
+      klass.class_eval do
+        define_method(method_name) do |*args|
+          if args.empty?
+            row.cells[name]
+          else
+            row.cell(name, *args)
           end
-        end unless klass.method_defined?(method_name)
-      end
-      
-      # Removes the definition for the given cell
-      def undef_cell(name)
-        method_name = name.gsub('-', '_')
-        
-        klass = class << self; self; end
-        klass.class_eval do
-          remove_method(method_name)
         end
-      end
+      end unless klass.method_defined?(method_name)
     end
     
-    # Represents a single row within a table.  A row can consist of either
-    # data cells or header cells.
-    class Row < HtmlElement
-      # The proxy class used externally to build the actual cells
-      attr_reader :builder
-      
-      # The current cells in this row, in the order in which they will be built
-      attr_reader :cells
+    # Removes the definition for the given cell
+    def undef_cell(name)
+      method_name = name.gsub('-', '_')
       
-      def initialize #:nodoc:
-        super
-        
-        @cells = ActiveSupport::OrderedHash.new
-        @builder = RowBuilder.new(self)
+      klass = class << self; self; end
+      klass.class_eval do
+        remove_method(method_name)
       end
+    end
+  end
+  
+  # Represents a single row within a table.  A row can consist of either
+  # data cells or header cells.
+  class Row < HtmlElement
+    # The proxy class used externally to build the actual cells
+    attr_reader :builder
+    
+    # The current cells in this row, in the order in which they will be built
+    attr_reader :cells
+    
+    def initialize #:nodoc:
+      super
       
-      # Creates a new cell with the given name and generates shortcut
-      # accessors for the method.
-      def cell(name, *args)
-        name = name.to_s if name
-        
-        cell = Cell.new(name, *args)
-        cells[name] = cell
-        builder.define_cell(name) if name
-        
-        cell
-      end
+      @cells = ActiveSupport::OrderedHash.new
+      @builder = RowBuilder.new(self)
+    end
+    
+    # Creates a new cell with the given name and generates shortcut
+    # accessors for the method.
+    def cell(name, *args)
+      name = name.to_s if name
       
-      # The names of all cells in this row
-      def cell_names
-        cells.keys
-      end
+      cell = Cell.new(name, *args)
+      cells[name] = cell
+      builder.define_cell(name) if name
       
-      # Clears all of the current cells from the row
-      def clear
-        # Remove all of the shortcut methods
-        cell_names.each {|name| builder.undef_cell(name)}
-        cells.clear
+      cell
+    end
+    
+    # The names of all cells in this row
+    def cell_names
+      cells.keys
+    end
+    
+    # Clears all of the current cells from the row
+    def clear
+      # Remove all of the shortcut methods
+      cell_names.each {|name| builder.undef_cell(name)}
+      cells.clear
+    end
+    
+    private
+      def tag_name
+        'tr'
       end
       
-      private
-        def tag_name
-          'tr'
-        end
-        
-        def content
-          cells.values.map(&:html).join
-        end
-    end
+      def content
+        cells.values.map(&:html).join
+      end
   end
 end

data/test/{table_helper_test.rb → helpers/table_helper_test.rb}

@@ -1,7 +1,7 @@
-require File.dirname(__FILE__) + '/test_helper'
+require File.expand_path(File.dirname(__FILE__) + '/../test_helper')
 
-class TableHelperTest < Test::Unit::TestCase
-  include PluginAWeek::TableHelper
+class TableHelperTest < ActionView::TestCase
+  tests TableHelper
   
   def test_should_build_collection_table
     html = collection_table(['first', 'second', 'last']) do |header, body|
@@ -12,7 +12,7 @@ class TableHelperTest < Test::Unit::TestCase
       end
     end
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <table cellpadding="0" cellspacing="0">
         <thead>
           <tr>
@@ -31,7 +31,7 @@ class TableHelperTest < Test::Unit::TestCase
           </tr>
         </tbody>
       </table>
-    end_eval
+    end_str
     assert_html_equal expected, html
   end
 end

data/test/test_helper.rb

@@ -1,12 +1,9 @@
-require 'test/unit'
+# Load the plugin testing framework
+$:.unshift("#{File.dirname(__FILE__)}/../../plugin_test_helper/lib")
 require 'rubygems'
-require 'action_controller'
-require 'action_view'
+require 'plugin_test_helper'
 
-$:.unshift(File.dirname(__FILE__) + '/../lib')
-require File.dirname(__FILE__) + '/../init'
-
-class Test::Unit::TestCase #:nodoc:
+Test::Unit::TestCase.class_eval do
   private
     def assert_html_equal(expected, actual)
       assert_equal expected.strip.gsub(/\n\s*/, ''), actual

data/test/{body_row_test.rb → unit/body_row_test.rb}

@@ -1,9 +1,9 @@
-require File.dirname(__FILE__) + '/test_helper'
+require File.expand_path(File.dirname(__FILE__) + '/../test_helper')
 
 class BodyRowByDefaultTest < Test::Unit::TestCase
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
-    @row = PluginAWeek::TableHelper::BodyRow.new(Object.new, header)
+    header = TableHelper::Header.new([])
+    @row = TableHelper::BodyRow.new(Object.new, header)
   end
   
   def test_should_not_alternate
@@ -23,9 +23,9 @@ class BodyRowTest < Test::Unit::TestCase
   end
   
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
+    header = TableHelper::Header.new([])
     header.column :title
-    @row = PluginAWeek::TableHelper::BodyRow.new(Post.new, header)
+    @row = TableHelper::BodyRow.new(Post.new, header)
   end
   
   def test_should_generate_cell_accessors
@@ -40,8 +40,8 @@ end
 
 class BodyRowWithNoColumnsTest < Test::Unit::TestCase
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
-    @row = PluginAWeek::TableHelper::BodyRow.new(Object.new, header)
+    header = TableHelper::Header.new([])
+    @row = TableHelper::BodyRow.new(Object.new, header)
   end
   
   def test_should_not_build_cells
@@ -57,10 +57,10 @@ class BodyRowWithCustomAttributeTest < Test::Unit::TestCase
   end
   
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
+    header = TableHelper::Header.new([])
     header.column :title
     header.column :author_name
-    @row = PluginAWeek::TableHelper::BodyRow.new(Post.new, header)
+    @row = TableHelper::BodyRow.new(Post.new, header)
   end
   
   def test_should_use_attribute_values_as_cell_content
@@ -71,10 +71,10 @@ end
 
 class BodyRowWithMissingCellsTest < Test::Unit::TestCase
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
+    header = TableHelper::Header.new([])
     header.column :title
     header.column :author_name
-    @row = PluginAWeek::TableHelper::BodyRow.new(Object.new, header)
+    @row = TableHelper::BodyRow.new(Object.new, header)
   end
   
   def test_should_build_missing_cells_if_cells_not_specified
@@ -94,8 +94,8 @@ end
 
 class BodyRowAlternatingTest < Test::Unit::TestCase
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
-    @row = PluginAWeek::TableHelper::BodyRow.new(Object.new, header)
+    header = TableHelper::Header.new([])
+    @row = TableHelper::BodyRow.new(Object.new, header)
     @row.alternate = true
   end
   

data/test/{body_test.rb → unit/body_test.rb}

@@ -1,9 +1,9 @@
-require File.dirname(__FILE__) + '/test_helper'
+require File.expand_path(File.dirname(__FILE__) + '/../test_helper')
 
 class BodyByDefaultTest < Test::Unit::TestCase
   def setup
-    header = PluginAWeek::TableHelper::Header.new([])
-    @body = PluginAWeek::TableHelper::Body.new([], header)
+    header = TableHelper::Header.new([])
+    @body = TableHelper::Body.new([], header)
   end
   
   def test_should_not_alternate_rows
@@ -30,8 +30,8 @@ class BodyTest < Test::Unit::TestCase
       Post.new('second'),
       Post.new('last')
     ]
-    @header = PluginAWeek::TableHelper::Header.new(@collection)
-    @body = PluginAWeek::TableHelper::Body.new(@collection, @header)
+    @header = TableHelper::Header.new(@collection)
+    @body = TableHelper::Body.new(@collection, @header)
   end
   
   def test_should_raise_exception_if_invalid_alternate_rows_specified
@@ -57,11 +57,11 @@ class BodyTest < Test::Unit::TestCase
         assert_equal @collection.index(post), index
       end
       
-      expected = <<-end_eval
+      expected = <<-end_str
         <tr class="row">
           <td class="title">#{post.title}</td>
         </tr>
-      end_eval
+      end_str
       assert_html_equal expected, html
     end
   end
@@ -74,11 +74,11 @@ class BodyTest < Test::Unit::TestCase
       assert_equal 1, index
     end
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row">
         <td class="title">first</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, html
   end
   
@@ -86,19 +86,19 @@ class BodyTest < Test::Unit::TestCase
     @header.column :title
     @header.column :author_name
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row">
         <td class="title">first</td>
         <td class="author_name empty"></td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build_row(@collection.first)
   end
   
   def test_should_build_all_rows
     @header.column :title
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row">
         <td class="title">first</td>
       </tr>
@@ -108,7 +108,7 @@ class BodyTest < Test::Unit::TestCase
       <tr class="row">
         <td class="title">last</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build
   end
   
@@ -116,7 +116,7 @@ class BodyTest < Test::Unit::TestCase
     @header.column :title
     @body.build
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tbody>
         <tr class="row">
           <td class="title">first</td>
@@ -128,7 +128,7 @@ class BodyTest < Test::Unit::TestCase
           <td class="title">last</td>
         </tr>
       </tbody>
-    end_eval
+    end_str
     assert_html_equal expected, @body.html
   end
   
@@ -137,7 +137,7 @@ class BodyTest < Test::Unit::TestCase
     @body[:class] = 'pretty'
     @body.build
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tbody class="pretty">
         <tr class="row">
           <td class="title">first</td>
@@ -149,7 +149,7 @@ class BodyTest < Test::Unit::TestCase
           <td class="title">last</td>
         </tr>
       </tbody>
-    end_eval
+    end_str
     assert_html_equal expected, @body.html
   end
 end
@@ -157,16 +157,16 @@ end
 class BodyWithEmptyCollectionTest < Test::Unit::TestCase
   def setup
     @collection = []
-    @header = PluginAWeek::TableHelper::Header.new(@collection)
-    @body = PluginAWeek::TableHelper::Body.new(@collection, @header)
+    @header = TableHelper::Header.new(@collection)
+    @body = TableHelper::Body.new(@collection, @header)
   end
   
   def test_should_show_no_content
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="no_content">
         <td>No matches found.</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build
   end
   
@@ -179,11 +179,11 @@ class BodyWithEmptyCollectionTest < Test::Unit::TestCase
     @header.column :title
     @header.column :author_name
     
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="no_content">
         <td colspan="2">No matches found.</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build
   end
 end
@@ -203,28 +203,28 @@ class BodyWithAlternatingEvenRowsTest < Test::Unit::TestCase
       Post.new('second'),
       Post.new('last')
     ]
-    @header = PluginAWeek::TableHelper::Header.new(@collection)
+    @header = TableHelper::Header.new(@collection)
     @header.column :title
     
-    @body = PluginAWeek::TableHelper::Body.new(@collection, @header)
+    @body = TableHelper::Body.new(@collection, @header)
     @body.alternate_rows = :even
   end
   
   def test_should_alternate_even_row
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row alternate">
         <td class="title">first</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build_row(@collection.first)
   end
   
   def test_should_not_alternate_odd_row
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row">
         <td class="title">second</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build_row(@collection[1])
   end
 end
@@ -244,28 +244,28 @@ class BodyWithAlternatingOddRowsTest < Test::Unit::TestCase
       Post.new('second'),
       Post.new('last')
     ]
-    @header = PluginAWeek::TableHelper::Header.new(@collection)
+    @header = TableHelper::Header.new(@collection)
     @header.column :title
     
-    @body = PluginAWeek::TableHelper::Body.new(@collection, @header)
+    @body = TableHelper::Body.new(@collection, @header)
     @body.alternate_rows = :odd
   end
   
   def test_should_alternate_odd_row
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row alternate">
         <td class="title">second</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build_row(@collection[1])
   end
   
   def test_should_not_alternate_even_row
-    expected = <<-end_eval
+    expected = <<-end_str
       <tr class="row">
         <td class="title">first</td>
       </tr>
-    end_eval
+    end_str
     assert_html_equal expected, @body.build_row(@collection.first)
   end
 end