text-table 1.2.2 → 1.2.3

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 5637e2fb2a77c1c4c6f019b4120e6d1dfc547336
+  data.tar.gz: da83a6185a3c4acac5e857594dd187fe43544b76
+SHA512:
+  metadata.gz: da85599487daf6768f1a466e20950418cc65905782a22972a27e0b59c70b4d63df5ab41c945a59e378f065b7c224026a9d712e4b18b174523c0276d0bb67307a
+  data.tar.gz: 359c082e3f514a9b706ff5a82a1df813ab82c89ee0802c1883f8630eb8a87eff4e70aa400bfc79936c9bf36ae3b0d8c81e81b5d2585930c7dfbdb769c46bae50

data/.document

@@ -1,5 +1,4 @@
-README.rdoc
+CHANGELOG.rdoc
 lib/**/*.rb
-bin/*
-features/**/*.feature
+README.rdoc
 LICENSE

data/.gitignore

@@ -1,24 +1,6 @@
-## MAC OS
-.DS_Store
-
-## TEXTMATE
-*.tmproj
-tmtags
-
-## EMACS
-*~
-\#*
-.\#*
-
-## VIM
-*.swp
-
-## NETBEANS
-nbproject
-
-## PROJECT::GENERAL
-coverage
-rdoc
-pkg
-
-## PROJECT::SPECIFIC
+bin/
+.bundle/
+doc/
+Gemfile.lock
+tmp/
+vendor/bundle/

data/CHANGELOG.rdoc

@@ -1,6 +1,10 @@
+=== 1.2.3 (2013-04-15)
+
+* Fixed performance issue with large number of rows (thanks Kaoru Maeda)
+
 === 1.2.2 (2010-03-29)
 
-* Fix Ruby 1.9 warnings on shadowed outer local variables (thanks Claudio Bustos)
+* Fixed Ruby 1.9 warnings on shadowed outer local variables (thanks Claudio Bustos)
 
 === 1.2.1 (2010-01-05)
 
@@ -18,4 +22,4 @@
 
 === 1.0.1 (2009-12-21)
 
-* Initial stable release.
+* Initial stable release.

data/Gemfile

@@ -0,0 +1,3 @@
+source 'https://rubygems.org'
+
+gemspec

data/README.rdoc

@@ -17,13 +17,33 @@ Text::Table is compatible with Ruby 1.8.6, 1.8.7 and 1.9.1 and includes a compre
     gem install text-table
 
 
+== Examples
+
+    require 'text-table'
+
+    table = Text::Table.new
+    table.head = ['A', 'B']
+    table.rows = [['a1', 'b1']]
+    table.rows << ['a2', 'b2']
+
+    table.to_s
+
+    #    +----+----+
+    #    | A  | B  |
+    #    +----+----+
+    #    | a1 | b1 |
+    #    | a2 | b2 |
+    #    +----+----+
+
+You can also pass an options hash when instantiating a Text::Table:
+
+    table = Text::Table.new(:head => ['A', 'B'], :rows => [['a1', 'b1'], ['a2', 'b2']])
+
+
 == Calling the <tt>to_table</tt> Method
 
 Just call the <tt>to_table</tt> method (or <tt>to_text_table</tt> if the former is already defined) on Arrays (and other Enumerables).
 
-    require 'rubygems'
-    require 'text-table'
-
     array = [
       ['Student', 'Mid-Terms', 'Finals'],
       ['Sam', 94, 93],
@@ -66,30 +86,6 @@ You could also specify that the last row is the table footer.
     #    +---------+-----------+--------+
 
 
-== Creating a New Text::Table Object
-
-You could create a Text::Table object by passing an options hash:
-
-    table = Text::Table.new(:head => ['A', 'B'], :rows => [['a1', 'b1'], ['a2', 'b2']])
-
-Or by passing a block:
-
-    table = Text::Table.new do |t|
-      t.head = ['A', 'B']
-      t.rows = [['a1', 'b1']]
-      t.rows << ['a2', 'b2']
-    end
-
-    table.to_s
-
-    #    +----+----+
-    #    | A  | B  |
-    #    +----+----+
-    #    | a1 | b1 |
-    #    | a2 | b2 |
-    #    +----+----+
-
-
 == Aligning Cells and Spanning Columns
 
 Alignment and column span can be specified by passing a cell as a Hash object.
@@ -98,13 +94,12 @@ The acceptable aligments are <tt>:left</tt>, <tt>:center</tt> and <tt>:right</tt
 
 Cells and footers are aligned to the left by default, while headers are centered by default.
 
-    table = Text::Table.new do |t|
-      t.head = ['Heading A', 'Heading B']
-      t.rows << ['a1', 'b1']
-      t.rows << ['a2', {:value => 'b2', :align => :right}]
-      t.rows << ['a3', 'b3']
-      t.rows << [{:value => 'a4', :colspan => 2, :align => :center}]
-    end
+    table = Text::Table.new
+    table.head = ['Heading A', 'Heading B']
+    table.rows << ['a1', 'b1']
+    table.rows << ['a2', {:value => 'b2', :align => :right}]
+    table.rows << ['a3', 'b3']
+    table.rows << [{:value => 'a4', :colspan => 2, :align => :center}]
 
     puts table
 
@@ -187,9 +182,12 @@ Thanks to the authors and contributors of these projects.
 
 == Contributors
 
-* Claudio Bustos (clbustos[link:http://github.com/clbustos])
+* Claudio Bustos (clbustos[http://github.com/clbustos])
   * Fix Ruby 1.9 warnings on shadowed outer local variables
 
+* Kaoru Maeda (mad-p[https://github.com/mad-p])
+  * Fixed performance issue with large number of rows
+
 
 == Copyright
 

data/lib/text-table/table.rb

@@ -151,6 +151,7 @@ module Text #:nodoc:
     end
 
     def column_widths #:nodoc:
+      @column_widths ||= \
       all_text_table_rows.reject {|row| row.cells == :separator}.map do |row|
         row.cells.map {|cell| [(cell.value.length/cell.colspan.to_f).ceil] * cell.colspan}.flatten
       end.transpose.map(&:max)

data/lib/text-table/version.rb

@@ -0,0 +1,5 @@
+module Text
+  class Table
+    VERSION = '1.2.3'
+  end
+end

data/spec/cell_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe Text::Table::Cell do
   before(:each) do

data/spec/enumerable_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe Enumerable do
   describe Array do
@@ -11,45 +11,45 @@ describe Enumerable do
     end
 
     it "to Text::Table" do
-      @arr.to_text_table.to_s.should == <<-EOS.deindent
+      @arr.to_text_table.to_s.should == deindent(%q{
         +----+-----+------+
         | 11 | 2   | 3333 |
         | 44 | 56  | 6    |
         | 7  | 888 | 99   |
         +----+-----+------+
-      EOS
+      })
     end
 
     it "to Text::Table using flat array" do
-      [11, 44, 7].to_text_table.to_s.should == <<-EOS.deindent
+      [11, 44, 7].to_text_table.to_s.should == deindent(%q{
         +----+
         | 11 |
         | 44 |
         | 7  |
         +----+
-      EOS
+      })
     end
 
     it "to Text::Table, setting first row as head" do
-      @arr.to_text_table(:first_row_is_head => true).to_s.should == <<-EOS.deindent
+      @arr.to_text_table(:first_row_is_head => true).to_s.should == deindent(%q{
         +----+-----+------+
         | 11 |  2  | 3333 |
         +----+-----+------+
         | 44 | 56  | 6    |
         | 7  | 888 | 99   |
         +----+-----+------+
-      EOS
+      })
     end
 
     it "to Text::Table, setting last row as foot" do
-      @arr.to_text_table(:last_row_is_foot => true).to_s.should == <<-EOS.deindent
+      @arr.to_text_table(:last_row_is_foot => true).to_s.should == deindent(%q{
         +----+-----+------+
         | 11 | 2   | 3333 |
         | 44 | 56  | 6    |
         +----+-----+------+
         | 7  | 888 | 99   |
         +----+-----+------+
-      EOS
+      })
     end
   end
 
@@ -61,32 +61,32 @@ describe Enumerable do
       }
     end
     it "to Text::Table" do
-      @hsh.to_text_table.to_s.should == <<-EOS.deindent
+      @hsh.to_text_table.to_s.should == deindent(%q{
         +---+----+
         | k | vv |
         | k | vv |
         +---+----+
-      EOS
+      })
     end
 
     it "to Text::Table, setting first row as head" do
-      @hsh.to_text_table(:first_row_is_head => true).to_s.should == <<-EOS.deindent
+      @hsh.to_text_table(:first_row_is_head => true).to_s.should == deindent(%q{
         +---+----+
         | k | vv |
         +---+----+
         | k | vv |
         +---+----+
-      EOS
+      })
     end
 
     it "to Text::Table, setting last row as foot" do
-      @hsh.to_text_table(:last_row_is_foot => true).to_s.should == <<-EOS.deindent
+      @hsh.to_text_table(:last_row_is_foot => true).to_s.should == deindent(%q{
         +---+----+
         | k | vv |
         +---+----+
         | k | vv |
         +---+----+
-      EOS
+      })
     end
   end
 end

data/spec/integration/alignment_spec.rb

@@ -0,0 +1,183 @@
+require 'integration_helper'
+
+describe Text::Table do
+  let(:table) { Text::Table.new :rows => rows, :head => head, :foot => foot }
+  let(:rows) { @rows }
+  let(:head) { @head }
+  let(:foot) { @foot }
+  subject { table.to_s }
+
+  describe 'header alignment' do
+    let(:head) { @head.map { |heading| { :value => heading, :align => align } } }
+    let(:foot) { nil }
+
+    context 'when left' do
+      let(:align) { :left }
+
+      it { should == deindent(%q{
+        +-----+------+------+------+
+        | a   | bb   | ccc  | dddd |
+        +-----+------+------+------+
+        | aa  | bbb  | cccc | d    |
+        | aaa | bbbb | c    | dd   |
+        +-----+------+------+------+
+      }) }
+    end
+
+    context 'when center' do
+      let(:align) { :center }
+
+      it { should == deindent(%q{
+        +-----+------+------+------+
+        |  a  |  bb  | ccc  | dddd |
+        +-----+------+------+------+
+        | aa  | bbb  | cccc | d    |
+        | aaa | bbbb | c    | dd   |
+        +-----+------+------+------+
+      }) }
+    end
+
+    context 'when right' do
+      let(:align) { :right }
+
+      it { should == deindent(%q{
+        +-----+------+------+------+
+        |   a |   bb |  ccc | dddd |
+        +-----+------+------+------+
+        | aa  | bbb  | cccc | d    |
+        | aaa | bbbb | c    | dd   |
+        +-----+------+------+------+
+      }) }
+    end
+  end
+
+  describe 'cell alignment' do
+    let(:rows) { @rows.map { |row| row.map { |cell| { :value => cell, :align => align } } } }
+
+    context 'when left' do
+      let(:align) { :left }
+
+      it { should == deindent(%q{
+        +------+------+------+------+
+        |  a   |  bb  | ccc  | dddd |
+        +------+------+------+------+
+        | aa   | bbb  | cccc | d    |
+        | aaa  | bbbb | c    | dd   |
+        +------+------+------+------+
+        | aaaa | b    | cc   | ddd  |
+        +------+------+------+------+
+      }) }
+    end
+
+    context 'when center' do
+      let(:align) { :center }
+
+      it { should == deindent(%q{
+        +------+------+------+------+
+        |  a   |  bb  | ccc  | dddd |
+        +------+------+------+------+
+        |  aa  | bbb  | cccc |  d   |
+        | aaa  | bbbb |  c   |  dd  |
+        +------+------+------+------+
+        | aaaa | b    | cc   | ddd  |
+        +------+------+------+------+
+      }) }
+    end
+
+    context 'when right' do
+      let(:align) { :right }
+
+      it { should == deindent(%q{
+        +------+------+------+------+
+        |  a   |  bb  | ccc  | dddd |
+        +------+------+------+------+
+        |   aa |  bbb | cccc |    d |
+        |  aaa | bbbb |    c |   dd |
+        +------+------+------+------+
+        | aaaa | b    | cc   | ddd  |
+        +------+------+------+------+
+      }) }
+    end
+  end
+
+  describe 'footer alignment' do
+    let(:head) { nil }
+    let(:foot) { @foot.map { |footing| { :value => footing, :align => align } } }
+
+    context 'when left' do
+      let(:align) { :left }
+
+      it { should == deindent(%q{
+        +------+------+------+-----+
+        | aa   | bbb  | cccc | d   |
+        | aaa  | bbbb | c    | dd  |
+        +------+------+------+-----+
+        | aaaa | b    | cc   | ddd |
+        +------+------+------+-----+
+      }) }
+    end
+
+    context 'when center' do
+      let(:align) { :center }
+
+      it { should == deindent(%q{
+        +------+------+------+-----+
+        | aa   | bbb  | cccc | d   |
+        | aaa  | bbbb | c    | dd  |
+        +------+------+------+-----+
+        | aaaa |  b   |  cc  | ddd |
+        +------+------+------+-----+
+      }) }
+    end
+
+    context 'when right' do
+      let(:align) { :right }
+
+      it { should == deindent(%q{
+        +------+------+------+-----+
+        | aa   | bbb  | cccc | d   |
+        | aaa  | bbbb | c    | dd  |
+        +------+------+------+-----+
+        | aaaa |    b |   cc | ddd |
+        +------+------+------+-----+
+      }) }
+    end
+  end
+
+  describe 'table-wide alignment' do
+    context 'of columns' do
+      before do
+        table.rows[1][2] = {:value => 'c', :align => :center}
+        table.rows << [{:value => 'x', :colspan => 2}, 'c', 'd']
+        table.rows << ['a', 'b', {:value => 'x', :colspan => 2}]
+        table.align_column 2, :right
+        table.align_column 3, :right
+      end
+
+      it { should == deindent(%q{
+        +------+------+------+------+
+        |  a   |  bb  | ccc  | dddd |
+        +------+------+------+------+
+        | aa   |  bbb | cccc | d    |
+        | aaa  | bbbb |  c   | dd   |
+        | x           |    c | d    |
+        | a    |    b | x           |
+        +------+------+------+------+
+        | aaaa |    b |   cc | ddd  |
+        +------+------+------+------+
+      }) }
+    end
+
+    context 'of rows' do
+      pending
+    end
+
+    context 'of headers' do
+      pending
+    end
+
+    context 'of footers' do
+      pending
+    end
+  end
+end