test-prof 1.3.0 → 1.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0678bc968a267ebeece00e12338f6408683140428ecdd041ece64c70e6bb91c0'
-  data.tar.gz: 3ab5bc88b7b5dea557847f05a16d7fc3e4c6c9212115cd16984a56b7dd7b1737
+  metadata.gz: 3cfa0f1751f398092de8ffa46d5bff81bb9eadd7b90caaf9293c62c93c6b3e20
+  data.tar.gz: 3afcb831782f0e58881839f33af07a41b3b42d63ac8eb87c35b6fd0eb01537ea
 SHA512:
-  metadata.gz: 5b32e7e689cef27e72ce5f7e65ca0463bae7b83dfcaf77eba17aa0cc9bae1b74d6223b6b141e805debb7640b74b75be34bf238a8f41cb3efaf0386929935d814
-  data.tar.gz: 34200774bc30c39acc48109fbc5031fd8f48abb5f7175799a1e1bbfd7650a11a441c9ba63638bf186d78ef5cd15b7b33e64782c781a7f28bd43e43cb6d1e5588
+  metadata.gz: 2de8fb8232f5ed623167846c37cce6224b297761324acdf6a17a47338e2a3904aa480b48fc9793a51f5595d3d839c3ff3bc9fbb262c7900ad3d7c819113947a0
+  data.tar.gz: dcec92d722f9e7cce9f1d371a2c6d363aa9a78a6cf1149a73d8b836a269ebf6bd14e50d86a8470cccc56a861b94646ad23494f43de6d9beb0cc6a5b478f77e93

data/CHANGELOG.md

@@ -2,6 +2,78 @@
 
 ## master (unreleased)
 
+## 1.4.4 (2025-01-03)
+
+- Fix _stamping_ specs with single quotes with Rpsec Stamp. ([@elasticspoon][])
+
+## 1.4.3 (2024-12-18)
+
+- Fix handling new (lazy) connection pools in `before_all`. ([@palkan][])
+
+- Updates Rubocop::Cop code to comply with most modern API. ([@aseroff][])
+
+## 1.4.2 (2024-09-03) 🗓️
+
+- Ignore default scopes in `ActiveRecord::Base#refind`. ([@palkan][])
+
+## 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][])
+
+- Drop support for **Ruby <2.7** and **Rails <6**.
+
+- FactoryDefault: Add `#get_factory_default`. [[@john-h-k][]]
+
+- Add variations information to FactorProf reports. ([@lHydra][])
+
+Get info on traits/overrides used by running `FPROF=1 FPROF_VARS=1 <your test command>`.
+
+- Add support for `report_duplicates` config option for `let_it_be` ([@lHydra][])
+
+- Support latest Timecop patching `Process.clock_gettime`. ([@palkan][])
+
+- Vernier: Add hooks configuration parameter. ([@lHydra][])
+
+Now you can add more insights to the resulting report by adding event markers from Active Support Notifications.
+To do this, specify the `TEST_VERNIER_HOOKS=rails` env var or set it through `Vernier` configuration:
+
+```ruby
+TestProf::Vernier.configure do |config|
+  config.hooks = :rails
+end
+```
+
+- FactoryProf: Add threshold configuration parameter. ([@lHydra][])
+
+Now you can ignore factories which total number of calls is less than the provided threshold. To do this, specify
+the `FPROF_THRESHOLD=30` env var or set it through `FactoryProf` configuration:
+
+```ruby
+TestProf::FactoryProf.configure do |config|
+  config.threshold = 30
+end
+```
+
+## 1.3.3 (2024-04-19)
+
+- Fix MemProf bugs. ([@palkan][])
+
+## 1.3.2 (2024-03-08) 🌷
+
+- Add Minitest support for TagProf. ([@lioneldebauge][])
+
+## 1.3.1 (2023-12-12)
+
+- Add support for dumping FactoryProf results in JSON format. ([@uzushino][])
+
 ## 1.3.0 (2023-11-21)
 
 - Add Vernier integration. ([@palkan][])
@@ -376,3 +448,9 @@ See [changelog](https://github.com/test-prof/test-prof/blob/v0.8.0/CHANGELOG.md)
 [@peret]: https://github.com/peret
 [@bf4]: https://github.com/bf4
 [@Vankiru]: https://github.com/Vankiru
+[@uzushino]: https://github.com/uzushino
+[@lioneldebauge]: https://github.com/lioneldebauge
+[@lHydra]: https://github.com/lHydra
+[@john-h-k]: https://github.com/john-h-k
+[@devinburnette]: https://github.com/devinburnette
+[@elasticspoon]: https://github.com/elasticspoon

data/README.md

@@ -83,9 +83,9 @@ And that's it)
 
 Supported Ruby versions:
 
-- Ruby (MRI) >= 2.5.0 (**NOTE:** for Ruby 2.2 use TestProf < 0.7.0, Ruby 2.3 use TestProf ~> 0.7.0, Ruby 2.4 use TestProf <0.12.0)
+- Ruby (MRI) >= 2.7.0 (**NOTE:** for Ruby 2.2 use TestProf < 0.7.0, Ruby 2.3 use TestProf ~> 0.7.0, Ruby 2.4 use TestProf <0.12.0, Ruby 2.5-2.6 use TestProf < 1.3)
 
-- JRuby >= 9.1.0.0 (**NOTE:** refinements-dependent features might require 9.2.7+)
+- JRuby >= 9.3.0
 
 Supported RSpec version (for RSpec features only): >= 3.5.0 (for older RSpec versions use TestProf < 0.8.0).
 

data/lib/minitest/test_prof_plugin.rb

@@ -3,6 +3,7 @@
 require "test_prof/event_prof/minitest"
 require "test_prof/factory_doctor/minitest"
 require "test_prof/memory_prof/minitest"
+require "test_prof/tag_prof/minitest"
 
 module Minitest # :nodoc:
   module TestProf # :nodoc:
@@ -16,6 +17,7 @@ module Minitest # :nodoc:
         opts[:sample] = true if ENV["SAMPLE"] || ENV["SAMPLE_GROUPS"]
         opts[:mem_prof_mode] = ENV["TEST_MEM_PROF"] if ENV["TEST_MEM_PROF"]
         opts[:mem_prof_top_count] = ENV["TEST_MEM_PROF_COUNT"] if ENV["TEST_MEM_PROF_COUNT"]
+        opts[:tag_prof] = true if ENV["TAG_PROF"] == "type"
       end
     end
   end
@@ -50,6 +52,7 @@ module Minitest # :nodoc:
     reporter << TestProf::EventProfReporter.new(options[:io], options) if options[:event]
     reporter << TestProf::FactoryDoctorReporter.new(options[:io], options) if options[:fdoc]
     reporter << TestProf::MemoryProfReporter.new(options[:io], options) if options[:mem_prof_mode]
+    reporter << Minitest::TestProf::TagProfReporter.new(options[:io], options) if options[:tag_prof]
 
     ::TestProf::MinitestSample.call if options[:sample]
   end

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/dump.rb

@@ -69,7 +69,7 @@ module TestProf
         end
 
         def commit
-          return unless defined?(:@file)
+          return unless instance_variable_defined?(:@file)
 
           file.close
 

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/adapters/active_record.rb

@@ -8,44 +8,79 @@ module TestProf
         POOL_ARGS = ((::ActiveRecord::VERSION::MAJOR > 6) ? [:writing] : []).freeze
 
         class << self
-          def all_connections
-            @all_connections ||= if ::ActiveRecord::Base.respond_to? :connects_to
-              ::ActiveRecord::Base.connection_handler.connection_pool_list(*POOL_ARGS).filter_map { |pool|
-                begin
-                  pool.connection
-                rescue *pool_connection_errors => error
-                  log_pool_connection_error(pool, error)
-                  nil
-                end
-              }
-            else
-              Array.wrap(::ActiveRecord::Base.connection)
+          if ::ActiveRecord::Base.connection.pool.respond_to?(:pin_connection!)
+            def begin_transaction
+              subscribe!
+              ::ActiveRecord::Base.connection_handler.connection_pool_list(:writing).each do |pool|
+                pool.pin_connection!(true)
+              end
             end
-          end
 
-          def pool_connection_errors
-            @pool_connection_errors ||= []
-          end
+            def rollback_transaction
+              ::ActiveRecord::Base.connection_handler.connection_pool_list(:writing).each do |pool|
+                pool.unpin_connection!
+              end
+              unsubscribe!
+            end
 
-          def log_pool_connection_error(pool, error)
-            warn "Could not connect to pool #{pool.connection_class.name}. #{error.class}: #{error.message}"
-          end
+            def subscribe!
+              Thread.current[:before_all_connection_subscriber] = ActiveSupport::Notifications.subscribe("!connection.active_record") do |_, _, _, _, payload|
+                connection_name = payload[:connection_name] if payload.key?(:connection_name)
+                shard = payload[:shard] if payload.key?(:shard)
+                next unless connection_name
 
-          def begin_transaction
-            @all_connections = nil
-            all_connections.each do |connection|
-              connection.begin_transaction(joinable: false)
+                pool = ::ActiveRecord::Base.connection_handler.retrieve_connection_pool(connection_name, shard: shard)
+                next unless pool && pool.role == :writing
+
+                pool.pin_connection!(true)
+              end
             end
-          end
 
-          def rollback_transaction
-            all_connections.each do |connection|
-              if connection.open_transactions.zero?
-                warn "!!! before_all transaction has been already rollbacked and " \
-                      "could work incorrectly"
-                next
+            def unsubscribe!
+              return unless Thread.current[:before_all_connection_subscriber]
+              ActiveSupport::Notifications.unsubscribe(Thread.current[:before_all_connection_subscriber])
+              Thread.current[:before_all_connection_subscriber] = nil
+            end
+          else
+            def all_connections
+              @all_connections ||= if ::ActiveRecord::Base.respond_to? :connects_to
+                ::ActiveRecord::Base.connection_handler.connection_pool_list(*POOL_ARGS).filter_map { |pool|
+                  begin
+                    pool.connection
+                  rescue *pool_connection_errors => error
+                    log_pool_connection_error(pool, error)
+                    nil
+                  end
+                }
+              else
+                Array.wrap(::ActiveRecord::Base.connection)
+              end
+            end
+
+            def pool_connection_errors
+              @pool_connection_errors ||= []
+            end
+
+            def log_pool_connection_error(pool, error)
+              warn "Could not connect to pool #{pool.connection_class.name}. #{error.class}: #{error.message}"
+            end
+
+            def begin_transaction
+              @all_connections = nil
+              all_connections.each do |connection|
+                connection.begin_transaction(joinable: false)
+              end
+            end
+
+            def rollback_transaction
+              all_connections.each do |connection|
+                if connection.open_transactions.zero?
+                  warn "!!! before_all transaction has been already rollbacked and " \
+                        "could work incorrectly"
+                  next
+                end
+                connection.rollback_transaction
               end
-              connection.rollback_transaction
             end
           end
 
@@ -67,23 +102,23 @@ module TestProf
       end
     end
 
-    # avoid instance variable collisions with cats
-    PREFIX_RESTORE_LOCK_THREAD = "@😺"
-
-    configure do |config|
-      # Make sure ActiveRecord uses locked thread.
-      # It only gets locked in `before` / `setup` hook,
-      # thus using thread in `before_all` (e.g. ActiveJob async adapter)
-      # might lead to leaking connections
-      config.before(:begin) do
-        next unless ::ActiveRecord::Base.connection.pool.respond_to?(:lock_thread=)
-        instance_variable_set("#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread", ::ActiveRecord::Base.connection.pool.instance_variable_get(:@lock_thread)) unless instance_variable_defined? "#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread"
-        ::ActiveRecord::Base.connection.pool.lock_thread = true
-      end
+    unless ::ActiveRecord::Base.connection.pool.respond_to?(:pin_connection!)
+      # avoid instance variable collisions with cats
+      PREFIX_RESTORE_LOCK_THREAD = "@😺"
 
-      config.after(:rollback) do
-        next unless ::ActiveRecord::Base.connection.pool.respond_to?(:lock_thread=)
-        ::ActiveRecord::Base.connection.pool.lock_thread = instance_variable_get("#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread")
+      configure do |config|
+        # Make sure ActiveRecord uses locked thread.
+        # It only gets locked in `before` / `setup` hook,
+        # thus using thread in `before_all` (e.g. ActiveJob async adapter)
+        # might lead to leaking connections
+        config.before(:begin) do
+          instance_variable_set("#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread", ::ActiveRecord::Base.connection.pool.instance_variable_get(:@lock_thread)) unless instance_variable_defined? "#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread"
+          ::ActiveRecord::Base.connection.pool.lock_thread = true
+        end
+
+        config.after(:rollback) do
+          ::ActiveRecord::Base.connection.pool.lock_thread = instance_variable_get("#{PREFIX_RESTORE_LOCK_THREAD}_orig_lock_thread")
+        end
       end
     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:
@@ -66,7 +95,7 @@ module TestProf
       private
 
       def filters_apply?(metadata)
-        return true unless filters.present? && TestProf.rspec?
+        return true unless filters.is_a?(Hash) && TestProf.rspec?
 
         ::RSpec::Core::MetadataFilter.apply?(
           :all?,
@@ -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

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
         # @example `its`
         #
         #   # Supports regular `its` call with an attribute/method name,
@@ -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/line_range_helpers.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
         # @internal Support methods for keeping newlines around examples.
         module LineRangeHelpers
           include RangeHelp

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

@@ -5,7 +5,7 @@ require "test_prof/cops/rspec/language"
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
         # When aggregated, the expectations will fail when not supposed to or
         # have a risk of not failing when expected to. One example is
         # `validate_presence_of :comment` as it leaves an empty comment after

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

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
         # @internal
         #   Support methods for example metadata.
         #   Examples with similar metadata are grouped.

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

@@ -5,7 +5,7 @@ require "test_prof/cops/rspec/language"
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
         # @internal
         #   Node matchers and searchers.
         module NodeMatchers

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

@@ -108,7 +108,8 @@ module RuboCop
       #     expect(number).to be_odd
       #   end
       #
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < ::RuboCop::Cop::Base
+        extend AutoCorrector
         include LineRangeHelpers
         include MetadataHelpers
         include NodeMatchers
@@ -123,29 +124,22 @@ 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.drop(1).each do |example|
                 add_offense(example,
-                  location: :expression,
-                  message: message_for(example, examples[0]))
+                  message: message_for(example, examples[0])) do |corrector|
+                    clusters = example_clusters_for_autocorrect(example)
+                    clusters.each do |metadata, examples|
+                      range = range_for_replace(examples)
+                      replacement = aggregated_example(examples, metadata)
+                      corrector.replace(range, replacement)
+                      examples.drop(1).map { |example| drop_example(corrector, example) }
+                    end
+                  end
               end
             end
           end
         end
 
-        def autocorrect(example_node)
-          clusters = example_clusters_for_autocorrect(example_node)
-          return if clusters.empty?
-
-          lambda do |corrector|
-            clusters.each do |metadata, examples|
-              range = range_for_replace(examples)
-              replacement = aggregated_example(examples, metadata)
-              corrector.replace(range, replacement)
-              examples[1..-1].map { |example| drop_example(corrector, example) }
-            end
-          end
-        end
-
         private
 
         # Clusters of examples in the same example group, on the same nesting

data/lib/test_prof/core.rb

@@ -6,6 +6,29 @@ require "logger"
 require "test_prof/logging"
 require "test_prof/utils"
 
+# Add an alias for Process.clock_gettime "reserved" for TestProf
+# (in case some other tool would like to patch it)
+module ::Process
+  class << self
+    # Already patched by Timecop
+    if method_defined?(:clock_gettime_without_mock)
+      alias_method :clock_gettime_for_test_prof, :clock_gettime_without_mock
+    else
+      alias_method :clock_gettime_for_test_prof, :clock_gettime
+
+      def singleton_method_added(method_name)
+        return super unless method_name == :clock_gettime_without_mock
+
+        define_method(:clock_gettime_for_test_prof) { |*args| clock_gettime_without_mock(*args) }
+      end
+    end
+  end
+end
+
+# Main TestProf module
+#
+# Contains configuration and common methods
+
 # Ruby applications tests profiling tools.
 #
 # Contains tools to analyze factories usage, integrate with Ruby profilers,
@@ -52,9 +75,13 @@ 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(Process::CLOCK_MONOTONIC)
+      Process.clock_gettime_for_test_prof(Process::CLOCK_MONOTONIC)
     end
 
     # Require gem and shows a custom

data/lib/test_prof/event_prof/monitor.rb

@@ -48,16 +48,9 @@ module TestProf
 
           patch = Module.new do
             mids.each do |mid|
-              if RUBY_VERSION >= "2.7.0"
-                define_method(mid) do |*args, **kwargs, &block|
-                  next super(*args, **kwargs, &block) unless guard.nil? || instance_exec(*args, **kwargs, &guard)
-                  tracker.track { super(*args, **kwargs, &block) }
-                end
-              else
-                define_method(mid) do |*args, &block|
-                  next super(*args, &block) unless guard.nil? || instance_exec(*args, &guard)
-                  tracker.track { super(*args, &block) }
-                end
+              define_method(mid) do |*args, **kwargs, &block|
+                next super(*args, **kwargs, &block) unless guard.nil? || instance_exec(*args, **kwargs, &guard)
+                tracker.track { super(*args, **kwargs, &block) }
               end
             end
           end

data/lib/test_prof/ext/active_record_refind.rb

@@ -12,7 +12,7 @@ module TestProf
         #
         # We need it to make sure that the state is clean.
         def refind
-          self.class.find(send(self.class.primary_key))
+          self.class.unscoped.find(send(self.class.primary_key))
         end
       end
     end

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