test-prof 0.8.0 → 0.9.0

data/lib/test_prof/factory_doctor/fabrication_patch.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module TestProf
+  module FactoryDoctor
+    # Wrap #run method with FactoryDoctor tracking
+    module FabricationPatch
+      def create(*)
+        FactoryDoctor.within_factory(:create) { super }
+      end
+    end
+  end
+end

data/lib/test_prof/factory_doctor/factory_bot_patch.rb

@@ -1,11 +1,15 @@
 # frozen_string_literal: true
 
+require "test_prof/ext/factory_bot_strategy"
+
 module TestProf
   module FactoryDoctor
     # Wrap #run method with FactoryDoctor tracking
     module FactoryBotPatch
+      using TestProf::FactoryBotStrategy
+
       def run(strategy = @strategy)
-        FactoryDoctor.within_factory(strategy) { super }
+        FactoryDoctor.within_factory(strategy.create? ? :create : :other) { super }
       end
     end
   end

data/lib/test_prof/factory_doctor/rspec.rb

@@ -58,9 +58,15 @@ module TestProf
           MSG
 
         @example_groups.each do |group, examples|
-          msgs << "#{group.description} (#{group.metadata[:location]})\n"
+          group_time = examples.sum { |ex| ex.metadata[:time] }
+          group_count = examples.sum { |ex| ex.metadata[:factories] }
+
+          msgs << "#{group.description} (#{group.metadata[:location]}) " \
+                  "(#{pluralize_records(group_count)} created, " \
+                  "#{group_time.duration})\n"
+
           examples.each do |ex|
-            msgs << "  #{ex.description} (#{ex.metadata[:location]}) "\
+            msgs << "  #{ex.description} (#{ex.metadata[:location]}) " \
                     "– #{pluralize_records(ex.metadata[:factories])} created, "\
                     "#{ex.metadata[:time].duration}\n"
           end

data/lib/test_prof/factory_prof.rb

@@ -36,15 +36,16 @@ module TestProf
 
       # Returns sorted stats
       def stats
-        return @stats if instance_variable_defined?(:@stats)
+        @stats ||= @raw_stats.values
+          .sort_by { |el| -el[:total_count] }
+      end
 
-        @stats = @raw_stats.values
-          .sort_by { |el| -el[:total] }
+      def total_count
+        @total_count ||= @raw_stats.values.sum { |v| v[:total_count] }
       end
 
-      def total
-        return @total if instance_variable_defined?(:@total)
-        @total = @raw_stats.values.sum { |v| v[:total] }
+      def total_time
+        @total_time ||= @raw_stats.values.sum { |v| v[:total_time] }
       end
 
       private
@@ -103,13 +104,18 @@ module TestProf
 
       def track(factory)
         return yield unless running?
+        @depth += 1
+        @current_stack << factory if config.flamegraph?
+        @stats[factory][:total_count] += 1
+        @stats[factory][:top_level_count] += 1 if @depth == 1
+        t1 = TestProf.now
         begin
-          @depth += 1
-          @current_stack << factory if config.flamegraph?
-          @stats[factory][:total] += 1
-          @stats[factory][:top_level] += 1 if @depth == 1
           yield
         ensure
+          t2 = TestProf.now
+          elapsed = t2 - t1
+          @stats[factory][:total_time] += elapsed
+          @stats[factory][:top_level_time] += elapsed if @depth == 1
           @depth -= 1
           flush_stack if @depth.zero?
         end
@@ -120,7 +126,15 @@ module TestProf
       def reset!
         @stacks = [] if config.flamegraph?
         @depth = 0
-        @stats = Hash.new { |h, k| h[k] = {name: k, total: 0, top_level: 0} }
+        @stats = Hash.new do |h, k|
+          h[k] = {
+            name: k,
+            total_count: 0,
+            top_level_count: 0,
+            total_time: 0.0,
+            top_level_time: 0.0
+          }
+        end
         flush_stack
       end
 

data/lib/test_prof/factory_prof/factory_builders/factory_bot.rb

@@ -2,6 +2,7 @@
 
 require "test_prof/factory_prof/factory_bot_patch"
 require "test_prof/factory_bot"
+require "test_prof/ext/factory_bot_strategy"
 
 module TestProf
   module FactoryProf
@@ -9,23 +10,7 @@ module TestProf
       # implementation of #patch and #track methods
       # to provide unified interface for all factory-building gems
       class FactoryBot
-        # FactoryBot 5.0 uses strategy classes for associations,
-        # older versions and top-level invocations use Symbols
-        using(Module.new do
-          refine Symbol do
-            def create?
-              self == :create
-            end
-          end
-
-          if defined?(::FactoryBot::Strategy::Create)
-            refine Class do
-              def create?
-                self <= ::FactoryBot::Strategy::Create
-              end
-            end
-          end
-        end)
+        using TestProf::FactoryBotStrategy
 
         # Monkey-patch FactoryBot / FactoryGirl
         def self.patch

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

@@ -15,7 +15,7 @@ module TestProf::FactoryProf
           return log(:info, "No factories detected") if result.raw_stats == {}
           report_data = {
             total_stacks: result.stacks.size,
-            total: result.total
+            total: result.total_count
           }
 
           report_data[:roots] = convert_stacks(result)
@@ -45,7 +45,11 @@ module TestProf::FactoryProf
                 node = paths[path]
                 node[:value] += 1
               else
-                node = {name: sample, value: 1, total: result.raw_stats.fetch(sample)[:total]}
+                node = {
+                  name: sample,
+                  value: 1,
+                  total: result.raw_stats.fetch(sample)[:total_count]
+                }
                 paths[path] = node
 
                 if parent.nil?

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

@@ -10,23 +10,25 @@ module TestProf::FactoryProf
           return log(:info, "No factories detected") if result.raw_stats == {}
           msgs = []
 
-          total = result.stats.sum { |stat| stat[:total] }
-          total_top_level = result.stats.sum { |stat| stat[:top_level] }
+          total_count = result.stats.sum { |stat| stat[:total_count] }
+          total_top_level_count = result.stats.sum { |stat| stat[:top_level_count] }
+          total_time = result.stats.sum { |stat| stat[:top_level_time] }
           total_uniq_factories = result.stats.map { |stat| stat[:name] }.uniq.count
 
           msgs <<
             <<~MSG
               Factories usage
 
-               Total: #{total}
-               Total top-level: #{total_top_level}
+               Total: #{total_count}
+               Total top-level: #{total_top_level_count}
+               Total time: #{format("%.4f", total_time)}s
                Total uniq factories: #{total_uniq_factories}
 
-               total      top-level                            name
+                 total   top-level   total time   top-level time                           name
             MSG
 
           result.stats.each do |stat|
-            msgs << format("%6d    %11d  %30s", stat[:total], stat[:top_level], stat[:name])
+            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])
           end
 
           log :info, msgs.join("\n")

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

@@ -7,7 +7,26 @@ module TestProf
   # Just like `let`, but persist the result for the whole group.
   # NOTE: Experimental and magical, for more control use `before_all`.
   module LetItBe
+    class Configuration
+      # Define an alias for `let_it_be` with the predefined options:
+      #
+      #   TestProf::LetItBe.configure do |config|
+      #     config.alias_to :let_it_be_reloaded, reload: true
+      #   end
+      def alias_to(name, **default_args)
+        LetItBe.define_let_it_be_alias(name, **default_args)
+      end
+    end
+
     class << self
+      def config
+        @config ||= Configuration.new
+      end
+
+      def configure
+        yield config
+      end
+
       def module_for(group)
         modules[group] ||= begin
           Module.new.tap { |mod| group.prepend(mod) }
@@ -25,6 +44,12 @@ module TestProf
     # And we love cats!)
     PREFIX = RUBY_ENGINE == "jruby" ? "@__jruby_is_not_cat_friendly__" : "@😸"
 
+    def self.define_let_it_be_alias(name, **default_args)
+      define_method(name) do |identifier, **options, &blk|
+        let_it_be(identifier, default_args.merge(options), &blk)
+      end
+    end
+
     def let_it_be(identifier, **options, &block)
       initializer = proc do
         instance_variable_set(:"#{PREFIX}#{identifier}", instance_exec(&block))

data/lib/test_prof/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-13 00:00:00.000000000 Z
+date: 2019-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -80,20 +80,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.9'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.65.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.65.0
 - !ruby/object:Gem::Dependency
   name: rubocop-md
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +100,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.36
+        version: 0.0.39
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.36
+        version: 0.0.39
 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
@@ -170,6 +156,7 @@ files:
 - 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
 - lib/test_prof/ext/float_duration.rb
 - lib/test_prof/ext/string_truncate.rb
 - lib/test_prof/factory_all_stub.rb
@@ -178,6 +165,7 @@ files:
 - lib/test_prof/factory_default.rb
 - lib/test_prof/factory_default/factory_bot_patch.rb
 - lib/test_prof/factory_doctor.rb
+- lib/test_prof/factory_doctor/fabrication_patch.rb
 - lib/test_prof/factory_doctor/factory_bot_patch.rb
 - lib/test_prof/factory_doctor/minitest.rb
 - lib/test_prof/factory_doctor/rspec.rb
@@ -226,7 +214,12 @@ files:
 homepage: http://github.com/palkan/test-prof
 licenses:
 - MIT
-metadata: {}
+metadata:
+  bug_tracker_uri: http://github.com/palkan/test-prof/issues
+  changelog_uri: https://github.com/palkan/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: 
 rdoc_options: []
 require_paths:
@@ -242,7 +235,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Ruby applications tests profiling tools