feature: Add monkey-patch to allow input of serial dates

PR open at:
gimite/google-drive-ruby#379

Also
- Update the spreadsheet mock with a way to input serial dates.
- Remove core_extensions from coverage report.

Author: Drowze

lib/core_extensions.rb

@@ -28,9 +28,37 @@ def rows(skip = 0)
       def rows_with_numerics(skip = 0)
         rows(skip) { |row, col| numeric_value(row, col) || self[row, col] }
       end
+
+      def []=(*args) # rubocop:disable all
+        (row, col) = parse_cell_args(args[0...-1])
+        value = args[-1]
+
+        reload_cells unless @cells
+        if value.is_a?(Numeric)
+          @numeric_values[[row, col]] = value
+        else
+          value = value.to_s
+          validate_cell_value(value)
+          @numeric_values[[row, col]] = nil
+        end
+
+        @cells[[row, col]] = value
+        @input_values[[row, col]] = value
+        @modified.add([row, col])
+        self.max_rows = row if row > @max_rows
+        self.max_cols = col if col > @max_cols
+        if value == ''
+          @num_rows = nil
+          @num_cols = nil
+        else
+          @num_rows = row if @num_rows && row > @num_rows
+          @num_cols = col if @num_cols && col > @num_cols
+        end
+      end
     end
   end
 end
 
 # https://github.com/gimite/google-drive-ruby/issues/378
+# https://github.com/gimite/google-drive-ruby/issues/380
 GoogleDrive::Worksheet.prepend AspireBudget::CoreExtensions::Worksheet

lib/utils.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'date'
-require 'configuration'
 
 module AspireBudget
   module Utils

spec/spec_helper.rb

@@ -5,13 +5,14 @@
 if ENV['CI']
   require 'simplecov'
   SimpleCov.start do
+    add_filter 'lib/core_extensions.rb'
     add_filter %r{^/spec/}
   end
 end
 
 require 'pry'
 require 'support/google_drive_mock'
-require 'support/spreadsheet_version_helper'
+require 'support/spreadsheet_mock_helpers'
 require 'aspire_budget'
 
 RSpec.configure do |config|

spec/support/google_drive_mock.rb

@@ -86,10 +86,11 @@ def [](*args)
         else
           args
         end
-      rows[row - 1][col - 1].to_s
+
+      rows[row - 1][col - 1]
     end
 
-    def []=(row, col, value) # rubocop:disable Metrics/MethodLength
+    def []=(row, col, value) # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
       @dirty = true
 
       if row > num_rows
@@ -104,13 +105,18 @@ def []=(row, col, value) # rubocop:disable Metrics/MethodLength
         end
       end
 
+      value = value.to_s unless value.is_a?(Numeric)
+      value = treat_value(row, col, value) || value
+
       @rows[row - 1][col - 1] = value
     end
 
+    def treat_value(row, col, value); end
+
     # assumes that "," is not a decimal separator
     # assumes date format as defined above
     def numeric_value(*args)
-      value = self[*args]
+      value = self[*args].to_s
       if value.match?(CURRENCY_REGEX)
         value.gsub(/[€$,]/, '').to_f
       elsif value.match?(%r{\d+/\d+/\d+})

spec/support/spreadsheet_mock_helpers.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require 'utils'
+
+module SpreadsheetMockHelpers
+  def use_spreadsheet_version(version)
+    AspireBudget.configure do |config|
+      config.session = GoogleDrive.from_config('foo')
+      config.spreadsheet_key = version
+    end
+  end
+
+  # When inputing into a numeric cell, mock it first so we'll try to coerce its
+  # contents.
+  #
+  # @param currency [Array] currency cells ([row, col] format)
+  # @param date [Array] date cels ([row, col] format)
+  def treat_cells_as(currency: [], date: [])
+    allow_any_instance_of(GoogleDriveMock::Worksheet)
+      .to receive(:treat_value) do |_obj, row, col, value|
+        if date.include?([row, col])
+          AspireBudget::Utils.parse_date(Float(value)).strftime('%d/%-m/%y')
+        elsif currency.include?([row, col])
+          value = format('%<value>.2f', value: value)
+          "$#{format('%<value>.2f', value: value)}"
+        end
+      end
+  end
+end
+
+RSpec.configure do |config|
+  config.include SpreadsheetMockHelpers
+end

spec/support/spreadsheet_version_helper.rb

@@ -32,8 +32,12 @@
       end
 
       describe '#insert' do
+        before do
+          treat_cells_as(date: [[11, 2]], currency: [[11, 3]])
+        end
+
         let(:params) do
-          { date: '03/06/2020', amount: 120.43, from: 'Available to budget', to: 'Groceries', memo: 'ruby' }
+          { date: Date.parse('2020-06-03'), amount: 120.43, from: 'Available to budget', to: 'Groceries', memo: 'ruby' }
         end
         let(:new_record_attributes) do
           {
@@ -52,7 +56,7 @@
               .to change { subject.all.size }
               .by(1)
 
-            expect(new_record).to have_attributes(new_record_attributes)
+            expect(subject.all.last).to have_attributes(new_record_attributes)
 
             expect(subject).not_to be_dirty
           end
@@ -67,7 +71,7 @@
             allow(category_transfer)
               .to receive(:to_row)
               .with(%i[date amount from to memo])
-              .and_return(['03/06/20', '120.43', 'Available to budget', 'Groceries', 'ruby'])
+              .and_return([43_985.0, 120.43, 'Available to budget', 'Groceries', 'ruby'])
           end
 
           it 'inserts new data' do
@@ -76,7 +80,7 @@
               .to change { subject.all.size }
               .by(1)
 
-            expect(new_record).to have_attributes(new_record_attributes)
+            expect(subject.all.last).to have_attributes(new_record_attributes)
 
             expect(subject).not_to be_dirty
           end

spec/worksheets/category_transfers_spec.rb

@@ -56,9 +56,16 @@
       end
 
       describe '#insert' do
+        before do
+          treat_cells_as(
+            date: [[13, 2]],
+            currency: [[13, 3], [13, 4]]
+          )
+        end
+
         let(:params) do
           {
-            date: '03/06/2020',
+            date: Date.parse('2020-06-03'),
             outflow: 900,
             inflow: 800,
             category: 'Test',
@@ -101,7 +108,7 @@
             allow(transaction)
               .to receive(:to_row)
               .with(%i[date outflow inflow category account memo status])
-              .and_return(['03/06/20', '900.00', '800.00', 'Test', '💰 Checking', 'ruby', '✅'])
+              .and_return([43_985.0, 900.00, 800.00, 'Test', '💰 Checking', 'ruby', '✅'])
           end
 
           it 'inserts new data' do