test-prof 1.4.0.rc.4 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 46c375967bb8f18cb7a27c0852c3bbc277ea6cae7c0d9ed51d8fe709b1ec67e9
-  data.tar.gz: 133939cf5cd497246b36642d3552ac0883a3a558d32775be960a9f308e3e67a5
+  metadata.gz: 573636b6655a08f1cfa650a2cf0d0136bbbbc85b7e2b84cf76969698fb92b597
+  data.tar.gz: 55e1c58e1a1ed27fd56862c0be79737f3b2261d31bec306a1264b8c08e77c37a
 SHA512:
-  metadata.gz: 830b4ed5c186ecec5a5396ad764f4d8c96700c8c0c3e55b2f0634b937906ec74d39e61ce1e270d9dd93a72b5618aaad87a6c0b316dc42874c3522631e1e8bbc6
-  data.tar.gz: 8e81d280c8f86adf1df271758d7bbf91499d86f2493ecbc6435f23cef73adfc23ed1404f475b10ef749a84dd19d0e87eff42aa61f1040738aef04cdb270a2689
+  metadata.gz: f26c70dd8a5bc0379fc38f49834da4ae2b43bdd5d79f4290a5147a4b4c22255dc9eee1a8d0b4ecce2003625afd295656a6745b1111fce6c9e9dfd618b9c0fe91
+  data.tar.gz: b44cd50ef268582467f18956475d6d767cbea0aef3f78f5ddbdf966d1ba6a07749a9481390fce75be1a667cd9cc8d0e7673868357c7fd241a5cf1e5dea18a02b

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## master (unreleased)
 
+## 1.4.1 (2024-08-23)
+
+- Skips loading the ActiveRecord adapter for runs where RSpec --dry-run mode is enabled. ([@devinburnette][])
+
+## 1.4.0 (2024-08-12)
+
+- AnyFixture: Disable fixtures cache within _clean fixture_ context. Automatically _refind_ records when using `#fixture`. ([@palkan][])
+
 - Add new TPS (tests per second) profiler. ([@palkan][])
 
 - FactoryDefault: add Fabrication support. ([@palkan][])
@@ -430,3 +438,4 @@ See [changelog](https://github.com/test-prof/test-prof/blob/v0.8.0/CHANGELOG.md)
 [@lioneldebauge]: https://github.com/lioneldebauge
 [@lHydra]: https://github.com/lHydra
 [@john-h-k]: https://github.com/john-h-k
+[@devinburnette]: https://github.com/devinburnette

data/lib/test_prof/any_fixture/dsl.rb

@@ -1,5 +1,10 @@
 # frozen_string_literal: true
 
+if defined?(::ActiveRecord::Base)
+  require "test_prof/ext/active_record_refind"
+  using TestProf::Ext::ActiveRecordRefind
+end
+
 module TestProf
   module AnyFixture
     # Adds "global" `fixture`, `before_fixtures_reset` and `after_fixtures_reset` methods (through refinement)
@@ -9,7 +14,20 @@ module TestProf
       # - Rails added `Kernel.prepend` in 6.1: https://github.com/rails/rails/commit/3124007bd674dcdc9c3b5c6b2964dfb7a1a0733c
       refine ::Object do
         def fixture(id, &block)
-          ::TestProf::AnyFixture.register(:"#{id}", &block)
+          id = :"#{id}"
+          record = ::TestProf::AnyFixture.cached(id)
+
+          return ::TestProf::AnyFixture.register(id, &block) unless record
+
+          return record.refind if record.is_a?(::ActiveRecord::Base)
+
+          if record.respond_to?(:to_ary)
+            return record.map do |rec|
+              rec.is_a?(::ActiveRecord::Base) ? rec.refind : rec
+            end
+          end
+
+          record
         end
 
         def before_fixtures_reset(&block)

data/lib/test_prof/any_fixture.rb

@@ -108,14 +108,18 @@ module TestProf
         cached(id) do
           raise "No fixture named #{id} has been registered" unless block_given?
 
+          next yield if @disabled
+
           ActiveSupport::Notifications.subscribed(method(:subscriber), "sql.active_record") do
             yield
           end
         end
       end
 
-      def cached(id)
-        cache.fetch(id) { yield }
+      def cached(id, &block)
+        return (block_given? ? yield : nil) if @disabled
+
+        cache.fetch(id, &block)
       end
 
       # Create and register new SQL dump.
@@ -174,6 +178,14 @@ module TestProf
         callbacks.clear
       end
 
+      def disable!
+        @disabled = true
+      end
+
+      def enable!
+        @disabled = false
+      end
+
       def before_fixtures_reset(&block)
         callbacks[:before_fixtures_reset] << block
       end
@@ -266,5 +278,7 @@ module TestProf
         connection.disable_referential_integrity { yield }
       end
     end
+
+    enable!
   end
 end

data/lib/test_prof/before_all/isolator.rb

@@ -1,20 +1,11 @@
 # frozen_string_literal: true
 
-module TestProf
-  module BeforeAll
-    # Disable Isolator within before_all blocks
-    module Isolator
-      def begin_transaction(*)
-        ::Isolator.transactions_threshold += 1
-        super
-      end
+TestProf::BeforeAll.configure do |config|
+  config.before(:begin) do
+    ::Isolator.incr_thresholds!
+  end
 
-      def rollback_transaction(*)
-        super
-        ::Isolator.transactions_threshold -= 1
-      end
-    end
+  config.after(:rollback) do
+    ::Isolator.decr_thresholds!
   end
 end
-
-TestProf::BeforeAll.singleton_class.prepend(TestProf::BeforeAll::Isolator)

data/lib/test_prof/before_all.rb

@@ -14,8 +14,26 @@ module TestProf
       end
     end
 
+    # Used in dry-run mode
+    class NoopAdapter
+      class << self
+        def begin_transaction(...)
+        end
+
+        def rollback_transaction(...)
+        end
+
+        def setup_fixtures(...)
+        end
+      end
+    end
+
     class << self
-      attr_accessor :adapter
+      attr_writer :adapter
+
+      def adapter
+        @adapter ||= default_adapter
+      end
 
       def begin_transaction(scope = nil, metadata = [])
         raise AdapterMissing if adapter.nil?
@@ -47,6 +65,17 @@ module TestProf
       def configure
         yield config
       end
+
+      private
+
+      def default_adapter
+        return NoopAdapter if TestProf.dry_run?
+
+        if defined?(::ActiveRecord::Base)
+          require "test_prof/before_all/adapters/active_record"
+          Adapters::ActiveRecord
+        end
+      end
     end
 
     class HookEntry # :nodoc:
@@ -138,12 +167,6 @@ module TestProf
   end
 end
 
-if defined?(::ActiveRecord::Base)
-  require "test_prof/before_all/adapters/active_record"
-
-  TestProf::BeforeAll.adapter = TestProf::BeforeAll::Adapters::ActiveRecord
-end
-
 if defined?(::Isolator)
   require "test_prof/before_all/isolator"
 end

data/lib/test_prof/cops/rspec/aggregate_examples/its.rb

@@ -64,7 +64,7 @@ module RuboCop
             return super unless its?(example.send_node)
 
             # First parameter to `its` is not metadata.
-            example.send_node.arguments[1..-1]
+            example.send_node.arguments[1..]
           end
 
           def its?(node)

data/lib/test_prof/cops/rspec/aggregate_examples.rb

@@ -123,7 +123,7 @@ module RuboCop
         def on_block(node)
           example_group_with_several_examples(node) do |all_examples|
             example_clusters(all_examples).each do |_, examples|
-              examples[1..-1].each do |example|
+              examples[1..].each do |example|
                 add_offense(example,
                   location: :expression,
                   message: message_for(example, examples[0]))
@@ -141,7 +141,7 @@ module RuboCop
               range = range_for_replace(examples)
               replacement = aggregated_example(examples, metadata)
               corrector.replace(range, replacement)
-              examples[1..-1].map { |example| drop_example(corrector, example) }
+              examples[1..].map { |example| drop_example(corrector, example) }
             end
           end
         end

data/lib/test_prof/core.rb

@@ -75,6 +75,10 @@ module TestProf
       defined?(::Spring::Application) && (disabled.nil? || disabled.empty? || disabled == "0")
     end
 
+    def dry_run?
+      rspec? && ::RSpec.configuration.dry_run?
+    end
+
     # Returns the current process time
     def now
       Process.clock_gettime_for_test_prof(Process::CLOCK_MONOTONIC)

data/lib/test_prof/ext/string_truncate.rb

@@ -12,7 +12,7 @@ module TestProf
         head = ((limit - 3) / 2)
         tail = head + 3 - limit
 
-        "#{self[0..(head - 1)]}...#{self[tail..-1]}"
+        "#{self[0..(head - 1)]}...#{self[tail..]}"
       end
     end
   end

data/lib/test_prof/recipes/rspec/any_fixture.rb

@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
 require "test_prof/any_fixture"
+require "test_prof/any_fixture/dsl"
+require "test_prof/ext/active_record_refind"
+
 require "test_prof/recipes/rspec/before_all"
 
 RSpec.shared_context "any_fixture:clean" do
@@ -8,6 +11,11 @@ RSpec.shared_context "any_fixture:clean" do
 
   before_all do
     TestProf::AnyFixture.clean
+    TestProf::AnyFixture.disable!
+  end
+
+  after(:all) do
+    TestProf::AnyFixture.enable!
   end
 end
 

data/lib/test_prof/rspec_stamp/parser.rb

@@ -143,7 +143,7 @@ module TestProf
           elsif expr.first == :@const
             expr[1]
           elsif expr.first == :const_path_ref
-            expr[1..-1].map(&method(:parse_const)).join("::")
+            expr[1..].map(&method(:parse_const)).join("::")
           end
         end
       end

data/lib/test_prof/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TestProf
-  VERSION = "1.4.0.rc.4"
+  VERSION = "1.4.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 1.4.0.rc.4
+  version: 1.4.1
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-07-26 00:00:00.000000000 Z
+date: 2024-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -252,9 +252,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.4.19
 signing_key: