excel_utils 1.0.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 13a477a8679749794ff54f4bb0a76c653b354be459d50ed1d7c89a6de8a32509
-  data.tar.gz: 920a86b0b1fb587105d7fc78609cf5bdd98a231aa6bee22726ebdfc090b3daca
+  metadata.gz: 70807b62912a2e82da93e85e0275b6f3f6b6a8f010f70a7924546f31f0d93087
+  data.tar.gz: 6b78f90fd4174174044c9464299bf645bc13619251b1fde6f0ac9079a00b6d14
 SHA512:
-  metadata.gz: cfb43adfda51aa543e1dd13e00f952b9eb0d2c9cec9dd7b41119eea1b2e0b2db433e47040c224dda7069f0ff0ec0b6d17f43f215f50e60a66db4545ecc740f5a
-  data.tar.gz: c1f424a05025b504b92cd5bf5b67f53c60129ad1973045daf263acf837301e61cc2681a9ce87a3ad90ba20e82a8ac1ef51891e9b2f6d387f4c3756fa6145a74f
+  metadata.gz: c8619bbea23bbad8b25c4014e0ead0bb8b81d901dce0c0e355e47b3b1fd2742f7c307d25980afc7d5066515ef33547d89d8c71df586ffe06bcdd3bc5dc6f9de4
+  data.tar.gz: 98a84f2fa77259e4ce3bbb691d267d4427e7e887eb0a41385ee7b82e59b7e1e6d06e1da333a257f3cdc9c54768a530c7c709608c3e8b89726c89ae61185b46fa

data/excel_utils.gemspec

@@ -19,9 +19,10 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'inflecto', '~> 0.0'
-  spec.add_runtime_dependency 'roo', '~> 2.7'
-  spec.add_runtime_dependency 'roo-xls', '~> 1.1'
+  spec.add_runtime_dependency 'roo', '~> 2.8'
+  spec.add_runtime_dependency 'roo-xls', '~> 1.2'
   spec.add_runtime_dependency 'write_xlsx', '~> 0.85'
+  spec.add_runtime_dependency 'nesquikcsv', '~> 0.1'
 
   spec.add_development_dependency 'rake', '~> 12.0'
   spec.add_development_dependency 'minitest', '~> 5.0', '< 5.11'

data/lib/excel_utils.rb

@@ -4,20 +4,30 @@ require 'roo'
 require 'roo-xls'
 require 'write_xlsx'
 require 'inflecto'
+require 'nesquikcsv'
 
 require_relative 'excel_utils/version'
-require_relative 'excel_utils/workbook'
-require_relative 'excel_utils/sheet'
+require_relative 'excel_utils/workbooks/csv'
+require_relative 'excel_utils/workbooks/excel'
+require_relative 'excel_utils/sheets/base'
+require_relative 'excel_utils/sheets/csv'
+require_relative 'excel_utils/sheets/excel'
+require_relative 'excel_utils/sheets/excel_stream'
 require_relative 'excel_utils/writer'
 
 module ExcelUtils
-  
+
   def self.read(filename, **options)
-    Workbook.new filename, **options
+    extension = options.fetch(:extension, File.extname(filename)[1..-1])
+    if extension == 'csv'
+      Workbooks::CSV.new(filename, **options)
+    else
+      Workbooks::Excel.new(filename, **options)
+    end
   end
 
   def self.write(filename, data)
     Writer.write filename, data
   end
 
-end
+end

data/lib/excel_utils/sheets/base.rb

@@ -0,0 +1,41 @@
+module ExcelUtils
+  module Sheets
+    class Base
+
+      include Enumerable
+
+      attr_reader :name, :normalize_column_names
+
+      def initialize(name:, normalize_column_names: false)
+        @name = name
+        @normalize_column_names = normalize_column_names
+      end
+
+      def column_names
+        @column_names ||= normalize_column_names ? normalize_columns(first_row) : first_row
+      end
+
+      def each
+        if column_names.any?
+          each_row do |row|
+            break if empty_row? row
+            yield Hash[column_names.zip(row)]
+          end
+        end
+      end
+
+      private
+
+      def normalize_columns(names)
+        names.map do |name|
+          Inflecto.underscore(name.strip.gsub(' ', '_')).to_sym
+        end
+      end
+
+      def empty_row?(row)
+        row.all? { |cell| cell.to_s.strip.empty? }
+      end
+
+    end
+  end
+end

data/lib/excel_utils/sheets/csv.rb

@@ -0,0 +1,28 @@
+module ExcelUtils
+  module Sheets
+    class CSV < Base
+
+      def initialize(filename:, **options)
+        super(**options)
+        @filename = filename
+      end
+
+      private
+
+      attr_reader :filename
+
+      def first_row
+        NesquikCSV.open(filename) { |csv| csv.readline } || []
+      end
+
+      def each_row
+        first = true
+        NesquikCSV.foreach(filename) do |row|
+          yield row unless first
+          first = false
+        end
+      end
+
+    end
+  end
+end

data/lib/excel_utils/sheets/excel.rb

@@ -0,0 +1,35 @@
+module ExcelUtils
+  module Sheets
+    class Excel < Base
+
+      def initialize(spreadsheet:, **options)
+        super(**options)
+        @spreadsheet = spreadsheet
+      end
+
+      private
+
+      attr_reader :spreadsheet
+
+      def first_row
+        with_sheet do |sheet|
+          sheet.first_row ? sheet.row(sheet.first_row) : []
+        end
+      end
+
+      def each_row
+        with_sheet do |sheet|
+          (sheet.first_row + 1).upto(sheet.last_row) do |i|
+            yield sheet.row(i)
+          end
+        end
+      end
+
+      def with_sheet
+        yield spreadsheet.sheet(name)
+      end
+
+    end
+
+  end
+end

data/lib/excel_utils/sheets/excel_stream.rb

@@ -0,0 +1,35 @@
+module ExcelUtils
+  module Sheets
+    class ExcelStream < Base
+
+      def initialize(spreadsheet:, **options)
+        super(**options)
+        @spreadsheet = spreadsheet
+      end
+
+      private
+
+      attr_reader :spreadsheet
+
+      def first_row
+        row = sheet.each_row_streaming(pad_cells: true, max_rows: 0).first || []
+        normalize_row row
+      end
+
+      def each_row
+        sheet.each_row_streaming(pad_cells: true, offset: 1) do |row|
+          yield normalize_row(row)
+        end
+      end
+
+      def normalize_row(row)
+        row.map { |cell| cell ? cell.value : cell }
+      end
+
+      def sheet
+        spreadsheet.sheet name
+      end
+
+    end
+  end
+end

data/lib/excel_utils/version.rb

@@ -1,3 +1,3 @@
 module ExcelUtils
-  VERSION = '1.0.0'
-end
+  VERSION = '1.3.0'
+end

data/lib/excel_utils/workbooks/csv.rb

@@ -0,0 +1,36 @@
+module ExcelUtils
+  module Workbooks
+    class CSV
+
+      SHEET_NAME = 'default'.freeze
+
+      attr_reader :filename, :normalize_column_names
+
+      def initialize(filename, normalize_column_names: false)
+        @filename = filename
+        @normalize_column_names = normalize_column_names
+
+        @sheet = Sheets::CSV.new name: SHEET_NAME,
+                                 normalize_column_names: normalize_column_names,
+                                 filename: filename
+      end
+
+      def sheets
+        [sheet]
+      end
+
+      def [](sheet_name)
+        sheet_name == SHEET_NAME ? sheet : nil
+      end
+
+      def to_h
+        {SHEET_NAME => sheet.to_a}
+      end
+
+      private
+
+      attr_reader :sheet
+
+    end
+  end
+end

data/lib/excel_utils/workbooks/excel.rb

@@ -0,0 +1,41 @@
+module ExcelUtils
+  module Workbooks
+    class Excel
+
+      attr_reader :filename, :normalize_column_names
+
+      def initialize(filename, normalize_column_names: false, extension: nil)
+        @filename = filename
+        @normalize_column_names = normalize_column_names
+        @spreadsheet = Roo::Spreadsheet.open filename, extension: extension
+      end
+
+      def sheets
+        @sheets ||= spreadsheet.sheets.map do |name|
+          sheet_class.new name: name,
+                          normalize_column_names: normalize_column_names,
+                          spreadsheet: spreadsheet
+        end
+      end
+
+      def [](sheet_name)
+        sheets.detect { |sheet| sheet.name == sheet_name }
+      end
+
+      def to_h
+        sheets.each_with_object({}) do |sheet, hash|
+          hash[sheet.name] = sheet.to_a
+        end
+      end
+
+      private
+
+      attr_reader :spreadsheet
+
+      def sheet_class
+        @sheet_class ||= spreadsheet.respond_to?(:each_row_streaming) ? Sheets::ExcelStream : Sheets::Excel
+      end
+
+    end
+  end
+end

data/lib/excel_utils/writer.rb

@@ -1,20 +1,24 @@
 module ExcelUtils
   class Writer
 
-    TIME_FORMAT = '%Y-%m-%dT%H:%M:%S'
+    DEFAULT_SHEET_NAME = 'Sheet1'.freeze
+
+    TIME_FORMAT = '%Y-%m-%dT%H:%M:%S'.freeze
 
     EXCEL_FORMATS = {
       date: 'yyyy-mm-dd',
       date_time: 'yyyy-mm-dd hh:mm:ss'
-    }
+    }.freeze
 
     class << self
-    
+
       def write(filename, data)
         workbook = WriteXLSX.new filename, strings_to_urls: false
 
         formats = add_formats workbook
 
+        data = {DEFAULT_SHEET_NAME => data} if data.is_a? Array
+
         data.each do |sheet_name, sheet_data|
           add_sheet workbook, sheet_name, sheet_data, formats
         end
@@ -34,18 +38,23 @@ module ExcelUtils
         sheet = workbook.add_worksheet sheet_name
 
         if sheet_data.any?
-          header = sheet_data.first.keys
+          header = sheet_data.flat_map(&:keys).uniq
           sheet.write_row 0, 0, header.map(&:to_s)
 
           sheet_data.each_with_index do |row, r|
-            header.each_with_index do |column, c|
+            row_index = r + 1
+            header.each_with_index do |column, col_index|
               if row[column]
-                if row[column].respond_to? :to_time
+                if row[column].is_a?(String) || row[column].is_a?(Array)
+                  sheet.write_string row_index, col_index, row[column]
+
+                elsif row[column].respond_to? :to_time
                   time = row[column].to_time
                   type = date?(time) ? :date : :date_time
-                  sheet.write_date_time r + 1, c, time.to_time.strftime(TIME_FORMAT), formats[type]
+                  sheet.write_date_time row_index, col_index, time.to_time.strftime(TIME_FORMAT), formats[type]
+
                 else
-                  sheet.write r + 1, c, row[column]
+                  sheet.write row_index, col_index, row[column]
                 end
               end
             end

data/spec/minitest_helper.rb

@@ -3,4 +3,27 @@ require 'minitest/autorun'
 require 'minitest/colorin'
 require 'pry-nav'
 
-require 'excel_utils'
+require 'excel_utils'
+
+RESOURCES_PATH = File.expand_path '../resources', __FILE__
+
+TMP_PATH = File.expand_path '../../tmp', __FILE__
+FileUtils.mkpath TMP_PATH unless Dir.exists? TMP_PATH
+
+class Minitest::Test
+
+  def resource_path(relative_path)
+    File.join RESOURCES_PATH, relative_path
+  end
+
+  def tmp_path(relative_path)
+    File.join TMP_PATH, relative_path
+  end
+
+end
+
+Minitest.after_run do
+  Dir.glob(File.join(TMP_PATH, '*')).each do |filename|
+    FileUtils.remove_file filename
+  end
+end

data/spec/read_spec.rb

@@ -2,104 +2,151 @@ require 'minitest_helper'
 
 describe ExcelUtils, 'Read' do
 
-  def expected_rows(sheet)
-    rows_by_sheet[sheet.name].map { |r| Hash[columns_by_sheet[sheet.name].zip(r)] }
-  end
+  Dir.glob(File.join(RESOURCES_PATH, 'basic.*')).each do |filename|
 
-  let :rows_by_sheet do
-    {
-      'Sheet1' => [
-        [1.0, 'some text'],
-        [2.0, 1.35],
-        [3.0, Date.parse('2019-08-17')],
-        [4.0, nil]
-      ],
-      'Sheet2' => [
-        [123.0, 'Text 1'],
-        [456.0, 'Text 2']
-      ],
-      'Sheet3' => []
-    }
-  end
+    [true, false].each do |normalize_column_names|
+
+      describe File.basename(filename), "Workbook (normalize_column_names: #{normalize_column_names})" do
+
+        let(:workbook) { ExcelUtils.read filename, normalize_column_names: normalize_column_names }
+
+        let(:sheet) { workbook.sheets.first }
 
-  ['xls', 'xlsx'].each do |extension|
+        let(:csv?) { File.extname(filename) == '.csv' }
 
-    describe extension do
-  
-      let(:filename) { File.expand_path "../sample.#{extension}", __FILE__ }
+        let(:expected_sheet_name) { csv? ? 'default' : 'Sheet1' }
 
-      describe 'Original column names' do
+        let(:column_a) { normalize_column_names ? :column_a : 'Column A' }
+
+        let(:column_b) { normalize_column_names ? :column_b : 'Column B' }
+
+        let(:expected_columns) { [column_a, column_b] }
+
+        let :expected_rows do
+          if csv?
+            [
+              {column_a => '1', column_b => 'some text'},
+              {column_a => '2', column_b => '1,35'},
+              {column_a => '3', column_b => '17/08/2019'},
+              {column_a => '4', column_b => nil}
+            ]
+          else
+            [
+              {column_a => 1, column_b => 'some text'},
+              {column_a => 2, column_b => 1.35},
+              {column_a => 3, column_b => Date.parse('2019-08-17')},
+              {column_a => 4, column_b => nil}
+            ]
+          end
+        end
 
-        let(:workbook) { ExcelUtils.read filename }
-        
-        let :columns_by_sheet do
-          {
-            'Sheet1' => ['Column A', 'Column B'],
-            'Sheet2' => ['ID', 'Value'],
-            'Sheet3' => []
-          }
+        it 'filename' do
+          workbook.filename.must_equal filename
+        end
+
+        it 'normalize_column_names' do
+          workbook.normalize_column_names.must_equal normalize_column_names
+        end
+
+        it 'sheets' do
+          workbook.sheets.count.must_equal 1
+          workbook[workbook.sheets.first.name].must_equal sheet
         end
 
         it 'to_h' do
-          workbook.to_h.must_equal 'Sheet1' => expected_rows(workbook['Sheet1']),
-                                   'Sheet2' => expected_rows(workbook['Sheet2']),
-                                   'Sheet3' => []
+          workbook.to_h.must_equal workbook.sheets.first.name => sheet.to_a
         end
 
-        ['Sheet1', 'Sheet2', 'Sheet3'].each do |sheet_name|
+        describe 'Sheet' do
 
-          describe sheet_name do
+          it 'name' do
+            sheet.name.must_equal expected_sheet_name
+          end
 
-            let(:sheet) { workbook[sheet_name] }
+          it 'normalize_column_names' do
+            sheet.normalize_column_names.must_equal workbook.normalize_column_names
+          end
 
-            it 'Column names' do
-              sheet.column_names.must_equal columns_by_sheet[sheet_name]
-            end
+          it 'column_names' do
+            sheet.column_names.must_equal expected_columns
+          end
 
-            it 'Rows' do
-              sheet.to_a.must_equal expected_rows(sheet)
-            end
+          it 'count' do
+            sheet.count.must_equal 4
+          end
 
+          it 'to_a' do
+            sheet.to_a.must_equal expected_rows
           end
 
         end
 
       end
 
-      describe 'Normalized column names' do
+    end
+
+  end
 
-        let(:workbook) { ExcelUtils.read filename, normalize_column_names: true }
-        
-        let :columns_by_sheet do
-          {
-            'Sheet1' => [:column_a, :column_b],
-            'Sheet2' => [:id, :value],
-            'Sheet3' => []
-          }
-        end
+  it 'empty.csv' do
+    workbook = ExcelUtils.read resource_path('empty.csv')
 
-        ['Sheet1', 'Sheet2', 'Sheet3'].each do |sheet_name|
+    workbook.sheets.map(&:name).must_equal ['default']
 
-          describe sheet_name do
+    workbook['default'].column_names.must_equal []
+    workbook['default'].to_a.must_equal []
 
-            let(:sheet) { workbook[sheet_name] }
+    workbook.to_h.must_equal 'default' => []
+  end
 
-            it 'Column names' do
-              sheet.column_names.must_equal columns_by_sheet[sheet_name]
-            end
+  it 'only_headers.csv' do
+    workbook = ExcelUtils.read resource_path('only_headers.csv')
 
-            it 'Rows' do
-              sheet.to_a.must_equal expected_rows(sheet)
-            end
+    workbook.sheets.map(&:name).must_equal ['default']
 
-          end
+    workbook['default'].column_names.must_equal ['ID', 'Value']
+    workbook['default'].to_a.must_equal []
 
-        end
+    workbook.to_h.must_equal 'default' => []
+  end
 
-      end
+  it 'custom_extension.tmp' do
+    workbook = ExcelUtils.read resource_path('custom_extension.tmp'), extension: 'xlsx'
 
-    end
+    expected_rows = [
+      {'ID' => 1, 'Value' => 'Text 1'},
+      {'ID' => 2, 'Value' => 'Text 2'}
+    ]
+
+    workbook.sheets.map(&:name).must_equal ['Sheet1']
+
+    workbook['Sheet1'].column_names.must_equal ['ID', 'Value']
+    workbook['Sheet1'].to_a.must_equal expected_rows
+
+    workbook.to_h.must_equal 'Sheet1' => expected_rows
+  end
+
+  it 'multiple.xlsx' do
+    workbook = ExcelUtils.read resource_path('multiple.xlsx'), normalize_column_names: true
+
+    workbook.sheets.map(&:name).must_equal ['Sheet1', 'Sheet2', 'Sheet3']
+
+    expected_rows_sheet_1 = [
+      {id: 1, value: 'Text 1'},
+      {id: 2, value: 'Text 2'}
+    ]
+
+    workbook['Sheet1'].column_names.must_equal [:id, :value]
+    workbook['Sheet1'].to_a.must_equal expected_rows_sheet_1
+
+    workbook['Sheet2'].column_names.must_equal [:id, :value]
+    workbook['Sheet2'].to_a.must_equal []
+
+    workbook['Sheet3'].column_names.must_equal []
+    workbook['Sheet3'].to_a.must_equal []
 
+    workbook.to_h.must_equal 'Sheet1' => expected_rows_sheet_1,
+                             'Sheet2' => [],
+                             'Sheet3' => []
   end
 
 end

data/spec/resources/basic.csv

@@ -0,0 +1,5 @@
+Column A,Column B
+1,some text
+2,"1,35"
+3,17/08/2019
+4,

data/spec/resources/only_headers.csv

@@ -0,0 +1 @@
+ID,Value

data/spec/write_spec.rb

@@ -2,32 +2,95 @@ require 'minitest_helper'
 
 describe ExcelUtils do
 
-  let(:tmp_path) { File.expand_path '../../tmp', __FILE__ }
+  def assert_wrote(data, expected=data)
+    filename = tmp_path "#{SecureRandom.uuid}.xlsx"
 
-  let(:filename) { File.join tmp_path, "#{SecureRandom.uuid}.xlsx" }
+    ExcelUtils.write filename, data
+
+    workbook = ExcelUtils.read filename
+
+    expected = {'Sheet1' => expected} if expected.is_a?(Array)
+
+    workbook.to_h.must_equal expected
+  end
+
+  describe 'Single sheet' do
+
+    it 'Basic data' do
+      rows = [
+        {'column_a' => '1', 'column_b' => 'text 1'},
+        {'column_a' => '2.5', 'column_b' => nil, 'column_c' => 'text 2'},
+      ]
+
+      assert_wrote rows, rows.map { |r| {'column_c' => nil}.merge r }
+    end
+
+    it 'Symbolized column names' do
+      rows = [
+        {column_a: 'text 1'},
+        {column_a: 'text 2'},
+      ]
+
+      assert_wrote rows, rows.map { |r| {'column_a' => r[:column_a]} }
+    end
+
+    it 'Long values' do
+      long_url = 'https://external.xx.fbcdn.net/safe_image.php?d=AQCY1f77h1RtuFfa&w=720&h=720&url=https%3A%2F%2Fwww.mercadolibre.com%2Fjms%2Fmla%2Flgz%2Fbackground%2Fsession%2Farmor.c27f26204365785956aaf9b30c289d0d4b0a201b4b01dfc3d83162b82608973be9a82ac85c2097c3f23b522681adca38b143f5e1a10fda251b430051a1592bf19caba204c07179120c48d47a1ceb5a45.774f4b8036d4b909101bab92870d262a%3Fbackground%3Darmor.c27f26204365785956aaf9b30c289d0d4b0a201b4b01dfc3d83162b82608973be9a82ac85c2097c3f23b522681adca38b143f5e1a10fda251b430051a1592bf19caba204c07179120c48d47a1ceb5a45.774f4b8036d4b909101bab92870d262a%26message%3DeyJqc190eXBlIjoianNfY29va2llIiwidmFsdWUiOiJ4In0&cfs=1&_nc_hash=AQB9Sf1HyWCwCVhF'
+      long_text = 'AQCY1f77h1RtuFfa&w=720&h=720&url=https%3A%2F%2Fwww.mercadolibre.com%2Fjms%2Fmla%2Flgz%2Fbackground%2Fsession%2Farmor.c27f26204365785956aaf9b30c289d0d4b0a201b4b01dfc3d83162b82608973be9a82ac85c2097c3f23b522681adca38b143f5e1a10fda251b430051a1592bf19caba204c07179120c48d47a1ceb5a45.774f4b8036d4b909101bab92870d262a%3Fbackground%3Darmor.c27f26204365785956aaf9b30c289d0d4b0a201b4b01dfc3d83162b82608973be9a82ac85c2097c3f23b522681adca38b143f5e1a10fda251b430051a1592bf19caba204c07179120c48d47a1ceb5a45.774f4b8036d4b909101bab92870d262a%26message%3DeyJqc190eXBlIjoianNfY29va2llIiwidmFsdWUiOiJ4In0&cfs=1&_nc_hash=AQB9Sf1HyWCwCVhF'
+
+      rows = [
+        {'column_a' => long_url},
+        {'column_a' => long_text},
+      ]
+
+      assert_wrote rows
+    end
+
+    it 'Numbers' do
+      rows = [
+        {'column_a' => 1},
+        {'column_a' => 2.5},
+      ]
+
+      assert_wrote rows
+    end
+
+    it 'Date and DateTime' do
+      rows = [
+        {'column_a' => DateTime.parse('2019-05-25T16:30:00')},
+        {'column_a' => Date.parse('2019-07-09')},
+      ]
+
+      assert_wrote rows
+    end
+
+    it 'Objects' do
+      rows = [
+        {'column_a' => ['text', 1]},
+        {'column_a' => {key: 123}},
+        {'column_a' => Set.new(['text', 1])}
+      ]
+
+      assert_wrote rows, rows.map { |r| {'column_a' => r['column_a'].to_s} }
+    end
 
-  before do
-    FileUtils.rm_rf tmp_path
-    FileUtils.mkpath tmp_path
   end
 
-  it 'Write' do
+  it 'Multiple sheets' do
     data = {
-      'Sheet 1' => [
-        {'column_a' => 1.5, 'column_b' => 'text 1', 'column_c' => DateTime.parse('2019-05-25T16:30:00')},
-        {'column_b' => 'text 2', 'column_c' => Date.parse('2019-07-09'), 'column_a' => 2},
+      'strings' => [
+        {'column_a' => 'text 1', 'column_b' => 'text 2'},
+        {'column_a' => 'text 3', 'column_b' => 'text 4'}
       ],
-      'Sheet 2' => [
-        {'Column A' => 'Text A', 'Column B' => 'Text B'}
+      'numbers' => [
+        {'x' => 1, 'y' => 2},
+        {'x' => 3, 'y' => 4},
+        {'x' => 5, 'y' => 6}
       ],
-      'Sheet 3' => []
+      'empty' => [],
     }
 
-    ExcelUtils.write filename, data
-
-    workbook = ExcelUtils.read filename
-
-    workbook.to_h.must_equal data
+    assert_wrote data
   end
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: excel_utils
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Gabriel Naiman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-03 00:00:00.000000000 Z
+date: 2021-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: inflecto
@@ -30,28 +30,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: '2.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: '2.8'
 - !ruby/object:Gem::Dependency
   name: roo-xls
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: write_xlsx
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.85'
+- !ruby/object:Gem::Dependency
+  name: nesquikcsv
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -188,15 +202,26 @@ files:
 - Rakefile
 - excel_utils.gemspec
 - lib/excel_utils.rb
-- lib/excel_utils/sheet.rb
+- lib/excel_utils/sheets/base.rb
+- lib/excel_utils/sheets/csv.rb
+- lib/excel_utils/sheets/excel.rb
+- lib/excel_utils/sheets/excel_stream.rb
 - lib/excel_utils/version.rb
-- lib/excel_utils/workbook.rb
+- lib/excel_utils/workbooks/csv.rb
+- lib/excel_utils/workbooks/excel.rb
 - lib/excel_utils/writer.rb
 - spec/coverage_helper.rb
 - spec/minitest_helper.rb
 - spec/read_spec.rb
-- spec/sample.xls
-- spec/sample.xlsx
+- spec/resources/basic.csv
+- spec/resources/basic.ods
+- spec/resources/basic.xls
+- spec/resources/basic.xlsm
+- spec/resources/basic.xlsx
+- spec/resources/custom_extension.tmp
+- spec/resources/empty.csv
+- spec/resources/multiple.xlsx
+- spec/resources/only_headers.csv
 - spec/write_spec.rb
 homepage: https://github.com/gabynaiman/excel_utils
 licenses:
@@ -225,6 +250,13 @@ test_files:
 - spec/coverage_helper.rb
 - spec/minitest_helper.rb
 - spec/read_spec.rb
-- spec/sample.xls
-- spec/sample.xlsx
+- spec/resources/basic.csv
+- spec/resources/basic.ods
+- spec/resources/basic.xls
+- spec/resources/basic.xlsm
+- spec/resources/basic.xlsx
+- spec/resources/custom_extension.tmp
+- spec/resources/empty.csv
+- spec/resources/multiple.xlsx
+- spec/resources/only_headers.csv
 - spec/write_spec.rb

data/lib/excel_utils/sheet.rb

@@ -1,50 +0,0 @@
-module ExcelUtils
-  class Sheet
-
-    include Enumerable
-    
-    attr_reader :name, :normalize_column_names
-    
-    def initialize(name, spreadsheet, normalize_column_names: false)
-      @name = name
-      @spreadsheet = spreadsheet
-      @normalize_column_names = normalize_column_names
-    end
-
-    def column_names
-      @column_names ||= begin
-        if sheet.first_row
-          first_row = sheet.row sheet.first_row
-          normalize_column_names ? first_row.map { |n| Inflecto.underscore(n.strip.gsub(' ', '_')).to_sym } : first_row
-        else
-          []
-        end
-      end
-    end
-
-    def each(&block)
-      rows.each(&block)
-    end
-
-    private
-
-    attr_reader :spreadsheet
-
-    def sheet
-      spreadsheet.sheet name
-    end
-
-    def rows
-      @rows ||= begin
-        if sheet.first_row
-          sheet.to_a[1..-1].map do |row| 
-            Hash[column_names.zip(row)] 
-          end
-        else
-          []
-        end
-      end
-    end
-
-  end
-end

data/lib/excel_utils/workbook.rb

@@ -1,33 +0,0 @@
-module ExcelUtils
-  class Workbook
-
-    attr_reader :filename, :normalize_column_names
-    
-    def initialize(filename, normalize_column_names: false)
-      @filename = filename
-      @normalize_column_names = normalize_column_names
-      @spreadsheet = Roo::Spreadsheet.open filename
-    end
-
-    def sheets
-      @sheets ||= spreadsheet.sheets.map do |name| 
-        Sheet.new name, spreadsheet, normalize_column_names: normalize_column_names
-      end
-    end
-
-    def [](sheet_name)
-      sheets.detect { |sheet| sheet.name == sheet_name }
-    end
-
-    def to_h
-      sheets.each_with_object({}) do |sheet, hash|
-        hash[sheet.name] = sheet.to_a
-      end
-    end
-
-    private
-
-    attr_reader :spreadsheet
-
-  end
-end