rubyXL 3.3.29 → 3.4.33

data/rubyXL.gemspec

@@ -1,25 +1,28 @@
-# Generated by jeweler
+# Generated by juwelier
 # DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: rubyXL 3.3.29 ruby lib
+# stub: rubyXL 3.4.33 ruby lib
 
 Gem::Specification.new do |s|
-  s.name = "rubyXL"
-  s.version = "3.3.29"
+  s.name = "rubyXL".freeze
+  s.version = "3.4.33"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
-  s.authors = ["Vivek Bhagwat", "Wesha"]
-  s.date = "2018-02-26"
-  s.description = "rubyXL is a gem which allows the parsing, creation, and manipulation of Microsoft Excel (.xlsx/.xlsm) Documents"
-  s.email = "bhagwat.vivek@gmail.com"
+  s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib".freeze]
+  s.authors = ["Vivek Bhagwat".freeze, "Wesha".freeze]
+  s.date = "2024-12-20"
+  s.description = "rubyXL is a gem which allows the parsing, creation, and manipulation of Microsoft Excel (.xlsx/.xlsm) Documents".freeze
+  s.email = "bhagwat.vivek@gmail.com".freeze
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.rdoc"
   ]
   s.files = [
+    ".circleci/config.yml",
     ".codeclimate.yml",
+    ".rubocop.yml",
+    "CHANGELOG.md",
     "Gemfile",
     "LICENSE.txt",
     "README.rdoc",
@@ -28,6 +31,11 @@ Gem::Specification.new do |s|
     "lib/rubyXL.rb",
     "lib/rubyXL/cell.rb",
     "lib/rubyXL/convenience_methods.rb",
+    "lib/rubyXL/convenience_methods/cell.rb",
+    "lib/rubyXL/convenience_methods/color.rb",
+    "lib/rubyXL/convenience_methods/font.rb",
+    "lib/rubyXL/convenience_methods/workbook.rb",
+    "lib/rubyXL/convenience_methods/worksheet.rb",
     "lib/rubyXL/objects/border.rb",
     "lib/rubyXL/objects/calculation_chain.rb",
     "lib/rubyXL/objects/cell_style.rb",
@@ -35,6 +43,7 @@ Gem::Specification.new do |s|
     "lib/rubyXL/objects/color.rb",
     "lib/rubyXL/objects/column_range.rb",
     "lib/rubyXL/objects/comments.rb",
+    "lib/rubyXL/objects/connection.rb",
     "lib/rubyXL/objects/container_nodes.rb",
     "lib/rubyXL/objects/content_types.rb",
     "lib/rubyXL/objects/data_validation.rb",
@@ -46,6 +55,7 @@ Gem::Specification.new do |s|
     "lib/rubyXL/objects/font.rb",
     "lib/rubyXL/objects/formula.rb",
     "lib/rubyXL/objects/ooxml_object.rb",
+    "lib/rubyXL/objects/query_table.rb",
     "lib/rubyXL/objects/reference.rb",
     "lib/rubyXL/objects/relationships.rb",
     "lib/rubyXL/objects/root.rb",
@@ -70,6 +80,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/AdjustHandleList.html",
     "rdoc/RubyXL/Alignment.html",
     "rdoc/RubyXL/AlternateContent.html",
+    "rdoc/RubyXL/AlternateUrls.html",
     "rdoc/RubyXL/Authors.html",
     "rdoc/RubyXL/AutoFilter.html",
     "rdoc/RubyXL/AutoFilterColumn.html",
@@ -211,6 +222,10 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/ChartsheetView.html",
     "rdoc/RubyXL/ChartsheetViews.html",
     "rdoc/RubyXL/Color.html",
+    "rdoc/RubyXL/ColorConvenienceClasses.html",
+    "rdoc/RubyXL/ColorConvenienceClasses/HlsColor.html",
+    "rdoc/RubyXL/ColorConvenienceClasses/RgbColor.html",
+    "rdoc/RubyXL/ColorConvenienceMethods.html",
     "rdoc/RubyXL/ColorFilter.html",
     "rdoc/RubyXL/ColorScale.html",
     "rdoc/RubyXL/ColorSet.html",
@@ -223,6 +238,12 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/ConditionalFormatValue.html",
     "rdoc/RubyXL/ConditionalFormatting.html",
     "rdoc/RubyXL/ConditionalFormattingRule.html",
+    "rdoc/RubyXL/Connection.html",
+    "rdoc/RubyXL/ConnectionTable.html",
+    "rdoc/RubyXL/ConnectionTables.html",
+    "rdoc/RubyXL/ConnectionTextField.html",
+    "rdoc/RubyXL/ConnectionTextFields.html",
+    "rdoc/RubyXL/Connections.html",
     "rdoc/RubyXL/ContentTypeDefault.html",
     "rdoc/RubyXL/ContentTypeOverride.html",
     "rdoc/RubyXL/ContentTypes.html",
@@ -236,6 +257,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/CustomProperties.html",
     "rdoc/RubyXL/CustomPropertiesFile.html",
     "rdoc/RubyXL/CustomProperty.html",
+    "rdoc/RubyXL/CustomPropertyFile.html",
     "rdoc/RubyXL/CustomSheetView.html",
     "rdoc/RubyXL/CustomSheetViews.html",
     "rdoc/RubyXL/CustomWorkbookView.html",
@@ -318,7 +340,12 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/OOXMLObjectInstanceMethods.html",
     "rdoc/RubyXL/OOXMLRelationshipsFile.html",
     "rdoc/RubyXL/OOXMLTopLevelObject.html",
+    "rdoc/RubyXL/OdbcOleDbProperties.html",
     "rdoc/RubyXL/Offset.html",
+    "rdoc/RubyXL/OlapProperties.html",
+    "rdoc/RubyXL/OleItem.html",
+    "rdoc/RubyXL/OleItems.html",
+    "rdoc/RubyXL/OleLink.html",
     "rdoc/RubyXL/OutlineProperties.html",
     "rdoc/RubyXL/PageMargins.html",
     "rdoc/RubyXL/PageSetup.html",
@@ -326,6 +353,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/Pane.html",
     "rdoc/RubyXL/Parser.html",
     "rdoc/RubyXL/PatternFill.html",
+    "rdoc/RubyXL/PersonMetadata.html",
     "rdoc/RubyXL/PhoneticProperties.html",
     "rdoc/RubyXL/PhoneticRun.html",
     "rdoc/RubyXL/PivotArea.html",
@@ -343,6 +371,14 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/ProtectedRange.html",
     "rdoc/RubyXL/ProtectedRanges.html",
     "rdoc/RubyXL/Protection.html",
+    "rdoc/RubyXL/QueryParameter.html",
+    "rdoc/RubyXL/QueryParameters.html",
+    "rdoc/RubyXL/QueryTable.html",
+    "rdoc/RubyXL/QueryTableDeletedField.html",
+    "rdoc/RubyXL/QueryTableDeletedFields.html",
+    "rdoc/RubyXL/QueryTableField.html",
+    "rdoc/RubyXL/QueryTableFields.html",
+    "rdoc/RubyXL/QueryTableRefresh.html",
     "rdoc/RubyXL/RID.html",
     "rdoc/RubyXL/RawOOXML.html",
     "rdoc/RubyXL/Reference.html",
@@ -366,6 +402,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/SheetData.html",
     "rdoc/RubyXL/SheetDataExt.html",
     "rdoc/RubyXL/SheetDataSet.html",
+    "rdoc/RubyXL/SheetMetadata.html",
     "rdoc/RubyXL/SheetName.html",
     "rdoc/RubyXL/SheetNames.html",
     "rdoc/RubyXL/Sheets.html",
@@ -388,6 +425,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/TableStyle.html",
     "rdoc/RubyXL/TableStyles.html",
     "rdoc/RubyXL/Text.html",
+    "rdoc/RubyXL/TextImportSettings.html",
     "rdoc/RubyXL/Theme.html",
     "rdoc/RubyXL/ThemeElements.html",
     "rdoc/RubyXL/ThumbnailFile.html",
@@ -402,6 +440,7 @@ Gem::Specification.new do |s|
     "rdoc/RubyXL/WebPublishingItem.html",
     "rdoc/RubyXL/WebPublishingItems.html",
     "rdoc/RubyXL/WebPublishingProperties.html",
+    "rdoc/RubyXL/WebQueryProperties.html",
     "rdoc/RubyXL/Workbook.html",
     "rdoc/RubyXL/WorkbookConvenienceMethods.html",
     "rdoc/RubyXL/WorkbookProperties.html",
@@ -454,7 +493,6 @@ Gem::Specification.new do |s|
     "rdoc/images/zoom.png",
     "rdoc/index.html",
     "rdoc/js/darkfish.js",
-    "rdoc/js/jquery.js",
     "rdoc/js/navigation.js",
     "rdoc/js/navigation.js.gz",
     "rdoc/js/search.js",
@@ -468,6 +506,7 @@ Gem::Specification.new do |s|
     "spec/lib/color_spec.rb",
     "spec/lib/parser_spec.rb",
     "spec/lib/reference_spec.rb",
+    "spec/lib/rgb_color_spec.rb",
     "spec/lib/stylesheet_spec.rb",
     "spec/lib/text_spec.rb",
     "spec/lib/workbook_spec.rb",
@@ -478,42 +517,37 @@ Gem::Specification.new do |s|
     "test/test_parse_write.rb",
     "tmp/.gitignore"
   ]
-  s.homepage = "http://github.com/gilt/rubyXL"
-  s.licenses = ["MIT"]
-  s.rubygems_version = "2.5.1"
-  s.summary = "rubyXL is a gem which allows the parsing, creation, and manipulation of Microsoft Excel (.xlsx/.xlsm) Documents"
+  s.homepage = "http://github.com/gilt/rubyXL".freeze
+  s.licenses = ["MIT".freeze]
+  s.rubygems_version = "3.1.6".freeze
+  s.summary = "rubyXL is a gem which allows the parsing, creation, and manipulation of Microsoft Excel (.xlsx/.xlsm) Documents".freeze
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
+  end
 
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<nokogiri>, [">= 1.4.4"])
-      s.add_runtime_dependency(%q<rubyzip>, [">= 1.1.6"])
-      s.add_development_dependency(%q<bundler>, [">= 0"])
-      s.add_development_dependency(%q<rake>, [">= 0"])
-      s.add_development_dependency(%q<jeweler>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, [">= 0"])
-      s.add_development_dependency(%q<simplecov>, [">= 0"])
-      s.add_development_dependency(%q<ruby-prof>, [">= 0"])
-    else
-      s.add_dependency(%q<nokogiri>, [">= 1.4.4"])
-      s.add_dependency(%q<rubyzip>, [">= 1.1.6"])
-      s.add_dependency(%q<bundler>, [">= 0"])
-      s.add_dependency(%q<rake>, [">= 0"])
-      s.add_dependency(%q<jeweler>, [">= 0"])
-      s.add_dependency(%q<rspec>, [">= 0"])
-      s.add_dependency(%q<simplecov>, [">= 0"])
-      s.add_dependency(%q<ruby-prof>, [">= 0"])
-    end
+  if s.respond_to? :add_runtime_dependency then
+    s.add_runtime_dependency(%q<nokogiri>.freeze, [">= 1.10.8"])
+    s.add_runtime_dependency(%q<rubyzip>.freeze, [">= 1.3.0"])
+    s.add_development_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rake>.freeze, [">= 0"])
+    s.add_development_dependency(%q<juwelier>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rspec>.freeze, [">= 0"])
+    s.add_development_dependency(%q<simplecov>.freeze, [">= 0"])
+    s.add_development_dependency(%q<ruby-prof>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rspec_junit_formatter>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rubocop>.freeze, [">= 0"])
   else
-    s.add_dependency(%q<nokogiri>, [">= 1.4.4"])
-    s.add_dependency(%q<rubyzip>, [">= 1.1.6"])
-    s.add_dependency(%q<bundler>, [">= 0"])
-    s.add_dependency(%q<rake>, [">= 0"])
-    s.add_dependency(%q<jeweler>, [">= 0"])
-    s.add_dependency(%q<rspec>, [">= 0"])
-    s.add_dependency(%q<simplecov>, [">= 0"])
-    s.add_dependency(%q<ruby-prof>, [">= 0"])
+    s.add_dependency(%q<nokogiri>.freeze, [">= 1.10.8"])
+    s.add_dependency(%q<rubyzip>.freeze, [">= 1.3.0"])
+    s.add_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_dependency(%q<rake>.freeze, [">= 0"])
+    s.add_dependency(%q<juwelier>.freeze, [">= 0"])
+    s.add_dependency(%q<rspec>.freeze, [">= 0"])
+    s.add_dependency(%q<simplecov>.freeze, [">= 0"])
+    s.add_dependency(%q<ruby-prof>.freeze, [">= 0"])
+    s.add_dependency(%q<rspec_junit_formatter>.freeze, [">= 0"])
+    s.add_dependency(%q<rubocop>.freeze, [">= 0"])
   end
 end
 

data/spec/lib/cell_spec.rb

@@ -1,14 +1,17 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'bigdecimal'
+require 'rubyXL/convenience_methods/font'
+require 'rubyXL/convenience_methods/cell'
 
 describe RubyXL::Cell do
-
   before do
     @workbook  = RubyXL::Workbook.new
     @worksheet = @workbook.add_worksheet('Test Worksheet')
     @workbook.worksheets << @worksheet
-    (0..10).each do |i|
-      (0..10).each do |j|
+    11.times do |i|
+      11.times do |j|
         @worksheet.add_cell(i, j, "#{i}:#{j}")
       end
     end
@@ -29,6 +32,66 @@ describe RubyXL::Cell do
       cell = @worksheet.add_cell(r, c, RubyXL::RichText.new(:t => RubyXL::Text.new(:value => 'Hello')))
       expect(cell.datatype).to eq(RubyXL::DataType::INLINE_STRING)
     end
+
+    it 'should properly handle dates' do
+      r = 3
+      c = 3
+
+      dt = Date.today
+      cell = @worksheet.add_cell(r, c, dt)
+      cell.set_number_format('ddd mmm dd, yyyy')
+      expect(cell.value).to eq(dt)
+
+      tm = DateTime.now
+      cell = @worksheet.add_cell(r, c, tm)
+      cell.set_number_format('ddd mmm dd, yyyy HH:MM:SS')
+
+      # Due to rounding errors, we allow microsecond precision on DateTime.
+      expect((cell.value - tm).to_f).to be_within(1.0 / 86400e6).of(0)
+
+      tm = Time.now
+      cell = @worksheet.add_cell(r, c, tm)
+      cell.set_number_format('ddd mmm dd, yyyy HH:MM:SS')
+
+      # Due to rounding errors, we allow microsecond precision on Time.
+      expect(cell.value - tm.to_datetime).to be_within(1.0 / 86400e6).of(0)
+
+      expected_date = '2020-10-15T14:00:00Z'
+      expected_datetime = Time.parse(expected_date).utc
+      raw_value = '44119.583333333328' # Obtained from parsing a xlsx file with the expected date
+      cell = @worksheet.add_cell(r, c, Time.now) # Force a date cell type
+      cell.set_number_format('ddd mmm dd, yyyy HH:MM:SS')
+      cell.raw_value = raw_value
+      expect(cell.raw_value).to eq(raw_value), 'Wrong raw value'
+      cell.set_number_format('ddd mmm dd, yyyy HH:MM:SS')
+
+      # We expect exactly the same date
+      expect(cell.value.to_time.utc.iso8601).to eq(expected_datetime.iso8601)
+    end
+
+    it 'should raise against too long String' do
+      ok_data = 'A' * 32767  # The limit is 32767
+
+      expect {
+         @worksheet.add_cell(0, 1, ok_data) # 32767 -> OK
+      }.not_to raise_error
+      expect {
+        # 1 longer than the limit, so an exception must be thrown.
+        @worksheet.add_cell(0, 2, "#{ok_data}x")
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should raise against too long RichText' do
+      ok_data = 'A' * 32767  # The limit is 32767
+
+      expect {
+        @worksheet.add_cell(0, 1, RubyXL::RichText.new(:t => RubyXL::Text.new(:value => ok_data))) # 32767 -> OK
+      }.not_to raise_error
+      expect {
+        # 1 longer than the limit, so an exception must be thrown.
+        @worksheet.add_cell(0, 2, RubyXL::RichText.new(:t => RubyXL::Text.new(:value => "#{ok_data}x")))
+      }.to raise_error(ArgumentError)
+    end
   end
 
   describe '.change_fill' do
@@ -145,6 +208,22 @@ describe RubyXL::Cell do
     end
   end
 
+  describe '.change_text_indent' do
+    it 'should cause the cell to have the corresponding text indent' do
+      expect(@cell.text_indent).to be_nil
+      @cell.change_text_indent(2)
+      expect(@cell.text_indent).to eq(2)
+    end
+
+    it 'should not cause other cells with the same style to have text indent' do
+      another_cell = @worksheet[1][0]
+      another_cell.style_index = @cell.style_index
+      expect(another_cell.text_indent).to be_nil
+      @cell.change_text_indent(2)
+      expect(another_cell.text_indent).to be_nil
+    end
+  end
+
   describe '.change_border_color' do
     it 'should cause cell to have a colored top border' do
       expect(@cell.get_border_color(:top)).to be_nil
@@ -231,7 +310,7 @@ describe RubyXL::Cell do
       expect(cell.value).to eq('Hello')
     end
 
-    it "should properly handle numeric values" do
+    it 'should properly handle numeric values' do
       @cell.datatype = nil
       @cell.raw_value = '1'
       expect(@cell.value).to eq(1)
@@ -322,6 +401,12 @@ describe RubyXL::Cell do
       end
     end
 
+    context 'with RichText' do
+      it 'returns the value of the RichText' do
+        cell = RubyXL::Cell.new(is: RubyXL::RichText.new(t: RubyXL::Text.new(value: 'test')))
+        expect(cell.value).to eq('test')
+      end
+    end
   end
 
   describe '.change_contents' do
@@ -340,6 +425,15 @@ describe RubyXL::Cell do
       expect(@cell.formula).to be_nil
     end
 
+    it 'should cause cell value to match a time that is passed in' do
+      time = Time.parse('January 1, 2011')
+      @cell.change_contents(time)
+      expect(@cell).to receive(:is_date?).at_least(1).and_return(true)
+      expect(@cell.value).to eq(time.to_datetime)
+      expect(@cell.datatype).to be_nil
+      expect(@cell.formula).to be_nil
+    end
+
     it 'should case cell value to match a Float that is passed in' do
       number = 1.25
       @cell.change_contents(number)
@@ -356,8 +450,8 @@ describe RubyXL::Cell do
       expect(@cell.formula).to be_nil
     end
 
-    it 'should case cell value to match an BigDecimal that is passed in' do
-      number = BigDecimal.new('1234.5678')
+    it 'should cause cell value to match a BigDecimal that is passed in' do
+      number = BigDecimal('1234.5678')
       @cell.change_contents(number)
       expect(@cell.value).to eq(number)
       expect(@cell.datatype).to be_nil
@@ -444,6 +538,18 @@ describe RubyXL::Cell do
     it 'should return nil if no horizontal alignment has been specified for this cell' do
       expect(@cell.horizontal_alignment).to be_nil
     end
+
+    it 'should not create new XFs when changing alignment to already existing values' do
+      @cell.change_horizontal_alignment('left')
+      style_xf1 = @cell.style_index
+      @cell.change_horizontal_alignment('right')
+      expect(@cell.style_index).not_to eq(style_xf1)
+      style_xf2 = @cell.style_index
+      @cell.change_horizontal_alignment('left')
+      expect(@cell.style_index).to eq(style_xf1)
+      @cell.change_horizontal_alignment('right')
+      expect(@cell.style_index).to eq(style_xf2)
+    end
   end
 
   describe '.vertical_alignment' do
@@ -512,4 +618,11 @@ describe RubyXL::Cell do
     end
   end
 
+  describe '.text_rotation' do
+    it 'should correctly return the rotation for this cell' do
+      expect(@cell.text_rotation).to be_nil
+      @cell.change_text_rotation(45)
+      expect(@cell.text_rotation).to eq(45)
+    end
+  end
 end

data/spec/lib/color_spec.rb

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
+require 'rubyXL/convenience_methods/color'
 
 describe RubyXL::Color do
   describe '.validate_color' do
@@ -6,8 +9,13 @@ describe RubyXL::Color do
       expect(RubyXL::Color.validate_color('0fbCAd')).to eq(true)
     end
 
+    it 'should return true if a valid hex color with alpha is passed' do
+      expect(RubyXL::Color.validate_color('01AbCdeF')).to eq(true)
+    end
+
     it 'should cause an error if an invalid hex color code or one with a # is passed' do
-      expect {RubyXL::Color.validate_color('#G')}.to raise_error(RuntimeError)
+      expect { RubyXL::Color.validate_color('#G') }.to raise_error(RuntimeError)
+      expect { RubyXL::Color.validate_color('1234567') }.to raise_error(RuntimeError)
     end
   end
 end

data/spec/lib/parser_spec.rb

@@ -1,25 +1,28 @@
+# frozen_string_literal: true
+
+require 'English'
 require 'spec_helper'
 require 'tmpdir'
 
 describe RubyXL::Parser do
-
   before do
-    @test_sheet_name = "This is a very long sheet name that should be trimmed to 31 characters for compatibility with MS Excel"
+    @test_sheet_name = 'This is a very long sheet name that should be trimmed ' \
+                       'to 31 characters for compatibility with MS Excel'
     @workbook = RubyXL::Workbook.new
-    @workbook.add_worksheet("Test Worksheet")
+    @workbook.add_worksheet('Test Worksheet')
     @time = Time.at(Time.now.to_i) # Excel only saves times with 1-second precision.
     @time2 = @time + 123456
 
-    ws = @workbook.add_worksheet("Escape Test")
-    ws.add_cell(0, 0, "&")
-    ws.add_cell(0, 1, "<")
-    ws.add_cell(0, 2, ">")
-    ws.add_cell(0, 3, "")
+    ws = @workbook.add_worksheet('Escape Test')
+    ws.add_cell(0, 0, '&')
+    ws.add_cell(0, 1, '<')
+    ws.add_cell(0, 2, '>')
+    ws.add_cell(0, 3, '')
 
-    ws.add_cell(1, 0, "&")#TODO#.datatype = RubyXL::Cell::SHARED_STRING
-    ws.add_cell(1, 1, "<")#TODO#.datatype = RubyXL::Cell::SHARED_STRING
-    ws.add_cell(1, 2, ">")#TODO#.datatype = RubyXL::Cell::SHARED_STRING
-    ws.add_cell(1, 3, "")
+    ws.add_cell(1, 0, '&') #TODO# .datatype = RubyXL::Cell::SHARED_STRING
+    ws.add_cell(1, 1, '<') #TODO# .datatype = RubyXL::Cell::SHARED_STRING
+    ws.add_cell(1, 2, '>') #TODO# .datatype = RubyXL::Cell::SHARED_STRING
+    ws.add_cell(1, 3, '')
 
     ws.add_cell(2, 0, 0)
     ws.add_cell(2, 1, 12345)
@@ -35,13 +38,13 @@ describe RubyXL::Parser do
 
     @workbook.add_worksheet(@test_sheet_name)
 
-    @workbook.creator = "test creator"
-    @workbook.modifier = "test modifier"
+    @workbook.creator = 'test creator'
+    @workbook.modifier = 'test modifier'
     @workbook.created_at = @time
     @workbook.modified_at = @time2
 
     @time_str = Time.now.to_s
-    @file = "rubyXL-#{$$}-#{DateTime.now.strftime('%Q')}.xlsx"
+    @file = "rubyXL-#{$PROCESS_ID}-#{DateTime.now.strftime('%Q')}.xlsx"
     @workbook.write(@file)
   end
 
@@ -58,7 +61,7 @@ describe RubyXL::Parser do
     end
 
     it 'should cause an error if an xlsx or xlsm workbook is not passed' do
-      expect {@workbook2 = RubyXL::Parser.parse("nonexistent_file.tmp")}.to raise_error(Zip::Error)
+      expect { @workbook2 = RubyXL::Parser.parse('nonexistent_file.tmp') }.to raise_error(Zip::Error)
     end
 
     it 'should construct consistent number formats' do
@@ -69,19 +72,19 @@ describe RubyXL::Parser do
 
     it 'should unescape HTML entities properly' do
       @workbook2 = RubyXL::Parser.parse(@file)
-      expect(@workbook2["Escape Test"][0][0].value).to eq("&")
-      expect(@workbook2["Escape Test"][0][1].value).to eq("<")
-      expect(@workbook2["Escape Test"][0][2].value).to eq(">")
+      expect(@workbook2['Escape Test'][0][0].value).to eq('&')
+      expect(@workbook2['Escape Test'][0][1].value).to eq('<')
+      expect(@workbook2['Escape Test'][0][2].value).to eq('>')
 
-      expect(@workbook2["Escape Test"][1][0].value).to eq("&")
-      expect(@workbook2["Escape Test"][1][1].value).to eq("<")
-      expect(@workbook2["Escape Test"][1][2].value).to eq(">")
+      expect(@workbook2['Escape Test'][1][0].value).to eq('&')
+      expect(@workbook2['Escape Test'][1][1].value).to eq('<')
+      expect(@workbook2['Escape Test'][1][2].value).to eq('>')
     end
 
     it 'should parse Core properties correctly' do
       @workbook2 = RubyXL::Parser.parse(@file)
-      expect(@workbook2.creator).to eq("test creator")
-      expect(@workbook2.modifier).to eq("test modifier")
+      expect(@workbook2.creator).to eq('test creator')
+      expect(@workbook2.modifier).to eq('test modifier')
       expect(@workbook2.created_at).to eq(@time)
       expect(@workbook2.modified_at).to eq(@time2)
     end
@@ -91,11 +94,9 @@ describe RubyXL::Parser do
       expect(@workbook2[@test_sheet_name]).to be_nil
       expect(@workbook2[@test_sheet_name[0..30]]).not_to be_nil
     end
-
   end
 
   describe 'parse_buffer' do
-
     it 'should parse string buffer correctly' do
       buffer = File.read(@file)
       expect(buffer).to be_instance_of(String)
@@ -109,10 +110,9 @@ describe RubyXL::Parser do
       f = RubyXL::Parser.parse_buffer(io)
       expect(f).to be_instance_of(RubyXL::Workbook)
     end
-
   end
 
   after do
-    File.delete(@file) if File.exist?(@file)
+    FileUtils.rm_f(@file)
   end
 end

data/spec/lib/reference_spec.rb

@@ -1,9 +1,10 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe RubyXL::Reference do
-
-  describe '.ind2ref + .ref2ind' do
-    it 'should correctly return the "Excel Style" description of cells when given a row/column number' do
+  describe '.ind2ref' do
+    it "should correctly return the 'Excel Style' description of cells when given a row/column number" do
       expect(RubyXL::Reference.ind2ref(0, 26)).to eq('AA1')
       expect(RubyXL::Reference.ind2ref(99, 0)).to eq('A100')
       expect(RubyXL::Reference.ind2ref(0, 26)).to eq('AA1')
@@ -14,15 +15,50 @@ describe RubyXL::Reference do
       expect(RubyXL::Reference.ind2ref(0, 16383)).to eq('XFD1')
     end
 
-    it 'should correctly convert back and forth between "Excel Style" and index style cell references' do
+    it "should correctly convert back and forth between 'Excel Style' and index style cell references" do
       0.upto(16383) do |n|
         expect(RubyXL::Reference.ref2ind(RubyXL::Reference.ind2ref(n, 16383 - n))).to eq([ n, 16383 - n ])
       end
     end
+  end
+
+  describe '.valid?' do
+    it 'should return true for valid references' do
+      expect(RubyXL::Reference.new('C23').valid?).to be true
+    end
+
+    it 'should return false for invalid references' do
+      expect(RubyXL::Reference.new('C2A').valid?).to be false
+    end
+  end
 
+  describe '.ref2ind' do
     it 'should return [-1, -1] if the Excel index is not well-formed' do
       expect(RubyXL::Reference.ref2ind('A1B')).to eq([-1, -1])
     end
   end
 
-end
+  describe '.new' do
+    it 'should take a string parameter' do
+      new_ref = RubyXL::Reference.new('C23')
+      expect(new_ref.single_cell?).to be true
+      expect(new_ref.to_s).to eq 'C23'
+    end
+
+    it 'should take 2 coordinate parameters' do
+      new_ref = RubyXL::Reference.new(11, 22)
+      expect(new_ref.single_cell?).to be true
+      expect(new_ref.to_s).to eq 'W12'
+    end
+
+    it 'should take 4 coordinate parameters' do
+      new_ref = RubyXL::Reference.new(11, 22, 33, 44)
+      expect(new_ref.single_cell?).to be false
+      expect(new_ref.to_s).to eq 'AH12:AS23'
+    end
+
+    it 'should take named parameters' do
+      expect(RubyXL::Reference.new(row_from: 44, row_to: 33, col_from: 22, col_to: 11).to_s).to eq('W45:L34')
+    end
+  end
+end

data/spec/lib/rgb_color_spec.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'rubyXL/convenience_methods/color'
+
+describe RubyXL::RgbColor do
+  describe '.to_s' do
+    it 'should properly translate RGB color to string' do
+      rgb_color = RubyXL::RgbColor.new
+
+      rgb_color.r = 1
+      rgb_color.g = 2
+      rgb_color.b = 255
+
+      expect(rgb_color.to_s).to eq('0102ff')
+    end
+
+    it 'should properly translate RGB color with alpha value to string' do
+      rgb_color = RubyXL::RgbColor.new
+
+      rgb_color.r = 11
+      rgb_color.g = 22
+      rgb_color.b = 33
+      rgb_color.a = 255
+
+      expect(rgb_color.to_s).to eq('0b1621ff')
+    end
+  end
+end