test-prof 0.12.0 → 1.0.7

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

@@ -39,7 +39,7 @@ module TestProf
           acc << if v.nil?
             k.to_sym
           else
-            Hash[k.to_sym, v.to_sym]
+            {k.to_sym => v.to_sym}
           end
         end
       end

data/lib/test_prof/rubocop.rb

@@ -11,4 +11,3 @@ require "rubocop"
 
 require_relative "cops/inject"
 require "test_prof/cops/rspec/aggregate_examples"
-require "test_prof/cops/rspec/aggregate_failures"

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/tag_prof/result.rb

@@ -12,7 +12,7 @@ module TestProf
 
         @data = Hash.new do |h, k|
           h[k] = {value: k, count: 0, time: 0.0}
-          h[k].merge!(Hash[events.map { |event| [event, 0.0] }]) unless
+          h[k].merge!(events.map { |event| [event, 0.0] }.to_h) unless
             events.empty?
           h[k]
         end

data/lib/test_prof/tag_prof/rspec.rb

@@ -53,11 +53,9 @@ module TestProf
       def fetch_events_data
         return {} unless @events_profiler
 
-        Hash[
-          @events_profiler.profilers.map do |profiler|
-            [profiler.event, profiler.time]
-          end
-        ]
+        @events_profiler.profilers.map do |profiler|
+          [profiler.event, profiler.time]
+        end.to_h
       end
     end
   end

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 = "0.12.0"
+  VERSION = "1.0.7"
 end

data/lib/test_prof.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 require "fileutils"
+require "logger"
+
 require "test_prof/version"
 require "test_prof/logging"
 require "test_prof/utils"
@@ -44,6 +46,13 @@ module TestProf
       defined?(Minitest)
     end
 
+    # Returns true if Spring is used and not disabled
+    def spring?
+      # See https://github.com/rails/spring/blob/577cf01f232bb6dbd0ade7df2df2ac209697e741/lib/spring/binstub.rb
+      disabled = ENV["DISABLE_SPRING"]
+      defined?(::Spring::Application) && (disabled.nil? || disabled.empty? || disabled == "0")
+    end
+
     # Returns the current process time
     def now
       Process.clock_gettime(Process::CLOCK_MONOTONIC)
@@ -63,7 +72,7 @@ module TestProf
     # equal to the provided value (if any).
     # Contains workaround for applications using Spring.
     def activate(env_var, val = nil)
-      if defined?(::Spring::Application)
+      if spring?
         notify_spring_detected
         ::Spring.after_fork do
           activate!(env_var, val) do
@@ -130,7 +139,7 @@ module TestProf
 
   # TestProf configuration
   class Configuration
-    attr_accessor :output, # IO to write output messages.
+    attr_accessor :output, # IO to write logs
       :color, # Whether to colorize output or not
       :output_dir, # Directory to store artifacts
       :timestamps, # Whether to use timestamped names for artifacts,
@@ -145,12 +154,16 @@ module TestProf
     end
 
     def color?
-      color == true
+      color == true && output.is_a?(IO) && output.tty?
     end
 
     def timestamps?
       timestamps == true
     end
+
+    def logger
+      @logger ||= Logger.new(output, formatter: Logging::Formatter.new)
+    end
   end
 end
 
@@ -161,5 +174,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"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 1.0.7
 platform: ruby
 authors:
 - Vladimir Dementyev
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-17 00:00:00.000000000 Z
+date: 2021-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -119,6 +119,7 @@ files:
 - assets/tagprof.demo.html
 - assets/tagprof.template.html
 - config/default.yml
+- config/rubocop-rspec.yml
 - lib/minitest/base_reporter.rb
 - lib/minitest/event_prof_formatter.rb
 - lib/minitest/test_prof_plugin.rb
@@ -126,6 +127,11 @@ files:
 - lib/test_prof.rb
 - lib/test_prof/any_fixture.rb
 - lib/test_prof/any_fixture/dsl.rb
+- lib/test_prof/any_fixture/dump.rb
+- lib/test_prof/any_fixture/dump/base_adapter.rb
+- lib/test_prof/any_fixture/dump/digest.rb
+- lib/test_prof/any_fixture/dump/postgresql.rb
+- lib/test_prof/any_fixture/dump/sqlite.rb
 - lib/test_prof/before_all.rb
 - lib/test_prof/before_all/adapters/active_record.rb
 - lib/test_prof/before_all/isolator.rb
@@ -136,7 +142,6 @@ files:
 - lib/test_prof/cops/rspec/aggregate_examples/matchers_with_side_effects.rb
 - lib/test_prof/cops/rspec/aggregate_examples/metadata_helpers.rb
 - lib/test_prof/cops/rspec/aggregate_examples/node_matchers.rb
-- lib/test_prof/cops/rspec/aggregate_failures.rb
 - lib/test_prof/cops/rspec/language.rb
 - lib/test_prof/event_prof.rb
 - lib/test_prof/event_prof/custom_events.rb
@@ -148,7 +153,6 @@ files:
 - lib/test_prof/event_prof/monitor.rb
 - lib/test_prof/event_prof/profiler.rb
 - lib/test_prof/event_prof/rspec.rb
-- lib/test_prof/ext/active_record_3.rb
 - lib/test_prof/ext/active_record_refind.rb
 - lib/test_prof/ext/array_bsearch_index.rb
 - lib/test_prof/ext/factory_bot_strategy.rb
@@ -170,11 +174,11 @@ files:
 - lib/test_prof/factory_prof/factory_bot_patch.rb
 - lib/test_prof/factory_prof/factory_builders/fabrication.rb
 - lib/test_prof/factory_prof/factory_builders/factory_bot.rb
+- lib/test_prof/factory_prof/nate_heckler.rb
 - lib/test_prof/factory_prof/printers/flamegraph.rb
+- lib/test_prof/factory_prof/printers/nate_heckler.rb
 - lib/test_prof/factory_prof/printers/simple.rb
 - lib/test_prof/logging.rb
-- lib/test_prof/recipes/active_record_one_love.rb
-- lib/test_prof/recipes/active_record_shared_connection.rb
 - lib/test_prof/recipes/logging.rb
 - lib/test_prof/recipes/minitest/before_all.rb
 - lib/test_prof/recipes/minitest/sample.rb
@@ -208,16 +212,17 @@ files:
 - lib/test_prof/utils/rspec.rb
 - lib/test_prof/utils/sized_ordered_set.rb
 - lib/test_prof/version.rb
-homepage: http://github.com/palkan/test-prof
+homepage: http://github.com/test-prof/test-prof
 licenses:
 - MIT
 metadata:
-  bug_tracker_uri: http://github.com/palkan/test-prof/issues
-  changelog_uri: https://github.com/palkan/test-prof/blob/master/CHANGELOG.md
+  bug_tracker_uri: https://github.com/test-prof/test-prof/issues
+  changelog_uri: https://github.com/test-prof/test-prof/blob/master/CHANGELOG.md
   documentation_uri: https://test-prof.evilmartians.io/
   homepage_uri: https://test-prof.evilmartians.io/
-  source_code_uri: http://github.com/palkan/test-prof
-post_install_message: 
+  source_code_uri: https://github.com/test-prof/test-prof
+  funding_uri: https://github.com/sponsors/test-prof
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -232,8 +237,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: []

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

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      class AggregateExamples
-        def self.registry
-          RuboCop::Cop::Cop.registry
-        end
-      end
-
-      class AggregateFailures < AggregateExamples
-        def initialize(*)
-          super
-          self.class.just_once { warn "`AggregateFailures` cop has been renamed to `AggregateExamples`." }
-        end
-
-        def self.just_once
-          return if @already_done
-          yield
-          @already_done = true
-        end
-      end
-    end
-  end
-end

data/lib/test_prof/ext/active_record_3.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-module TestProf
-  # Add missing `begin_transaction` and `rollback_transaction` methods
-  module ActiveRecord3Transactions
-    refine ::ActiveRecord::ConnectionAdapters::AbstractAdapter do
-      def begin_transaction(joinable: true)
-        increment_open_transactions
-        if open_transactions > 0
-          create_savepoint
-        else
-          begin_db_transaction
-        end
-        self.transaction_joinable = joinable
-      end
-
-      def rollback_transaction(*)
-        if open_transactions > 1
-          rollback_to_savepoint
-        else
-          rollback_db_transaction
-        end
-        decrement_open_transactions
-      end
-    end
-  end
-end

data/lib/test_prof/recipes/active_record_one_love.rb

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "./active_record_shared_connection"
-
-# One ❤️
-TestProf::ActiveRecordOneLove = TestProf::ActiveRecordSharedConnection

data/lib/test_prof/recipes/active_record_shared_connection.rb

@@ -1,77 +0,0 @@
-# frozen_string_literal: true
-
-module TestProf
-  # Forces ActiveRecord to use the same connection between threads
-  module ActiveRecordSharedConnection # :nodoc: all
-    class << self
-      attr_reader :connection
-
-      def enable!
-        self.connection = ActiveRecord::Base.connection
-      end
-
-      def disable!
-        self.connection = nil
-      end
-
-      def ignore(&block)
-        raise ArgumentError, "Block is required" unless block_given?
-
-        @ignores ||= []
-
-        ignores << block
-      end
-
-      def ignored?(config)
-        !ignores.nil? && ignores.any? { |clbk| clbk.call(config) }
-      end
-
-      private
-
-      attr_reader :ignores
-
-      def connection=(conn)
-        @connection = conn
-        connection.singleton_class.prepend Connection
-        connection
-      end
-    end
-
-    module Connection
-      def shared_lock
-        @shared_lock ||= Mutex.new
-      end
-
-      def exec_cache(*)
-        shared_lock.synchronize { super }
-      end
-
-      def exec_no_cache(*)
-        shared_lock.synchronize { super }
-      end
-
-      def execute(*)
-        shared_lock.synchronize { super }
-      end
-    end
-
-    module Ext
-      def connection
-        return super if ActiveRecordSharedConnection.ignored?(connection_config)
-        ActiveRecordSharedConnection.connection || super
-      end
-    end
-  end
-end
-
-ActiveSupport.on_load(:active_record) do
-  if ::ActiveRecord::Base.connection.pool.respond_to?(:lock_thread=)
-    TestProf.log :warn, "You activated ActiveRecordSharedConnection patch for the Rails version,\n" \
-                        "which has a built-in support for the same functionality.\n" \
-                        "Consider removing it, 'cause this could result in unexpected behaviour.\n\n" \
-                        "Read more in the docs: https://test-prof.evilmartians.io/#/active_record_shared_connection"
-  end
-
-  TestProf::ActiveRecordSharedConnection.enable!
-  ActiveRecord::Base.singleton_class.prepend TestProf::ActiveRecordSharedConnection::Ext
-end