writeexcel 1.0.5 → 1.0.7

data/lib/writeexcel/storage_lite.rb

@@ -55,8 +55,6 @@ class OLEStorageLite       #:nodoc:
     end
     done << no
 
-    rootblock = info[:root_start]
-
     #1. Get Information about itself
     pps = _getNthPps(no, info, data)
 
@@ -76,7 +74,6 @@ class OLEStorageLite       #:nodoc:
   private :_getPpsTree
 
   def _getPpsSearch(no, info, name, data, icase, done = nil)
-    rootblock = info[:root_start]
     #1. Check it self
     if done
       return [] if done.include?(no)
@@ -589,7 +586,6 @@ class OLEStorageLitePPSRoot < OLEStorageLitePPS       #:nodoc:
       rh_info[:fileh] = sFile.binmode
     end
 
-    iBlk = 0
     #1. Make an array of PPS (for Save)
     aList=[]
     if bNoAs
@@ -728,7 +724,6 @@ class OLEStorageLitePPSRoot < OLEStorageLitePPS       #:nodoc:
   private :_saveHeader
 
   def _saveBigData(iStBlk, aList, rh_info)
-    iRes = 0
     file = rh_info[:fileh]
 
     #1.Write Big (ge 0x1000) Data into Block
@@ -810,7 +805,7 @@ class OLEStorageLitePPSRoot < OLEStorageLitePPS       #:nodoc:
       aPrev = aWk[0, iPos]
       aWk[0..iPos-1] = []
       aNext = aWk[1, iCnt - iPos - 1]
-      aWk[1..(1 + iCnt - iPos -1 -1)] = []
+      aWk[1..(1 + iCnt - iPos - 1 - 1)] = []
       pps_array[iPos].prev_pps = _savePpsSetPnt(aPrev, aList, rh_info)
       pps_array[iPos].next_pps = _savePpsSetPnt(aNext, aList, rh_info)
       pps_array[iPos].dir_pps  = _savePpsSetPnt(pps_array[iPos].child, aList, rh_info)
@@ -834,12 +829,10 @@ class OLEStorageLitePPSRoot < OLEStorageLitePPS       #:nodoc:
       return pps_array[0].no
     #1.3 Array
     else
-      iCnt = pps_array.size
       #1.3.1 Define Center
       iPos = 0  #int($iCnt/ 2);     #$iCnt
 
       aWk = pps_array.dup
-      aPrev = aWk[1, 1]
       aWk[1..1] = []
       aNext = aWk[1..aWk.size]      #, $iCnt - $iPos -1);
       pps_array[iPos].prev_pps = _savePpsSetPnt2(pps_array, aList, rh_info)
@@ -975,7 +968,7 @@ class OLEStorageLitePPSFile < OLEStorageLitePPS       #:nodoc:
     @pps_file.binmode
   end
 
-  def append (data)
+  def append(data)
     return if data.nil?
     if @pps_file
       @pps_file << data

data/lib/writeexcel/version.rb

@@ -1,5 +1,3 @@
-require 'writeexcel'
-
-class WriteExcel < Workbook
-  VERSION = "1.0.5"
+module Writeexcel
+  VERSION = "1.0.7"
 end

data/lib/writeexcel/workbook.rb

@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# frozen_string_literal: true
 ###############################################################################
 #
 # Workbook - A writer class for Excel Workbooks.
@@ -13,20 +14,9 @@
 #
 require 'nkf'
 require 'forwardable'
-require 'writeexcel/biffwriter'
-require 'writeexcel/worksheet'
-require 'writeexcel/chart'
-require 'writeexcel/format'
-require 'writeexcel/formula'
-require 'writeexcel/olewriter'
-require 'writeexcel/storage_lite'
-require 'writeexcel/compatibility'
 require 'writeexcel/shared_string_table'
-require 'writeexcel/worksheets'
 
 class Workbook < BIFFWriter
-  require 'writeexcel/properties'
-  require 'writeexcel/helper'
 
   extend Forwardable
 
@@ -115,6 +105,7 @@ class Workbook < BIFFWriter
     @localtime             = Time.now
 
     @defined_names         = []
+    @fileclosed            = nil
 
     setup_built_in_formats(default_formats)
 
@@ -652,7 +643,6 @@ class Workbook < BIFFWriter
   #
   def define_name(name, formula, encoding = 0)
     sheet_index = 0
-    full_name   = name.downcase
 
     if name =~ /^(.*)!(.*)$/
       sheetname   = $1
@@ -1266,7 +1256,7 @@ class Workbook < BIFFWriter
     # Add the length of the MSODRAWINGGROUP records including an extra 4 bytes
     # for any CONTINUE headers. See add_mso_drawing_group_continue().
     mso_size = @mso_size
-    mso_size += 4 * Integer((mso_size -1) / Float(@limit))
+    mso_size += 4 * Integer((mso_size - 1) / Float(@limit))
     offset   += mso_size
 
     @worksheets.each do |sheet|
@@ -1374,7 +1364,7 @@ class Workbook < BIFFWriter
           image_id += 1
         else
           # We've processed this file already.
-          index = images_seen[image.filename] -1
+          index = images_seen[image.filename] - 1
 
           # Increase image reference count.
           image_data[index].ref_count += 1
@@ -2053,7 +2043,6 @@ class Workbook < BIFFWriter
   def store_shared_strings       #:nodoc:
     record              = 0x00FC   # Record identifier
     length              = 0x0008   # Number of bytes to follow
-    total               = 0x0000
 
     # Iterate through the strings to calculate the CONTINUE block sizes
     continue_limit = 8208
@@ -2136,10 +2125,10 @@ class Workbook < BIFFWriter
           append(tmp)
 
           # The remainder will be written in the next block(s)
-          string = string[space_remaining .. string.length-1]
+          string = string[space_remaining .. string.length - 1]
 
           # Reduce the current block length by the amount written
-          block_length -= continue_limit -continue -align
+          block_length -= continue_limit - continue - align
 
           # If the current string was split then the next CONTINUE block
           # should have the string continue flag (grbit) set unless the
@@ -2152,7 +2141,7 @@ class Workbook < BIFFWriter
           end
         else
           # Not enough space to start the string in the current block
-          block_length -= continue_limit -space_remaining -continue
+          block_length -= continue_limit - space_remaining - continue
           continue = 0
         end
 
@@ -2206,7 +2195,7 @@ class Workbook < BIFFWriter
       bucket_size = 1 + Integer(unique_strings / 128.0)
     end
 
-    buckets = Integer((unique_strings + bucket_size -1)  / Float(bucket_size))
+    buckets = Integer((unique_strings + bucket_size - 1) / Float(bucket_size))
 
     @extsst_buckets        = buckets
     @extsst_bucket_size    = bucket_size
@@ -2278,7 +2267,7 @@ class Workbook < BIFFWriter
   #     Case 3:  >  2*8224 bytes      2 MSODRAWINGGROUP + n CONTINUE
   #
   def add_mso_drawing_group_continue(data)       #:nodoc:
-    limit       = 8228 -4
+    limit       = 8228 - 4
     mso_group   = 0x00EB # Record identifier
     continue    = 0x003C # Record identifier
     block_count = 1
@@ -2334,7 +2323,7 @@ class Workbook < BIFFWriter
     version     = 15
     instance    = 0
     data        = ''
-    length      = @mso_size -12 # -4 (biff header) -8 (for this).
+    length      = @mso_size - 12 # -4 (biff header) -8 (for this).
 
     add_mso_generic(type, version, instance, data, length)
   end
@@ -2372,7 +2361,7 @@ class Workbook < BIFFWriter
     version     = 15
     instance    = @images_data.size          # Number of images.
     data        = ''
-    length      = @images_size +8 *instance
+    length      = @images_size + 8 * instance
 
     add_mso_generic(type, version, instance, data, length)
   end
@@ -2399,12 +2388,12 @@ class Workbook < BIFFWriter
     type        = 0xF007
     version     = 2
     instance    = image_type
-    length      = size +61
+    length      = size + 61
     data        = [image_type].pack('C')  +    # Win32
     [image_type].pack('C')  +    # Mac
     [checksum1].pack('H*')  +    # Uid checksum
     [0xFF].pack('v')        +    # Tag
-    [size +25].pack('V')    +    # Next Blip size
+    [size + 25].pack('V')   +    # Next Blip size
     [ref_count].pack('V')   +    # Image ref count
     [0x00000000].pack('V')  +    # File offset
     [0x00].pack('C')        +    # Usage
@@ -2430,7 +2419,7 @@ class Workbook < BIFFWriter
 
     type        = 0xF018 + image_type
     version     = 0x0000
-    length      = size +17
+    length      = size + 17
     data        = [checksum1].pack('H*')  +     # Uid checksum
     [0xFF].pack('C')        +     # Tag
     image_data                   # Image

data/lib/writeexcel/worksheet.rb

@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+# frozen_string_literal: true
 ###############################################################################
 #
 # Worksheet - A writer class for Excel Worksheets.
@@ -102,6 +103,7 @@ class Worksheet < BIFFWriter
     @title_range         = TitleRange.new(self)
     @print_range         = PrintRange.new(self)
 
+    @print_headers       = 0
     @print_gridlines     = 1
     @screen_gridlines    = 1
 
@@ -154,6 +156,18 @@ class Worksheet < BIFFWriter
 
     @table               = []
     @row_data            = {}
+    @header              = ''
+    @header_encoding     = nil
+    @footer              = ''
+    @footer_encoding     = nil
+    @hidden              = nil
+    @selected            = nil
+    @protect             = nil
+    @hcenter             = nil
+    @vcenter             = nil
+    @display_arabic      = nil
+    @frozen              = nil
+    @hide_zeros          = nil
   end
 
   #
@@ -352,8 +366,8 @@ class Worksheet < BIFFWriter
   #     worksheet2.activate
   #     worksheet1.hide
   #
-  def hide
-    @hidden = true
+  def hide(hidden = :hidden)
+    @hidden = hidden
 
     # A hidden worksheet shouldn't be active or selected.
     @selected  = false
@@ -536,7 +550,7 @@ class Worksheet < BIFFWriter
     # not be obtained exactly due to rounding in Excel.
     #
     if height
-      miyRw = height *20
+      miyRw = height * 20
     else
       miyRw = 0xff # The default row height
       height = 0
@@ -672,8 +686,11 @@ class Worksheet < BIFFWriter
   #
   def set_column(*args)
     # Check for a cell reference in A1 notation and substitute row and column
-    if args[0] =~ /^\D/
-      row1, firstcol, row2, lastcol, *data = substitute_cellref(*args)
+    if args[0].respond_to?(:match) && args[0] =~ /^\D/
+      array = substitute_cellref(*args)
+      firstcol = array[1]
+      lastcol = array[3]
+      *data = array[4, array.length - 4]
     else
       firstcol, lastcol, *data = args
     end
@@ -959,11 +976,11 @@ class Worksheet < BIFFWriter
     format   = args[5]
     encoding = args[6] ? 1 : 0
 
-    merge_range_core(rwFirst, colFirst, rwLast, colLast, string, format, encoding) do |rwFirst, colFirst, string, format, encoding|
-      if encoding != 0
-        write_utf16be_string(rwFirst, colFirst, string, format)
+    merge_range_core(rwFirst, colFirst, rwLast, colLast, string, format, encoding) do |_rwFirst, _colFirst, _string, _format, _encoding|
+      if _encoding != 0
+        write_utf16be_string(_rwFirst, _colFirst, _string, _format)
       else
-        write(rwFirst, colFirst, string, format)
+        write(_rwFirst, _colFirst, _string, _format)
       end
     end
   end
@@ -990,8 +1007,8 @@ class Worksheet < BIFFWriter
     format   = args[5]
     encoding = nil
 
-    merge_range_core(rwFirst, colFirst, rwLast, colLast, string, format, encoding) do |rwFirst, colFirst, string, format, encoding|
-      write_date_time(rwFirst, colFirst, string, format)
+    merge_range_core(rwFirst, colFirst, rwLast, colLast, string, format, encoding) do |_rwFirst, _colFirst, _string, _format, _encoding|
+      write_date_time(_rwFirst, _colFirst, _string, _format)
     end
   end
 
@@ -1211,7 +1228,7 @@ class Worksheet < BIFFWriter
 
     # Check for a column reference in A1 notation and substitute.
     # Convert col ref to a cell ref and then to a col number.
-    dummy, col = substitute_cellref("#{col}1") if col =~ /^\D/
+    col = substitute_cellref("#{col}1")[1] if col.to_s =~ /^\D/
 
     # Reject column if it is outside filter range.
     unless @filter_area.inside?(col)
@@ -1615,8 +1632,10 @@ class Worksheet < BIFFWriter
   #
   def repeat_columns(*args)
     # Check for a cell reference in A1 notation and substitute row and column
-    if args[0] =~ /^\D/
-      row1, firstcol, row2, lastcol = substitute_cellref(*args)
+    if args[0].to_s =~ /^\D/
+      array = substitute_cellref(*args)
+      firstcol = array[1]
+      lastcol = array[3]
     else
       firstcol, lastcol = args
     end
@@ -2209,10 +2228,8 @@ class Worksheet < BIFFWriter
     # Check for a cell reference in A1 notation and substitute row and column
     args = row_col_notation(args)
 
-    token = args[2]
-
     # Handle undefs as blanks
-    token ||= ''
+    token = args[2] || ''
 
     # First try user defined matches.
     @write_match.each do |aref|
@@ -2223,7 +2240,7 @@ class Worksheet < BIFFWriter
         match = eval("#{sub} self, args")
         return match if match
       end
-    end
+    end if token.respond_to?(:match)
 
     # Match an array ref.
     if token.respond_to?(:to_ary)
@@ -2231,16 +2248,16 @@ class Worksheet < BIFFWriter
     elsif token.respond_to?(:coerce)  # Numeric
       write_number(*args)
       # Match http, https or ftp URL
-    elsif token =~ %r|^[fh]tt?ps?://|
+    elsif token.respond_to?(:match) && token =~ %r|^[fh]tt?ps?://|
       write_url(*args)
       # Match mailto:
-    elsif token =~ %r|^mailto:|
+    elsif token.respond_to?(:match) && token =~ %r|^mailto:|
       write_url(*args)
       # Match internal or external sheet link
-    elsif token =~ %r!^(?:in|ex)ternal:!
+    elsif token.respond_to?(:match) && token =~ %r!^(?:in|ex)ternal:!
       write_url(*args)
       # Match formula
-    elsif token =~ /^=/
+    elsif token.respond_to?(:match) && token =~ /^=/
       write_formula(*args)
       # Match blank
     elsif token == ''
@@ -3495,7 +3512,8 @@ class Worksheet < BIFFWriter
 
     return -1 if args.size < 3   # Check the number of args
 
-    row, col, comment, params = args
+    row, col = args
+    params = args[3]
 
     # Check for pairs of optional arguments, i.e. an odd number of args.
     # raise "Uneven number of additional arguments" if args.size % 2 == 0
@@ -4308,7 +4326,7 @@ class Worksheet < BIFFWriter
   end
 
   def hidden?  # :nodoc:
-    @hidden
+    @hidden == :hidden
   end
 
   def hidden=(val)  # :nodoc:
@@ -4457,10 +4475,10 @@ class Worksheet < BIFFWriter
     y2 =  256.0 * height / size_row(row_end)
 
     # Simulate ceil() without calling POSIX::ceil().
-    x1 = (x1 +0.5).to_i
-    y1 = (y1 +0.5).to_i
-    x2 = (x2 +0.5).to_i
-    y2 = (y2 +0.5).to_i
+    x1 = (x1 + 0.5).to_i
+    y1 = (y1 + 0.5).to_i
+    x2 = (x2 + 0.5).to_i
+    y2 = (y2 + 0.5).to_i
 
     [
       col_start, x1,
@@ -4615,7 +4633,7 @@ class Worksheet < BIFFWriter
     drawings_saved += 1
 
     # For each sheet start the spids at the next 1024 interval.
-    max_spid   = 1024 * (1 + Integer((max_spid -1)/1024.0))
+    max_spid   = 1024 * (1 + Integer((max_spid - 1) / 1024.0))
     start_spid = max_spid
 
     # Max spid for each sheet and eventually for the workbook.
@@ -4626,7 +4644,7 @@ class Worksheet < BIFFWriter
     push_cluster(num_shapes, drawings_saved, clusters)
 
     # Pass calculated values back to the worksheet
-    @object_ids = ObjectIds.new(start_spid, drawings_saved, num_shapes, max_spid -1)
+    @object_ids = ObjectIds.new(start_spid, drawings_saved, num_shapes, max_spid - 1)
 
     [mso_size, drawings_saved, max_spid, start_spid]
   end
@@ -4666,7 +4684,7 @@ class Worksheet < BIFFWriter
   def set_header_footer_common(type, string, margin, encoding)  # :nodoc:
     ruby_19 { string = convert_to_ascii_if_ascii(string) }
 
-    limit    = encoding != 0 ? 255 *2 : 255
+    limit    = encoding != 0 ? 255 * 2 : 255
 
     # Handle utf8 strings
     if is_utf8?(string)
@@ -4821,7 +4839,7 @@ class Worksheet < BIFFWriter
     # Special handling of "Top" filter expressions.
     if tokens[0] =~ /^top|bottom$/i
       value = tokens[1]
-      if (value =~ /\D/ or value.to_i < 1 or value.to_i > 500)
+      if (value.to_s =~ /\D/ or value.to_i < 1 or value.to_i > 500)
         raise "The value '#{value}' in expression '#{expression}' " +
         "must be in the range 1 to 500"
       end
@@ -4850,7 +4868,7 @@ class Worksheet < BIFFWriter
     end
 
     # Special handling for Blanks/NonBlanks.
-    if (token =~ /^blanks|nonblanks$/i)
+    if (token.to_s =~ /^blanks|nonblanks$/i)
       # Only allow Equals or NotEqual in this context.
       if (operator != 2 and operator != 5)
         raise "The operator '#{tokens[1]}' in expression '#{expression}' " +
@@ -4876,7 +4894,7 @@ class Worksheet < BIFFWriter
 
     # if the string token contains an Excel match character then change the
     # operator type to indicate a non "simple" equality.
-    if (operator == 2 and token =~ /[*?]/)
+    if (operator == 2 and token.to_s =~ /[*?]/)
       operator = 22
     end
 
@@ -4936,31 +4954,27 @@ class Worksheet < BIFFWriter
   def substitute_cellref(cell, *args)       #:nodoc:
     return [cell, *args] if cell.respond_to?(:coerce) # Numeric
 
-    cell.upcase!
+    normalized_cell = cell.upcase
 
+    case normalized_cell
     # Convert a column range: 'A:A' or 'B:G'.
     # A range such as A:A is equivalent to A1:65536, so add rows as required
-    if cell =~ /\$?([A-I]?[A-Z]):\$?([A-I]?[A-Z])/
-      row1, col1 =  cell_to_rowcol($1 +'1')
-      row2, col2 =  cell_to_rowcol($2 +'65536')
+    when /\$?([A-I]?[A-Z]):\$?([A-I]?[A-Z])/
+      row1, col1 =  cell_to_rowcol($1 + '1')
+      row2, col2 =  cell_to_rowcol($2 + '65536')
       return [row1, col1, row2, col2, *args]
-    end
-
     # Convert a cell range: 'A1:B7'
-    if cell =~ /\$?([A-I]?[A-Z]\$?\d+):\$?([A-I]?[A-Z]\$?\d+)/
+    when /\$?([A-I]?[A-Z]\$?\d+):\$?([A-I]?[A-Z]\$?\d+)/
       row1, col1 =  cell_to_rowcol($1)
       row2, col2 =  cell_to_rowcol($2)
       return [row1, col1, row2, col2, *args]
-    end
-
     # Convert a cell reference: 'A1' or 'AD2000'
-    if (cell =~ /\$?([A-I]?[A-Z]\$?\d+)/)
+    when /\$?([A-I]?[A-Z]\$?\d+)/
       row1, col1 =  cell_to_rowcol($1)
       return [row1, col1, *args]
-
+    else
+      raise("Unknown cell reference #{normalized_cell}")
     end
-
-    raise("Unknown cell reference #{cell}")
   end
 
   #
@@ -5177,8 +5191,8 @@ class Worksheet < BIFFWriter
     # Convert an Ascii URL type and to a null terminated wchar string.
     if encoding == 0
       url  =
-        ruby_18 { url + "\0" } ||
-        ruby_19 { url.force_encoding('BINARY') + "\0".force_encoding('BINARY') }
+        ruby_18 { url.dup + "\0" } ||
+        ruby_19 { url.dup.force_encoding('BINARY') + "\0" }
       url  = url.unpack('c*').pack('v*')
     end
 
@@ -5266,12 +5280,12 @@ class Worksheet < BIFFWriter
   end
 
   def sheetname_from_url(url)
-    sheetname, cell = url.split('!')
+    sheetname = url.split('!')[0]
     sheetname
   end
 
   def cell_from_url(url)
-    sheetname, cell = url.split('!')
+    cell = url.split('!')[1]
     cell
   end
 
@@ -6368,9 +6382,9 @@ class Worksheet < BIFFWriter
 
       # The relationship is different for user units less than 1.
       if width < 1
-        (width *12).to_i
+        (width * 12).to_i
       else
-        (width *7 +5 ).to_i
+        (width * 7 + 5 ).to_i
       end
     else
       64
@@ -6427,7 +6441,7 @@ class Worksheet < BIFFWriter
 
     col1.upto(col2) do |i|
       # Reverse order since records are being pre-pended.
-      col = col2 -i
+      col = col2 - i
 
       # Skip if column doesn't have an active filter.
       next unless @filter_cols[col]
@@ -6546,7 +6560,7 @@ class Worksheet < BIFFWriter
       # Handle utf8 strings
       if is_utf8?(string)
         string = utf8_to_16be(string)
-        encodign = 1
+        encoding = 1
       end
 
       string =
@@ -6950,7 +6964,10 @@ class Worksheet < BIFFWriter
 
   # Check for a cell reference in A1 notation and substitute row and column
   def row_col_notation(args)   # :nodoc:
-    if args[0] =~ /^\D/
+    # ruby 3.2 no longer handles =~ for various types
+    return args unless args[0].respond_to?(:=~)
+
+    if args[0].respond_to?(:match) && args[0] =~ /^\D/
       substitute_cellref(*args)
     else
       args

data/lib/writeexcel/worksheets.rb

@@ -1,6 +1,4 @@
 class Workbook < BIFFWriter
-  require 'writeexcel/properties'
-  require 'writeexcel/helper'
 
   class Worksheets < Array
     attr_accessor :activesheet
@@ -8,6 +6,7 @@ class Workbook < BIFFWriter
 
     def initialize
       @activesheet = nil
+      @firstsheet  = nil
     end
 
     def activesheet_index

data/lib/writeexcel.rb

@@ -10,12 +10,13 @@
 # original written in Perl by John McNamara
 # converted to Ruby by Hideo Nakamura, cxn03651@msj.biglobe.ne.jp
 #
+require 'writeexcel/version'
 require 'writeexcel/biffwriter'
 require 'writeexcel/olewriter'
 require 'writeexcel/formula'
 require 'writeexcel/format'
-require 'writeexcel/worksheet'
-require "writeexcel/workbook"
+require 'writeexcel/worksheets'
+require 'writeexcel/workbook'
 require 'writeexcel/chart'
 require 'writeexcel/charts/area'
 require 'writeexcel/charts/bar'
@@ -1153,7 +1154,4 @@ require 'writeexcel/debug_info'
 #       12 until another chart with the title set is viewed.
 #
 class WriteExcel < Workbook
-  if RUBY_VERSION < '1.9'
-    $KCODE = 'u'
-  end
 end

data/test/helper.rb

@@ -1,14 +1,22 @@
 # -*- coding: utf-8 -*-
-require 'simplecov'
-require 'test/unit'
+# frozen_string_literal: true
 
-SimpleCov.start
+require 'bundler'
+
+begin
+  Bundler.setup(:default, :development)
+rescue Bundler::BundlerError => e
+  warn e.message
+  warn "Run `bundle install` to install missing gems"
+  exit e.status_code
+end
+require 'minitest/autorun'
 
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 require 'writeexcel'
 
-class Test::Unit::TestCase
+class Minitest::Test
   ###############################################################################
   #
   # Unpack the binary data into a format suitable for printing in tests.

data/test/test_00_IEEE_double.rb

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 require 'helper'
 
-class TC_BIFFWriter < Test::Unit::TestCase
+class TC_BIFFWriter < Minitest::Test
 
   def test_IEEE_double
     teststr = [1.2345].pack("d")

data/test/test_04_dimensions.rb

@@ -14,7 +14,7 @@
 require 'helper'
 require 'stringio'
 
-class TC_dimensions < Test::Unit::TestCase
+class TC_dimensions < Minitest::Test
 
   def setup
     @workbook            = WriteExcel.new(StringIO.new)
@@ -25,11 +25,11 @@ class TC_dimensions < Test::Unit::TestCase
   end
 
   def teardown
-    if @workbook.instance_variable_get(:@filehandle)
-      @workbook.instance_variable_get(:@filehandle).close(true)
+    if @workbook.instance_variable_get("@filehandle")
+      @workbook.instance_variable_get("@filehandle").close(true)
     end
-    if @worksheet.instance_variable_get(:@filehandle)
-      @worksheet.instance_variable_get(:@filehandle).close(true)
+    if @worksheet.instance_variable_get("@filehandle")
+      @worksheet.instance_variable_get("@filehandle").close(true)
     end
   end
 
@@ -386,7 +386,7 @@ class TC_dimensions < Test::Unit::TestCase
   end
 
   def test_merge_range
-    formula = @worksheet.__send__("store_formula", '=A1 * 3 + 50')
+    @worksheet.__send__("store_formula", '=A1 * 3 + 50')
     @worksheet.merge_range('C6:E8', 'Test', @format)
 
     data     = @worksheet.__send__("store_dimensions")