test-prof 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d33f8b474fb308b33c253a2213cc86f192c976b9b6289e73511f2d4f5b9dc5f
-  data.tar.gz: c150fd5213ba2aefa34f3d56c75bcb6d69d7172a281d090b1d14a7a5b9f389d1
+  metadata.gz: c3ea5b7078b12209f32a60554f25acffccd184bdf852ebfe14cad161ded2c180
+  data.tar.gz: 34e61ec6d58e5688b441ea09c1be524ae563f9ca82f334541c3f2c3197787cbd
 SHA512:
-  metadata.gz: 18295b823f620a143a867e183ccf203f08b08a9e8e344ed1b95c076a0cfe98b84688e090cb8cbe8a33f60d17b249a1828afb57d7bdeba962207f253f4f6b4e5a
-  data.tar.gz: 86ff82bdd8f3f6a17f15d44234034c71847c2a459d907d8c6acad11c52bcfac98b8fe60b584bbcb7c5b73ac718c6e90e17b6c9fc1741d19d34e8bcebf01f5310
+  metadata.gz: d866bfe03650d81cb2dd5b698b5de5703a8fa505c7c8d0b0e4283b8130eb774ac43c5cca6a750fef6f8342acdec7e518613489e898fd354f133e1793df833e45
+  data.tar.gz: cf92cef8973837d2e09ff3bcc4449fb0675ebfca3f10fc75a314f10d885ad92aaae2f6fac0cd99e7250eb57985208e6d6f53b88d101f177709a172b8fcae1815

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## master (unrealeased)
 
+## 1.0.3 (2021-04-30)
+
+- Minor fixes.
+
 ## 1.0.2 (2021-02-26)
 
 - Make `before_all(setup_fixtures: true)` compatible with Rails 6.1. ([@palkan][])

data/README.md

@@ -32,7 +32,7 @@ TestProf toolbox aims to help you identify bottlenecks in your test suite. It co
 📑 [Documentation](https://test-prof.evilmartians.io)
 
 <p align="center">
-  <a href="http://bit.ly/test-prof-map">
+  <a href="http://bit.ly/test-prof-map-v1">
     <img src="./docs/assets/images/coggle.png" alt="TestProf map" width="738">
   </a>
 </p>

data/lib/test_prof.rb

@@ -161,5 +161,5 @@ require "test_prof/factory_doctor"
 require "test_prof/factory_prof"
 require "test_prof/rspec_stamp"
 require "test_prof/tag_prof"
-require "test_prof/rspec_dissect"
+require "test_prof/rspec_dissect" if TestProf.rspec?
 require "test_prof/factory_all_stub"

data/lib/test_prof/any_fixture.rb

@@ -6,7 +6,7 @@ require "test_prof/any_fixture/dump"
 module TestProf
   # Make DB fixtures from blocks.
   module AnyFixture
-    INSERT_RXP = /^INSERT INTO ([\S]+)/.freeze
+    INSERT_RXP = /^INSERT INTO (\S+)/.freeze
 
     using FloatDuration
 
@@ -16,8 +16,8 @@ module TestProf
         :import_dump_via_cli, :dump_matching_queries, :force_matching_dumps
       attr_reader :default_dump_watch_paths
 
-      alias reporting_enabled? reporting_enabled
-      alias import_dump_via_cli? import_dump_via_cli
+      alias_method :reporting_enabled?, :reporting_enabled
+      alias_method :import_dump_via_cli?, :import_dump_via_cli
 
       def initialize
         @reporting_enabled = ENV["ANYFIXTURE_REPORT"] == "1"
@@ -42,7 +42,7 @@ module TestProf
       end
 
       def before_dump(&block)
-        if block_given?
+        if block
           @before_dump << block
         else
           @before_dump
@@ -50,7 +50,7 @@ module TestProf
       end
 
       def after_dump(&block)
-        if block_given?
+        if block
           @after_dump << block
         else
           @after_dump
@@ -112,7 +112,7 @@ module TestProf
         config.reporting_enabled
       end
 
-      alias reporting_enabled? reporting_enabled
+      alias_method :reporting_enabled?, :reporting_enabled
 
       # Register a block of code as a fixture,
       # returns the result of the block execution

data/lib/test_prof/any_fixture/dump.rb

@@ -6,9 +6,9 @@ require "set"
 
 module TestProf
   module AnyFixture
-    MODIFY_RXP = /^(INSERT INTO|UPDATE|DELETE FROM) ([\S]+)/i.freeze
-    ANY_FIXTURE_RXP = /(\/\*|\-\-).*\bany_fixture:dump/.freeze
-    ANY_FIXTURE_IGNORE_RXP = /(\/\*|\-\-).*\bany_fixture:ignore/.freeze
+    MODIFY_RXP = /^(INSERT INTO|UPDATE|DELETE FROM) (\S+)/i.freeze
+    ANY_FIXTURE_RXP = /(\/\*|--).*\bany_fixture:dump/.freeze
+    ANY_FIXTURE_IGNORE_RXP = /(\/\*|--).*\bany_fixture:ignore/.freeze
 
     using(Module.new do
       refine Object do
@@ -111,7 +111,7 @@ module TestProf
       end
 
       attr_reader :name, :digest, :path, :subscriber, :success
-      alias success? success
+      alias_method :success?, :success
 
       def initialize(name, watch: [], cache_key: nil)
         @name = name

data/lib/test_prof/before_all.rb

@@ -79,7 +79,7 @@ module TestProf
       #   config.before(:rollback) { ... }
       def before(type, &block)
         validate_hook_type!(type)
-        hooks[type].before << block if block_given?
+        hooks[type].before << block if block
       end
 
       # Add `after` hook for `begin` or
@@ -88,7 +88,7 @@ module TestProf
       #   config.after(:begin) { ... }
       def after(type, &block)
         validate_hook_type!(type)
-        hooks[type].after << block if block_given?
+        hooks[type].after << block if block
       end
 
       def run_hooks(type) # :nodoc:

data/lib/test_prof/event_prof/custom_events.rb

@@ -6,7 +6,7 @@ module TestProf
     module CustomEvents
       class << self
         def register(event, &block)
-          raise ArgumentError, "Block is required!" unless block_given?
+          raise ArgumentError, "Block is required!" unless block
           registrations[event] = block
         end
 

data/lib/test_prof/event_prof/instrumentations/active_support.rb

@@ -26,7 +26,7 @@ module TestProf::EventProf
 
       class << self
         def subscribe(event, &block)
-          raise ArgumentError, "Block is required!" unless block_given?
+          raise ArgumentError, "Block is required!" unless block
 
           ::ActiveSupport::Notifications.subscribe(event, Subscriber.new(block))
         end

data/lib/test_prof/event_prof/profiler.rb

@@ -6,7 +6,7 @@ module TestProf
       attr_reader :event, :total_count, :total_time, :rank_by, :top_count, :per_example,
         :time, :count, :example_time, :example_count, :absolute_run_time
 
-      alias per_example? per_example
+      alias_method :per_example?, :per_example
 
       def initialize(event:, instrumenter:, rank_by: :time, top_count: 5, per_example: false)
         @event = event
@@ -88,14 +88,13 @@ module TestProf
       end
 
       def results
-        results = {
+        {
           groups: @groups.to_a
         }.tap do |data|
           next unless per_example?
 
           data[:examples] = @examples.to_a
         end
-        results
       end
 
       def take_time(start_ts)
@@ -130,7 +129,7 @@ module TestProf
       end
 
       def each(&block)
-        if block_given?
+        if block
           @profilers.each(&block)
         else
           @profilers.each

data/lib/test_prof/factory_prof.rb

@@ -10,7 +10,7 @@ module TestProf
   # flamegraphs or detect most popular factories
   module FactoryProf
     FACTORY_BUILDERS = [FactoryBuilders::FactoryBot,
-                        FactoryBuilders::Fabrication].freeze
+      FactoryBuilders::Fabrication].freeze
 
     # FactoryProf configuration
     class Configuration

data/lib/test_prof/recipes/logging.rb

@@ -12,7 +12,7 @@ module TestProf
         def logger
           return @logger if instance_variable_defined?(:@logger)
 
-          @logger = Logger.new(STDOUT)
+          @logger = Logger.new($stdout)
         end
 
         def ar_loggables

data/lib/test_prof/recipes/minitest/before_all.rb

@@ -11,8 +11,8 @@ module TestProf
         attr_reader :active, :block, :captured_ivars, :teardown_block, :current_test_object,
           :setup_fixtures
 
-        alias active? active
-        alias setup_fixtures? setup_fixtures
+        alias_method :active?, :active
+        alias_method :setup_fixtures?, :setup_fixtures
 
         def initialize(setup_fixtures: false, &block)
           @setup_fixtures = setup_fixtures

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

@@ -7,7 +7,7 @@ module TestProf
     # Helper to wrap the whole example group into a transaction
     module RSpec
       def before_all(setup_fixtures: BeforeAll.config.setup_fixtures, &block)
-        raise ArgumentError, "Block is required!" unless block_given?
+        raise ArgumentError, "Block is required!" unless block
 
         if within_before_all?
           before(:all) do

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

@@ -55,9 +55,7 @@ module TestProf
       end
 
       def module_for(group)
-        modules[group] ||= begin
-          Module.new.tap { |mod| group.prepend(mod) }
-        end
+        modules[group] ||= Module.new.tap { |mod| group.prepend(mod) }
       end
 
       private
@@ -91,8 +89,7 @@ module TestProf
       initializer = proc do
         instance_variable_set(:"#{TestProf::LetItBe::PREFIX}#{identifier}", instance_exec(&block))
       rescue FrozenError => e
-        e.message << TestProf::LetItBe::FROZEN_ERROR_HINT
-        raise
+        raise e.exception("#{e.message}#{TestProf::LetItBe::FROZEN_ERROR_HINT}")
       end
 
       default_options = LetItBe.config.default_modifiers.dup
@@ -243,7 +240,7 @@ end
 RSpec.configure do |config|
   config.after(:example) do |example|
     if example.exception&.is_a?(FrozenError)
-      example.exception.message << TestProf::LetItBe::FROZEN_ERROR_HINT
+      example.exception.message << TestProf::LetItBe::FROZEN_ERROR_HINT unless example.exception.message.frozen?
     end
   end
 end

data/lib/test_prof/rspec_dissect.rb

@@ -38,7 +38,7 @@ module TestProf
       attr_accessor :top_count, :let_stats_enabled,
         :let_top_count
 
-      alias let_stats_enabled? let_stats_enabled
+      alias_method :let_stats_enabled?, :let_stats_enabled
 
       attr_reader :mode
 
@@ -139,7 +139,7 @@ end
 
 require "test_prof/rspec_dissect/collectors/let"
 require "test_prof/rspec_dissect/collectors/before"
-require "test_prof/rspec_dissect/rspec" if TestProf.rspec?
+require "test_prof/rspec_dissect/rspec"
 
 TestProf.activate("RD_PROF") do
   TestProf::RSpecDissect.init

data/lib/test_prof/ruby_prof.rb

@@ -239,16 +239,18 @@ module TestProf
       end
 
       def exclude_common_methods(profiler)
-        profiler.exclude_methods!(
-          TSort,
-          :tsort_each
-        )
+        if defined?(TSort)
+          profiler.exclude_methods!(
+            TSort,
+            :tsort_each
+          )
 
-        profiler.exclude_methods!(
-          TSort.singleton_class,
-          :tsort_each, :each_strongly_connected_component,
-          :each_strongly_connected_component_from
-        )
+          profiler.exclude_methods!(
+            TSort.singleton_class,
+            :tsort_each, :each_strongly_connected_component,
+            :each_strongly_connected_component_from
+          )
+        end
 
         profiler.exclude_methods!(
           BasicObject,

data/lib/test_prof/utils/sized_ordered_set.rb

@@ -14,7 +14,7 @@ module TestProf
       def initialize(max_size, sort_by: nil, &block)
         @max_size = max_size
         @comparator =
-          if block_given?
+          if block
             block
           elsif !sort_by.nil?
             ->(x, y) { x[sort_by] >= y[sort_by] }
@@ -42,7 +42,7 @@ module TestProf
       end
 
       def each(&block)
-        if block_given?
+        if block
           data.each(&block)
         else
           data.each

data/lib/test_prof/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Vladimir Dementyev
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-26 00:00:00.000000000 Z
+date: 2021-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -219,7 +219,7 @@ metadata:
   documentation_uri: https://test-prof.evilmartians.io/
   homepage_uri: https://test-prof.evilmartians.io/
   source_code_uri: http://github.com/test-prof/test-prof
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -234,8 +234,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: Ruby applications tests profiling tools
 test_files: []