test-prof 1.0.9 → 1.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9680aa33c1d917182a0be6e8cfd0656149d510b634db637b8023feda29f10886
-  data.tar.gz: 721767f61684ed87eff0f2e4c62080b9bb2a9e24bf6f47db194fc687bedbbe9b
+  metadata.gz: 1cd40e8800d63a753008f4d9b27418db0015bd052ae3b12eb773c7d7380bc1af
+  data.tar.gz: 48cc4a7c0c5105e7b23bf4793419dffa049d1154ebbf124f0a24f83ec2a4d79a
 SHA512:
-  metadata.gz: 5a1d037cdac034f7c58f7bd090de128a81d41c3927787c953cacd33e8ecec1dd11cbda282f64fd542c2679b5176aa7bce4fe2ecb49806abac895d2cf9f478c00
-  data.tar.gz: baf837db1165a0d5631f07c9f4a978234afcce65fd9be886ef80bcf0a020fd43926e57eb5f295f0daddb22027c821ee72960fb01cd2cd4c7db6d18100d404343
+  metadata.gz: abdda954d6dce088d9dd015394ebfdca5904973244f2a0c194a2203e8a2ef0a565b2926831770deb4f6e957781a635c2d302860e59148967bf9bc8c10439cd72
+  data.tar.gz: 0db8384bf621fb4a1857dabc02e6e516d99d702ee81a8f1f777c2ffd9e9890dc8f312be1f55172b942b93385168e432bb1f5df0ad03779e21f80f96f868a9889

data/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 ## master (unreleased)
 
+## 1.0.11 (2022-10-27)
+
+- Fix monitoring methods with keyword args in Ruby 3+. ([@palkan][])
+
+- Disable garbage collection frames when `TEST_STACK_PROF_IGNORE_GC` env variable is set ([@cbliard][])
+
+- Fixed restoring lock_thread value in nested contexts ([@ygelfand][])
+
+## 1.0.10 (2022-08-12)
+
+- Allow overriding global logger. ([@palkan][])
+
+```ruby
+require "test_prof/recipes/logging"
+
+TestProf::Rails::LoggingHelpers.logger = CustomLogger.new
+```
+
 ## 1.0.9 (2022-05-05)
 
 - Add `AnyFixture.before_fixtures_reset` and `AnyFixture.after_fixtures_reset` callbacks. ([@ruslanshakirov][])
@@ -284,3 +302,5 @@ See [changelog](https://github.com/test-prof/test-prof/blob/v0.8.0/CHANGELOG.md)
 [@grillermo]: https://github.com/grillermo
 [@cou929]: https://github.com/cou929
 [@ruslanshakirov]: https://github.com/ruslanshakirov
+[@ygelfand]: https://github.com/ygelfand
+[@cbliard]: https://github.com/cbliard

data/lib/test_prof/before_all/adapters/active_record.rb

@@ -47,7 +47,7 @@ module TestProf
       # 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))
+        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
 

data/lib/test_prof/event_prof/monitor.rb

@@ -48,9 +48,16 @@ module TestProf
 
           patch = Module.new do
             mids.each do |mid|
-              define_method(mid) do |*args, &block|
-                next super(*args, &block) unless guard.nil? || instance_exec(*args, &guard)
-                tracker.track { super(*args, &block) }
+              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
               end
             end
           end

data/lib/test_prof/event_prof.rb

@@ -8,7 +8,7 @@ require "test_prof/utils/sized_ordered_set"
 
 module TestProf
   # EventProf profiles your tests and suites against custom events,
-  # such as ActiveSupport::Notifacations.
+  # such as ActiveSupport::Notifications.
   #
   # It works very similar to `rspec --profile` but can track arbitrary events.
   #

data/lib/test_prof/factory_doctor/minitest.rb

@@ -74,8 +74,8 @@ module Minitest
         @example_groups.each do |group, examples|
           msgs << "#{group[:description]} (#{group[:location]})\n"
           examples.each do |ex|
-            msgs << "  #{ex[:description]} (#{ex[:location]}) "\
-                    "– #{pluralize_records(ex[:factories])} created, "\
+            msgs << "  #{ex[:description]} (#{ex[:location]}) " \
+                    "– #{pluralize_records(ex[:factories])} created, " \
                     "#{ex[:time].duration}\n"
           end
           msgs << "\n"

data/lib/test_prof/factory_doctor/rspec.rb

@@ -67,7 +67,7 @@ module TestProf
 
           examples.each do |ex|
             msgs << "  #{ex.description} (#{ex.metadata[:location]}) " \
-                    "– #{pluralize_records(ex.metadata[:factories])} created, "\
+                    "– #{pluralize_records(ex.metadata[:factories])} created, " \
                     "#{ex.metadata[:time].duration}\n"
           end
           msgs << "\n"

data/lib/test_prof/logging.rb

@@ -11,7 +11,7 @@ module TestProf
 
     class Formatter
       def call(severity, _time, progname, msg)
-        colorize(severity.to_sym, "[#{progname} #{severity}] #{msg}\n")
+        colorize(severity.to_sym, "[#{progname} #{severity}] #{msg}") + "\n"
       end
 
       private

data/lib/test_prof/recipes/logging.rb

@@ -7,12 +7,38 @@ module TestProf
     # Add `with_logging` and `with_ar_logging helpers`
     module LoggingHelpers
       class << self
-        attr_writer :logger
+        def logger=(logger)
+          @logger = logger
+
+          # swap global loggers
+          global_loggables.each do |loggable|
+            loggable.logger = logger
+          end
+        end
 
         def logger
           return @logger if instance_variable_defined?(:@logger)
 
-          @logger = Logger.new($stdout)
+          @logger = if defined?(ActiveSupport::TaggedLogging)
+            ActiveSupport::TaggedLogging.new(ActiveSupport::Logger.new($stdout))
+          elsif defined?(ActiveSupport::Logger)
+            ActiveSupport::Logger.new($stdout)
+          else
+            Logger.new($stdout)
+          end
+        end
+
+        def global_loggables
+          return @global_loggables if instance_variable_defined?(:@global_loggables)
+
+          @global_loggables = []
+        end
+
+        def swap_logger!(loggables)
+          loggables.each do |loggable|
+            loggable.logger = logger
+            global_loggables << loggable
+          end
         end
 
         def ar_loggables
@@ -77,7 +103,9 @@ end
 if TestProf.rspec?
   RSpec.shared_context "logging:verbose" do
     around(:each) do |ex|
-      with_logging(&ex)
+      next with_logging(&ex) if ex.metadata[:log] == true || ex.metadata[:log] == :all
+
+      ex.call
     end
   end
 
@@ -96,13 +124,10 @@ end
 
 TestProf.activate("LOG", "all") do
   TestProf.log :info, "Rails verbose logging enabled"
-  ActiveSupport::LogSubscriber.logger =
-    Rails.logger =
-      ActiveRecord::Base.logger = TestProf::Rails::LoggingHelpers.logger
+  TestProf::Rails::LoggingHelpers.swap_logger!(TestProf::Rails::LoggingHelpers.all_loggables)
 end
 
 TestProf.activate("LOG", "ar") do
   TestProf.log :info, "Active Record verbose logging enabled"
-  ActiveSupport::LogSubscriber.logger =
-    ActiveRecord::Base.logger = TestProf::Rails::LoggingHelpers.logger
+  TestProf::Rails::LoggingHelpers.swap_logger!(TestProf::Rails::LoggingHelpers.ar_loggables)
 end

data/lib/test_prof/rspec_stamp.rb

@@ -153,8 +153,8 @@ module TestProf
           end
         end
 
-        replacement = "\\1#{parsed.fname}#{need_parens ? "(" : " "}"\
-                      "#{[desc, tags_str, htags_str].compact.join(", ")}"\
+        replacement = "\\1#{parsed.fname}#{need_parens ? "(" : " "}" \
+                      "#{[desc, tags_str, htags_str].compact.join(", ")}" \
                       "#{need_parens ? ") " : " "}\\3"
 
         if config.dry_run?

data/lib/test_prof/stack_prof.rb

@@ -24,8 +24,7 @@ module TestProf
     class Configuration
       FORMATS = %w[html json].freeze
 
-      attr_accessor :mode, :interval, :raw, :target, :format
-
+      attr_accessor :mode, :raw, :target, :format, :interval, :ignore_gc
       def initialize
         @mode = ENV.fetch("TEST_STACK_PROF_MODE", :wall).to_sym
         @target = ENV["TEST_STACK_PROF"] == "boot" ? :boot : :suite
@@ -39,6 +38,7 @@ module TestProf
 
         sample_interval = ENV["TEST_STACK_PROF_INTERVAL"].to_i
         @interval = sample_interval > 0 ? sample_interval : nil
+        @ignore_gc = !ENV["TEST_STACK_PROF_IGNORE_GC"].nil?
       end
 
       def raw?
@@ -96,6 +96,7 @@ module TestProf
         }
 
         options[:interval] = config.interval if config.interval
+        options[:ignore_gc] = true if config.ignore_gc
 
         if block_given?
           options[:out] = build_path(name)

data/lib/test_prof/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TestProf
-  VERSION = "1.0.9"
+  VERSION = "1.0.11"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 1.0.9
+  version: 1.0.11
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-06 00:00:00.000000000 Z
+date: 2022-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -237,7 +237,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.11
 signing_key:
 specification_version: 4
 summary: Ruby applications tests profiling tools