minitest-utils 0.5.0 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee05007ab6791116cc1ea0e62b39d94b2e11cdfebcd02f66c993717ca620c1f4
-  data.tar.gz: a535e5c9454fa423faab453f8719ca1978711e9a07c9423de8aba8778c762446
+  metadata.gz: 88ad6d0826de366086152cedfef5fabbbc338b4aeb8a664ed62d9762002d841d
+  data.tar.gz: 7256702f812f28c4320bf3f42a05733302c963fd6c8093e35b107e21f8248e55
 SHA512:
-  metadata.gz: 5cfdbc7786514bcad2dcf4d43ba3f0377018895027b7600c4a8f2aad954588851df558291db149f69c73ab44b0f0026eaf39393a68789809aaa9a69e49e1a0fb
-  data.tar.gz: 3d2cf46cbc08556df35d4ea73a59782c1ef702a27ad7472b3b79fa145b3c0e01c2efaf5a03254f3a0d86126a3bb32398167d6cee9e39e0610a29b955f070fa68
+  metadata.gz: 4979039508fcb466137ac584c0b8f1cae0115f83ddf3866ec92b26384cde28567e72f17bd7f10499d41b23745c66f0fd49945b677daa08cbbcd87ba227134d47
+  data.tar.gz: ad984963d7267ece2f9f32f93aa752aeca90ee3d4fdd6e35a86a05fd2108b470e7317ff86c67c3f36f25a1bd6644264c5d58b5d8423dc10a3be3dc0c43ef5ae1

data/lib/minitest/utils/extension.rb

@@ -19,27 +19,33 @@ module Minitest
     include ::Minitest::Utils::Assertions
 
     def self.tests
-      @tests ||= []
+      @tests ||= {}
     end
 
     def self.test(name, &block)
       source_location = caller_locations(1..1).first
+      source_location = [
+        Pathname(source_location.path).relative_path_from(Pathname(Dir.pwd)),
+        source_location.lineno
+      ]
+
+      klass = self.name
       method_name = name.downcase
                         .gsub(/[^a-z0-9]+/, "_")
                         .gsub(/^_+/, "")
                         .gsub(/_+$/, "").squeeze("_")
       test_name = "test_#{method_name}".to_sym
-      defined = method_defined? test_name
+      defined = method_defined?(test_name)
+
+      Test.tests["#{klass}##{test_name}"] = {
+        name: name,
+        source_location:,
+        benchmark: nil
+      }
+
       testable = proc do
         benchmark = Benchmark.measure { instance_eval(&block) }
-
-        Test.tests << {
-          class: self.class,
-          name: name,
-          test_name: test_name,
-          benchmark: benchmark,
-          source_location:
-        }
+        Test.tests["#{klass}##{test_name}"][:benchmark] = benchmark
       end
 
       raise "#{test_name} is already defined in #{self}" if defined

data/lib/minitest/utils/reporter.rb

@@ -68,20 +68,22 @@ module Minitest
           failing_results.each {|result| display_replay_command(result) }
           io.puts "\n\n"
         else
-          io.puts "\nSlow Tests:\n"
           threshold = 0.1 # 100ms
           test_results =
             Test
             .tests
+            .values
+            .select { _1[:benchmark] }
             .sort_by { _1[:benchmark].total }
             .reverse
             .take(10).filter { _1[:benchmark].total > threshold }
-          pwd = Pathname(Dir.pwd)
+
+          return unless test_results.any?
+
+          io.puts "\nSlow Tests:\n"
 
           test_results.each_with_index do |info, index|
-            relative_path = Pathname(info[:source_location].absolute_path)
-                            .relative_path_from(pwd)
-            location = "#{relative_path}:#{info[:source_location].lineno}"
+            location = info[:source_location].join(":")
             duration = humanize_duration(info[:benchmark].total * 1_000_000_000)
 
             prefix = "#{index + 1}) "
@@ -164,15 +166,9 @@ module Minitest
       end
 
       private def find_test_file(result)
-        location, line = if result.respond_to?(:source_location)
-                           result.source_location
-                         else
-                           result.method(result.name).source_location
-                         end
+        info = Test.tests.fetch("#{result.klass}##{result.name}")
 
-        location = location.gsub(%r{^.*?/((?:test|spec)/.*?)$}, "\\1")
-
-        [location, line]
+        info[:source_location]
       end
 
       private def backtrace(backtrace)
@@ -191,7 +187,11 @@ module Minitest
         return matches[1] if matches
 
         regex = include_line_number ? /^([^:]+:\d+)/ : /^([^:]+)/
-        location = File.expand_path(location[regex, 1])
+        path = location[regex, 1]
+
+        return location unless path
+
+        location = File.expand_path
 
         return location unless location.start_with?(Dir.pwd)
 
@@ -199,7 +199,8 @@ module Minitest
       end
 
       private def filter_backtrace(backtrace)
-        Minitest.backtrace_filter.filter(backtrace)
+        # drop the last line, which is from benchmark.
+        Minitest.backtrace_filter.filter(backtrace)[0..-2]
       end
 
       private def result_name(name)

data/lib/minitest/utils/version.rb

@@ -2,6 +2,6 @@
 
 module Minitest
   module Utils
-    VERSION = "0.5.0"
+    VERSION = "0.5.2"
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: minitest-utils
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.2
 platform: ruby
 authors:
 - Nando Vieira
 bindir: exe
 cert_chain: []
-date: 2025-02-24 00:00:00.000000000 Z
+date: 2025-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark