ndr_support 5.7.1 → 5.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e95a05bf2c1dd28e2bfe10716b35e761c5db50ac58947f168585692b588c499
-  data.tar.gz: 49ed22feef4c4a22ea2094883263eaf81d24a497cff6d0977fa23c6889518e77
+  metadata.gz: 528b4fe69aceac0e252cab457de3d2b01436ef6f188308858e014a4ceee43299
+  data.tar.gz: c15d6e7487920ccf74d4ff1711cbc062346de5c6457c7d9552b216536428e5e3
 SHA512:
-  metadata.gz: aaa5b6b7e04962f01738e60815f338c3c54ca38c5862ab5b5a373a9f9628712670d75db1f7bae9e2e2830e6cf511a36b9bc0ec304822352e0e64b94caa375160
-  data.tar.gz: e204589b6cd0951139bea6d2d96d23d4ec0bfe7d0c2be501bdd0ed1668802d4dbb3a7866e73242d004bce25cfc9d8407b934051a3c5afe29b101fe813cee49d5
+  metadata.gz: 5759f651f4ddea307afb890f7163972da66d3704b8d904b77ff60a88ffd77704419a1347ee96425c3fe5a60f1c4e558cc69c9411630d6e6121e54ad03148b842
+  data.tar.gz: 5837a1948066929be0486edc36f3571bffefb7b086f88a7d2f9c1bb047a3df1970221d97e683a3dcb89517f17822d60967f2f25381b85a18384e09d444860b7f

data/CHANGELOG.md

@@ -1,6 +1,27 @@
 ## [Unreleased]
 *no unreleased changes*
 
+## 5.8.3 / 2021-11-25
+## Fixed
+* remove circular dependencies between Ourdate and Daterange
+
+## 5.8.2 / 2021-10-11
+## Fixed
+* fix issue with blank date causing error for dateranges
+
+## 5.8.1 / 2021-10-11
+### Patched
+* Added fix for failing threat scanner tests (#22)
+
+## 5.8.0 / 2021-04-19
+## Added
+
+* Add ability to disable date reversing in Daterange
+
+## Fixed
+* Support Ruby 2.6-3.0.
+* Fix ruby warnings
+
 ## 5.7.1 / 2021-01-03
 ## Fixed
 * Postcodeize old Newport postcodes

data/README.md

@@ -1,4 +1,4 @@
-# NdrSupport [![Build Status](https://travis-ci.org/PublicHealthEngland/ndr_support.svg?branch=master)](https://travis-ci.org/PublicHealthEngland/ndr_support) [![Gem Version](https://badge.fury.io/rb/ndr_support.svg)](https://badge.fury.io/rb/ndr_support)
+# NdrSupport [![Build Status](https://github.com/publichealthengland/ndr_support/workflows/Test/badge.svg)](https://github.com/publichealthengland/ndr_support/actions?query=workflow%3Atest) [![Gem Version](https://badge.fury.io/rb/ndr_support.svg)](https://rubygems.org/gems/ndr_support)
 
 This is the Public Health England (PHE) National Disease Registers (NDR) Support ruby gem, providing:
 

data/code_safety.yml

@@ -1,5 +1,17 @@
 ---
 file safety:
+  ".github/CODEOWNERS":
+    comments: 
+    reviewed_by: ollietulloch
+    safe_revision: 993f7719b57eb6ddd8dffa7230ed9ffc69f48749
+  ".github/workflows/lint.yml":
+    comments: 
+    reviewed_by: ollietulloch
+    safe_revision: 993f7719b57eb6ddd8dffa7230ed9ffc69f48749
+  ".github/workflows/test.yml":
+    comments: 
+    reviewed_by: ollietulloch
+    safe_revision: 993f7719b57eb6ddd8dffa7230ed9ffc69f48749
   ".gitignore":
     comments: 
     reviewed_by: josh.pencheon
@@ -8,16 +20,10 @@ file safety:
     comments: 
     reviewed_by: josh.pencheon
     safe_revision: a02f14cbf1fc36fbaf820365fdedfdc9af326db1
-  ".travis.yml":
-    comments: 'diff failed; reviewed whole file. Revision c88f5ac8c12fd657bb142144e0d3afaa7f9bc361:
-      Slack notification secret is public/private key encrypted, so not a disclosure
-      risk. As a secondary protection, it''s excluded from the .gem file by ndr_support.gemspec'
-    reviewed_by: josh.pencheon
-    safe_revision: 0cfcfac5ddcd721a67dcd23f04cbf6bc2278b922
   CHANGELOG.md:
     comments: 
     reviewed_by: brian.shand
-    safe_revision: f7cfb534917c991a6e1811f1eb4cff2083c141d2
+    safe_revision: efa8b316f7ee8169baf8b74b4d8e9d3bf02499d5
   CODE_OF_CONDUCT.md:
     comments: 
     reviewed_by: timgentry
@@ -36,8 +42,8 @@ file safety:
     safe_revision: 90328cca8494539257e192a63b240a91c89f0616
   README.md:
     comments: 
-    reviewed_by: josh.pencheon
-    safe_revision: 2685f35c907af6968a69eabb5ab9424b490d0f40
+    reviewed_by: ollietulloch
+    safe_revision: 993f7719b57eb6ddd8dffa7230ed9ffc69f48749
   Rakefile:
     comments: 
     reviewed_by: timgentry
@@ -68,8 +74,8 @@ file safety:
     safe_revision: c1158eccb843b637b4dd82815dc7c081f12b93e4
   lib/ndr_support/daterange.rb:
     comments: 
-    reviewed_by: ollietulloch
-    safe_revision: 9a4b8e53f477725c2c2be5000b3c9e66f07d7389
+    reviewed_by: brian.shand
+    safe_revision: 9d19593cc45a9f1724c4b34a1a05a5ff48003084
   lib/ndr_support/hash.rb:
     comments: 
     reviewed_by: pauleves
@@ -78,10 +84,6 @@ file safety:
     comments: 
     reviewed_by: josh.pencheon
     safe_revision: 2685f35c907af6968a69eabb5ab9424b490d0f40
-  lib/ndr_support/integer/julian_date_conversions.rb:
-    comments: 
-    reviewed_by: josh.pencheon
-    safe_revision: 2685f35c907af6968a69eabb5ab9424b490d0f40
   lib/ndr_support/integer/rounding.rb:
     comments: 
     reviewed_by: josh.pencheon
@@ -100,12 +102,16 @@ file safety:
     safe_revision: 50a4293953f9ae227823c390b069a3ed4825074c
   lib/ndr_support/ourdate.rb:
     comments: 
-    reviewed_by: pauleves
-    safe_revision: a18fb5c6c360806a177a9c5584458586e90b3530
-  lib/ndr_support/ourtime.rb:
+    reviewed_by: brian.shand
+    safe_revision: 9d19593cc45a9f1724c4b34a1a05a5ff48003084
+  lib/ndr_support/ourdate/build_datetime.rb:
     comments: 
     reviewed_by: brian.shand
-    safe_revision: 73527378c53b7ac00d1037155572fe3f01324992
+    safe_revision: 9d19593cc45a9f1724c4b34a1a05a5ff48003084
+  lib/ndr_support/ourtime.rb:
+    comments: 
+    reviewed_by: ollietulloch
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/password.rb:
     comments: with input from brian.shand
     reviewed_by: josh.pencheon
@@ -124,8 +130,8 @@ file safety:
     safe_revision: 2685f35c907af6968a69eabb5ab9424b490d0f40
   lib/ndr_support/safe_path.rb:
     comments: 
-    reviewed_by: timgentry
-    safe_revision: 4492da77f6a8b678c40dd571fdd46fd936052cca
+    reviewed_by: ollietulloch
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/string/cleaning.rb:
     comments: 
     reviewed_by: brianshand
@@ -133,7 +139,7 @@ file safety:
   lib/ndr_support/string/conversions.rb:
     comments: 
     reviewed_by: ollietulloch
-    safe_revision: 9a4b8e53f477725c2c2be5000b3c9e66f07d7389
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/tasks.rb:
     comments: 
     reviewed_by: timgentry
@@ -148,20 +154,20 @@ file safety:
     safe_revision: 8e024a4bacfd36e172aad4b800f1960faae0b4b6
   lib/ndr_support/utf8_encoding/control_characters.rb:
     comments: 
-    reviewed_by: josh.pencheon
-    safe_revision: 8e024a4bacfd36e172aad4b800f1960faae0b4b6
+    reviewed_by: ollietulloch
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/utf8_encoding/force_binary.rb:
     comments: 
-    reviewed_by: timgentry
-    safe_revision: 29595e6431587ff9b7db6e3ad3abbb3577bff99c
+    reviewed_by: ollietulloch
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/utf8_encoding/object_support.rb:
     comments: 
-    reviewed_by: timgentry
-    safe_revision: f7adf44fc2772e1926df37abfd9041d41c303328
+    reviewed_by: ollietulloch
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   lib/ndr_support/version.rb:
     comments: 
     reviewed_by: brian.shand
-    safe_revision: f7cfb534917c991a6e1811f1eb4cff2083c141d2
+    safe_revision: efa8b316f7ee8169baf8b74b4d8e9d3bf02499d5
   lib/ndr_support/working_days.rb:
     comments: 
     reviewed_by: josh.pencheon
@@ -172,8 +178,8 @@ file safety:
     safe_revision: 29595e6431587ff9b7db6e3ad3abbb3577bff99c
   ndr_support.gemspec:
     comments: 
-    reviewed_by: brianshand
-    safe_revision: befa1d4f27448d5be4e133f442c2b5fbac7d8f25
+    reviewed_by: ollietulloch
+    safe_revision: 993f7719b57eb6ddd8dffa7230ed9ffc69f48749
   test/array_test.rb:
     comments: 
     reviewed_by: timgentry
@@ -185,7 +191,7 @@ file safety:
   test/daterange_test.rb:
     comments: 
     reviewed_by: ollietulloch
-    safe_revision: 9a4b8e53f477725c2c2be5000b3c9e66f07d7389
+    safe_revision: ccca1dc67e3742efb154e01e01243e7b38180936
   test/hash_test.rb:
     comments: 
     reviewed_by: timgentry
@@ -245,15 +251,15 @@ file safety:
   test/string/conversions_test.rb:
     comments: 
     reviewed_by: ollietulloch
-    safe_revision: 9a4b8e53f477725c2c2be5000b3c9e66f07d7389
+    safe_revision: '09b3f97fb4c1afc88d0d070c39796b1f60c39d25'
   test/test_helper.rb:
     comments: 
     reviewed_by: josh.pencheon
     safe_revision: 34ae80f2952f0bfd18688541862774cbaaa6c5f5
   test/threat_scanner_test.rb:
     comments: 
-    reviewed_by: josh.pencheon
-    safe_revision: ed8ff421cc9353456af13a37049a07ce2545aac1
+    reviewed_by: ollietulloch
+    safe_revision: fbb6ef9804040aa2e38257d9f5c487bde18a490d
   test/utf8_encoding/control_characters_test.rb:
     comments: 
     reviewed_by: timgentry

data/lib/ndr_support/daterange.rb

@@ -1,6 +1,6 @@
 require 'active_support/core_ext/enumerable'
 require 'active_support/time'
-require 'ndr_support/integer/julian_date_conversions'
+require 'ndr_support/ourdate/build_datetime'
 
 # Our "vague date" class, which can represent a single date or a date range.
 class Daterange
@@ -8,6 +8,9 @@ class Daterange
 
   OKYEARS = 1880..2030
 
+  # Raised when dates are not supplied in the correct order if do_not_flip_dates is passed as true:
+  class WrongDateOrderError < StandardError; end
+
   def self.extract(dates_string)
     dates_string.to_s.split(',').map { |str| new(str) }
   end
@@ -17,15 +20,15 @@ class Daterange
     new(ranges.map(&:date1).compact.min, ranges.map(&:date2).compact.max)
   end
 
-  def initialize(x1 = nil, x2 = nil)
+  def initialize(x1 = nil, x2 = nil, do_not_sort_dates: false)
     x1 = x1.to_datetime if x1.is_a?(Date) || x1.is_a?(Time)
     x2 = x2.to_datetime if x2.is_a?(Date) || x2.is_a?(Time)
 
     if x1.is_a?(DateTime) && x2.is_a?(DateTime)
-      @date1 = [x1, x2].min
-      @date2 = [x1, x2].max
+      @date1 = do_not_sort_dates ? x1 : [x1, x2].min
+      @date2 = do_not_sort_dates ? x2 : [x1, x2].max
       @source = nil
-    elsif x1.is_a?(Daterange) && x2.nil?  # Patient model line 645
+    elsif x1.is_a?(Daterange) && x2.nil? # Patient model line 645
       @date1 = x1.date1
       @date2 = x1.date2
       @source = x1.source
@@ -34,15 +37,24 @@ class Daterange
       @date2 = x1
       @source = nil
     elsif x1.is_a?(String) && x2.nil?
-      self.source = (x1)
+      self.send(:source=, x1, do_not_sort_dates)
     else
       @date1 = nil
       @date2 = nil
       @source = nil
     end
+    raise WrongDateOrderError, 'Invalid date range order' if wrong_date_order?(do_not_sort_dates)
+
     self.freeze
   end
 
+  def wrong_date_order?(do_not_sort_dates)
+    return false unless do_not_sort_dates
+    return false unless @date1.present? && @date2.present?
+
+    @date1 > @date2
+  end
+
   # If we have a valid date range, return a string representation of it
   # TODO: possibly add support for to_s(format) e.g. to_s(:short)
   def to_s
@@ -137,22 +149,23 @@ class Daterange
   # +s+ consists of one or more dates separated with spaces.
   # Each date can be in various formats, e.g. d/m/yyyy, ddmmyyyy, yyyy-mm-dd, dd-mon-yyyy
   # Each date can omit days or months, e.g. yyyy, dd/yyyy, yyyy-mm, mon-yyyy
-  def source=(s)
+  def source=(s, do_not_sort_dates)
     @source = s
     ss = s.upcase.sub(/TO/, ' ') # accept default _to_s format
-    if ss =~ /[^\w0-9\-\/\. ]/i # only allow letters, digits, hyphen, slash, dot, space
+    if ss =~ %r{[^A-Z0-9\-/\. ]}i # only allow letters, digits, hyphen, slash, dot, space
       @date1 = @date2 = nil
     else
       da = [] # temporary array of arrays of dates
-      ss.split(' ').each do |vaguedate|
+      ss.split.each do |vaguedate|
         da << str_to_date_array(vaguedate)
       end
       da.flatten!
       if da.include?(nil)
-        @date1 = @date2 =  nil
+        @date1 = @date2 = nil
       else
-        da.sort!
-        @date1, @date2 = da.first, da.last
+        da.sort! unless do_not_sort_dates
+        @date1 = da.first
+        @date2 = da.last
       end
     end
   end
@@ -175,7 +188,10 @@ class Daterange
       j1 = j2 = Date.new(parts[0], parts[1], parts[2]).jd
     end
 
-    [j1.jd_to_datetime, j2.jd_to_datetime]
+    [j1, j2].collect do |jd|
+      date = Date.jd(jd) # Julian date number to Ruby Date
+      Ourdate.build_datetime(date.year, date.month, date.day)
+    end
   rescue
     nil
   end

data/lib/ndr_support/ourdate/build_datetime.rb

@@ -0,0 +1,36 @@
+require 'active_record'
+require 'active_support/time'
+
+class Ourdate
+  # Construct a daylight saving time safe datetime, with arguments
+  #--
+  # FIXME: Note that the arguments should be numbers, not strings -- it works
+  # with strings arguments only after the 1970 epoch; before, it returns nil.
+  #++
+  def self.build_datetime(year, month = 1, day = 1, hour = 0, min = 0, sec = 0, usec = 0)
+    return nil if year.nil?
+
+    if ActiveRecord::Base.default_timezone == :local
+      # Time.local_time(year, month, day, hour, min, sec, usec).to_datetime
+      # Behave like oracle_adapter.rb
+      seconds = sec + Rational(usec, 10**6)
+      time_array = [year, month, day, hour, min, seconds]
+      begin
+        #--
+        # TODO: Fails unit tests unless we .to_datetime here
+        #       but the risk is we lose the usec component unnecesssarily.
+        #       Investigate removing .to_datetime below.
+        #++
+        Time.send(ActiveRecord::Base.default_timezone, *time_array).to_datetime
+      rescue
+        zone_offset = ActiveRecord::Base.default_timezone == :local ? DateTime.now.offset : 0
+        # Append zero calendar reform start to account for dates skipped by calendar reform
+        DateTime.new(*time_array[0..5] << zone_offset << 0) rescue nil
+      end
+    else
+      # Only supports fake GMT time -- needs improvement
+      # Maybe use Time.zone.local or Time.local_time(year, month, day)
+      Time.utc(year, month, day, hour, min, sec, usec).to_datetime
+    end
+  end
+end

data/lib/ndr_support/ourdate.rb

@@ -1,5 +1,6 @@
 require 'active_record'
 require 'active_support/time'
+require 'ndr_support/ourdate/build_datetime'
 require 'ndr_support/date_and_time_extensions'
 require 'ndr_support/daterange'
 
@@ -25,36 +26,6 @@ class Ourdate
     end
   end
 
-  # Construct a daylight saving time safe datetime, with arguments
-  #--
-  # FIXME: Note that the arguments should be numbers, not strings -- it works
-  # with strings arguments only after the 1970 epoch; before, it returns nil.
-  #++
-  def self.build_datetime(year, month = 1, day = 1, hour = 0, min = 0, sec = 0, usec = 0)
-    if ActiveRecord::Base.default_timezone == :local
-      # Time.local_time(year, month, day, hour, min, sec, usec).to_datetime
-      # Behave like oracle_adapter.rb
-      seconds = sec + Rational(usec, 10**6)
-      time_array = [year, month, day, hour, min, seconds]
-      begin
-        #--
-        # TODO: Fails unit tests unless we .to_datetime here
-        #       but the risk is we lose the usec component unnecesssarily.
-        #       Investigate removing .to_datetime below.
-        #++
-        Time.send(ActiveRecord::Base.default_timezone, *time_array).to_datetime
-      rescue
-        zone_offset = ActiveRecord::Base.default_timezone == :local ? DateTime.now.offset : 0
-        # Append zero calendar reform start to account for dates skipped by calendar reform
-        DateTime.new(*time_array[0..5] << zone_offset << 0) rescue nil
-      end
-    else
-      # Only supports fake GMT time -- needs improvement
-      # Maybe use Time.zone.local or Time.local_time(year, month, day)
-      Time.utc(year, month, day, hour, min, sec, usec).to_datetime
-    end
-  end
-
   def initialize(x = nil)
     if x.is_a?(Date)
       @thedate = x
@@ -72,8 +43,8 @@ class Ourdate
   end
 
   def empty?
-    # An unspecified date will be empty. A valid or invalid date will not.
-    @thedate.nil? && @source.blank?
+    # An unspecified date will be empty. A valid date will not.
+    @thedate.nil?
   end
 
   def source=(s)

data/lib/ndr_support/ourtime.rb

@@ -28,8 +28,8 @@ class Ourtime
   end
 
   def empty?
-    # An unspecified time will be empty. A valid or invalid time will not.
-    @thetime.nil? && @source.blank?
+    # An unspecified time will be empty. A valid time will not.
+    @thetime.nil?
   end
 
   private

data/lib/ndr_support/safe_path.rb

@@ -118,6 +118,7 @@ class SafePath
 
     @path_space = path_space
     @maximum_prms = fs_paths[platform][path_space]['prms']
+    @prm = nil
 
     self.path = @root
   end

data/lib/ndr_support/string/conversions.rb

@@ -84,7 +84,7 @@ class String
 
   # Try to convert the string value into a date.
   # If given a pattern, use it to parse date, otherwise use default setting to parse it
-  def to_date(pattern = nil)
+  redefine_method :to_date do |pattern = nil|
     return nil if blank?
 
     pattern = '%d%m%Y' if 'ddmmyyyy' == pattern

data/lib/ndr_support/utf8_encoding/control_characters.rb

@@ -1,5 +1,3 @@
-require 'ndr_support/utf8_encoding'
-
 module UTF8Encoding
   # Allows any supported object to have control characters
   # escaped, using standard replacement scheme.

data/lib/ndr_support/utf8_encoding/force_binary.rb

@@ -1,5 +1,3 @@
-require 'ndr_support/utf8_encoding'
-
 module UTF8Encoding
   # Allows any supported object to have any high-ascii string
   # content to be force-encoded from UTF-8 to BINARY (/ASCII-8BIT).

data/lib/ndr_support/utf8_encoding/object_support.rb

@@ -1,5 +1,3 @@
-require 'ndr_support/utf8_encoding'
-
 # Allows any object (if supported) to have all related
 # strings encoded in place to UTF-8.
 module UTF8Encoding

data/lib/ndr_support/version.rb

@@ -3,5 +3,5 @@
 # This defines the NdrSupport version. If you change it, rebuild and commit the gem.
 # Use "rake build" to build the gem, see rake -T for all bundler rake tasks.
 module NdrSupport
-  VERSION = '5.7.1'
+  VERSION = '5.8.3'
 end

data/ndr_support.gemspec

@@ -1,4 +1,4 @@
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'ndr_support/version'
 
@@ -13,9 +13,7 @@ Gem::Specification.new do |spec|
   spec.license       = 'MIT'
 
   # Exclude older versions of this gem from the package.
-  spec.files         = `git ls-files -z`.split("\x0").reject { |s| s =~ /^pkg\// }
-  # SECURE BNS 2018-08-06: Minimise sharing of (public-key encrypted) slack secrets in .travis.yml
-  spec.files         -= %w[.travis.yml] # Not needed in the gem
+  spec.files         = `git ls-files -z`.split("\x0").reject { |s| s =~ %r{^(\.github|pkg/)} }
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
@@ -26,7 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake', '>= 12.3.3'
 
-  spec.required_ruby_version = '>= 2.5.0'
+  spec.required_ruby_version = '>= 2.6.0'
 
   # Avoid std-lib minitest (which has different namespace)
   spec.add_development_dependency 'minitest', '>= 5.0.0'

data/test/daterange_test.rb

@@ -24,19 +24,53 @@ class DaterangeTest < Minitest::Test
     # assert_match(/\d\d\.\d\d\.\d\d\d\d to \d\d.\d\d.\d\d\d\d/, dr.to_s)
   end
 
-  def test_dates_in_reverse_order
-    d = Date.today
-    dr = Daterange.new d, d + 1
-    dr2 = Daterange.new d + 1, d
+  def test_date_time_dates_in_reverse_order_with_do_not_sort_dates_false
+    d = Time.zone.today
+    dr = Daterange.new(d, d + 1)
+    dr2 = Daterange.new(d + 1, d)
     assert_equal(dr.to_s, dr2.to_s)
   end
 
+  def test_blank_does_not_trigger_error_and_returns_nil_in_dates
+    dr = Daterange.new('', do_not_sort_dates: true)
+    assert_nil dr.date1
+    assert_nil dr.date2
+  end
+
+  def test_date_time_dates_in_reverse_order_with_do_not_sort_dates_true
+    d = Time.zone.today
+    Daterange.new(d, d + 1, do_not_sort_dates: true)
+    assert_raises(Daterange::WrongDateOrderError) do
+      Daterange.new(d + 1, d, do_not_sort_dates: true)
+    end
+  end
+
+  def test_string_dates_in_reverse_order_with_do_not_sort_dates_false
+    dr = Daterange.new('01/01/2019 to 01/01/2020')
+    dr2 = Daterange.new('01/01/2020 to 01/01/2019')
+    assert_equal(dr.to_s, dr2.to_s)
+  end
+
+  def test_string_dates_in_reverse_order_with_do_not_sort_dates_true
+    Daterange.new('01/01/2019 to 01/01/2020', do_not_sort_dates: true)
+    assert_raises(Daterange::WrongDateOrderError) do
+      Daterange.new('01/01/2020 to 01/01/2019', do_not_sort_dates: true)
+    end
+  end
+
   def test_illegal_strings
     dr = Daterange.new('01/o1/2000')
     assert_equal '', dr.to_s
     assert_nil dr.date1
     assert_nil dr.date2
-    refute dr.empty?  # Illegal dates do not count as empty / blank,
+    refute dr.empty? # Illegal dates do not count as empty / blank,
+    refute_nil dr.source # but the illegal string is preserved
+
+    dr = Daterange.new('01_01_2000')
+    assert_equal '', dr.to_s
+    assert_nil dr.date1
+    assert_nil dr.date2
+    refute dr.empty? # Illegal dates do not count as empty / blank,
     refute_nil dr.source # but the illegal string is preserved
   end
 

data/test/string/conversions_test.rb

@@ -357,9 +357,9 @@ class String::ConversionsTest < Minitest::Test
   end
 
   def assert_ymd(ymd, date)
-    assert_equal ymd.first,  date.year,  'years were not equal'
-    assert_equal ymd.second, date.month, 'months were not equal'
-    assert_equal ymd.third,  date.day,   'days were not equal'
+    assert_equal ymd[0],  date.year,  'years were not equal'
+    assert_equal ymd[1], date.month, 'months were not equal'
+    assert_equal ymd[2],  date.day,   'days were not equal'
   end
 
   def assert_ymd_parsed(ymd, parse_results)

data/test/threat_scanner_test.rb

@@ -8,6 +8,8 @@ class ThreatScannerTest < Minitest::Test
 
     ThreatScanner.stubs(installed?: true)
     ThreatScanner.any_instance.stubs(:`)
+
+    `pwd` # Ensure that $CHILD_STATUS is not nil, even if we never run any child processes
   end
 
   def teardown

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ndr_support
 version: !ruby/object:Gem::Version
-  version: 5.7.1
+  version: 5.8.3
 platform: ruby
 authors:
 - NCRS Development Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-03 00:00:00.000000000 Z
+date: 2021-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -249,12 +249,12 @@ files:
 - lib/ndr_support/daterange.rb
 - lib/ndr_support/hash.rb
 - lib/ndr_support/integer/calculations.rb
-- lib/ndr_support/integer/julian_date_conversions.rb
 - lib/ndr_support/integer/rounding.rb
 - lib/ndr_support/integer/working_days.rb
 - lib/ndr_support/nil.rb
 - lib/ndr_support/obfuscator.rb
 - lib/ndr_support/ourdate.rb
+- lib/ndr_support/ourdate/build_datetime.rb
 - lib/ndr_support/ourtime.rb
 - lib/ndr_support/password.rb
 - lib/ndr_support/password/constants.rb
@@ -309,14 +309,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: NDR Support library

data/lib/ndr_support/integer/julian_date_conversions.rb

@@ -1,14 +0,0 @@
-require 'ndr_support/ourdate'
-
-# Extend Integer for use in our Daterange class
-class Integer
-  # Julian date number to Ruby Date
-  def jd_to_date
-    Date.jd(self)
-  end
-
-  def jd_to_datetime
-    date = jd_to_date
-    Ourdate.build_datetime(date.year, date.month, date.day)
-  end
-end