test-prof 0.10.0 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 19888b286ec85650d1d6c24c8d92ad1643e46cc506c2f9ce5e65c2f787f43949
-  data.tar.gz: ad6477b9980cd68782ceb2c2b138b9f01a3248f09ebf23e60acbe77e3124fc63
+  metadata.gz: ccd09eb7997710ebeda54cadf25fa4f9bc4f3d9fce1a180113b29c5d92b22ea8
+  data.tar.gz: 32f50fb3af431b6d33622e03e0dcf720a3dfbcf34f52391be421753c8dede348
 SHA512:
-  metadata.gz: 3da02b7d11494242e30360c10c2f69123e2c43f1b3e3ebd6edea567f4f298bb572daa726d8520d524602cb5b5ce2a6b8d68d59ad21238d9d6c1060b4d2132bf4
-  data.tar.gz: 9b5831c47845a4c4337898b5e1a58f495e60bca3e1726813b30469162544b77f715f4f0a07d901026d991304d65077abe16ae3d61c990d8fe136937e3cd4ceb3
+  metadata.gz: f22b272e49c8423c1b3270202a4594b8acc082082f98418dc4ff781fa9711ea8c1dd2c89097bd8bdda972619a720bfc9c0258534fed80dd5c112186f02c254a4
+  data.tar.gz: f64bd5dbdd363d37346fe56eb3c2fb8223d38596a327b5b32408b5a13c9fd0649d3ebe4b25783923e41974e3eb5f6af69e260501d56c124f8fb05c6bcffaf3dd

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## master (unreleased)
 
+## 0.10.1 (2019-10-17)
+
+- Fix AnyFixture DSL when using with Rails 6.1+. ([@palkan][])
+
+- Fix loading `let_it_be` without ActiveRecord present. ([@palkan][])
+
+- Fix compatibility of `before_all` with [`isolator`](https://github.com/palkan/isolator) gem to handle correct usages of non-atomic interactions outside DB transactions. ([@Envek][])
+
+- Updates FactoryProf to show the amount of time taken per factory call. ([@tyleriguchi][])
+
 ## 0.10.0 (2019-08-19)
 
 - Use RSpec example ID instead of full description for RubyProf/Stackprof report names. ([@palkan][])
@@ -498,3 +508,5 @@ Fixes [#10](https://github.com/palkan/test-prof/issues/10).
 [@mkldon]: https://github.com/mkldon
 [@dmagro]: https://github.com/dmagro
 [@danielwaterworth]: https://github.com/danielwaterworth
+[@Envek]: https://github.com/Envek
+[@tyleriguchi]: https://github.com/tyleriguchi

data/README.md

@@ -1,5 +1,7 @@
 [![Cult Of Martians](http://cultofmartians.com/assets/badges/badge.svg)](http://cultofmartians.com)
-[![Gem Version](https://badge.fury.io/rb/test-prof.svg)](https://rubygems.org/gems/test-prof) [![Build Status](https://travis-ci.org/palkan/test-prof.svg?branch=master)](https://travis-ci.org/palkan/test-prof) [![Code Triagers Badge](https://www.codetriage.com/palkan/test-prof/badges/users.svg)](https://www.codetriage.com/palkan/test-prof)
+[![Gem Version](https://badge.fury.io/rb/test-prof.svg)](https://rubygems.org/gems/test-prof) [![Build](https://github.com/palkan/test-prof/workflows/Build/badge.svg)](https://github.com/palkan/test-prof/actions)
+[![JRuby Build](https://github.com/palkan/test-prof/workflows/JRuby%20Build/badge.svg)](https://github.com/palkan/test-prof/actions)
+[![Code Triagers Badge](https://www.codetriage.com/palkan/test-prof/badges/users.svg)](https://www.codetriage.com/palkan/test-prof)
 [![Documentation](https://img.shields.io/badge/docs-link-brightgreen.svg)](https://test-prof.evilmartians.io)
 
 # Ruby Tests Profiling Toolbox

data/lib/test_prof.rb

@@ -7,7 +7,7 @@ require "test_prof/utils"
 
 # Ruby applications tests profiling tools.
 #
-# Contains tools to anylyze factories usage, integrate with Ruby profilers,
+# Contains tools to analyze factories usage, integrate with Ruby profilers,
 # profile your examples using ActiveSupport notifications (if any) and
 # statically analyze your code with custom RuboCop cops.
 #
@@ -131,10 +131,10 @@ module TestProf
   # TestProf configuration
   class Configuration
     attr_accessor :output, # IO to write output messages.
-                  :color, # Whether to colorize output or not
-                  :output_dir, # Directory to store artifacts
-                  :timestamps, # Whethere to use timestamped names for artifacts,
-                  :report_suffix # Custom suffix for reports/artifacts
+      :color, # Whether to colorize output or not
+      :output_dir, # Directory to store artifacts
+      :timestamps, # Whether to use timestamped names for artifacts,
+      :report_suffix # Custom suffix for reports/artifacts
 
     def initialize
       @output = $stdout

data/lib/test_prof/any_fixture/dsl.rb

@@ -4,15 +4,14 @@ module TestProf
   module AnyFixture
     # Adds "global" `fixture` method (through refinement)
     module DSL
-      module Ext # :nodoc:
+      # Refine object, 'cause refining modules (Kernel) is vulnerable to prepend:
+      # - https://bugs.ruby-lang.org/issues/13446
+      # - 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)
         end
       end
-
-      refine Kernel do
-        include Ext
-      end
     end
   end
 end

data/lib/test_prof/before_all.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module TestProf
-  # `before_all` helper configiration
+  # `before_all` helper configuration
   module BeforeAll
     class AdapterMissing < StandardError # :nodoc:
       MSG = "Please, provide an adapter for `before_all` " \

data/lib/test_prof/before_all/isolator.rb

@@ -5,11 +5,13 @@ module TestProf
     # Disable Isolator within before_all blocks
     module Isolator
       def begin_transaction(*)
-        ::Isolator.disable { super }
+        ::Isolator.transactions_threshold += 1
+        super
       end
 
-      def within_transaction(*)
-        ::Isolator.disable { super }
+      def rollback_transaction(*)
+        super
+        ::Isolator.transactions_threshold -= 1
       end
     end
   end

data/lib/test_prof/event_prof.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
 require "test_prof/rspec_stamp"
 require "test_prof/event_prof/profiler"
 require "test_prof/event_prof/instrumentations/active_support"
@@ -35,7 +34,7 @@ module TestProf
       }.freeze
 
       attr_accessor :instrumenter, :top_count, :per_example,
-                    :rank_by, :event
+        :rank_by, :event
 
       def initialize
         @event = ENV["EVENT_PROF"]

data/lib/test_prof/event_prof/profiler.rb

@@ -4,7 +4,7 @@ module TestProf
   module EventProf
     class Profiler # :nodoc:
       attr_reader :event, :total_count, :total_time, :rank_by, :top_count, :per_example,
-                  :time, :count, :example_time, :example_count, :absolute_run_time
+        :time, :count, :example_time, :example_count, :absolute_run_time
 
       alias per_example? per_example
 

data/lib/test_prof/ext/array_bsearch_index.rb

@@ -2,8 +2,8 @@
 
 module TestProf
   # Ruby 2.3 #bsearch_index method (for usage with older Rubies)
-  # Straighforward and non-optimal implementation,
-  # just for compatiblity
+  # Straightforward and non-optimal implementation,
+  # just for compatibility
   module ArrayBSearchIndex
     refine Array do
       def bsearch_index(&block)

data/lib/test_prof/factory_doctor.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
 require "test_prof/factory_bot"
 require "test_prof/factory_doctor/factory_bot_patch"
 require "test_prof/factory_doctor/fabrication_patch"
@@ -67,6 +66,8 @@ module TestProf
       def init
         reset!
 
+        @running = false
+
         log :info, "FactoryDoctor enabled (event: \"#{config.event}\", threshold: #{config.threshold})"
 
         # Monkey-patch FactoryBot / FactoryGirl

data/lib/test_prof/factory_prof/printers/simple.rb

@@ -24,11 +24,13 @@ module TestProf::FactoryProf
                Total time: #{format("%.4f", total_time)}s
                Total uniq factories: #{total_uniq_factories}
 
-                 total   top-level   total time   top-level time                           name
+                 total   top-level     total time      time per call      top-level time               name
             MSG
 
           result.stats.each do |stat|
-            msgs << format("%8d %11d %11.4fs %15.4fs %30s", stat[:total_count], stat[:top_level_count], stat[:total_time], stat[:top_level_time], stat[:name])
+            time_per_call = stat[:total_time] / stat[:total_count]
+
+            msgs << format("%8d %11d %13.4fs %17.4fs %18.4fs %18s", stat[:total_count], stat[:top_level_count], stat[:total_time], time_per_call, stat[:top_level_time], stat[:name])
           end
 
           log :info, msgs.join("\n")

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

@@ -127,20 +127,22 @@ module TestProf
   end
 end
 
-require "test_prof/ext/active_record_refind"
-using TestProf::Ext::ActiveRecordRefind
-
-TestProf::LetItBe.configure do |config|
-  config.register_modifier :reload do |record, val|
-    next record unless val
-    next record unless record.is_a?(::ActiveRecord::Base)
-    record.reload
-  end
+if defined?(::ActiveRecord)
+  require "test_prof/ext/active_record_refind"
+  using TestProf::Ext::ActiveRecordRefind
+
+  TestProf::LetItBe.configure do |config|
+    config.register_modifier :reload do |record, val|
+      next record unless val
+      next record unless record.is_a?(::ActiveRecord::Base)
+      record.reload
+    end
 
-  config.register_modifier :refind do |record, val|
-    next record unless val
-    next record unless record.is_a?(::ActiveRecord::Base)
-    record.refind
+    config.register_modifier :refind do |record, val|
+      next record unless val
+      next record unless record.is_a?(::ActiveRecord::Base)
+      record.refind
+    end
   end
 end
 

data/lib/test_prof/rspec_dissect.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
 require "test_prof/rspec_stamp"
 require "test_prof/logging"
 
@@ -37,7 +36,7 @@ module TestProf
       MODES = %w[all let before].freeze
 
       attr_accessor :top_count, :let_stats_enabled,
-                    :let_top_count
+        :let_top_count
 
       alias let_stats_enabled? let_stats_enabled
 

data/lib/test_prof/rspec_stamp.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
 require "test_prof/logging"
 require "test_prof/rspec_stamp/parser"
 

data/lib/test_prof/ruby_prof.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
-
 module TestProf
   # RubyProf wrapper.
   #
@@ -48,9 +46,9 @@ module TestProf
       LOGFILE_PREFIX = "ruby-prof-report"
 
       attr_accessor :printer, :mode, :min_percent,
-                    :include_threads, :exclude_common_methods,
-                    :test_prof_exclusions_enabled,
-                    :custom_exclusions
+        :include_threads, :exclude_common_methods,
+        :test_prof_exclusions_enabled,
+        :custom_exclusions
 
       def initialize
         @printer = ENV["TEST_RUBY_PROF"].to_sym if PRINTERS.key?(ENV["TEST_RUBY_PROF"])

data/lib/test_prof/stack_prof.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
-
 module TestProf
   # StackProf wrapper.
   #
@@ -133,6 +131,7 @@ module TestProf
 
       def init_stack_prof
         return @initialized if instance_variable_defined?(:@initialized)
+        @locked = false
         @initialized = TestProf.require(
           "stackprof",
           <<~MSG

data/lib/test_prof/tag_prof.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof"
-
 require "test_prof/tag_prof/result"
 require "test_prof/tag_prof/printers/simple"
 require "test_prof/tag_prof/printers/html"

data/lib/test_prof/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.10.1
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-20 00:00:00.000000000 Z
+date: 2019-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '1.16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '1.16'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -81,33 +81,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '5.9'
 - !ruby/object:Gem::Dependency
-  name: rubocop-md
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: 0.72.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
-- !ruby/object:Gem::Dependency
-  name: standard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.1.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.1.0
+        version: 0.72.0
 description: "\n    Ruby applications tests profiling tools.\n\n    Contains tools
   to analyze factories usage, integrate with Ruby profilers,\n    profile your examples
   using ActiveSupport notifications (if any) and\n    statically analyze your code