whodunit 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 323bc72651528d7bf5e205c0388462078acc718ea39b14b5775517644a2115fc
-  data.tar.gz: d20344c823d5716ae218497ca92f61f41c6660c8e8f27a5479b0e52bc64b6b25
+  metadata.gz: b118c7a04ca24e4cfd6aa759e57b5c92e51be74f87b35a89ae6e5654f3a56051
+  data.tar.gz: db5c8c236bd170f7ce42b14682c3c44c6eaaae2dc125b6c8e9cdf9ef7a9a9596
 SHA512:
-  metadata.gz: f893afd5307dfa5cdabd58c8de25baa2e979809f9d2c810cdb5dd029a90ebb07f79a61ab24c149e9545159aafef09b3494e2fe15cdee93a6c66e1ae35c44bd2e
-  data.tar.gz: 9df8c395516af0720e71fd2a5e8b03a3471f4a85684f8b8a7b274222f74a19feeb1bb9a2b5b3e4795fbcf8938aef769a87e1dd380668838198efad31215d4e3b
+  metadata.gz: 4324dc93c29055aaa6dcc02b1999484d8108a4212eb8474185d5c0dee47e6d99fc88ed938d344f17bf75747a2eb5023ef2d6295d5308d9c838fd9b2f00c349dd
+  data.tar.gz: 6f31152d504e0b80a81992c3f62f190ac9de4aba73d0a964e24c116052ea5093629b6182eff58d6c9f811c2b3bec95b5385f49ef45429cc8f11d66a6d190d988

data/.rubocop.yml

@@ -6,7 +6,7 @@ plugins:
 
 AllCops:
   NewCops: enable
-  TargetRubyVersion: 3.1
+  TargetRubyVersion: 3.2
   Exclude:
     - 'vendor/**/*'
     - 'tmp/**/*'
@@ -114,12 +114,12 @@ Metrics/AbcSize:
     - 'spec/**/*'
 
 Metrics/CyclomaticComplexity:
-  Max: 8
+  Max: 10
   Exclude:
     - 'spec/**/*'
 
 Metrics/PerceivedComplexity:
-  Max: 8
+  Max: 10
   Exclude:
     - 'spec/**/*'
 Gemspec/DevelopmentDependencies:

data/CHANGELOG.md

@@ -5,7 +5,45 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased]
+## [0.4.0] - 2025-05-20
+
+### Breaking Changes
+
+- Dropped support for Ruby 3.1. Minimum supported Ruby version is now **3.2**.
+
+### Fixed
+
+- `TableDefinitionExtension`: replaced `included { attr_accessor :_whodunit_stamps_added }` with
+  explicit reader/writer methods so the accessor is defined correctly when the module is **prepended**
+  (as the Railtie does) rather than included. Previously the accessor was silently missing in
+  production, causing `@_whodunit_stamps_added` to be inaccessible via the public API.
+
+### Changed
+
+- **Test suite**: eliminated all hand-rolled ActiveRecord and Railtie mocks in favour of genuine
+  infrastructure — real `ActiveRecord::Base` subclasses backed by an in-memory SQLite3 database,
+  and real `ActiveRecord::ConnectionAdapters::TableDefinition` instances for migration-helper tests.
+  Callbacks, dirty tracking, `belongs_to` reflections, and `column_names` now go through actual AR
+  code paths rather than no-op stubs.
+  - `spec/support/rails_mocks.rb` — emptied; `activesupport` is a runtime dependency and needs no
+    re-implementation in tests.
+  - `spec/support/test_models.rb` — replaced `MockActiveRecord` stub class with real AR models
+    (`WhodunitRecord`, `WhodunitSoftDeleteRecord`) on an in-memory SQLite3 schema; backward-compat
+    aliases preserved.
+  - `spec/spec_helper.rb` — added `rescue Bundler::GemNotFound` guard on `require "bundler/setup"`
+    to allow running specs directly without `bundle exec`; wrapped every example in an AR transaction
+    that rolls back on completion; added missing config keys to `ORIGINAL_WHODUNIT_CONFIG`.
+  - `spec/whodunit/table_definition_extension_spec.rb` — now tests the real
+    `ActiveRecord::ConnectionAdapters::TableDefinition` with `TableDefinitionExtension` prepended,
+    rather than an anonymous class that reimplemented the extension logic internally.
+  - `spec/whodunit/stampable_spec.rb` — `being_soft_deleted?` tests use real AR dirty tracking via
+    genuine attribute writes; callback-integration tests persist records and assert database values
+    after `reload`.
+  - `spec/whodunit/migration_helpers_spec.rb` — `whodunit_stamps(table_def)` path uses a real
+    `TableDefinitionExtension`-prepended `TableDefinition` instance instead of a double.
+  - `spec/whodunit/per_model_config_spec.rb`, `railtie_spec.rb`,
+    `reverse_associations_integration_spec.rb` — all converted to real AR anonymous subclasses;
+    removed stubs for `before_create`, `belongs_to`, and `column_names`.
 
 ## [0.3.0] - 2025-01-24
 

data/README.md

@@ -3,7 +3,7 @@
 [![Gem Version](https://badge.fury.io/rb/whodunit.svg)](https://badge.fury.io/rb/whodunit)
 [![CI](https://github.com/kanutocd/whodunit/workflows/CI/badge.svg)](https://github.com/kanutocd/whodunit/actions)
 [![Coverage Status](https://codecov.io/gh/kanutocd/whodunit/branch/main/graph/badge.svg)](https://codecov.io/gh/kanutocd/whodunit)
-[![Ruby Version](https://img.shields.io/badge/ruby-%3E%3D%203.1.0-ruby.svg)](https://www.ruby-lang.org/en/)
+[![Ruby Version](https://img.shields.io/badge/ruby-%3E%3D%203.2-ruby.svg)](https://www.ruby-lang.org/en/)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 
 Lightweight creator/updater/deleter tracking for Rails ActiveRecord models.
@@ -16,7 +16,7 @@ Whodunit provides simple auditing for Rails applications by tracking who created
 
 ## Requirements
 
-- Ruby 3.1.1+ (tested on 3.1.1, 3.2.0, 3.3.0, 3.4). See the [the ruby-version matrix strategy of the CI workflow](https://github.com/kanutocd/whodunit/blob/main/.github/workflows/ci.yml#L15).
+- Ruby 3.2+ (tested on 3.2, 3.3, 3.4, 4.0). See the [the ruby-version matrix strategy of the CI workflow](https://github.com/kanutocd/whodunit/blob/main/.github/workflows/ci.yml#L15).
 - Rails 7.2+ (tested on 7.2, 8.2, and edge)
 - ActiveRecord for database operations
 
@@ -405,7 +405,7 @@ end
 | User tracking         | ✅       | ✅         | ✅      |
 | Change history        | Via [whodunit-chronicles](https://github.com/kanutocd/whodunit-chronicles) | ✅         | ✅      |
 | Performance overhead  | None     | High       | Medium  |
-| Soft-delete detection | ✅       | ❌         | ❌      |
+| Soft-delete support   | ✅       | ❌         | ❌      |
 | Setup complexity      | Low      | Medium     | Medium  |
 
 ## Documentation

data/gemfiles/rails_7_2.gemfile

@@ -3,8 +3,8 @@
 source "https://rubygems.org"
 
 # Rails 7.2 testing
-gem "activerecord", "~> 7.2.0"
-gem "activesupport", "~> 7.2.0"
+gem "activerecord", ">= 7.2.0"
+gem "activesupport", ">= 7.2.0"
 
 # Specify your gem's dependencies in whodunit.gemspec
 gemspec path: "../"

data/gemfiles/rails_8_2.gemfile

@@ -3,8 +3,8 @@
 source "https://rubygems.org"
 
 # Rails 8.0.2 testing
-gem "activerecord", "~> 8.0.0"
-gem "activesupport", "~> 8.0.0"
+gem "activerecord", ">= 8.0.0"
+gem "activesupport", ">= 8.0.0"
 
 # Specify your gem's dependencies in whodunit.gemspec
 gemspec path: "../"

data/lib/whodunit/current.rb

@@ -28,7 +28,7 @@ module Whodunit
   # @since 0.1.0
   class Current < ActiveSupport::CurrentAttributes
     # @!attribute [rw] user
-    #   @return [Integer, nil] the current user ID
+    # @return [Integer, nil] the current user ID
     attribute :user
 
     class << self

data/lib/whodunit/railtie.rb

@@ -22,19 +22,19 @@ module Whodunit
 
       # Extend ActiveRecord with migration helpers.
       #
-      # This initializer adds the MigrationHelpers module to ActiveRecord,
+      # This initializer adds the MigrationHelpers module to ActiveRecord::Migration,
       # making methods like add_whodunit_stamps available in migrations.
       # It also extends TableDefinition for automatic whodunit_stamps injection.
       #
       # @api private
       initializer "whodunit.extend_active_record" do |_app|
         ActiveSupport.on_load(:active_record) do
-          extend Whodunit::MigrationHelpers
+          ActiveRecord::Migration.include(Whodunit::MigrationHelpers)
 
-          # Extend TableDefinition for automatic whodunit_stamps injection
-          ActiveRecord::ConnectionAdapters::TableDefinition.include(
-            Whodunit::TableDefinitionExtension
-          )
+          # We use ActiveRecord::ConnectionAdapters::TableDefinition#timestamps to automatically inject whodunit_stamps
+          ActiveRecord::ConnectionAdapters::TableDefinition.prepend(Whodunit::TableDefinitionExtension)
+          # Reuse the `whodunit_stamps` flow in Whodunit::MigrationHelpers module
+          ActiveRecord::ConnectionAdapters::TableDefinition.extend(Whodunit::MigrationHelpers)
         end
       end
 

data/lib/whodunit/stampable.rb

@@ -145,6 +145,8 @@ module Whodunit
       private
 
       def setup_whodunit_associations
+        return if abstract_class?
+
         setup_creator_association if creator_column_exists? && model_creator_enabled?
         setup_updater_association if updater_column_exists? && model_updater_enabled?
         setup_deleter_association if deleter_column_exists? && model_deleter_enabled? && soft_delete_enabled?

data/lib/whodunit/table_definition_extension.rb

@@ -28,30 +28,33 @@ module Whodunit
   module TableDefinitionExtension
     extend ActiveSupport::Concern
 
-    included do
-      # Track whether whodunit_stamps have been automatically added
-      attr_accessor :_whodunit_stamps_added
+    def _whodunit_stamps_added
+      @_whodunit_stamps_added
+    end
+
+    def _whodunit_stamps_added=(value)
+      @_whodunit_stamps_added = value
     end
 
     # Override timestamps to trigger automatic whodunit_stamps injection
     def timestamps(**options)
+      options = options.dup
+      skip = options.delete(:skip_whodunit_stamps)
       result = super
 
-      # Auto-inject whodunit_stamps after timestamps if enabled and not already added
-      if Whodunit.auto_inject_whodunit_stamps &&
-         !@_whodunit_stamps_added &&
-         !options[:skip_whodunit_stamps]
+      if Whodunit.auto_inject_whodunit_stamps && !_whodunit_stamps_added && !skip
         whodunit_stamps(include_deleter: :auto)
-        @_whodunit_stamps_added = true
       end
 
       result
     end
 
-    # Also override whodunit_stamps to track that they've been added
-    def whodunit_stamps(**options)
-      @_whodunit_stamps_added = true
-      super
+    # assign true tp the tracker `@_whodunit_stamps_added` before the
+    # reuse/call of the `whodunit_stamps` flow from the Whodunit::MigrationHelpers module (see railtie.rb)
+    #
+    def whodunit_stamps(include_deleter: :auto, creator_type: nil, updater_type: nil, deleter_type: nil)
+      self._whodunit_stamps_added = true
+      self.class.whodunit_stamps(self, include_deleter:, creator_type:, updater_type:, deleter_type:)
     end
   end
 end

data/lib/whodunit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Whodunit
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: whodunit
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Ken C. Demanawa
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-07-24 00:00:00.000000000 Z
+date: 2026-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: 7.2.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: 7.2.2.2
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: 7.2.3.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: 7.2.3.1
 - !ruby/object:Gem::Dependency
   name: bundler-audit
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '2.5'
+- !ruby/object:Gem::Dependency
+  name: ostruct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -178,6 +192,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.9'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -245,9 +273,9 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '3.1'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="