fixture_kit 0.18.0 → 0.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f929fa8e3c8ecc52b9c9fdb0bacd2b94042f1cd55d91528427af72bb752f033a
-  data.tar.gz: 563261390ebdc5cd01dc17b1e6cf1fa3d304a871ff76ffe03ff54935e886940f
+  metadata.gz: 90cf1c283943f0ec2cf0c82b5f5b6b65de810a1e4e5ed8d15fd12c4b2ba64879
+  data.tar.gz: 2eb935b512b373c399220ed60a6250b3d32349aa9dc5e4c31801d3db45a94b94
 SHA512:
-  metadata.gz: db45522c0b30284ef5bb37ece365d899c351d67e01a3a864a9e0b8909a61f0fdf5a3c0afb18352c86c9187a4400ed83c24364ee2e53331273a712dc6ba6d1659
-  data.tar.gz: 668150429508f1ec34d2c9a82b3610036c3645acd135ebc9894922ea8e08838f619446a38646bca1b0859f1e99b3dc7c7dc72c9241f02317f05fb7084493458d
+  metadata.gz: 8a4f49bf2e6422be97e8a316acfd623c9e27c861bc63c29465dc6d8fb331ddbff85a84aeede77d654e4e544096061b62c8aa9daa3bfac23ec5ae7d6dfc8db957
+  data.tar.gz: a799c0b665d6396837fb82ee50d6e193e3adde8114f0edb63d3f3ed5540205b486587a5ff1065ea5811c2b61fa0ed9cb10f0c43fba18da9b15eb0a913b76f660

data/lib/fixture_kit/adapters/minitest_adapter.rb

@@ -7,15 +7,18 @@ require "active_support/inflector"
 module FixtureKit
   class MinitestAdapter < FixtureKit::Adapter
     TEST_NAME = "fixture kit cache pregeneration"
+    TEST_METHOD = :"test_#{TEST_NAME.tr(" ", "_")}"
 
     def execute(&block)
-      test_class = build_test_class
-      test_method = test_class.test(TEST_NAME) do
+      test = test_class.new(TEST_METHOD)
+      # Minitest runs a test by sending its name to the instance, so defining it
+      # here leaves the reused class untouched and the block dies with the test.
+      test.define_singleton_method(TEST_METHOD) do
         block.call(self)
         pass
       end
 
-      result = test_class.new(test_method).run
+      result = test.run
       return if result.passed?
 
       raise result.failures.first.error
@@ -27,8 +30,11 @@ module FixtureKit
 
     private
 
-    def build_test_class
-      Class.new(ActiveSupport::TestCase) do
+    # Reused rather than built per generation: including
+    # ActiveRecord::TestFixtures appends the class to ActiveSupport's
+    # :active_record_fixtures load hooks, and that never shrinks.
+    def test_class
+      @test_class ||= Class.new(ActiveSupport::TestCase) do
         ::Minitest::Runnable.runnables.delete(self)
         include(::ActiveRecord::TestFixtures)
       end

data/lib/fixture_kit/adapters/rspec_adapter.rb

@@ -7,12 +7,14 @@ module FixtureKit
     def execute(&block)
       previous_example = ::RSpec.current_example
       previous_scope = ::RSpec.current_scope
-      example_group = build_example_group
-      example = example_group.example { block.call(self) }
+      group = example_group
+      example = group.example { block.call(self) }
       succeeded =
         begin
-          example.run(example_group.new, ::RSpec::Core::NullReporter)
+          example.run(group.new, ::RSpec::Core::NullReporter)
         ensure
+          # The group is reused, and the example it holds retains this block.
+          group.examples.clear
           ::RSpec.current_example = previous_example
           ::RSpec.current_scope = previous_scope
         end
@@ -27,8 +29,11 @@ module FixtureKit
 
     private
 
-    def build_example_group
-      ::RSpec::Core::ExampleGroup.subclass(
+    # Reused rather than built per generation: rspec-rails includes
+    # ActiveRecord::TestFixtures into every group, which appends it to
+    # ActiveSupport's :active_record_fixtures load hooks, and that never shrinks.
+    def example_group
+      @example_group ||= ::RSpec::Core::ExampleGroup.subclass(
         ::RSpec::Core::ExampleGroup,
         "FixtureKit",
         [],

data/lib/fixture_kit/cache.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
+require "digest"
+
 module FixtureKit
   class Cache
     ANONYMOUS_DIRECTORY = "_anonymous"
+    DIGEST_LENGTH = 12
 
     include ConfigurationHelper
 
@@ -22,7 +25,8 @@ module FixtureKit
         if raw_identifier.is_a?(String)
           raw_identifier
         else
-          File.join(ANONYMOUS_DIRECTORY, FixtureKit.runner.adapter.identifier_for(raw_identifier))
+          normalized_scope = FixtureKit.runner.adapter.identifier_for(raw_identifier)
+          File.join(ANONYMOUS_DIRECTORY, "#{normalized_scope}.#{definition_digest}")
         end
       end
     end
@@ -53,7 +57,7 @@ module FixtureKit
       FixtureKit.runner.adapter.execute do |context|
         @content = MemoryCache.new(
           data: evaluate(FixtureKit.runner.coders, context),
-          exposed: file_cache.serialize_exposed(fixture.definition.exposed)
+          exposed: fixture.definition.exposed
         )
       end
 
@@ -62,6 +66,26 @@ module FixtureKit
 
     private
 
+    # The scope name alone does not identify a declaration site. Test
+    # frameworks derive it from the group description, which is lossy -- RSpec
+    # strips every non-alphanumeric character, so `Foo::Bar` and `"Foo Bar"`
+    # both become `FooBar` -- and, worse, not even unique within a process:
+    # test runners that load specs in batches call RSpec::Core::World#reset
+    # between them, which drops the constants RSpec disambiguates against, so
+    # the first group of a given description in *every* batch takes the
+    # unsuffixed name. The cache directory is cleared once per process, so
+    # without the digest two spec files sharing a top-level description share
+    # this entry, and the second fixture to generate silently mounts the
+    # first's records.
+    #
+    # Joined to the scope with a dot rather than an underscore: scope names are
+    # snake_case, so an underscore leaves no boundary between the two, while a
+    # dot cannot appear in one -- RSpec strips non-alphanumerics from the group
+    # description before it is underscored.
+    def definition_digest
+      Digest::SHA256.hexdigest(fixture.definition.fingerprint)[0, DIGEST_LENGTH]
+    end
+
     def evaluate(coders, context, data = {}, &block)
       if coders.empty?
         fixture.definition.evaluate(context, parent: fixture.parent&.mount)

data/lib/fixture_kit/definition.rb

@@ -14,6 +14,17 @@ module FixtureKit
       @definition.source_location.first
     end
 
+    def location
+      file, line = @definition.source_location
+      "#{file}:#{line}"
+    end
+
+    # Two definitions sharing a fingerprint evaluate the same block against the
+    # same parent, so they are interchangeable and may share a cache entry.
+    def fingerprint
+      "#{location}:#{extends}"
+    end
+
     def evaluate(context, parent: nil)
       context.singleton_class.prepend(mixin(parent))
       context.instance_exec(&@definition)
@@ -25,12 +36,31 @@ module FixtureKit
           raise FixtureKit::DuplicateNameError, "Name #{name} already exposed"
         end
 
-        @exposed[name] = record
+        @exposed[name] = serialize(name, record)
       end
     end
 
     private
 
+    def serialize(name, record)
+      if record.is_a?(Array)
+        record.map { |item| reference(name, item) }
+      else
+        reference(name, record)
+      end
+    end
+
+    def reference(name, record)
+      unless record.persisted?
+        raise FixtureKit::UnpersistedRecordError,
+          "cannot expose #{name.inspect}: the #{record.class} is not persisted. " \
+          "Exposed records are captured as class/id pairs when `expose` is called, " \
+          "so save the record before exposing it."
+      end
+
+      { record.class => record.id }
+    end
+
     def mixin(parent)
       definition = self
 

data/lib/fixture_kit/file_cache.rb

@@ -48,16 +48,6 @@ module FixtureKit
       File.write(path, JSON.pretty_generate(content))
     end
 
-    def serialize_exposed(exposed)
-      exposed.each_with_object({}) do |(name, record), hash|
-        if record.is_a?(Array)
-          hash[name] = record.map { |record| { record.class => record.id } }
-        else
-          hash[name] = { record.class => record.id }
-        end
-      end
-    end
-
     private
 
     def coder_for(class_name)

data/lib/fixture_kit/fixture.rb

@@ -19,6 +19,8 @@ module FixtureKit
     end
 
     def generate(force: false)
+      FixtureKit.runner.registry.claim_cache_identifier(self)
+
       return if @cache.exists? && !force
 
       parent&.generate

data/lib/fixture_kit/registry.rb

@@ -8,6 +8,7 @@ module FixtureKit
       @declarations = {}
       @fixtures = {}
       @resolving = []
+      @cache_owners = {}
     end
 
     def add(name_or_definition, scope = nil)
@@ -34,6 +35,23 @@ module FixtureKit
       @fixtures.values
     end
 
+    # Fixtures sharing a cache identifier share a cache file: the second one to
+    # generate finds the file already written, skips generation, and mounts the
+    # other's records. That is only safe when both declarations are the same, so
+    # anything else has to fail here -- otherwise it surfaces as a confusing
+    # NoMethodError on Repository, or not at all when both expose the same names.
+    # Identifiers are derived from the declaration site, so this is a backstop
+    # against a digest collision or a regression in that derivation.
+    def claim_cache_identifier(fixture)
+      owner = (@cache_owners[fixture.cache.identifier] ||= fixture)
+      return if owner.equal?(fixture)
+      return if owner.definition.fingerprint == fixture.definition.fingerprint
+
+      raise FixtureKit::CacheIdentifierCollision,
+        "fixtures declared at #{owner.definition.location} and #{fixture.definition.location} " \
+        "both resolve to the cache identifier '#{fixture.cache.identifier}'"
+    end
+
     private
 
     def fetch_named_fixture(name)

data/lib/fixture_kit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FixtureKit
-  VERSION = "0.18.0"
+  VERSION = "0.20.0"
 end

data/lib/fixture_kit.rb

@@ -6,9 +6,11 @@ module FixtureKit
   class InvalidFixtureDeclaration < Error; end
   class MultipleFixtures < Error; end
   class CacheMissingError < Error; end
+  class CacheIdentifierCollision < Error; end
   class FixtureDefinitionNotFound < Error; end
   class RunnerAlreadyStartedError < Error; end
   class CircularFixtureInheritance < Error; end
+  class UnpersistedRecordError < Error; end
 
   autoload :VERSION, File.expand_path("fixture_kit/version", __dir__)
   autoload :Configuration, File.expand_path("fixture_kit/configuration", __dir__)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fixture_kit
 version: !ruby/object:Gem::Version
-  version: 0.18.0
+  version: 0.20.0
 platform: ruby
 authors:
 - Ngan Pham
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-07-20 00:00:00.000000000 Z
+date: 2026-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport