colszowka-cucumber_table_formatter 0.2.0 → 0.2.1

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
-:patch: 0
+:patch: 1
 :major: 0
 :minor: 2

data/cucumber_table_formatter.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{cucumber_table_formatter}
-  s.version = "0.2.0"
+  s.version = "0.2.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Christoph Olszowka"]

data/lib/cucumber_table_formatter.rb

@@ -27,8 +27,13 @@ class CucumberTableFormatter
     end
   end
   
-  def initialize(table_string)
-    self.lines = table_string.reject { |l| l.strip.chomp.length == 0 }.map(&:chomp)
+  def initialize(table)
+    # Convert string by splitting by newlines if string given. If array, this can be skipped
+    table = table.split("\n") if table.kind_of?(String)
+    # Remove all empty lines and put all others into lines instance variable, cleaned up from newline chars
+    # TODO: Leave lines without a pipe character as they are so whole feature-files can be processed
+    self.lines = table.reject { |l| l.strip.chomp.length == 0 }.map(&:chomp)
+    # Get the whitespace 
     @whitespace = self.lines.first.split("|")[0].length
     read_columns!
   end

data/test/cucumber_table_formatter_test.rb

@@ -1,7 +1,52 @@
 require 'test_helper'
 
 class CucumberTableFormatterTest < Test::Unit::TestCase
-  should "probably rename this file and start testing for real" do
-    flunk "hey buddy, you should probably rename this file and start testing for real"
+  context "A simple table" do
+    setup { @table = "| a | b | c |" }
+    
+    should "be returned as-is" do
+      assert_equal @table, format(@table)
+    end
+  end
+  
+  context "A simple table with missing whitespace" do
+    setup { @table = "|a|b | c|" }
+    
+    should "be returned with whitespace" do
+      assert_equal "| a | b | c |", format(@table)
+    end
+  end
+  
+  context "A more complex table without leading whitespace" do
+    setup do
+      @table = "| a | b | c |\n|def|ghi|jkl|\n|foobar|baz|blur|"
+    end
+    
+    should "be returned in proper format" do
+      expectation = "|      a |   b |    c |\n|    def | ghi |  jkl |\n| foobar | baz | blur |"
+      assert_equal expectation, format(@table)
+    end
+  end
+  
+  context "A more complex table with leading whitespace" do
+    setup do
+      @table = "    | a | b | c |\n    |def|ghi|jkl|\n    |foobar|baz|blur|"
+    end
+    
+    should "be returned in proper format" do
+      expectation = "    |      a |   b |    c |\n    |    def | ghi |  jkl |\n    | foobar | baz | blur |"
+      assert_equal expectation, format(@table)
+    end
+  end
+  
+  context "A more complex table with unicode-characters" do
+    setup do
+      @table = "| Ö | b | c |\n|def|ghi|jkl|\n|foobar|baz|blur|"
+    end
+    
+    should "be returned in proper format" do
+      expectation = "|      Ö |   b |    c |\n|    def | ghi |  jkl |\n| foobar | baz | blur |"
+      assert_equal expectation, format(@table)
+    end
   end
 end

data/test/test_helper.rb

@@ -7,4 +7,8 @@ $LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'cucumber_table_formatter'
 
 class Test::Unit::TestCase
+  # Shorthand for CucumberTableFormatter.new(table).formatted
+  def format(table)
+    CucumberTableFormatter.new(table).formatted
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: colszowka-cucumber_table_formatter
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors: 
 - Christoph Olszowka