[WIP] [ci-skip] Test

Close #388

Author: tagliala

lib/chrono_model/patches/join_node.rb

@@ -2,40 +2,16 @@
 
 module ChronoModel
   module Patches
-    # This class supports the AR < 7.1 code that expects to receive an
-    # Arel::Table as the left join node. We need to replace the node
-    # with a virtual table that fetches from the history at a given
-    # point in time, we replace the join node with a SqlLiteral node
-    # that does not respond to the methods that AR expects.
-    #
-    # This class provides AR with an object implementing the methods
-    # it expects, yet producing SQL that fetches from history tables
-    # as-of-time.
-    #
-    # TODO: Remove when dropping Rails < 7.1 compatibility
+    # Replaces the Arel::Table left join node with a virtual table that fetches
+    # data from the history at a given point in time.
     class JoinNode < Arel::Nodes::SqlLiteral
-      attr_reader :name, :table_name, :table_alias, :as_of_time
+      attr_reader :as_of_time
 
       def initialize(join_node, history_model, as_of_time)
-        # Handle both Arel::Table (which has .name method) and other join node types
-        # (which have .table_name method) to extract the table name consistently
-        table_name =
-          if join_node.respond_to?(:table_name)
-            join_node.table_name
-          elsif join_node.respond_to?(:name)
-            join_node.name
-          else
-            raise ArgumentError, "Cannot determine table name from #{join_node.class}: expected :table_name or :name method"
-          end
-
-        @name        = table_name
-        @table_name  = table_name
-        @table_alias = join_node.table_alias if join_node.respond_to?(:table_alias)
-
         @as_of_time  = as_of_time
 
-        virtual_table = history_model
-                        .virtual_table_at(@as_of_time, table_name: @table_alias || @table_name)
+        table_name = join_node.table_alias || join_node.name
+        virtual_table = history_model.virtual_table_at(@as_of_time, table_name: table_name)
 
         super(virtual_table)
       end

lib/chrono_model/patches/relation.rb

@@ -68,26 +68,15 @@ def build_arel(*)
 
       # Replaces a join with the current data with another that
       # loads records As-Of time against the history data.
-      #
       def chrono_join_history(join)
-        # This case happens with nested includes, where the below
-        # code has already replaced the join.left with a JoinNode.
-        #
-        return if join.left.respond_to?(:as_of_time)
-
-        # Handle both Arel::Table (which has .name method) and other join node types
-        # (which have .table_name method) to extract the table name consistently
-        model =
-          if join.left.respond_to?(:table_name)
-            ChronoModel.history_models[join.left.table_name]
-          elsif join.left.respond_to?(:name)
-            ChronoModel.history_models[join.left.name]
-          end
+        join_left = join.left
+        return if join_left.respond_to?(:as_of_time)
 
+        model = ChronoModel.history_models[join_left.name] if join_left.respond_to?(:name)
         return unless model
 
         join.left = ChronoModel::Patches::JoinNode.new(
-          join.left, model.history, @_as_of_time
+          join_left, model.history, @_as_of_time
         )
       end