tabletastic 0.2.0.pre2 → 0.2.0.pre3

data/lib/tabletastic/helper.rb

@@ -33,4 +33,7 @@ module Tabletastic
   end
 end
 
-ActionView::Base.send :include, Tabletastic::Helper
+ActiveSupport.on_load(:action_view) do
+  include Tabletastic::Helper
+end
+

data/lib/tabletastic/version.rb

@@ -1,3 +1,3 @@
 module Tabletastic
-  VERSION = "0.2.0.pre2"
+  VERSION = "0.2.0.pre3"
 end

data/spec/spec_helper.rb

@@ -1,116 +1,114 @@
-$LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'rubygems'
+require 'spork'
 
-def smart_require(gem_name, gem_version = '>= 0.0.0', lib_name = nil)
-  lib_name ||= gem_name
-  begin
-    require lib_name if lib_name
-  rescue LoadError
-    if gem_name
-      gem gem_name, gem_version
-      require lib_name if lib_name
-    end
-  end
-end
+Spork.prefork do
+  ## Use bundler to exec the specs
+  $LOAD_PATH.unshift(File.dirname(__FILE__))
+  require 'bundler'
+  Bundler.setup
+  require 'rspec'
 
-smart_require 'rspec', '>= 1.3.0', 'spec'
-require 'spec/autorun'
-smart_require 'nokogiri'
-smart_require 'rspec_tag_matchers', '>= 1.0.0'
-smart_require 'activesupport', '>= 3.0.0.beta3', 'active_support'
-smart_require 'actionpack',    '>= 3.0.0.beta3', 'action_pack'
-smart_require 'activerecord',  '>= 3.0.0.beta3', 'active_record'
-require 'action_controller'
-require 'action_view/base'
-require 'action_view/template'
-require 'action_view/helpers'
-Spec::Runner.configure do |config|
-  config.include(RspecTagMatchers)
-end
+  require 'rspec_tag_matchers'
+  require 'active_record'
+  require 'action_controller'
 
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-require 'tabletastic'
-
-module TabletasticSpecHelper
-  include ActiveSupport
-  include ActionView
-  include ActionView::Helpers::UrlHelper
-  include ActionView::Helpers::TagHelper
-  include ActionView::Helpers::TextHelper
-  include ActionView::Helpers::ActiveModelHelper
-  include ActionView::Helpers::RecordIdentificationHelper
-  include ActionView::Helpers::RecordTagHelper
-  include ActionView::Helpers::CaptureHelper
-  include ActionView::Helpers::RawOutputHelper
-  include ActionController::PolymorphicRoutes
-
-  def self.included(base)
-    base.class_eval do
-      attr_accessor :output_buffer
-    end
+  require 'action_view/base'
+  require 'action_view/template'
+  require 'action_view/helpers'
+
+  RSpec.configure do |config|
+    config.include(RspecTagMatchers)
   end
 
-  module ::RspecTagMatchers
-    def have_table_with_tag(selector, inner_text_or_options = nil, options = {}, &block)
-      HaveTag.new("table", nil, {}) &&
-        HaveTag.new(selector, inner_text_or_options, options, &block)
+  $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+  require 'tabletastic'
+
+  module TabletasticSpecHelper
+    include ActiveSupport
+    include ActionView
+    include ActionView::Helpers::UrlHelper
+    include ActionView::Helpers::TagHelper
+    include ActionView::Helpers::TextHelper
+    include ActionView::Helpers::ActiveModelHelper
+    include ActionView::Helpers::RecordIdentificationHelper
+    include ActionView::Helpers::RecordTagHelper
+    include ActionView::Helpers::CaptureHelper
+    include ActionView::Helpers::RawOutputHelper
+    include ActionController::PolymorphicRoutes
+
+    def self.included(base)
+      base.class_eval do
+        attr_accessor :output_buffer
+      end
     end
-  end
 
-  class ::Post
-    def id
+    module ::RspecTagMatchers
+      def have_table_with_tag(selector, inner_text_or_options = nil, options = {}, &block)
+        HaveTag.new("table", nil, {}) &&
+          HaveTag.new(selector, inner_text_or_options, options, &block)
+      end
     end
-  end
-  class ::Author
-  end
 
-  def mock_everything
-    def post_path(post); "/posts/#{post.id}"; end
-    def admin_post_path(post); "/admin/posts/#{post.id}"; end
-    def edit_post_path(post); "/posts/#{post.id}/edit"; end
-    def edit_admin_post_path(post); "/admin/posts/#{post.id}/edit"; end
-
-    # Sometimes we need a mock @post object and some Authors for belongs_to
-    @post = mock('post')
-    @post.stub!(:class).and_return(::Post)
-    @post.stub!(:id).and_return(nil)
-    @post.stub!(:author)
-    @post.stub!(:to_key).and_return([2])
-    ::Post.stub!(:human_attribute_name).and_return { |column_name| column_name.humanize }
-    ::Post.stub!(:model_name).and_return(ActiveModel::Name.new(::Post))
-
-    @fred = mock('user')
-    @fred.stub!(:class).and_return(::Author)
-    @fred.stub!(:name).and_return('Fred Smith')
-    @fred.stub!(:id).and_return(37)
-
-    ::Author.stub!(:find).and_return([@fred])
-    ::Author.stub!(:human_attribute_name).and_return { |column_name| column_name.humanize }
-    ::Author.stub!(:human_name).and_return('Author')
-    ::Author.stub!(:model_name).and_return(ActiveModel::Name.new(::Author))
-    ::Author.stub!(:reflect_on_association).and_return { |column_name| mock('reflection', :options => {}, :klass => Post, :macro => :has_many) if column_name == :posts }
-
-    @freds_post = mock('post')
-    @freds_post.stub!(:class).and_return(::Post)
-    @freds_post.stub!(:title).and_return('Fred\'s Post')
-    @freds_post.stub!(:body)
-    @freds_post.stub!(:id).and_return(19)
-    @freds_post.stub!(:to_key).and_return([19])
-    @freds_post.stub!(:author).and_return(@fred)
-    @freds_post.stub!(:author_id).and_return(@fred.id)
-    @fred.stub!(:posts).and_return([@freds_post])
-    @fred.stub!(:post_ids).and_return([@freds_post.id])
-
-    @mock_reflection_belongs_to_author = mock('reflection', :options => {}, :name => :author, :klass => ::Author, :macro => :belongs_to)
-
-    ::Post.stub!(:reflect_on_association).and_return do |column_name|
-      @mock_reflection_belongs_to_author if column_name == :author
+    class ::Post
+      def id
+      end
+    end
+    class ::Author
     end
 
-    ::Post.stub!(:reflect_on_all_associations).with(:belongs_to).and_return([])
+    def mock_everything
+      def post_path(post); "/posts/#{post.id}"; end
+      def admin_post_path(post); "/admin/posts/#{post.id}"; end
+      def edit_post_path(post); "/posts/#{post.id}/edit"; end
+      def edit_admin_post_path(post); "/admin/posts/#{post.id}/edit"; end
+
+      # Sometimes we need a mock @post object and some Authors for belongs_to
+      @post = mock('post')
+      @post.stub!(:class).and_return(::Post)
+      @post.stub!(:id).and_return(nil)
+      @post.stub!(:author)
+      @post.stub!(:to_key).and_return([2])
+      ::Post.stub!(:human_attribute_name).and_return { |column_name| column_name.humanize }
+      ::Post.stub!(:model_name).and_return(ActiveModel::Name.new(::Post))
+
+      @fred = mock('user')
+      @fred.stub!(:class).and_return(::Author)
+      @fred.stub!(:name).and_return('Fred Smith')
+      @fred.stub!(:id).and_return(37)
+
+      ::Author.stub!(:find).and_return([@fred])
+      ::Author.stub!(:human_attribute_name).and_return { |column_name| column_name.humanize }
+      ::Author.stub!(:human_name).and_return('Author')
+      ::Author.stub!(:model_name).and_return(ActiveModel::Name.new(::Author))
+      ::Author.stub!(:reflect_on_association).and_return { |column_name| mock('reflection', :options => {}, :klass => Post, :macro => :has_many) if column_name == :posts }
+
+      @freds_post = mock('post')
+      @freds_post.stub!(:class).and_return(::Post)
+      @freds_post.stub!(:title).and_return('Fred\'s Post')
+      @freds_post.stub!(:body)
+      @freds_post.stub!(:id).and_return(19)
+      @freds_post.stub!(:to_key).and_return([19])
+      @freds_post.stub!(:author).and_return(@fred)
+      @freds_post.stub!(:author_id).and_return(@fred.id)
+      @fred.stub!(:posts).and_return([@freds_post])
+      @fred.stub!(:post_ids).and_return([@freds_post.id])
+
+      @mock_reflection_belongs_to_author = mock('reflection', :options => {}, :name => :author, :klass => ::Author, :macro => :belongs_to)
+
+      ::Post.stub!(:reflect_on_association).and_return do |column_name|
+        @mock_reflection_belongs_to_author if column_name == :author
+      end
+
+      ::Post.stub!(:reflect_on_all_associations).with(:belongs_to).and_return([])
+    end
   end
+
+  include TabletasticSpecHelper
+  include Tabletastic
+  include Tabletastic::Helper
 end
 
-include TabletasticSpecHelper
-include Tabletastic
-include Tabletastic::Helper
+Spork.each_run do
+  # This code will be run each time you run your specs.
+
+end

data/spec/tabletastic/table_builder_spec.rb

@@ -0,0 +1,316 @@
+require 'spec_helper'
+
+describe Tabletastic::TableBuilder do
+
+  before do
+    @output_buffer = ActiveSupport::SafeBuffer.new
+
+    mock_everything
+    ::Post.stub!(:content_columns).and_return([mock('column', :name => 'title'), mock('column', :name => 'body'), mock('column', :name => 'created_at')])
+    @post.stub!(:title).and_return("The title of the post")
+    @post.stub!(:body).and_return("Lorem ipsum")
+    @post.stub!(:created_at).and_return(Time.now)
+    @post.stub!(:id).and_return(2)
+    @posts = [@post]
+  end
+
+  context "without a block" do
+    context "with no other arguments" do
+      before do
+        concat table_for(@posts) { |t| t.data }
+      end
+
+      it "should output table with id of the class of the collection" do
+        output_buffer.should have_tag("table#posts")
+      end
+
+      it "should output head" do
+        output_buffer.should have_table_with_tag("thead")
+      end
+
+      it "should have a <th> for each attribute" do
+        # title and body
+        output_buffer.should have_table_with_tag("th", :count => 2)
+      end
+
+      it "should include header for Title" do
+        output_buffer.should have_table_with_tag("th", "Title")
+      end
+
+      it "should include header for Body" do
+        output_buffer.should have_table_with_tag("th", "Body")
+      end
+
+      it "should output body" do
+        output_buffer.should have_table_with_tag("tbody")
+      end
+
+      it "should include a row for each record" do
+        output_buffer.should have_table_with_tag("tbody") do |tbody|
+          tbody.should have_tag("tr", :count => 1)
+        end
+      end
+
+      it "should have data for each field" do
+        output_buffer.should have_table_with_tag("td", "The title of the post")
+        output_buffer.should have_table_with_tag("td", "Lorem ipsum")
+      end
+
+      it "should include the id for the <tr> for each record" do
+        output_buffer.should have_table_with_tag("tr#post_#{@post.id}")
+      end
+
+      it "should cycle row classes" do
+        @output_buffer = ""
+        @posts = [@post, @post]
+        table_for(@posts) do |t|
+          concat(t.data)
+        end
+        output_buffer.should have_table_with_tag("tr.odd")
+        output_buffer.should have_table_with_tag("tr.even")
+      end
+
+      context "when collection has associations" do
+        it "should handle belongs_to associations" do
+          ::Post.stub!(:reflect_on_all_associations).with(:belongs_to).and_return([@mock_reflection_belongs_to_author])
+          @posts = [@freds_post]
+          @output_buffer = ""
+          concat table_for(@posts) { |t| t.data }
+          output_buffer.should have_table_with_tag("th", "Author")
+          output_buffer.should have_table_with_tag("td", "Fred Smith")
+        end
+      end
+    end
+
+    context "with options[:actions]" do
+      it "includes path to post for :show" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => :show))
+        end
+        output_buffer.should have_table_with_tag("a[@href=\"/posts/#{@post.id}\"]")
+        output_buffer.should have_table_with_tag("th", "")
+      end
+
+      it "should have a cell with default class 'actions' and the action name" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => :show))
+        end
+        output_buffer.should have_tag("td.actions.show_link") do |td|
+          td.should have_tag("a")
+        end
+      end
+
+      it "includes path to post for :edit" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => :edit))
+        end
+        output_buffer.should have_tag("a[@href=\"/posts/#{@post.id}/edit\"]", "Edit")
+      end
+
+      it "includes path to post for :destroy" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => :destroy))
+        end
+        output_buffer.should have_table_with_tag("a[@href=\"/posts/#{@post.id}\"]")
+        output_buffer.should have_table_with_tag("th", "")
+      end
+
+      it "includes path to post for :show and :edit" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => [:show, :edit]))
+        end
+        output_buffer.should have_tag("td:nth-child(3) a[@href=\"/posts/#{@post.id}\"]", "Show")
+        output_buffer.should have_tag("td:nth-child(4) a[@href=\"/posts/#{@post.id}/edit\"]", "Edit")
+      end
+
+      it "includes path to post for :all" do
+        table_for(@posts) do |t|
+          concat(t.data(:actions => :all))
+        end
+        output_buffer.should have_tag("td:nth-child(3) a[@href=\"/posts/#{@post.id}\"]", "Show")
+        output_buffer.should have_tag("td:nth-child(4) a[@href=\"/posts/#{@post.id}/edit\"]", "Edit")
+        output_buffer.should have_tag("td:nth-child(5) a[@href=\"/posts/#{@post.id}\"]", "Destroy")
+      end
+
+      context "with options[:actions_prefix]" do
+        it "includes path to admin post for :show" do
+          table_for(@posts) do |t|
+            concat(t.data(:actions => :show, :action_prefix => :admin))
+          end
+          output_buffer.should have_tag("td:nth-child(3) a[@href=\"/admin/posts/#{@post.id}\"]", "Show")
+        end
+
+        it "includes path to admin post for :edit" do
+          table_for(@posts) do |t|
+            concat(t.data(:actions => :edit, :action_prefix => :admin))
+          end
+          output_buffer.should have_tag("td:nth-child(3) a[@href=\"/admin/posts/#{@post.id}/edit\"]", "Edit")
+        end
+
+        it "includes path to admin post for :destroy" do
+          table_for(@posts) do |t|
+            concat(t.data(:actions => :destroy, :action_prefix => :admin))
+          end
+          output_buffer.should have_tag("td:nth-child(3) a[@href=\"/admin/posts/#{@post.id}\"]", "Destroy")
+        end
+
+        it "includes path to admin for all actions" do
+          table_for(@posts) do |t|
+            concat(t.data(:actions => :all, :action_prefix => :admin))
+          end
+          output_buffer.should have_tag("td:nth-child(3) a[@href=\"/admin/posts/#{@post.id}\"]", "Show")
+          output_buffer.should have_tag("td:nth-child(4) a[@href=\"/admin/posts/#{@post.id}/edit\"]", "Edit")
+          output_buffer.should have_tag("td:nth-child(5) a[@href=\"/admin/posts/#{@post.id}\"]", "Destroy")
+        end
+      end
+    end
+
+    context "with a list of attributes" do
+      before do
+        table_for(@posts) do |t|
+          concat(t.data(:title, :created_at))
+        end
+      end
+
+      it "should call each method passed in, and only those methods" do
+        output_buffer.should have_table_with_tag("th", "Title")
+        output_buffer.should have_table_with_tag("th", "Created at")
+        output_buffer.should_not have_table_with_tag("th", "Body")
+      end
+    end
+
+    context "with a list of attributes and options[:actions]" do
+      it "includes path to post for :show" do
+        table_for(@posts) do |t|
+          concat(t.data(:title, :created_at, :actions => :show))
+        end
+        output_buffer.should have_tag("th:nth-child(1)", "Title")
+        output_buffer.should have_tag("th:nth-child(2)", "Created at")
+        output_buffer.should have_tag("th:nth-child(3)", "")
+        output_buffer.should_not have_tag("th", "Body")
+
+        output_buffer.should have_tag("td:nth-child(3) a[@href=\"/posts/#{@post.id}\"]")
+      end
+    end
+  end
+
+  context "with a block" do
+    context "and normal columns" do
+      before do
+        concat(table_for(@posts) do |t|
+          t.data do
+            t.cell(:title)
+            t.cell(:body)
+          end
+        end)
+      end
+
+      it "should include the data for the fields passed in" do
+        output_buffer.should have_table_with_tag("th", "Title")
+        output_buffer.should have_tag("td", "The title of the post")
+        output_buffer.should have_tag("td", "Lorem ipsum")
+      end
+    end
+
+    context "with custom cell options" do
+      before do
+        concat(table_for(@posts) do |t|
+          t.data do
+            t.cell(:title, :heading => "FooBar")
+            t.cell(:body, :cell_html => {:class => "batquux"})
+          end
+        end)
+      end
+
+      it "should change the heading label for :heading option" do
+        output_buffer.should have_table_with_tag("th", "FooBar")
+        output_buffer.should have_table_with_tag("th", "Body")
+      end
+
+      it "should pass :cell_html to the cell" do
+        output_buffer.should have_table_with_tag("td.batquux")
+      end
+    end
+
+    context "with custom cell options" do
+      before do
+        concat(table_for(@posts) do |t|
+          t.data do
+            t.cell(:title) {|p| link_to p.title, "/" }
+            t.cell(:body, :heading => "Content") {|p| p.body }
+          end
+        end)
+      end
+
+      it "accepts a block as a lazy attribute" do
+        output_buffer.should have_table_with_tag("th:nth-child(1)", "Title")
+        output_buffer.should have_table_with_tag("td:nth-child(1)") do |td|
+          td.should have_tag("a", "The title of the post")
+        end
+      end
+
+      it "accepts a block as a lazy attribute (2)" do
+        output_buffer.should have_table_with_tag("th:nth-child(2)", "Content")
+        output_buffer.should have_table_with_tag("td:nth-child(2)", "Lorem ipsum")
+      end
+    end
+
+    context "with options[:actions]" do
+      it "includes path to post for :show" do
+        concat(table_for(@posts) do |t|
+          t.data(:actions => :show) do
+            t.cell(:title)
+            t.cell(:body)
+          end
+        end)
+        output_buffer.should have_table_with_tag("td:nth-child(3) a[@href=\"/posts/#{@post.id}\"]")
+      end
+    end
+
+    context "and normal/association columns" do
+      before do
+        ::Post.stub!(:reflect_on_all_associations).with(:belongs_to).and_return([@mock_reflection_belongs_to_author])
+        @posts = [@freds_post]
+        concat(table_for(@posts) do |t|
+          t.data do
+            t.cell(:title)
+            t.cell(:author)
+          end
+        end)
+      end
+
+      it "should include normal columns" do
+        output_buffer.should have_table_with_tag("th:nth-child(1)", "Title")
+        output_buffer.should have_table_with_tag("td:nth-child(1)", "Fred's Post")
+      end
+
+      it "should include belongs_to associations" do
+        output_buffer.should have_table_with_tag("th:nth-child(2)", "Author")
+        output_buffer.should have_table_with_tag("td:nth-child(2)", "Fred Smith")
+      end
+    end
+  end
+  
+  context "using human_attribute_names" do
+    it "should work" do
+      ::Post.stub!(:human_attribute_name).with('body').and_return("Blah blue")
+
+      concat(table_for(@posts) do |t|
+        t.data do
+          t.cell(:title)
+          t.cell(:body)
+        end
+      end)
+
+      output_buffer.should have_table_with_tag("th", "Blah blue")
+    end
+  end
+  
+  context "when table_for is not passed a block" do
+    it "the data should use the default option" do
+      Tabletastic.default_table_block = lambda {|table| table.data}
+      concat table_for(@posts)
+      output_buffer.should have_table_with_tag("td", "The title of the post")
+    end
+  end
+end

data/spec/tabletastic/table_field_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe Tabletastic::TableField do
+  it "should guess its heading automatically" do
+    tf = TableField.new(:method)
+    tf.method_or_proc.should == :method
+    tf.heading.should == "Method"
+  end
+
+  it "should know its heading when provided" do
+    tf = TableField.new(:method, :heading => 'Foo', :klass => ::Post)
+    tf.heading.should == "Foo"
+  end
+
+  it "should know what to do with a record" do
+    tf = TableField.new(:downcase)
+    tf.cell_data("HELLO").should == "hello"
+  end
+
+  it "should use a human_attribute_name whenever possible" do
+    ::Post.stub!(:human_attribute_name).with('method').and_return("Blah blue")
+    tf = TableField.new(:method, :klass => ::Post)
+    tf.heading.should == "Blah blue"
+  end
+
+  it "should know what to do with a record (proc)" do
+    tf = TableField.new(:fake) do |record|
+      record.upcase
+    end
+    tf.cell_data("hello").should == "HELLO"
+  end
+end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: tabletastic
 version: !ruby/object:Gem::Version 
+  hash: -1876988178
   prerelease: true
   segments: 
   - 0
   - 2
   - 0
-  - pre2
-  version: 0.2.0.pre2
+  - pre3
+  version: 0.2.0.pre3
 platform: ruby
 authors: 
 - Joshua Davey
@@ -15,21 +16,40 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-18 00:00:00 -05:00
+date: 2010-06-28 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: rspec
+  name: activesupport
   prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: -1848230022
+        segments: 
+        - 3
+        - 0
+        - 0
+        - beta2
+        version: 3.0.0.beta2
+  type: :runtime
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: rspec
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
         segments: 
         - 0
         version: "0"
   type: :development
-  version_requirements: *id001
+  version_requirements: *id002
 description: A table builder for active record collections that produces semantically rich and accessible table markup
 email: josh@joshuadavey.com
 executables: []
@@ -49,6 +69,10 @@ files:
 - README.rdoc
 - CHANGELOG.rdoc
 - init.rb
+- spec/tabletastic/table_builder_spec.rb
+- spec/tabletastic/table_field_spec.rb
+- spec/tabletastic_spec.rb
+- spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/jgdavey/tabletastic
 licenses: []
@@ -59,16 +83,20 @@ rdoc_options: []
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
       segments: 
       - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 23
       segments: 
       - 1
       - 3
@@ -77,10 +105,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.6
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
 summary: A smarter table builder for Rails collections
 test_files: 
-- spec/spec_helper.rb
+- spec/tabletastic/table_builder_spec.rb
+- spec/tabletastic/table_field_spec.rb
 - spec/tabletastic_spec.rb
+- spec/spec_helper.rb