paginated_table 0.0.6 → 0.0.7

data/test/integration/paginated_table_integration_test.rb

@@ -42,6 +42,12 @@ describe "paginated_table integration" do
       end
     end
 
+    it "cycles the 'odd' and 'even' css classes on the data rows" do
+      (1..10).each do |row|
+        page.has_xpath?("#{tr_xpath(row)}[@class='#{row.odd? ? 'odd' : 'even'}']").must_equal true
+      end
+    end
+
     describe "with javascript" do
       before do
         Capybara.current_driver = Capybara.javascript_driver
@@ -60,13 +66,13 @@ describe "paginated_table integration" do
     describe "without javascript" do
       it "displays one page of results" do
         visit "/data"
-        pagination_info_text.must_equal "Displaying data controller/data 1 - 10 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 1 - 10 of 100 in total"
       end
 
       it "follows the link to the second page of results" do
         visit "/data"
         click_link "2"
-        pagination_info_text.must_equal "Displaying data controller/data 11 - 20 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 11 - 20 of 100 in total"
       end
     end
 
@@ -77,14 +83,14 @@ describe "paginated_table integration" do
 
       it "displays one page of results" do
         visit "/data"
-        pagination_info_text.must_equal "Displaying data controller/data 1 - 10 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 1 - 10 of 100 in total"
       end
 
       it "follows the link to the second page of results" do
         visit "/data"
         click_link "2"
         wait_for_ajax_request
-        pagination_info_text.must_equal "Displaying data controller/data 11 - 20 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 11 - 20 of 100 in total"
       end
 
       # Ensures the AJAX content is decorated with event handlers
@@ -92,10 +98,10 @@ describe "paginated_table integration" do
         visit "/data"
         click_link "4"
         wait_for_ajax_request
-        pagination_info_text.must_equal "Displaying data controller/data 31 - 40 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 31 - 40 of 100 in total"
         click_link "3"
         wait_for_ajax_request
-        pagination_info_text.must_equal "Displaying data controller/data 21 - 30 of 100 in total"
+        pagination_info_text.must_equal "Displaying Data 21 - 30 of 100 in total"
       end
     end
   end
@@ -195,7 +201,7 @@ describe "paginated_table integration" do
   describe "searching" do
     it "limits the results" do
       visit "/data?search=secondhalf"
-      pagination_info_text.must_equal "Displaying data controller/data 1 - 10 of 50 in total"
+      pagination_info_text.must_equal "Displaying Data 1 - 10 of 50 in total"
       (51..60).each_with_index do |id, index|
         page.has_xpath?("#{tr_xpath(index + 1)}/td[1][.='Name #{id}']").must_equal true
       end
@@ -204,7 +210,7 @@ describe "paginated_table integration" do
     it "following the page links preserves the search criteria" do
       visit "/data?search=secondhalf"
       click_link "2"
-      pagination_info_text.must_equal "Displaying data controller/data 11 - 20 of 50 in total"
+      pagination_info_text.must_equal "Displaying Data 11 - 20 of 50 in total"
       (61..70).each_with_index do |id, index|
         page.has_xpath?("#{tr_xpath(index + 1)}/td[1][.='Name #{id}']").must_equal true
       end
@@ -213,7 +219,7 @@ describe "paginated_table integration" do
     it "following the sort links preserves the search criteria" do
       visit "/data?search=secondhalf"
       click_link "Name"
-      pagination_info_text.must_equal "Displaying data controller/data 1 - 10 of 50 in total"
+      pagination_info_text.must_equal "Displaying Data 1 - 10 of 50 in total"
       [100, 51, 52, 53, 54, 55, 56, 57, 58, 59].each_with_index do |id, index|
         page.has_xpath?("#{tr_xpath(index + 1)}/td[1][.='Name #{id}']").must_equal true
       end

data/test/units/column_description_test.rb

@@ -0,0 +1,97 @@
+require 'test_helper'
+
+module PaginatedTable
+  describe ColumnDescription do
+    let(:options) { {} }
+    let(:row) { stub("row") }
+    let(:name) { 'foo' }
+    let(:description) { ColumnDescription.new(row, name, options) }
+
+    describe "#initialize" do
+      it "creates a new instance with a row, a name and an optional block" do
+        ColumnDescription.new(row, name) { true }
+      end
+
+      it "accepts an options hash" do
+        ColumnDescription.new(row, name, :baz => 'bat')
+      end
+    end
+
+    describe "#sortable?" do
+      it "returns true by default" do
+        description.sortable?.must_equal true
+      end
+
+      it "returns the :sortable option" do
+        options[:sortable] = sortable = stub("sortable")
+        description.sortable?.must_equal sortable
+      end
+    end
+
+    describe "#span" do
+      it "returns false by default" do
+        description.span.must_equal false
+      end
+
+      it "returns the :span option" do
+        options[:span] = span = stub("span")
+        description.span.must_equal span
+      end
+    end
+
+    describe "#html_attributes" do
+      it "returns an empty hash by default" do
+        description.html_attributes.must_equal({})
+      end
+
+      it "adds the css classes given by the :class option" do
+        options[:class] = %w(bar baz)
+        description.html_attributes.must_equal(:class => 'bar baz')
+      end
+
+      it "adds the css styles given by the :style option" do
+        options[:style] = 'font-face: bold'
+        description.html_attributes.must_equal(:style => 'font-face: bold')
+      end
+
+      it "sets the colspan when span is :all" do
+        span = stub("span")
+        row.stubs(:colspan).with(span).returns("5")
+        options[:span] = span
+        description.html_attributes.must_equal(:colspan => '5')
+      end
+    end
+
+    describe "#render_header" do
+      it "returns the titleized name" do
+        description.render_header.must_equal "Foo"
+      end
+
+      it "returns the :title option if given" do
+        options[:title] = 'bar'
+        description.render_header.must_equal "bar"
+      end
+    end
+
+    describe "#render_cell" do
+      let(:results) { stub("results") }
+
+      describe "on a column with no block" do
+        it "sends its name to the datum" do
+          datum = stub("datum", 'foo' => results)
+          description.render_cell(datum).must_equal results
+        end
+      end
+
+      describe "on a column with a block" do
+        it "calls its block with the datum" do
+          datum = stub("datum")
+          column = ColumnDescription.new(row, name) do |block_arg|
+            results if block_arg == datum
+          end
+          column.render_cell(datum).must_equal results
+        end
+      end
+    end
+  end
+end

data/test/units/config_test.rb

@@ -1,3 +1,5 @@
+require 'test_helper'
+
 module PaginatedTable
   describe "configuration" do
     let(:configuration) { PaginatedTable.configuration }

data/test/units/controller_helpers_test.rb

@@ -1,3 +1,5 @@
+require 'test_helper'
+
 module PaginatedTable
   describe ControllerHelpers do
     let(:params) { stub("params") }
@@ -32,6 +34,13 @@ module PaginatedTable
         controller.paginated_table(name, collection)
       end
 
+      it "renders the given partial without layout if request is xhr?" do
+        partial = stub("partial")
+        request.stubs(:xhr? => true)
+        controller.expects(:render).with(:partial => partial, :layout => false)
+        controller.paginated_table(name, collection, :partial => partial)
+      end
+
       it "does not render if request is not xhr?" do
         request.stubs(:xhr? => false)
         controller.expects(:render).never

data/test/units/data_page_test.rb

@@ -0,0 +1,36 @@
+require 'test_helper'
+
+module PaginatedTable
+  describe DataPage do
+    describe "#data" do
+      let(:page) {
+        Page.new(
+          :number => 2,
+          :rows => 5,
+          :sort_column => 'name',
+          :sort_direction => 'asc'
+        )
+      }
+      let(:collection) {
+        collection = (1..10).map { |i| "Name #{i}" }
+        def collection.order(clause)
+          raise unless clause == "name asc"
+          sort
+        end
+        collection
+      }
+
+      it "sorts the collection and pages to the given page number" do
+        DataPage.new(collection, page).data.must_equal(
+          ["Name 5", "Name 6", "Name 7", "Name 8", "Name 9"]
+        )
+      end
+
+      describe "#page" do
+        it "provides a reference to the given page" do
+          DataPage.new(collection, page).page.must_equal page
+        end
+      end
+    end
+  end
+end

data/test/units/link_renderer_test.rb

@@ -0,0 +1,44 @@
+require 'test_helper'
+
+module PaginatedTable
+  describe LinkRenderer do
+    let(:page) { Page.new(:number => 2, :rows => 5, :sort_column => 'to_s', :sort_direction => 'desc') }
+    let(:data) { (1..10).to_a }
+    let(:data_page) { data.paginate(:page => 2, :per_page => 5) }
+    let(:view) { stub("view") }
+    let(:renderer) do
+      renderer = LinkRenderer.new(page)
+      renderer.prepare(data, {}, view)
+      renderer
+    end
+    let(:text) { stub("text") }
+    let(:href) { stub("href") }
+    let(:link) { stub("link") }
+
+    describe "#sort_link" do
+      it "calls link_to on the view with the sort url and the :remote option" do
+        view.stubs("url_for").
+          with(:sort_direction => 'asc', :per_page => '5', :page => '1', :sort_column => 'to_s').
+          returns(href)
+        view.stubs("link_to").with(text, href, :remote => true).returns(link)
+        renderer.sort_link(text, 'to_s').must_equal link
+      end
+    end
+
+    describe "#tag" do
+      it "calls link_to on the view with the :remote option for :a tags" do
+        html_safe_text = stub("html_safe_text")
+        text = stub("text", :to_s => stub("string", :html_safe => html_safe_text))
+        view.expects(:link_to).
+          with(html_safe_text, href, { :class => 'highlight', :remote => true }).
+          returns(link)
+        renderer.tag(:a, text, :class => 'highlight', :href => href).must_equal link
+      end
+
+      it "delegates to its parent for all other tags" do
+        view.expects(:link_to).never
+        renderer.tag(:span, "foo")
+      end
+    end
+  end
+end

data/test/units/page_params_test.rb

@@ -0,0 +1,48 @@
+require 'test_helper'
+
+module PaginatedTable
+  describe PageParams do
+    describe ".create_page" do
+      it "returns a new page created from the request params" do
+        page = PageParams.create_page(
+          :page => '2',
+          :per_page => '5',
+          :sort_column => 'name',
+          :sort_direction => 'desc'
+        )
+        page.number.must_equal 2
+        page.rows.must_equal 5
+        page.sort_column.must_equal 'name'
+        page.sort_direction.must_equal 'desc'
+      end
+
+      it "returns a new page created from the request params and the defaults" do
+        page = PageParams.create_page(
+          { :page => '2', :per_page => '5' },
+          { :sort_column => 'name', :sort_direction => 'desc' }
+        )
+        page.number.must_equal 2
+        page.rows.must_equal 5
+        page.sort_column.must_equal 'name'
+        page.sort_direction.must_equal 'desc'
+      end
+    end
+
+    describe ".to_params" do
+      it "creates a params hash from the page" do
+        page = Page.new(
+          :number => 2,
+          :rows => 5,
+          :sort_column => 'name',
+          :sort_direction => 'desc'
+        )
+        PageParams.to_params(page).must_equal(
+          :page => '2',
+          :per_page => '5',
+          :sort_column => 'name',
+          :sort_direction => 'desc'
+        )
+      end
+    end
+  end
+end

data/test/units/page_test.rb

@@ -1,3 +1,5 @@
+require 'test_helper'
+
 module PaginatedTable
   describe Page do
     let(:page) { Page.new(:number => 2, :rows => 5, :sort_column => 'name', :sort_direction => 'desc') }
@@ -121,83 +123,4 @@ module PaginatedTable
       end
     end
   end
-
-  describe PageParams do
-    describe ".create_page" do
-      it "returns a new page created from the request params" do
-        page = PageParams.create_page(
-          :page => '2',
-          :per_page => '5',
-          :sort_column => 'name',
-          :sort_direction => 'desc'
-        )
-        page.number.must_equal 2
-        page.rows.must_equal 5
-        page.sort_column.must_equal 'name'
-        page.sort_direction.must_equal 'desc'
-      end
-
-      it "returns a new page created from the request params and the defaults" do
-        page = PageParams.create_page(
-          { :page => '2', :per_page => '5' },
-          { :sort_column => 'name', :sort_direction => 'desc' }
-        )
-        page.number.must_equal 2
-        page.rows.must_equal 5
-        page.sort_column.must_equal 'name'
-        page.sort_direction.must_equal 'desc'
-      end
-    end
-
-    describe ".to_params" do
-      it "creates a params hash from the page" do
-        page = Page.new(
-          :number => 2,
-          :rows => 5,
-          :sort_column => 'name',
-          :sort_direction => 'desc'
-        )
-        PageParams.to_params(page).must_equal(
-          :page => '2',
-          :per_page => '5',
-          :sort_column => 'name',
-          :sort_direction => 'desc'
-        )
-      end
-    end
-  end
-
-  describe DataPage do
-    describe "#data" do
-      let(:page) {
-        Page.new(
-          :number => 2,
-          :rows => 5,
-          :sort_column => 'name',
-          :sort_direction => 'asc'
-        )
-      }
-      let(:collection) {
-        collection = (1..10).map { |i| "Name #{i}" }
-        def collection.order(clause)
-          raise unless clause == "name asc"
-          sort
-        end
-        collection
-      }
-
-      it "sorts the collection and pages to the given page number" do
-        DataPage.new(collection, page).data.must_equal(
-          ["Name 5", "Name 6", "Name 7", "Name 8", "Name 9"]
-        )
-      end
-
-      describe "#page" do
-        it "provides a reference to the given page" do
-          DataPage.new(collection, page).page.must_equal page
-        end
-      end
-    end
-  end
-
 end

data/test/units/row_description_test.rb

@@ -0,0 +1,71 @@
+require 'test_helper'
+
+module PaginatedTable
+  describe RowDescription do
+    let(:table) { TableDescription.new }
+    let(:description_proc) { lambda { |row| nil } }
+    let(:options) { {} }
+    let(:description) {
+      RowDescription.new(table, options, description_proc)
+    }
+
+    describe "#initialize" do
+      it "creates a new instance with empty columns" do
+        description.columns.must_equal []
+      end
+
+      it "calls the given block with itself" do
+        fake_proc = stub("proc")
+        fake_proc.expects(:call)
+        RowDescription.new(table, options, fake_proc)
+      end
+    end
+
+    describe "#title" do
+      it "returns the title option" do
+        options[:title] = title = stub("title")
+        description.title.must_equal title
+      end
+    end
+
+    describe "#cycle" do
+      it "returns the cycle option" do
+        options[:cycle] = cycle = stub("cycle")
+        description.cycle.must_equal cycle
+      end
+    end
+
+    describe "#hidden" do
+      it "returns the hidden option" do
+        options[:hidden] = hidden = stub("hidden")
+        description.hidden.must_equal hidden
+      end
+    end
+
+    describe "#data_type" do
+      it "returns the data_type option" do
+        options[:data_type] = data_type = stub("data_type")
+        description.data_type.must_equal data_type
+      end
+    end
+
+    describe "#colspan" do
+      it "delegates to the table description" do
+        colspan = stub("colspan")
+        arg = stub("arg")
+        table.stubs(:colspan).with(arg).returns(colspan)
+        description.colspan(arg).must_equal colspan
+      end
+    end
+
+    describe "#column" do
+      it "constructs a new ColumnDescription and appends it to the columns array" do
+        column = stub("column")
+        name = stub("name")
+        ColumnDescription.stubs(:new).with(description, name).returns(column)
+        description.column(name)
+        description.columns.must_equal [column]
+      end
+    end
+  end
+end