historiographer 4.4.2 → 4.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66fb9c7d713df84dd4ad5470c6ace5730e5a762d378d0d1fc523a00767b94a2b
-  data.tar.gz: e9b1cfaa4911d4e596aaa8808d1e4cf69b7e57a40101bae4cf826ec48832d3c9
+  metadata.gz: 60587470dc09b843fa049daa514e78c23492a10e1652535762e05838e827d75a
+  data.tar.gz: 248a9a9b6dc13d2a7fa9a7926e6bad37c22e2d70cb2a27ba85ca57c079d788ef
 SHA512:
-  metadata.gz: a6291260e457cc89940cbdd622e1e2075e05ad7682005f4bd4dab9e63d302e2cd1dd3d55d494f336b442fe36827b5934251f3fb155480ab46e1bf7e84cb77414
-  data.tar.gz: e1a2afe3202d4c3949424afda77a5c9b307baa50c1e43bde0c488ecff3bd85aff08b5ce57c6664dba058b188603e35c845e508487769848fa811003174b751e0
+  metadata.gz: 558b8819149579b0f9c5a21bb1c16082036661e37791fd40b9adbca3df395af99795cc9afad7d3c6e83dbfbb6a4e5b664380c3bbe73aa952366eb752d5e38db9
+  data.tar.gz: 8dc727ba00be315e3d4067de27b93e5544f4bf854b9f63ab80ff86a7c53927018b6072d0f5590fbc23cd022544e004f908108a22c1ff58b83679d9594794cd97

data/DEVELOPMENT.md

@@ -0,0 +1,124 @@
+# Historiographer Development Guide
+
+## Quick Start
+
+```bash
+# Initial setup
+bin/setup
+
+# Run tests
+bin/test        # Regular test suite (fast)
+bin/test-rails  # Rails integration tests (slower)
+bin/test-all    # Both test suites
+
+# Interactive console
+bin/console
+```
+
+## Available Commands
+
+### Using Rake Tasks
+
+```bash
+# Testing
+rake spec           # Run regular specs (default)
+rake spec:regular   # Run regular specs explicitly
+rake spec:rails     # Run Rails integration specs
+rake spec:all       # Run all specs
+
+# Database
+rake db:create      # Create test database
+rake db:migrate     # Run migrations
+rake db:rollback    # Rollback migrations
+rake test_setup     # Setup test database
+rake test_reset     # Reset test database
+
+# Other
+rake console        # Launch console with gem loaded
+rake help           # List all available tasks
+```
+
+### Using Bin Scripts
+
+All scripts are in the `bin/` directory:
+
+- `bin/setup` - Initial development setup
+- `bin/test` - Run regular test suite
+- `bin/test-rails` - Run Rails integration tests  
+- `bin/test-all` - Run all tests
+- `bin/console` - Interactive console
+
+### Direct Commands
+
+```bash
+# Run specific tests
+bundle exec rspec spec/historiographer_spec.rb
+bundle exec rspec spec/historiographer_spec.rb:100  # Run specific line
+
+# Rails integration tests only
+bundle exec rspec spec/rails_integration
+
+# Database operations
+bundle exec rake db:create
+bundle exec rake db:migrate
+bundle exec rake db:rollback
+```
+
+## Test Organization
+
+The test suite is split into two parts:
+
+1. **Regular Tests** (`spec/*.rb`, `spec/models/*.rb`, etc.)
+   - Fast, lightweight Rails simulation
+   - Uses standalone_migrations
+   - Default when running `rake spec` or `bin/test`
+
+2. **Rails Integration Tests** (`spec/rails_integration/*.rb`)
+   - Uses Combustion to create a real Rails app
+   - Tests Rails-specific behaviors like autoloading
+   - Run separately with `rake spec:rails` or `bin/test-rails`
+
+## Database Setup
+
+The gem uses PostgreSQL for testing. Two databases are used:
+
+1. `historiographer_test` - Main test database
+2. `historiographer_combustion_test` - Rails integration test database
+
+Configure database connection in:
+- `spec/db/database.yml` - Main test database
+- `spec/internal/config/database.yml` - Combustion test database
+
+## Adding New Tests
+
+### Regular Tests
+Place in `spec/` directory. Models go in `spec/models/`.
+
+### Rails Integration Tests  
+Place in `spec/rails_integration/`. These tests use Combustion and require:
+```ruby
+require 'combustion_helper'
+```
+
+## Debugging
+
+```bash
+# Launch console with all models loaded
+bin/console
+
+# Or with rake
+rake console
+
+# Or manually
+bundle exec pry -r ./init.rb
+```
+
+## Gem Development
+
+```bash
+# Build gem
+rake build
+
+# Release new version
+rake release
+```

data/Gemfile

@@ -25,9 +25,11 @@ group :development do
 end
 
 group :test do
+  gem 'combustion', '~> 1.3'
   gem 'database_cleaner'
   gem 'factory_bot_rails'
   gem 'guard'
   gem 'guard-rspec'
   gem 'rspec'
+  gem 'rspec-rails'
 end

data/README.md

@@ -358,7 +358,22 @@ For contributors on OSX, you may have difficulty installing mysql:
 gem install mysql2 -v '0.4.10' --source 'https://rubygems.org/' -- --with-ldflags=-L/usr/local/opt/openssl/lib --with-cppflags=-I/usr/local/opt/openssl/include
 ```
 
+== Testing
+
+```bash
+# Initial setup
+bin/setup
+
+# Run tests
+bin/test        # Regular test suite (fast)
+bin/test-rails  # Rails integration tests (slower)
+bin/test-all    # Both test suites
+
+# Interactive console
+bin/console
+```
+
 == Copyright
 
-Copyright (c) 2016-2020 brettshollenberger. See LICENSE.txt for
+Copyright (c) 2016-2025 brettshollenberger. See LICENSE.txt for
 further details.

data/Rakefile

@@ -22,6 +22,20 @@ Jeweler::Tasks.new do |gem|
   gem.description = %Q{Creates separate tables for each history table}
   gem.email = "brett.shollenberger@gmail.com"
   gem.authors = ["brettshollenberger"]
+  
+  # Use glob patterns to automatically include all relevant files
+  gem.files = `git ls-files`.split("\n").reject { |f| 
+    f.match(/^(test|spec|features)\//) || 
+    f.match(/\.gem$/) ||
+    f.match(/^\./) ||
+    f == 'Gemfile.lock'
+  }
+  
+  # Include spec files for development/testing
+  gem.files += Dir.glob("spec/**/*")
+  
+  # Ensure we have all the executables
+  gem.executables = Dir.glob("bin/*").map { |f| File.basename(f) }
 end
 Jeweler::RubygemsDotOrgTasks.new
 
@@ -52,3 +66,57 @@ end
 
 require 'standalone_migrations'
 StandaloneMigrations::Tasks.load_tasks
+
+# Custom tasks for common operations
+namespace :spec do
+  require 'rspec/core/rake_task'
+  
+  desc "Run regular specs (excluding Rails integration)"
+  RSpec::Core::RakeTask.new(:regular) do |t|
+    t.rspec_opts = "--exclude-pattern spec/rails_integration/**/*_spec.rb"
+  end
+  
+  desc "Run Rails integration specs with Combustion"
+  RSpec::Core::RakeTask.new(:rails) do |t|
+    t.pattern = "spec/rails_integration/**/*_spec.rb"
+  end
+  
+  desc "Run all specs (regular and Rails integration separately)"
+  task :all do
+    puts "\n========== Running Regular Specs ==========\n"
+    Rake::Task['spec:regular'].invoke
+    puts "\n========== Running Rails Integration Specs ==========\n"
+    Rake::Task['spec:rails'].invoke
+  end
+end
+
+desc "Run regular test suite (default)"
+task :spec => 'spec:regular'
+
+desc "Setup test database"
+task :test_setup do
+  sh "bundle exec rake db:create"
+  sh "bundle exec rake db:migrate"
+end
+
+desc "Reset test database"
+task :test_reset do
+  sh "bundle exec rake db:drop"
+  sh "bundle exec rake db:create"
+  sh "bundle exec rake db:migrate"
+end
+
+desc "Run linting and type checking"
+task :lint do
+  puts "No linting configured yet. Consider adding rubocop."
+end
+
+desc "Console with the gem loaded"
+task :console do
+  sh "bundle exec pry -r ./init.rb"
+end
+
+desc "List all available tasks"
+task :help do
+  sh "rake -T"
+end

data/VERSION

@@ -1 +1 @@
-4.4.2
+4.4.4

data/bin/console

@@ -0,0 +1,10 @@
+#!/usr/bin/env ruby
+# Launch an interactive console with the gem loaded
+
+require "bundler/setup"
+require "pry"
+require_relative "../init"
+
+puts "Loading Historiographer console..."
+puts "Models available: Post, Author, Comment, User, etc."
+Pry.start

data/bin/setup

@@ -0,0 +1,15 @@
+#!/usr/bin/env ruby
+# Setup script for development
+
+puts "Setting up Historiographer development environment..."
+
+system("bundle install")
+system("bundle exec rake db:create")
+system("bundle exec rake db:migrate")
+system("createdb historiographer_combustion_test 2>/dev/null")
+
+puts "\nSetup complete! You can now run:"
+puts "  bin/test        - Run regular tests"
+puts "  bin/test-rails  - Run Rails integration tests"
+puts "  bin/test-all    - Run all tests"
+puts "  bin/console     - Launch interactive console"

data/bin/test

@@ -0,0 +1,5 @@
+#!/usr/bin/env ruby
+# Run the regular test suite (excluding Rails integration tests)
+
+puts "Running regular test suite..."
+exec "bundle exec rspec --exclude-pattern spec/rails_integration/**/*_spec.rb"

data/bin/test-all

@@ -0,0 +1,10 @@
+#!/usr/bin/env ruby
+# Run all tests (regular and Rails integration)
+
+puts "\n========== Running Regular Test Suite ==========\n"
+system("bundle exec rspec --exclude-pattern spec/rails_integration/**/*_spec.rb")
+
+puts "\n========== Running Rails Integration Tests ==========\n"
+system("bundle exec rspec spec/rails_integration")
+
+exit($?.exitstatus || 0)

data/bin/test-rails

@@ -0,0 +1,5 @@
+#!/usr/bin/env ruby
+# Run Rails integration tests with Combustion
+
+puts "Running Rails integration tests..."
+exec "bundle exec rspec spec/rails_integration/historiographer_rails_integration_spec.rb"

data/historiographer.gemspec

@@ -2,38 +2,36 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: historiographer 4.4.2 ruby lib
+# stub: historiographer 4.4.4 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "historiographer".freeze
-  s.version = "4.4.2"
+  s.version = "4.4.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["brettshollenberger".freeze]
-  s.date = "2025-08-22"
+  s.date = "2025-09-19"
   s.description = "Creates separate tables for each history table".freeze
   s.email = "brett.shollenberger@gmail.com".freeze
+  s.executables = ["console".freeze, "setup".freeze, "test".freeze, "test-all".freeze, "test-rails".freeze]
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.md"
   ]
   s.files = [
-    ".document",
-    ".rspec",
-    ".ruby-version",
-    ".standalone_migrations",
+    "DEVELOPMENT.md",
     "Gemfile",
-    "Gemfile.lock",
     "Guardfile",
     "LICENSE.txt",
     "README.md",
     "Rakefile",
     "VERSION",
-    "historiographer-4.1.12.gem",
-    "historiographer-4.1.13.gem",
-    "historiographer-4.1.14.gem",
-    "historiographer-4.3.0.gem",
+    "bin/console",
+    "bin/setup",
+    "bin/test",
+    "bin/test-all",
+    "bin/test-rails",
     "historiographer.gemspec",
     "init.rb",
     "instructions/implementation.md",
@@ -49,6 +47,7 @@ Gem::Specification.new do |s|
     "lib/historiographer/safe.rb",
     "lib/historiographer/silent.rb",
     "lib/historiographer/version.rb",
+    "spec/combustion_helper.rb",
     "spec/db/database.yml",
     "spec/db/migrate/20161121212228_create_posts.rb",
     "spec/db/migrate/20161121212229_create_post_histories.rb",
@@ -71,14 +70,39 @@ Gem::Specification.new do |s|
     "spec/db/migrate/20250824000000_create_test_articles.rb",
     "spec/db/migrate/20250824000001_create_test_categories.rb",
     "spec/db/migrate/20250825000000_create_bylines.rb",
+    "spec/db/migrate/20250826000000_create_test_users.rb",
+    "spec/db/migrate/20250826000001_create_test_user_histories.rb",
+    "spec/db/migrate/20250826000002_create_test_websites.rb",
+    "spec/db/migrate/20250826000003_create_test_website_histories.rb",
+    "spec/db/migrate/20250827000000_create_templates.rb",
+    "spec/db/migrate/20250827000001_create_template_histories.rb",
+    "spec/db/migrate/20250827000002_create_websites.rb",
+    "spec/db/migrate/20250827000003_create_website_histories.rb",
+    "spec/db/migrate/20250827000004_create_template_files.rb",
+    "spec/db/migrate/20250827000005_create_template_file_histories.rb",
+    "spec/db/migrate/20250827000006_create_website_files.rb",
+    "spec/db/migrate/20250827000007_create_website_file_histories.rb",
+    "spec/db/migrate/20250827000008_create_code_files_view.rb",
     "spec/db/schema.rb",
+    "spec/examples.txt",
     "spec/factories/post.rb",
-    "spec/foreign_key_spec.rb",
     "spec/historiographer_spec.rb",
+    "spec/integration/historiographer_safe_integration_spec.rb",
+    "spec/internal/app/models/application_record.rb",
+    "spec/internal/app/models/deploy.rb",
+    "spec/internal/app/models/user.rb",
+    "spec/internal/app/models/website.rb",
+    "spec/internal/app/models/website_history.rb",
+    "spec/internal/config/database.yml",
+    "spec/internal/config/routes.rb",
+    "spec/internal/db/schema.rb",
+    "spec/internal/log/development.log",
+    "spec/internal/log/test.log",
     "spec/models/application_record.rb",
     "spec/models/author.rb",
     "spec/models/author_history.rb",
     "spec/models/byline.rb",
+    "spec/models/code_file.rb",
     "spec/models/comment.rb",
     "spec/models/comment_history.rb",
     "spec/models/easy_ml/column.rb",
@@ -93,15 +117,29 @@ Gem::Specification.new do |s|
     "spec/models/safe_post_history.rb",
     "spec/models/silent_post.rb",
     "spec/models/silent_post_history.rb",
+    "spec/models/template.rb",
+    "spec/models/template_file.rb",
+    "spec/models/template_file_history.rb",
+    "spec/models/template_history.rb",
     "spec/models/test_article.rb",
     "spec/models/test_article_history.rb",
     "spec/models/test_category.rb",
     "spec/models/test_category_history.rb",
+    "spec/models/test_user.rb",
+    "spec/models/test_user_history.rb",
+    "spec/models/test_website.rb",
+    "spec/models/test_website_history.rb",
     "spec/models/thing_with_compound_index.rb",
     "spec/models/thing_with_compound_index_history.rb",
     "spec/models/thing_without_history.rb",
     "spec/models/user.rb",
-    "spec/spec_helper.rb"
+    "spec/models/website.rb",
+    "spec/models/website_file.rb",
+    "spec/models/website_file_history.rb",
+    "spec/models/website_history.rb",
+    "spec/rails_integration/historiographer_rails_integration_spec.rb",
+    "spec/spec_helper.rb",
+    "spec/view_backed_model_spec.rb"
   ]
   s.homepage = "http://github.com/brettshollenberger/historiographer".freeze
   s.licenses = ["MIT".freeze]

data/lib/historiographer/history.rb

@@ -78,11 +78,24 @@ module Historiographer
       # "RetailerProductHistory."
       #
       foreign_class_name = base.name.gsub(/History$/) {}        # e.g. "RetailerProductHistory" => "RetailerProduct"
-      foreign_class = foreign_class_name.constantize
       association_name   = foreign_class_name.split("::").last.underscore.to_sym # e.g. "RetailerProduct" => :retailer_product
 
-      # Store the original class for method delegation
-      class_variable_set(:@@original_class, foreign_class)
+      # Defer foreign class resolution to avoid load order issues
+      base.define_singleton_method :foreign_class do
+        return class_variable_get(:@@foreign_class) if class_variable_defined?(:@@foreign_class)
+        begin
+          foreign_class = foreign_class_name.constantize
+          class_variable_set(:@@foreign_class, foreign_class)
+          foreign_class
+        rescue NameError => e
+          # If the class isn't loaded yet, return nil and it will be retried later
+          nil
+        end
+      end
+
+      # Store the foreign class name for later use
+      class_variable_set(:@@foreign_class_name, foreign_class_name)
+      class_variable_set(:@@association_name, association_name)
 
       #
       # A History class will be linked to the user
@@ -94,12 +107,20 @@ module Historiographer
       #
       # To use histories, a user class must be defined.
       #
-      unless foreign_class.ancestors.include?(Historiographer::Silent)
+      # Set up user association unless Silent module is included
+      # Defer this check until foreign_class is available
+      unless base.foreign_class && base.foreign_class.ancestors.include?(Historiographer::Silent)
         belongs_to :user, foreign_key: :history_user_id
       end
 
-      # Add method_added hook to the original class
-      foreign_class.singleton_class.class_eval do
+      # Add method_added hook to the original class when it's available
+      # This needs to be deferred until the foreign class is loaded
+      base.define_singleton_method :setup_method_delegation do
+        return unless foreign_class
+        return if class_variable_defined?(:@@method_delegation_setup) && class_variable_get(:@@method_delegation_setup)
+        class_variable_set(:@@method_delegation_setup, true)
+        
+        foreign_class.singleton_class.class_eval do
         # Keep track of original method_added if it exists
         if method_defined?(:method_added)
           alias_method :original_method_added, :method_added
@@ -120,9 +141,9 @@ module Historiographer
             return unless method_obj.owner == self
 
             # Skip if we've already defined this method in the history class
-            return if foreign_class.history_class.method_defined?(method_name)
+            return if self.history_class.method_defined?(method_name)
 
-            foreign_class.history_class.class_eval do
+            self.history_class.class_eval do
               define_method(method_name) do |*args, **kwargs, &block|
                 forward_method(method_name, *args, **kwargs, &block)
               end
@@ -134,19 +155,29 @@ module Historiographer
       end
 
       begin
-        (foreign_class.columns.map(&:name) - ["id"]).each do |method_name|
-          define_method(method_name) do |*args, **kwargs, &block|
-            forward_method(method_name, *args, **kwargs, &block)
+        end
+      end
+      
+      # Try to set up method delegation if foreign class is available
+      base.setup_method_delegation if base.foreign_class
+      
+      # Also delegate existing methods from the foreign class
+      if base.foreign_class
+        begin
+          (base.foreign_class.columns.map(&:name) - ["id"]).each do |method_name|
+            define_method(method_name) do |*args, **kwargs, &block|
+              forward_method(method_name, *args, **kwargs, &block)
+            end
           end
+        rescue ActiveRecord::StatementInvalid, ActiveRecord::ConnectionNotEstablished
+          # Table might not exist yet during setup
         end
-      rescue ActiveRecord::StatementInvalid, ActiveRecord::ConnectionNotEstablished
-        # Table might not exist yet during setup
       end
 
       # Add method_missing for any methods we might have missed
       def method_missing(method_name, *args, **kwargs, &block)
-        original_class = self.class.class_variable_get(:@@original_class)
-        if original_class.method_defined?(method_name)
+        original_class = self.class.foreign_class
+        if original_class && original_class.method_defined?(method_name)
           forward_method(method_name, *args, **kwargs, &block)
         else
           super
@@ -154,8 +185,8 @@ module Historiographer
       end
 
       def respond_to_missing?(method_name, include_private = false)
-        original_class = self.class.class_variable_get(:@@original_class)
-        original_class.method_defined?(method_name) || super
+        original_class = self.class.foreign_class
+        (original_class && original_class.method_defined?(method_name)) || super
       end
 
       #
@@ -241,29 +272,36 @@ module Historiographer
       history_classes = Thread.current[:historiographer_history_classes] ||= []
       history_classes << base
       
+      # Always define the setup_history_associations method
+      base.define_singleton_method :setup_history_associations do |force = false|
+        return if !force && class_variable_defined?(:@@associations_set_up) && class_variable_get(:@@associations_set_up)
+        class_variable_set(:@@associations_set_up, true)
+        
+        return unless foreign_class
+        
+        # Also set up method delegation if not already done
+        setup_method_delegation if respond_to?(:setup_method_delegation)
+        
+        foreign_class.reflect_on_all_associations.each do |association|
+          begin
+            define_history_association(association)
+          rescue => e
+            # Log but don't fail
+            puts "Warning: Could not define history association #{association.name}: #{e.message}" if ENV['DEBUG']
+          end
+        end
+      end
+      
       # Set up the after_initialize hook if we're in a Rails app
       if defined?(Rails) && Rails.respond_to?(:application) && Rails.application && Rails.application.config.respond_to?(:after_initialize)
         Rails.application.config.after_initialize do
           history_classes.each do |history_class|
+            history_class.setup_method_delegation if history_class.respond_to?(:setup_method_delegation)
             history_class.setup_history_associations
           end
         end
       else
-        # For non-Rails environments (like our test suite), set up associations immediately
-        # but defer if models aren't loaded yet
-        base.define_singleton_method :setup_history_associations do |force = false|
-          return if !force && class_variable_defined?(:@@associations_set_up) && class_variable_get(:@@associations_set_up)
-          class_variable_set(:@@associations_set_up, true)
-          
-          foreign_class.reflect_on_all_associations.each do |association|
-            begin
-              define_history_association(association)
-            rescue => e
-              # Log but don't fail
-              puts "Warning: Could not define history association #{association.name}: #{e.message}" if ENV['DEBUG']
-            end
-          end
-        end
+        # For non-Rails environments, try to set up associations immediately
         
         # Try to set up now if possible
         begin
@@ -301,11 +339,8 @@ module Historiographer
       end
 
       def original_class
-        unless class_variable_defined?(:@@original_class)
-          class_variable_set(:@@original_class, self.name.gsub(/History$/, '').constantize)
-        end
-
-        class_variable_get(:@@original_class)
+        # Use the foreign_class method we defined earlier
+        foreign_class
       end
 
       def define_history_association(association)

data/lib/historiographer.rb

@@ -304,6 +304,11 @@ module Historiographer
 
         # Recursively snapshot associations, avoiding infinite loops
         self.class.reflect_on_all_associations.each do |association|
+          # Skip associations to models without primary keys (e.g., database views)
+          association_class = association.klass rescue nil
+          next if association_class.nil?
+          next if association_class.primary_key.nil?
+          
           associated_records = send(association.name)&.reload
           if associated_records.respond_to?(:order)
             associated_records = associated_records.order(id: :asc)

data/spec/combustion_helper.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+ENV['RAILS_ENV'] ||= 'test'
+
+require 'bundler'
+Bundler.require :default, :test
+
+require 'historiographer'
+require 'combustion'
+
+Combustion.path = 'spec/internal'
+Combustion.initialize! :active_record do
+  config.load_defaults Rails::VERSION::STRING.to_f
+end
+
+require 'rspec/rails'
+require 'database_cleaner'
+
+RSpec.configure do |config|
+  config.use_transactional_fixtures = false
+  
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+  
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+  
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/db/migrate/20250826000000_create_test_users.rb

@@ -0,0 +1,8 @@
+class CreateTestUsers < ActiveRecord::Migration[7.0]
+  def change
+    create_table :test_users do |t|
+      t.string :name
+      t.timestamps
+    end
+  end
+end