Replace DateTime usage with Time, DateTime is deprecated (ruby/date@58ca6e6)

Author: JonathanTron

.github/workflows/ci.yml

@@ -32,6 +32,8 @@ jobs:
           - "3.1"
           - "3.2"
           - "3.3"
+          - "3.4"
+          - "4.0"
     name: Ruby ${{ matrix.ruby }}
 
     env:

.ruby-version

@@ -1 +1 @@
-2.5.3
+4.0.2

lib/sequel/plugins/bitemporal.rb

@@ -1,34 +1,36 @@
-require "date"
+require "time"
 
 module Sequel
   module Plugins
     module Bitemporal
+      VALIDITY_TIME_MAX = Time.utc 9999
+
       THREAD_POINT_IN_TIME_KEY = :sequel_plugins_bitemporal_point_in_time
       def self.as_we_knew_it(time)
         previous = Thread.current[THREAD_POINT_IN_TIME_KEY]
         raise ArgumentError, "requires a block" unless block_given?
-        Thread.current[THREAD_POINT_IN_TIME_KEY] = time.to_datetime
+        Thread.current[THREAD_POINT_IN_TIME_KEY] = time.to_time.utc
         yield
       ensure
         Thread.current[THREAD_POINT_IN_TIME_KEY] = previous
       end
 
       def self.point_in_time
-        Thread.current[THREAD_POINT_IN_TIME_KEY] || DateTime.now
+        Thread.current[THREAD_POINT_IN_TIME_KEY] || Time.now.utc
       end
 
       THREAD_NOW_KEY = :sequel_plugins_bitemporal_now
       def self.at(time)
         previous = Thread.current[THREAD_NOW_KEY]
         raise ArgumentError, "requires a block" unless block_given?
-        Thread.current[THREAD_NOW_KEY] = time.to_datetime
+        Thread.current[THREAD_NOW_KEY] = time.to_time.utc
         yield
       ensure
         Thread.current[THREAD_NOW_KEY] = previous
       end
 
       def self.now
-        Thread.current[THREAD_NOW_KEY] || DateTime.now
+        Thread.current[THREAD_NOW_KEY] || Time.now.utc
       end
 
       def self.version_foreign_keys(master = nil)
@@ -185,14 +187,14 @@ def current?
             t = ::Sequel::Plugins::Bitemporal.point_in_time
             n = ::Sequel::Plugins::Bitemporal.now
             !new? &&
-            created_at.to_datetime<=t &&
-            (expired_at.nil? || expired_at.to_datetime>t) &&
-            valid_from.to_datetime<=n &&
-            valid_to.to_datetime>n
+            created_at.to_time<=t &&
+            (expired_at.nil? || expired_at.to_time>t) &&
+            valid_from.to_time<=n &&
+            valid_to.to_time>n
           end
           def destroy(opts={})
             expand_previous_version = opts.fetch(:expand_previous_version){
-              valid_from.to_datetime>::Sequel::Plugins::Bitemporal.now
+              valid_from.to_time>::Sequel::Plugins::Bitemporal.now
             }
             master.destroy_version self, expand_previous_version
           end
@@ -396,11 +398,11 @@ def destroy
         def destroy_version(version, expand_previous_version)
           now = ::Sequel::Plugins::Bitemporal.now
           point_in_time = ::Sequel::Plugins::Bitemporal.point_in_time
-          return false if version.valid_to.to_datetime<=now
+          return false if version.valid_to.to_time<=now
           associations.delete :current_version
           model.db.transaction do
             success = true
-            version_was_valid = now>=version.valid_from.to_datetime
+            version_was_valid = now>=version.valid_from.to_time
             if expand_previous_version
               previous = versions_dataset.where({
                 expired_at: nil,
@@ -513,7 +515,7 @@ def expire_previous_versions
           expired = expired.exclude Sequel.lit("valid_from=valid_to")
           expired = expired.exclude Sequel.lit("valid_to<=?", pending_version.valid_from)
           pending_version.valid_to ||= expired.where(Sequel.lit("valid_from>?", pending_version.valid_from)).min(:valid_from)
-          pending_version.valid_to ||= Time.utc 9999
+          pending_version.valid_to ||= Sequel::Plugins::Bitemporal::VALIDITY_TIME_MAX
           expired = expired.exclude Sequel.lit("valid_from>=?", pending_version.valid_to)
           expired = expired.all
           expired.each do |expired_version|

sequel_bitemporal.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency "sequel", ">= 4.0", "< 6.0"
 
-  s.add_development_dependency "rspec", "~> 3.13.0"
+  s.add_development_dependency "rspec", "~> 3.13.2"
   s.add_development_dependency "timecop"
   s.add_development_dependency "rake"
 end

spec/bitemporal_date_spec.rb

@@ -1,3 +1,4 @@
+require "date"
 require "spec_helper"
 
 RSpec.describe "Sequel::Plugins::Bitemporal" do

spec/bitemporal_date_with_range_spec.rb

@@ -1,4 +1,6 @@
+require "date"
 require "spec_helper"
+
 if DbHelpers.pg?
   RSpec.describe "Sequel::Plugins::Bitemporal", "with ranges" do
     before :all do

spec/bitemporal_serialization_spec.rb

@@ -1,5 +1,6 @@
-require "spec_helper"
+require "date"
 require "json"
+require "spec_helper"
 
 RSpec.describe "Sequel::Plugins::Bitemporal", :skip_jdbc_sqlite do
   before :all do

spec/support/db.rb

@@ -24,7 +24,7 @@ def self.pg_ruby_connect_uri
   def db_setup(opts={})
     use_time = opts[:use_time]
     DB.drop_table(:room_versions) if DB.table_exists?(:room_versions)
-    DB.drop_table(:rooms) if DB.table_exists?(:rooms)
+    DB.drop_table(:rooms, cascade: true) if DB.table_exists?(:rooms)
     DB.create_table! :rooms do
       primary_key :id
       Boolean     :disabled, null: false, default: false