fast_cov 0.3.0 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a8e169598d457f54b52e872c65d55a9e87164bcac66654d7b21d9497aeba43a
-  data.tar.gz: 678150e27c7fb664bc5786764b063c1abb75218048f226b39495bc97c012ce75
+  metadata.gz: 8d5eb20089cfe981168592cd5672e1919746a5186c61f151aff87eea7319eb1c
+  data.tar.gz: af45acb129fd680beaa1a906248ad1aedf03e10a5d40d02fbeaa0971e9112b44
 SHA512:
-  metadata.gz: 68e671eab2e3583313b60e408f179006d77aa0d571d1b258b971916b38642fcedae05b8b728ff0588000bd69ada0183a3502b8f6b4e90a37fda3847be3dd6fcd
-  data.tar.gz: d4531b99a2ca2cfb1a566f63bed51444a91bedfd0356a33d50101f18aafa0020ed40d652199de538cb351c6ac2e5e6a7185c1a3aaef316bc444779767a19ad23
+  metadata.gz: 97cdc30f83fb1970d97d6cddc04758463043a85ee81ccd5a22011e0b017a04d8b9b82be90931b42bdf5565ef163ac84cc1544ec43ecae2cc1727041770adfc52
+  data.tar.gz: baa28e4ae365439250404b285eb0577a167de1cb907a3c37b33f09d3a99bf53bf4e6ca0ac6fd2ff84360a048f2795de7ca709ffd11c1236b0846ae66a4f4a75a

data/ext/fast_cov/fast_cov.c

@@ -140,80 +140,6 @@ static void on_line_event(rb_event_flag_t event, VALUE self_data, VALUE self,
   record_impacted_file(data, filename);
 }
 
-// ---- Utils module methods (FastCov::Utils) ------------------------------
-
-// Utils.path_within?(path, directory) -> true/false
-// Check if path is within directory, correctly handling:
-// - Trailing slashes on directory
-// - Sibling directories with longer names (e.g., /a/b/c vs /a/b/cd)
-static VALUE utils_path_within(VALUE self, VALUE path, VALUE directory) {
-  Check_Type(path, T_STRING);
-  Check_Type(directory, T_STRING);
-
-  // Freeze strings to prevent GC compaction from moving them
-  rb_str_freeze(path);
-  rb_str_freeze(directory);
-
-  bool result = fast_cov_is_within_root(
-      RSTRING_PTR(path), RSTRING_LEN(path),
-      RSTRING_PTR(directory), RSTRING_LEN(directory));
-
-  return result ? Qtrue : Qfalse;
-}
-
-// Utils.relativize_paths(set, root) -> set
-// Mutates set in place: converts absolute paths to relative paths from root.
-// Paths not within root are left unchanged.
-static VALUE utils_relativize_paths(VALUE self, VALUE set, VALUE root) {
-  Check_Type(root, T_STRING);
-
-  // Freeze root to prevent GC from moving it during compaction
-  rb_str_freeze(root);
-
-  const char *root_ptr = RSTRING_PTR(root);
-  long root_len = RSTRING_LEN(root);
-
-  // Normalize: strip trailing slash for offset calculation
-  long effective_root_len = root_len;
-  if (effective_root_len > 0 && root_ptr[effective_root_len - 1] == '/') {
-    effective_root_len--;
-  }
-
-  // Collect paths to transform (can't modify set while iterating)
-  VALUE paths = rb_funcall(set, rb_intern("to_a"), 0);
-  long num_paths = RARRAY_LEN(paths);
-
-  for (long i = 0; i < num_paths; i++) {
-    VALUE abs_path = rb_ary_entry(paths, i);
-    if (!RB_TYPE_P(abs_path, T_STRING)) continue;
-
-    // Freeze to prevent GC moving it
-    rb_str_freeze(abs_path);
-
-    const char *path_ptr = RSTRING_PTR(abs_path);
-    long path_len = RSTRING_LEN(abs_path);
-
-    // Use proper within_root check
-    if (!fast_cov_is_within_root(path_ptr, path_len, root_ptr, root_len)) {
-      continue;
-    }
-
-    // Calculate offset (skip root + separator)
-    long offset = effective_root_len;
-    if (offset < path_len && path_ptr[offset] == '/') offset++;
-
-    // Create relative path
-    VALUE rel_path = rb_str_substr(abs_path, offset, path_len - offset);
-
-    // Delete old path, add new path
-    rb_funcall(set, rb_intern("delete"), 1, abs_path);
-    rb_funcall(set, rb_intern("add"), 1, rel_path);
-  }
-
-  RB_GC_GUARD(paths);
-  return set;
-}
-
 // ---- Ruby instance methods ----------------------------------------------
 
 static VALUE fast_cov_initialize(int argc, VALUE *argv, VALUE self) {
@@ -366,8 +292,4 @@ void Init_fast_cov(void) {
   rb_define_method(cCoverage, "start", fast_cov_start, 0);
   rb_define_method(cCoverage, "stop", fast_cov_stop, 0);
 
-  // FastCov::Utils module (C-defined methods)
-  VALUE mUtils = rb_define_module_under(mFastCov, "Utils");
-  rb_define_module_function(mUtils, "path_within?", utils_path_within, 2);
-  rb_define_module_function(mUtils, "relativize_paths", utils_relativize_paths, 2);
 }

data/lib/fast_cov/trackers/abstract_tracker.rb

@@ -15,12 +15,18 @@ module FastCov
   # - threads: true  -> record from ALL threads (global tracking)
   # - threads: false -> only record from the thread that called start
   class AbstractTracker
+    attr_reader :coverage_map
+
     def initialize(coverage_map, **_options)
       @coverage_map = coverage_map
       @files = nil
       @started_thread = nil
     end
 
+    def root
+      coverage_map.root
+    end
+
     # Public API - called by FastCov framework
 
     def start
@@ -70,12 +76,12 @@ module FastCov
     class << self
       attr_accessor :active
 
-      def record(to: nil)
+      def record(path, to: nil)
         return unless active
-        return unless block_given?
+        return unless path
 
-        path = yield
-        active.record(path, to: to) if path
+        to ||= Utils.resolve_caller(caller_locations(1, 20), active.root)
+        active.record(path, to: to)
       end
 
       def reset

data/lib/fast_cov/trackers/const_get_tracker.rb

@@ -23,9 +23,8 @@ module FastCov
 
     module ConstGetPatch
       def const_get(name, inherit = true)
-        source = caller_locations(1, 1).first&.absolute_path
         result = super
-        FastCov::ConstGetTracker.record(to: source) { const_source_location(name, inherit)&.first }
+        FastCov::ConstGetTracker.record(const_source_location(name, inherit)&.first)
         result
       end
     end

data/lib/fast_cov/trackers/factory_bot_tracker.rb

@@ -12,9 +12,7 @@ module FastCov
   # Register via: coverage_map.use(FastCov::FactoryBotTracker)
   class FactoryBotTracker < AbstractTracker
     def install
-      unless defined?(::FactoryBot)
-        raise LoadError, "FactoryBotTracker requires the factory_bot gem to be installed"
-      end
+      gem "factory_bot"
 
       return if ::FactoryBot.factories.singleton_class.ancestors.include?(RegistryPatch)
 
@@ -42,7 +40,7 @@ module FastCov
           next unless block.is_a?(Proc)
 
           location = block.source_location
-          record { location&.first }
+          record(location&.first)
         end
       end
     end

data/lib/fast_cov/trackers/file_tracker.rb

@@ -4,21 +4,27 @@ require_relative "abstract_tracker"
 
 module FastCov
   # Tracks files read from disk during coverage (JSON, YAML, .rb templates, etc.)
-  # via File.read and File.open.
+  # via File.read, File.open, and YAML load methods.
+  #
+  # YAML methods are patched separately because Bootsnap's compile cache
+  # overrides YAML.load_file/safe_load_file to bypass File.open entirely.
   #
   # Register via: coverage_map.use(FastCov::FileTracker)
   class FileTracker < AbstractTracker
     def install
-      return if File.singleton_class.ancestors.include?(FilePatch)
+      unless File.singleton_class.ancestors.include?(FilePatch)
+        File.singleton_class.prepend(FilePatch)
+      end
 
-      File.singleton_class.prepend(FilePatch)
+      if defined?(::YAML) && !::YAML.singleton_class.ancestors.include?(YamlPatch)
+        ::YAML.singleton_class.prepend(YamlPatch)
+      end
     end
 
     module FilePatch
       def read(name, *args, **kwargs, &block)
-        source = caller_locations(1, 1).first&.absolute_path
         super.tap do
-          FastCov::FileTracker.record(to: source) { File.expand_path(name) }
+          FastCov::FileTracker.record(File.expand_path(name))
         end
       end
 
@@ -26,9 +32,28 @@ module FastCov
         mode = args[0]
         is_read = mode.nil? || (mode.is_a?(String) && mode.start_with?("r")) ||
                   (mode.is_a?(Integer) && (mode & (File::WRONLY | File::RDWR)).zero?)
-        source = caller_locations(1, 1).first&.absolute_path
         super.tap do
-          FastCov::FileTracker.record(to: source) { File.expand_path(name) } if is_read
+          FastCov::FileTracker.record(File.expand_path(name)) if is_read
+        end
+      end
+    end
+
+    module YamlPatch
+      def load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
+        end
+      end
+
+      def safe_load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
+        end
+      end
+
+      def unsafe_load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
         end
       end
     end

data/lib/fast_cov/trackers/fixture_kit_tracker.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require_relative "abstract_tracker"
+
+module FastCov
+  # Tracks FixtureKit fixture definition files when fixtures are used.
+  #
+  # Fixture definitions run once during cache generation (before(:context)),
+  # then every test replays cached SQL without executing Ruby. This tracker
+  # uses FixtureKit's callback hooks to:
+  #
+  # 1. Connect the fixture file to all source files touched during generation
+  # 2. Record the fixture definition file when a test mounts a fixture
+  #
+  # Requires fixture_kit >= 0.14.0 (Event-based callbacks).
+  #
+  # Register via: coverage_map.use(FastCov::FixtureKitTracker)
+  class FixtureKitTracker < AbstractTracker
+    def install
+      gem "fixture_kit", ">= 0.14.0"
+
+      tracker = self
+
+      FixtureKit.configure do |config|
+        # When a fixture is about to be generated, start the coverage map
+        # to track what files the fixture definition touches.
+        # This runs in before(:context), before the formatter starts coverage.
+        config.on_cache_save do |_fixture|
+          tracker.coverage_map.start
+        end
+
+        # After generation, stop coverage and connect the fixture file
+        # to everything it touched.
+        config.on_cache_saved do |fixture, _duration|
+          files = tracker.coverage_map.stop
+          files.each do |file|
+            tracker.coverage_map.connect(from: fixture.path, to: file)
+          end
+        end
+
+        # When a test mounts a fixture, record the fixture definition file
+        # and any parent fixture files in the chain.
+        config.on_cache_mount do |event|
+          tracker.class.record(event.path)
+          parent = event.fixture.parent
+          while parent
+            tracker.class.record(parent.definition.path)
+            parent = parent.parent
+          end
+        end
+      end
+    end
+  end
+end

data/lib/fast_cov/utils.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module FastCov
+  module Utils
+    # Check if path is within directory, correctly handling:
+    # - Trailing slashes on directory
+    # - Sibling directories with longer names (e.g., /a/b/c vs /a/b/cd)
+    def self.path_within?(path, directory)
+      dir = directory.end_with?("/") ? directory.chop : directory
+      return true if path == dir
+
+      path.start_with?("#{dir}/")
+    end
+
+    # Mutates set in place: converts absolute paths to relative paths from root.
+    # Paths not within root are left unchanged.
+    def self.relativize_paths(set, root)
+      prefix = root.end_with?("/") ? root : "#{root}/"
+
+      set.to_a.each do |abs_path|
+        next unless abs_path.is_a?(String)
+        next unless abs_path.start_with?(prefix) || abs_path == root.chomp("/")
+
+        set.delete(abs_path)
+        set.add(abs_path.delete_prefix(prefix))
+      end
+
+      set
+    end
+
+    # Walk caller locations to find the first frame whose file is within root.
+    # Handles indirect calls (e.g., YAML.load_file -> File.open) where the
+    # immediate caller is a stdlib/gem file outside the project.
+    def self.resolve_caller(locations, root)
+      locations.each do |loc|
+        path = loc.absolute_path
+        next unless path
+
+        return path if path_within?(path, root)
+      end
+      nil
+    end
+  end
+end

data/lib/fast_cov/version.rb

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

data/lib/fast_cov.rb

@@ -3,6 +3,7 @@
 require "fast_cov/fast_cov.#{RUBY_VERSION}"
 
 module FastCov
+  autoload :Utils, File.expand_path("fast_cov/utils", __dir__)
   autoload :VERSION, File.expand_path("fast_cov/version", __dir__)
   autoload :ConnectedDependencies, File.expand_path("fast_cov/connected_dependencies", __dir__)
   autoload :CoverageMap, File.expand_path("fast_cov/coverage_map", __dir__)
@@ -10,5 +11,6 @@ module FastCov
   autoload :FileTracker, File.expand_path("fast_cov/trackers/file_tracker", __dir__)
   autoload :FactoryBotTracker, File.expand_path("fast_cov/trackers/factory_bot_tracker", __dir__)
   autoload :ConstGetTracker, File.expand_path("fast_cov/trackers/const_get_tracker", __dir__)
+  autoload :FixtureKitTracker, File.expand_path("fast_cov/trackers/fixture_kit_tracker", __dir__)
   autoload :StaticMap, File.expand_path("fast_cov/static_map", __dir__)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fast_cov
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.2
 platform: ruby
 authors:
 - Ngan Pham
@@ -65,6 +65,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '6.0'
+- !ruby/object:Gem::Dependency
+  name: fixture_kit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.14.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.14.0
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  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'
 description: A high-performance C extension that tracks which Ruby source files are
   executed during test runs, enabling test impact analysis.
 executables: []
@@ -91,6 +119,8 @@ files:
 - lib/fast_cov/trackers/const_get_tracker.rb
 - lib/fast_cov/trackers/factory_bot_tracker.rb
 - lib/fast_cov/trackers/file_tracker.rb
+- lib/fast_cov/trackers/fixture_kit_tracker.rb
+- lib/fast_cov/utils.rb
 - lib/fast_cov/version.rb
 homepage: https://github.com/Gusto/fast_cov
 licenses: