spreadsheet 1.3.3 → 1.3.4

data/test/excel/writer/workbook.rb

@@ -1,10 +1,10 @@
 #!/usr/bin/env ruby
 # Excel::Writer::TestWorkbook -- Spreadsheet -- 20.07.2011 -- vanderhoorn@gmail.com
 
-$: << File.expand_path('../../../lib', File.dirname(__FILE__))
+$: << File.expand_path("../../../lib", File.dirname(__FILE__))
 
-require 'test/unit'
-require 'spreadsheet'
+require "test/unit"
+require "spreadsheet"
 
 module Spreadsheet
   module Excel
@@ -16,44 +16,47 @@ module Spreadsheet
           assert_equal @book.worksheets.size, 0
           @workbook_writer = Excel::Writer::Workbook.new @book
         end
+
         def test_sanitize_worksheets
           assert_nothing_raised { @workbook_writer.sanitize_worksheets @book.worksheets }
         end
+
         def test_collect_formats
           assert_equal 17, @workbook_writer.collect_formats(@book).length # Expected for vanilla
           sheet = @book.create_worksheet
-          rowi=-1
+          rowi = -1
 
           f1 = Spreadsheet::Format.new
-          sheet.row(rowi+=1).default_format = f1
+          sheet.row(rowi += 1).default_format = f1
           assert_equal 18, @workbook_writer.collect_formats(@book).length
 
           f2 = Spreadsheet::Format.new
-          sheet.row(rowi+=1).default_format = f2
+          sheet.row(rowi += 1).default_format = f2
           assert_equal 19, @workbook_writer.collect_formats(@book).length
 
-          sheet.row(rowi+=1).default_format = f2
+          sheet.row(rowi + 1).default_format = f2
           assert_equal 19, @workbook_writer.collect_formats(@book).length # Ignores duplicates
         end
+
         def test_xf_index
           sheet = @book.create_worksheet
-          rowi=-1
+          rowi = -1
 
           f1 = Spreadsheet::Format.new
-          sheet.row(rowi+=1).default_format = f1
+          sheet.row(rowi += 1).default_format = f1
           @workbook_writer.collect_formats(@book)
           assert_equal 17, @workbook_writer.xf_index(@book, f1)
 
           f2 = Spreadsheet::Format.new
-          sheet.row(rowi+=1).default_format = f2
+          sheet.row(rowi + 1).default_format = f2
           @workbook_writer.collect_formats(@book)
           assert_equal 18, @workbook_writer.xf_index(@book, f2)
-
         end
+
         def test_write_fonts
           num_written = 0
           sheet = @book.create_worksheet
-          rowi=-1
+          rowi = -1
           # Stub inner #write_font as a counter:
           (class << @workbook_writer; self; end).send(:define_method, :write_font) do |*args|
             num_written += 1
@@ -66,7 +69,7 @@ module Spreadsheet
           assert_equal 1, num_written # Default format's font
 
           f1 = Spreadsheet::Format.new
-          sheet.row(rowi+=1).default_format = f1
+          sheet.row(rowi += 1).default_format = f1
           num_written = 0
           @workbook_writer.collect_formats(@book)
           @workbook_writer.write_fonts(@book, io)
@@ -74,7 +77,7 @@ module Spreadsheet
 
           f2 = Spreadsheet::Format.new
           f2.font = Spreadsheet::Font.new("Foo")
-          sheet.row(rowi+=1).default_format = f2
+          sheet.row(rowi += 1).default_format = f2
           num_written = 0
           @workbook_writer.collect_formats(@book)
           @workbook_writer.write_fonts(@book, io)
@@ -82,12 +85,11 @@ module Spreadsheet
 
           f3 = Spreadsheet::Format.new
           f3.font = f2.font # Re-use previous font
-          sheet.row(rowi+=1).default_format = f3
+          sheet.row(rowi + 1).default_format = f3
           num_written = 0
           @workbook_writer.collect_formats(@book)
           @workbook_writer.write_fonts(@book, io)
           assert_equal 2, num_written # 2 distinct fonts total still
-
         end
       end
     end

data/test/excel/writer/worksheet.rb

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 # Excel::Writer::TestWorksheet -- Spreadheet -- 21.11.2007 -- hwyss@ywesee.com
 
-require 'test/unit'
-require 'spreadsheet/excel/writer/worksheet'
+require "test/unit"
+require "spreadsheet/excel/writer/worksheet"
 
 module Spreadsheet
   module Excel
@@ -23,7 +23,11 @@ module Spreadsheet
           attr_accessor :idx, :first_used, :first_unused, :height, :outline_level
 
           def initialize
-            @idx, @first_used, @first_unused, @height, @outline_level = 0,0,0,0,1
+            @idx, @first_used, @first_unused, @height, @outline_level = 0, 0, 0, 0, 1
+          end
+
+          def respond_to_missing? *args
+            super
           end
 
           def method_missing name, *args
@@ -38,7 +42,7 @@ module Spreadsheet
 
           sheet.write_row row
 
-          assert_equal '', sheet.data
+          assert_equal "", sheet.data
         end
 
         def test_write_row_should_write_if_any_column_is_used
@@ -61,20 +65,18 @@ module Spreadsheet
             writer.strings
           )
 
-          sheet.row(rowi+=1).concat(["Hello", "World"])
+          sheet.row(rowi += 1).concat(["Hello", "World"])
           assert_equal(
             {"Hello" => 1, "World" => 1},
             writer.strings
           )
 
-          sheet.row(rowi+=1).concat(["Goodbye", "Cruel", "World", 2012])
+          sheet.row(rowi + 1).concat(["Goodbye", "Cruel", "World", 2012])
           assert_equal(
             {"Hello" => 1, "Goodbye" => 1, "Cruel" => 1, "World" => 2},
             writer.strings
           )
-
         end
-
       end
     end
   end

data/test/font.rb

@@ -1,16 +1,17 @@
 #!/usr/bin/env ruby
 # TestFont -- Spreadsheet -- 09.10.2008 -- hwyss@ywesee.com
 
-$: << File.expand_path('../lib', File.dirname(__FILE__))
+$: << File.expand_path("../lib", File.dirname(__FILE__))
 
-require 'test/unit'
-require 'spreadsheet'
+require "test/unit"
+require "spreadsheet"
 
 module Spreadsheet
   class TestFont < Test::Unit::TestCase
     def setup
-      @font = Font.new 'Arial'
+      @font = Font.new "Arial"
     end
+
     def test_italic
       assert_equal false, @font.italic
       @font.italic!
@@ -20,33 +21,37 @@ module Spreadsheet
       @font.italic = 1
       assert_equal true, @font.italic
     end
+
     def test_encoding
       assert_equal :default, @font.encoding
       @font.encoding = :apple_roman
       assert_equal :apple_roman, @font.encoding
-      @font.encoding = 'Chinese Simplified'
+      @font.encoding = "Chinese Simplified"
       assert_equal :chinese_simplified, @font.encoding
-      assert_raises ArgumentError do @font.size = 'ascii' end
+      assert_raises ArgumentError do @font.size = "ascii" end # standard:disable BlockSingleLineBraces
       assert_equal :chinese_simplified, @font.encoding
       @font.encoding = nil
       assert_equal :default, @font.encoding
     end
+
     def test_family
       assert_equal :none, @font.family
       @font.family = :roman
       assert_equal :roman, @font.family
-      @font.family = 'Swiss'
+      @font.family = "Swiss"
       assert_equal :swiss, @font.family
-      assert_raises ArgumentError do @font.size = :greek end
+      assert_raises ArgumentError do @font.size = :greek end # standard:disable BlockSingleLineBraces
       assert_equal :swiss, @font.family
       @font.family = nil
       assert_equal :none, @font.family
     end
+
     def test_name
-      assert_equal 'Arial', @font.name
-      @font.name = 'Helvetica'
-      assert_equal 'Helvetica', @font.name
+      assert_equal "Arial", @font.name
+      @font.name = "Helvetica"
+      assert_equal "Helvetica", @font.name
     end
+
     def test_outline
       assert_equal false, @font.outline
       @font.outline!
@@ -56,17 +61,19 @@ module Spreadsheet
       @font.outline = 1
       assert_equal true, @font.outline
     end
+
     def test_escapement
       assert_equal :normal, @font.escapement
       @font.escapement = :superscript
       assert_equal :superscript, @font.escapement
-      @font.escapement = 'sub'
+      @font.escapement = "sub"
       assert_equal :subscript, @font.escapement
-      assert_raises ArgumentError do @font.size = "upwards" end
+      assert_raises ArgumentError do @font.size = "upwards" end # standard:disable BlockSingleLineBraces
       assert_equal :subscript, @font.escapement
       @font.escapement = nil
       assert_equal :normal, @font.escapement
     end
+
     def test_shadow
       assert_equal false, @font.shadow
       @font.shadow!
@@ -76,14 +83,16 @@ module Spreadsheet
       @font.shadow = 1
       assert_equal true, @font.shadow
     end
+
     def test_size
       assert_equal 10, @font.size
       @font.size = 12
       assert_equal 12, @font.size
       @font.size = 11.2
       assert_equal 11.2, @font.size
-      assert_raises ArgumentError do @font.size = "123" end
+      assert_raises ArgumentError do @font.size = "123" end # standard:disable BlockSingleLineBraces
     end
+
     def test_strikeout
       assert_equal false, @font.strikeout
       @font.strikeout!
@@ -93,70 +102,73 @@ module Spreadsheet
       @font.strikeout = 1
       assert_equal true, @font.strikeout
     end
+
     def test_underline
       assert_equal :none, @font.underline
       @font.underline = :single
       assert_equal :single, @font.underline
-      @font.underline = 'double accounting'
+      @font.underline = "double accounting"
       assert_equal :double_accounting, @font.underline
-      assert_raises ArgumentError do @font.size = :triple end
+      assert_raises ArgumentError do @font.size = :triple end # standard:disable BlockSingleLineBraces
       assert_equal :double_accounting, @font.underline
       @font.underline = nil
       assert_equal :none, @font.underline
       @font.underline = true
       assert_equal :single, @font.underline
     end
+
     def test_weight
       assert_equal :normal, @font.weight
       @font.weight = :bold
       assert_equal :bold, @font.weight
       @font.weight = 100
       assert_equal 100, @font.weight
-      assert_raises ArgumentError do @font.weight = Object.new end
+      assert_raises ArgumentError do @font.weight = Object.new end # standard:disable BlockSingleLineBraces
       assert_equal 100, @font.weight
-      @font.weight = 'bold'
+      @font.weight = "bold"
       assert_equal :bold, @font.weight
       @font.weight = nil
       assert_equal :normal, @font.weight
     end
+
     def test_key
-      expected = 'Arial_10_normal_normal_none_text_none_default'
+      expected = "Arial_10_normal_normal_none_text_none_default"
       assert_equal expected, @font.key
-      @font.name = 'Helvetica'
-      expected = 'Helvetica_10_normal_normal_none_text_none_default'
+      @font.name = "Helvetica"
+      expected = "Helvetica_10_normal_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.size = 12
-      expected = 'Helvetica_12_normal_normal_none_text_none_default'
+      expected = "Helvetica_12_normal_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.weight = :bold
-      expected = 'Helvetica_12_bold_normal_none_text_none_default'
+      expected = "Helvetica_12_bold_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.italic!
-      expected = 'Helvetica_12_bold_italic_normal_none_text_none_default'
+      expected = "Helvetica_12_bold_italic_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.strikeout!
-      expected = 'Helvetica_12_bold_italic_strikeout_normal_none_text_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.outline!
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_normal_none_text_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.shadow!
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_normal_none_text_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_normal_none_text_none_default"
       assert_equal expected, @font.key
       @font.escapement = :super
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_none_text_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_none_text_none_default"
       assert_equal expected, @font.key
       @font.underline = :double
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_text_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_text_none_default"
       assert_equal expected, @font.key
       @font.color = :blue
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_none_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_none_default"
       assert_equal expected, @font.key
       @font.family = :swiss
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_swiss_default'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_swiss_default"
       assert_equal expected, @font.key
       @font.encoding = :iso_latin1
-      expected = 'Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_swiss_iso_latin1'
+      expected = "Helvetica_12_bold_italic_strikeout_outline_shadow_superscript_double_blue_swiss_iso_latin1"
       assert_equal expected, @font.key
     end
   end

data/test/format.rb

@@ -1,16 +1,17 @@
 #!/usr/bin/env ruby
 # TestFormat -- Spreadsheet -- 06.11.2012 -- mina.git@naguib.ca
 
-$: << File.expand_path('../lib', File.dirname(__FILE__))
+$: << File.expand_path("../lib", File.dirname(__FILE__))
 
-require 'test/unit'
-require 'spreadsheet'
+require "test/unit"
+require "spreadsheet"
 
 module Spreadsheet
   class TestFormat < Test::Unit::TestCase
     def setup
       @format = Format.new
     end
+
     def test_date?
       assert_equal false, @format.date?
       @format.number_format = "hms"
@@ -28,6 +29,7 @@ module Spreadsheet
       @format.number_format = "[$-C0A]dd\\-mmm\\-yy"
       assert_equal true, @format.date?
     end
+
     def test_date_or_time?
       assert_equal false, @format.date_or_time?
       @format.number_format = "hms"
@@ -43,6 +45,7 @@ module Spreadsheet
       @format.number_format = "0.00;[RED]\\-0.00)"
       assert_equal false, @format.date_or_time?
     end
+
     def test_datetime?
       assert_equal false, @format.datetime?
       @format.number_format = "H"
@@ -58,6 +61,7 @@ module Spreadsheet
       @format.number_format = "0.00;[RED]\\-0.00)"
       assert_equal false, @format.datetime?
     end
+
     def test_time?
       assert_equal false, @format.time?
       @format.number_format = "YMD"
@@ -77,35 +81,36 @@ module Spreadsheet
       @format.number_format = "0.00;[RED]\\-0.00)"
       assert_equal false, @format.time?
     end
-		def test_borders?
-			assert_equal [:none, :none, :none, :none], @format.border
-			@format.border = :thick
-			assert_equal [:thick, :thick, :thick, :thick], @format.border
-			@format.left = :hair
-			assert_equal [:thick, :thick, :thick, :hair], @format.border
-			@format.right = :hair
-			assert_equal [:thick, :thick, :hair, :hair], @format.border
-			@format.top = :hair
-			assert_equal [:thick, :hair, :hair, :hair], @format.border
-			@format.bottom = :hair
-			assert_equal [:hair, :hair, :hair, :hair], @format.border
-			assert_raises(ArgumentError) do			
-				@format.bottom = :bogus
-			end
-			assert_equal [:black, :black, :black, :black], @format.border_color
-			@format.border_color = :green
-			assert_equal [:green, :green, :green, :green], @format.border_color
-			@format.left_color = :red
-			assert_equal [:green, :green, :green, :red], @format.border_color
-			@format.right_color = :red
-			assert_equal [:green, :green, :red, :red], @format.border_color
-			@format.top_color = :red
-			assert_equal [:green, :red, :red, :red], @format.border_color
-			@format.bottom_color = :red
-			assert_equal [:red, :red, :red, :red], @format.border_color
-			assert_raises(ArgumentError) do			
-				@format.bottom_color = :bogus
-			end
-		end
+
+    def test_borders?
+      assert_equal [:none, :none, :none, :none], @format.border
+      @format.border = :thick
+      assert_equal [:thick, :thick, :thick, :thick], @format.border
+      @format.left = :hair
+      assert_equal [:thick, :thick, :thick, :hair], @format.border
+      @format.right = :hair
+      assert_equal [:thick, :thick, :hair, :hair], @format.border
+      @format.top = :hair
+      assert_equal [:thick, :hair, :hair, :hair], @format.border
+      @format.bottom = :hair
+      assert_equal [:hair, :hair, :hair, :hair], @format.border
+      assert_raises(ArgumentError) do
+        @format.bottom = :bogus
+      end
+      assert_equal [:black, :black, :black, :black], @format.border_color
+      @format.border_color = :green
+      assert_equal [:green, :green, :green, :green], @format.border_color
+      @format.left_color = :red
+      assert_equal [:green, :green, :green, :red], @format.border_color
+      @format.right_color = :red
+      assert_equal [:green, :green, :red, :red], @format.border_color
+      @format.top_color = :red
+      assert_equal [:green, :red, :red, :red], @format.border_color
+      @format.bottom_color = :red
+      assert_equal [:red, :red, :red, :red], @format.border_color
+      assert_raises(ArgumentError) do
+        @format.bottom_color = :bogus
+      end
+    end
   end
 end