reckon 0.9.3 → 0.9.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc9cf03c584638f19ffd111497105288a9002a71ec4dd649655a7fa1b8c6566c
-  data.tar.gz: 75453fe81b3f131a72dae8ab1f592cf0bcfb61fb380c5486cb984ffb504fa64b
+  metadata.gz: 648b51c8481100983f8d8eff4d9b86ea2ad1962e0908268a6450f618d90f3267
+  data.tar.gz: fb19e70b53da00299caa495e24d8fa01ac20686306e383f598a8888f5118dae3
 SHA512:
-  metadata.gz: 9cc31d230176f3d70609d7560ce5b948295d3eef33402ea12aa91749ecb48cf4c36e9885af4ea07e451c8641ae1acacd5421b093621cbde65f88da116992be30
-  data.tar.gz: d4e743fa0f3104bc4fe14774ea5e7ce8b7862c0865b725f6e7fac5f0a68607c7baa45c95a99d755c46a03dd90389d69e6addfd06c990103e4bc12c9959e6bf65
+  metadata.gz: 56a4558fe3aadfbf9a0a9f4399f8e70b549b90a08057235fafe5839203390c462b502511fc60b748490398c75f46b5a4d8ccbb4ac9b07b4e6058d72bf615f4ac
+  data.tar.gz: dc3acb9fdae98efc8cd4a8891037cc5bfff716b2c5db74f8b1f86edaa2da0e60843ffbcb0eeeef49358738e0a63bded19e2fa3f9955cce40566a3241cbc86aff

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    reckon (0.9.3)
+    reckon (0.9.4)
       chronic (>= 0.3.0)
       highline (>= 1.5.2)
       matrix (>= 0.4.2)

data/lib/reckon/date_column.rb

@@ -81,12 +81,15 @@ module Reckon
       date_score += 30 if entry =~ /^\d+[:\/\.-]\d+[:\/\.-]\d+([ :]\d+[:\/\.]\d+)?$/
       date_score += 10 if entry =~ /^\d+\[\d+:GMT\]$/i
 
+      # ruby 2.6.0 doesn't have Date::Error, but Date::Error is a subclass of
+      # ArgumentError
+      #
+      # Sometimes DateTime.parse can throw a RangeError
+      # See https://github.com/cantino/reckon/issues/126
       begin
         DateTime.parse(entry)
         date_score += 20
-      # ruby 2.6.0 doesn't have Date::Error, but Date::Error is a subclass of
-      # ArgumentError
-      rescue ArgumentError
+      rescue StandardError
         # we don't need do anything here since the column didn't parse as a date
         nil
       end

data/lib/reckon/version.rb

@@ -1,3 +1,3 @@
 module Reckon
-  VERSION = "0.9.3"
+  VERSION = "0.9.4"
 end

data/spec/reckon/date_column_spec.rb

@@ -1,59 +1,72 @@
 #!/usr/bin/env ruby
-# encoding: utf-8
+# frozen_string_literal: true
 
 require "spec_helper"
-require 'rubygems'
-require 'reckon'
-
-describe Reckon::DateColumn do
-  describe "initialize" do
-    it "should detect us and world time" do
-      Reckon::DateColumn.new( ["01/02/2013", "01/14/2013"] ).endian_precedence.should == [:middle]
-      Reckon::DateColumn.new( ["01/02/2013", "14/01/2013"] ).endian_precedence.should == [:little]
-    end
-    it "should set endian_precedence to default when date format cannot be misinterpreted" do
-      Reckon::DateColumn.new( ["2013/01/02"] ).endian_precedence.should == [:middle,:little]
-    end
-    it "should raise an error when in doubt" do
-      expect{ Reckon::DateColumn.new( ["01/02/2013", "01/03/2013"] )}.to raise_error( StandardError )
-    end
-  end
-  describe "for" do
-    it "should detect the date" do
-      expect(Reckon::DateColumn.new(%w[13/12/2013]).for(0))
-        .to eq(Date.new(2013, 12, 13))
-      expect(Reckon::DateColumn.new(%w[01/14/2013]).for(0))
-        .to eq(Date.new(2013, 1, 14))
-      expect(Reckon::DateColumn.new(%w[13/12/2013 21/11/2013]).for(1))
-        .to eq(Date.new(2013, 11, 21))
-      expect(Reckon::DateColumn.new( ["2013-11-21"] ).for( 0 ))
-        .to eq(Date.new(2013, 11, 21))
+require "rubygems"
+require "reckon"
 
+# datecolumn specs
+module Reckon
+  describe DateColumn do
+    describe "#initialize" do
+      it "should detect us and world time" do
+        expect(DateColumn.new(%w[01/02/2013 01/14/2013]).endian_precedence)
+          .to eq [:middle]
+        expect(DateColumn.new(%w[01/02/2013 14/01/2013]).endian_precedence)
+          .to eq [:little]
+      end
+      it "should set endian_precedence to default when date format cannot be misinterpreted" do
+        expect(DateColumn.new(["2013/01/02"]).endian_precedence)
+          .to eq %i[middle little]
+      end
+      it "should raise an error when in doubt" do
+        expect { DateColumn.new(["01/02/2013", "01/03/2013"]) }
+          .to raise_error(StandardError)
+      end
     end
 
-    it "should correctly use endian_precedence" do
-      expect(Reckon::DateColumn.new(%w[01/02/2013 01/14/2013]).for(0))
-        .to eq(Date.new(2013, 1, 2))
-      expect(Reckon::DateColumn.new(%w[01/02/2013 14/01/2013]).for(0))
-        .to eq(Date.new(2013, 2, 1))
-    end
-  end
+    describe "#for" do
+      it "should detect the date" do
+        expect(DateColumn.new(%w[13/12/2013]).for(0)).to eq(Date.new(2013, 12, 13))
+        expect(DateColumn.new(%w[01/14/2013]).for(0)).to eq(Date.new(2013, 1, 14))
+        expect(DateColumn.new(%w[13/12/2013 21/11/2013]).for(1))
+          .to eq(Date.new(2013, 11, 21))
+        expect(DateColumn.new(["2013-11-21"]).for(0)).to eq(Date.new(2013, 11, 21))
+      end
 
-  describe "#pretty_for" do
-    it 'should use ledger_date_format' do
-      expect(Reckon::DateColumn.new(%w[13/02/2013], {ledger_date_format: '%d/%m/%Y'}).pretty_for(0))
-               .to eq('13/02/2013')
+      it "should correctly use endian_precedence" do
+        expect(DateColumn.new(%w[01/02/2013 01/14/2013]).for(0))
+          .to eq(Date.new(2013, 1, 2))
+        expect(DateColumn.new(%w[01/02/2013 14/01/2013]).for(0))
+          .to eq(Date.new(2013, 2, 1))
+      end
     end
 
-    it 'should default to is' do
-      expect(Reckon::DateColumn.new(%w[13/12/2013]).pretty_for(0))
-        .to eq('2013-12-13')
+    describe "#pretty_for" do
+      it "should use ledger_date_format" do
+        expect(
+          DateColumn.new(["13/02/2013"],
+                         { ledger_date_format: "%d/%m/%Y" }).pretty_for(0)
+        )
+          .to eq("13/02/2013")
+      end
+
+      it "should default to is" do
+        expect(DateColumn.new(["13/12/2013"]).pretty_for(0))
+          .to eq("2013-12-13")
+      end
     end
-  end
 
-  describe "#likelihood" do
-    it "should prefer numbers that looks like dates" do
-      expect(Reckon::DateColumn.likelihood("123456789")).to be < Reckon::DateColumn.likelihood("20160102")
+    describe "#likelihood" do
+      it "should prefer numbers that looks like dates" do
+        expect(DateColumn.likelihood("123456789"))
+          .to be < DateColumn.likelihood("20160102")
+      end
+
+      # See https://github.com/cantino/reckon/issues/126
+      it "Issue #126 - it shouldn't fail on invalid dates" do
+        expect(DateColumn.likelihood("303909302970-07-2023")).to be > 0
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reckon
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.4
 platform: ruby
 authors:
 - Andrew Cantino
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-25 00:00:00.000000000 Z
+date: 2023-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec