test-prof 0.2.5 → 0.3.0.beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0df1119b6d47cf2641984fe6bcc7387c228353ff
-  data.tar.gz: d472861344964d895fc964afb099f3d3a35dbfae
+  metadata.gz: 1c450b597564326b40288fbee6f6ceb27f6d9ec8
+  data.tar.gz: '080d7077ea0713b74a223a421c4a9af6dfc6d8f7'
 SHA512:
-  metadata.gz: ccde20ed06a18ff23b85ee1d2b42269214d8ea8dbaf88fda55a78093bd76c99cafef050b3db70e76f98570709b7adfec46d45e9cbb1140233f688c80bc158115
-  data.tar.gz: 2bda30e78808f8edfa279b88abafa617d416fa6f1bc9f235c883ab106a18d9619c8a5f4db3c412ede368307eade325db0d3559788e2015f2a7aeac6445cdeaed
+  metadata.gz: fcbfd05e6c5889d302c309f55a6285e0764838111dcb60af31f0b8ce20cd65619281dc3433c3be35cc9de748a65a3b63ccc58a6bb3506ecc8d490d8652b554e9
+  data.tar.gz: 0e5c02a21fe5ebeeb9b457abbb230158a6289fa66e5e694778f7acb0ce0293f50e9ab9c2f97c60b8d9fc78b9738b39e88fa6ed18f6b22f613f198ce2d19bd404

data/CHANGELOG.md

@@ -1,8 +1,17 @@
 # Change log
 
-## 0.2.5
+## 0.3.0
 
-- [#16](https://github.com/palkan/test-prof/pull/16) Support Ruby >= 2.2.0 (was >= 2.3.0). ([@palkan][])
+Features:
+
+- [#14](https://github.com/palkan/test-prof/pull/14) RSpecDissect profiler. ([@palkan][])
+
+RSpecDissect tracks how much time do you spend in `before` hooks 
+and memoization helpers (i.e. `let`) in your tests.
+
+- [#13](https://github.com/palkan/test-prof/pull/13) RSpec `let_it_be` method. ([@palkan][])
+
+Just like `let`, but persist the result for the whole group (i.e. `let` + `before_all`).
 
 ## 0.2.4
 

data/README.md

@@ -24,12 +24,6 @@ Of course, we have some [solutions](#tips-and-tricks) for common performance iss
 
 See [Table of Contents](#table-of-contents) for more.
 
-Supported Ruby versions:
-
-- Ruby (MRI) >= 2.2.0
-
-- JRuby >= 9.1.0.0
-
 <a href="https://evilmartians.com/">
 <img src="https://evilmartians.com/badges/sponsored-by-evil-martians.svg" alt="Sponsored by Evil Martians" width="236" height="54"></a>
 
@@ -69,6 +63,8 @@ Checkout our guides for each specific tool:
 
 - [Factory Profiler](https://github.com/palkan/test-prof/tree/master/guides/factory_prof.md)
 
+- [RSpecDissect Profiler](https://github.com/palkan/test-prof/tree/master/guides/rspec_dissect.md)
+
 - [Rubocop Cops](https://github.com/palkan/test-prof/tree/master/guides/rubocop.md)
 
 ## Tips and Tricks (or _Recipes_)
@@ -77,6 +73,8 @@ We also want to share some small code tricks which can help you to improve your
 
 - [`before_all` Hook](https://github.com/palkan/test-prof/tree/master/guides/before_all.md)
 
+- [`let_it_be` Helper](https://github.com/palkan/test-prof/tree/master/guides/let_it_be.md)
+
 - [AnyFixture](https://github.com/palkan/test-prof/tree/master/guides/any_fixture.md)
 
 - [FactoryDefault](https://github.com/palkan/test-prof/tree/master/guides/factory_default.md)

data/guides/let_it_be.md

@@ -0,0 +1,96 @@
+# Let It Be!
+
+Let's bring a little bit of magic and introduce a new way to setup a _shared_ test data.
+
+Suppose you have the following setup:
+
+```ruby
+describe BeatleWeightedSearchQuery do
+  let!(:paul) { create(:beatle, name: 'Paul') }
+  let!(:ringo) { create(:beatle, name: 'Ringo') }
+  let!(:george) { create(:beatle, name: 'George') }
+  let!(:john) { create(:beatle, name: 'John') }
+
+  specify { expect(subject.call('joh')).to contain_exactly(john) }
+
+  # and more examples here
+end
+```
+
+We don't need to re-create the Fab Four for every example, do we?
+
+We already have [`before_all`](https://github.com/palkan/test-prof/tree/master/guides/before_all.md) to solve the problem of _repeatable_ data:
+
+```ruby
+describe BeatleWeightedSearchQuery do
+  before_all do
+    @paul = create(:beatle, name: 'Paul')
+    ...
+  end
+
+  specify { expect(subject.call('joh')).to contain_exactly(@john) }
+
+  ...
+end
+```
+
+That technique works pretty good but requires us to use instance variables and define everything at once. Thus it's not easy to refactor existing tests which use `let/let!` instead.
+
+With `let_it_be` you can do the following:
+
+```ruby
+describe BeatleWeightedSearchQuery do
+  let_it_be(:paul) { create(:beatle, name: 'Paul') }
+  let_it_be(:ringo) { create(:beatle, name: 'Ringo') }
+  let_it_be(:george) { create(:beatle, name: 'George') }
+  let_it_be(:john) { create(:beatle, name: 'John') }
+
+  specify { expect(subject.call('joh')).to contain_exactly(john) }
+
+  # and more examples here
+end
+```
+
+That's it! Just replace `let!` with `let_it_be`. That's equal to the `before_all` approach but requires less refactoring.
+
+
+## Instructions
+
+In your `spec_helper.rb`:
+
+```ruby
+require "test_prof/recipes/rspec/let_it_be"
+```
+
+In your tests:
+
+```ruby
+describe MySuperDryService do
+  let_it_be(:user) { create(:user) }
+
+  ...
+end
+```
+
+## Caveats
+
+If you modify objects generated within a `let_it_be` block in your examples, you maybe have to re-initiate them.
+We have a built-in support for that:
+
+
+```ruby
+# Use reload: true option to reload user object (assuming it's an instance of ActiveRecord)
+# for every example
+let_it_be(:user, reload: true) { create(:user) }
+
+# it is almost equal to
+before_all { @user = create(:user) }
+let(:user) { @user.reload }
+
+# You can also specify refind: true option to hard-reload the record
+let_it_be(:user, refind: true) { create(:user) }
+
+# it is almost equal to
+before_all { @user = create(:user) }
+let(:user) { User.find(@user.id) }
+```

data/guides/rspec_dissect.md

@@ -0,0 +1,49 @@
+# RSpecDissect
+
+Do you know how much time you spend in `before` hooks? Or memoization helpers such as `let`? Usually, the most of the whole test suite time.
+
+_RSpecDissect_ provides this kind of information and also shows you the worst example groups. The main purpose of RSpecDissect is to identify these slow groups and refactor them using [`before_all`](https://github.com/palkan/test-prof/tree/master/guides/before_all.md) or [`let_it_be`](https://github.com/palkan/test-prof/tree/master/guides/let_it_be.md) recipes.
+
+Example output:
+
+```sh
+[TEST PROF INFO] RSpecDissect enabled
+
+Total time: 25:14.870
+Total `before(:each)` time: 14:36.482
+Total `let` time: 19:20.259
+
+Top 5 slowest suites (by `before(:each)` time):
+
+Webhooks::DispatchTransition (./spec/services/webhooks/dispatch_transition_spec.rb:3) – 00:29.895 of 00:33.706 (327)
+FunnelsController (./spec/controllers/funnels_controller_spec.rb:3) – 00:22.117 of 00:43.649 (133)
+ApplicantsController (./spec/controllers/applicants_controller_spec.rb:3) – 00:21.220 of 00:41.407 (222)
+BookedSlotsController (./spec/controllers/booked_slots_controller_spec.rb:3) – 00:15.729 of 00:27.893 (50)
+Analytics::Wor...rsion::Summary (./spec/services/analytics/workflow_conversion/summary_spec.rb:3) – 00:15.383 of 00:15.914 (12)
+
+
+Top 5 slowest suites (by `let` time):
+
+FunnelsController (./spec/controllers/funnels_controller_spec.rb:3) – 00:38.532 of 00:43.649 (133)
+ApplicantsController (./spec/controllers/applicants_controller_spec.rb:3) – 00:33.252 of 00:41.407 (222)
+Webhooks::DispatchTransition (./spec/services/webhooks/dispatch_transition_spec.rb:3) – 00:30.320 of 00:33.706 (327)
+BookedSlotsController (./spec/controllers/booked_slots_controller_spec.rb:3) – 00:25.710 of 00:27.893 (50)
+AvailableSlotsController (./spec/controllers/available_slots_controller_spec.rb:3) – 00:18.481 of 00:23.366 (85)
+```
+
+## Instructions
+
+RSpecDissect can only be used with RSpec (which is clear from the name).
+
+To activate RSpecDissect use `RD` environment variable:
+
+```sh
+RD=1 rspec ...
+```
+
+You can also specify the number of top slow groups through `RD_TOP` variable:
+
+```sh
+RD=1 RD_TOP=10 rspec ...
+```
+

data/lib/test_prof.rb

@@ -127,3 +127,4 @@ 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"

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

@@ -39,7 +39,7 @@ module RuboCop
 
         def on_block(node)
           method, _args, body = *node
-          return unless body && body.begin_type?
+          return unless body&.begin_type?
 
           _receiver, method_name, _object = *method
           return unless GROUP_BLOCKS.include?(method_name)
@@ -109,7 +109,7 @@ module RuboCop
         end
 
         def oneliner?(node)
-          node && node.block_type? &&
+          node&.block_type? &&
             (node.source.lines.size == 1) &&
             example_node?(node)
         end

data/lib/test_prof/event_prof/custom_events/factory_create.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
-using TestProf::StringStripHeredoc
-
 module TestProf::EventProf::CustomEvents
   module FactoryCreate # :nodoc: all
     module RunnerPatch
@@ -46,7 +42,7 @@ end
 TestProf.activate('EVENT_PROF', 'factory.create') do
   if TestProf.require(
     'factory_girl',
-    <<-MSG.strip_heredoc
+    <<~MSG
       Failed to load FactoryGirl.
 
       Make sure that "factory_girl" gem is in your Gemfile.

data/lib/test_prof/event_prof/custom_events/sidekiq_inline.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
-using TestProf::StringStripHeredoc
-
 module TestProf::EventProf::CustomEvents
   module SidekiqInline # :nodoc: all
     module ClientPatch
@@ -43,7 +39,7 @@ end
 TestProf.activate('EVENT_PROF', 'sidekiq.inline') do
   if TestProf.require(
     'sidekiq/testing',
-    <<-MSG.strip_heredoc
+    <<~MSG
       Failed to load Sidekiq.
 
       Make sure that "sidekiq" gem is in your Gemfile.

data/lib/test_prof/event_prof/custom_events/sidekiq_jobs.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
-using TestProf::StringStripHeredoc
-
 module TestProf::EventProf::CustomEvents
   module SidekiqJobs # :nodoc: all
     module ClientPatch
@@ -30,7 +26,7 @@ end
 TestProf.activate('EVENT_PROF', 'sidekiq.jobs') do
   if TestProf.require(
     'sidekiq/testing',
-    <<-MSG.strip_heredoc
+    <<~MSG
       Failed to load Sidekiq.
 
       Make sure that "sidekiq" gem is in your Gemfile.

data/lib/test_prof/event_prof/rspec.rb

@@ -2,7 +2,6 @@
 
 require "test_prof/ext/float_duration"
 require "test_prof/ext/string_truncate"
-require "test_prof/ext/string_strip_heredoc"
 
 module TestProf
   module EventProf
@@ -10,7 +9,6 @@ module TestProf
       include Logging
       using FloatDuration
       using StringTruncate
-      using StringStripHeredoc
 
       NOTIFICATIONS = %i[
         example_group_started
@@ -47,7 +45,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             EventProf results for #{@profiler.event}
 
             Total time: #{@profiler.total_time.duration}
@@ -62,7 +60,7 @@ module TestProf
           location = group[:id].metadata[:location]
 
           msgs <<
-            <<-GROUP.strip_heredoc
+            <<~GROUP
               #{description.truncate} (#{location}) – #{group[:time].duration} (#{group[:count]} / #{group[:examples]})
             GROUP
         end
@@ -74,7 +72,7 @@ module TestProf
             description = example[:id].description
             location = example[:id].metadata[:location]
             msgs <<
-              <<-GROUP.strip_heredoc
+              <<~GROUP
                 #{description.truncate} (#{location}) – #{example[:time].duration} (#{example[:count]})
               GROUP
           end

data/lib/test_prof/factory_doctor.rb

@@ -16,7 +16,7 @@ module TestProf
       end
 
       def bad?
-        count > 0 && queries_count.zero?
+        count.positive? && queries_count.zero?
       end
     end
 
@@ -111,7 +111,7 @@ module TestProf
       end
 
       def within_factory?
-        @depth > 0
+        @depth.positive?
       end
 
       def ignore?

data/lib/test_prof/factory_doctor/rspec.rb

@@ -1,16 +1,14 @@
 # frozen_string_literal: true
 
 require "test_prof/ext/float_duration"
-require "test_prof/ext/string_strip_heredoc"
 
 module TestProf
   module FactoryDoctor
     class RSpecListener # :nodoc:
       include Logging
       using FloatDuration
-      using StringStripHeredoc
 
-      SUCCESS_MESSAGE = 'FactoryDoctor says: "Looks good to me!"'.freeze
+      SUCCESS_MESSAGE = 'FactoryDoctor says: "Looks good to me!"'
 
       NOTIFICATIONS = %i[
         example_started
@@ -51,7 +49,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             FactoryDoctor report
 
             Total (potentially) bad examples: #{@count}

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

@@ -1,19 +1,16 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
 module TestProf::FactoryProf
   module Printers
     module Simple # :nodoc: all
       class << self
         include TestProf::Logging
-        using TestProf::StringStripHeredoc
 
         def dump(result)
           msgs = []
 
           msgs <<
-            <<-MSG.strip_heredoc
+            <<~MSG
               Factories usage
 
                total      top-level                            name

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

@@ -12,7 +12,7 @@ RSpec.shared_context "any_fixture:clean", with_clean_fixture: true do
 
   def open_transaction?
     pool = ActiveRecord::Base.connection_pool
-    pool.active_connection? && pool.connection.open_transactions > 0
+    pool.active_connection? && pool.connection.open_transactions.positive?
   end
 end
 

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

@@ -6,6 +6,10 @@ module TestProf
     def before_all(&block)
       raise ArgumentError, "Block is required!" unless block_given?
 
+      return if within_before_all?
+
+      @__before_all_activated__ = true
+
       before(:all) do
         ActiveRecord::Base.connection.begin_transaction(joinable: false)
         instance_eval(&block)
@@ -15,6 +19,10 @@ module TestProf
         ActiveRecord::Base.connection.rollback_transaction
       end
     end
+
+    def within_before_all?
+      instance_variable_defined?(:@__before_all_activated__)
+    end
   end
 end
 

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

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+require_relative "./before_all"
+
+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 << self
+      def module_for(group)
+        modules.fetch(group) do
+          Module.new.tap { |mod| group.prepend(mod) }
+        end
+      end
+
+      private
+
+      def modules
+        @modules ||= {}
+      end
+    end
+    # Use uniq prefix for instance variables to avoid collisions
+    # We want to use the power of Ruby's unicode support)
+    # And we love cats!)
+    PREFIX = "@😸"
+
+    def let_it_be(identifier, **options, &block)
+      initializer = proc do
+        instance_variable_set(:"#{PREFIX}#{identifier}", instance_exec(&block))
+      end
+
+      if within_before_all?
+        before(:all, &initializer)
+      else
+        before_all(&initializer)
+      end
+
+      define_let_it_be_methods(identifier, **options)
+    end
+
+    def define_let_it_be_methods(identifier, reload: false, refind: false)
+      let_accessor = -> { instance_variable_get(:"#{PREFIX}#{identifier}") }
+
+      let_accessor = -> { instance_variable_get(:"#{PREFIX}#{identifier}")&.reload } if reload
+
+      if refind
+        let_accessor = lambda do
+          record = instance_variable_get(:"#{PREFIX}#{identifier}")
+          next unless record.is_a?(::ActiveRecord::Base)
+
+          record.class.find(record.send(record.class.primary_key))
+        end
+      end
+
+      LetItBe.module_for(self).module_eval do
+        define_method(identifier) do
+          # Trying to detect the context (couldn't find other way so far)
+          if @__inspect_output =~ /\(:context\)/
+            instance_variable_get(:"#{PREFIX}#{identifier}")
+          else
+            # Fallback to let definition
+            super()
+          end
+        end
+      end
+
+      let(identifier, &let_accessor)
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.extend TestProf::LetItBe
+end

data/lib/test_prof/rspec_dissect.rb

@@ -0,0 +1,104 @@
+# frozen_string_literal: true
+
+require "test_prof/logging"
+
+module TestProf
+  # RSpecDissect tracks how much time do you spend in `before` hooks
+  # and memoization helpers (i.e. `let`) in your tests.
+  module RSpecDissect
+    module ExampleInstrumentation # :nodoc:
+      def run_before_example(*)
+        RSpecDissect.track(:before) { super }
+      end
+    end
+
+    module MemoizedInstrumentation # :nodoc:
+      def fetch_or_store(*)
+        res = nil
+        Thread.current[:_rspec_dissect_memo_depth] ||= 0
+        Thread.current[:_rspec_dissect_memo_depth] += 1
+        begin
+          res = if Thread.current[:_rspec_dissect_memo_depth] == 1
+                  RSpecDissect.track(:memo) { super }
+                else
+                  super
+                end
+        ensure
+          Thread.current[:_rspec_dissect_memo_depth] -= 1
+        end
+        res
+      end
+    end
+
+    # RSpecDisect configuration
+    class Configuration
+      attr_accessor :top_count
+
+      def initialize
+        @top_count = (ENV['RD_TOP'] || 5).to_i
+      end
+    end
+
+    METRICS = %w[before memo].freeze
+
+    class << self
+      include Logging
+
+      def config
+        @config ||= Configuration.new
+      end
+
+      def configure
+        yield config
+      end
+
+      def init
+        RSpec::Core::Example.prepend(ExampleInstrumentation)
+        RSpec::Core::MemoizedHelpers::ThreadsafeMemoized.prepend(MemoizedInstrumentation)
+        RSpec::Core::MemoizedHelpers::NonThreadSafeMemoized.prepend(MemoizedInstrumentation)
+
+        @data = {}
+
+        METRICS.each do |type|
+          @data["total_#{type}"] = 0.0
+        end
+
+        reset!
+
+        log :info, "RSpecDissect enabled"
+      end
+
+      def track(type)
+        start = TestProf.now
+        res = yield
+        delta = (TestProf.now - start)
+        type = type.to_s
+        @data[type] += delta
+        @data["total_#{type}"] += delta
+        res
+      end
+
+      def reset!
+        METRICS.each do |type|
+          @data[type.to_s] = 0.0
+        end
+      end
+
+      METRICS.each do |type|
+        define_method("#{type}_time") do
+          @data[type.to_s]
+        end
+
+        define_method("total_#{type}_time") do
+          @data["total_#{type}"]
+        end
+      end
+    end
+  end
+end
+
+require "test_prof/rspec_dissect/rspec" if defined?(RSpec::Core)
+
+TestProf.activate('RD') do
+  TestProf::RSpecDissect.init
+end

data/lib/test_prof/rspec_dissect/rspec.rb

@@ -0,0 +1,121 @@
+# frozen_string_literal: true
+
+require "test_prof/ext/float_duration"
+require "test_prof/ext/string_truncate"
+require "test_prof/utils/sized_ordered_set"
+
+module TestProf
+  module RSpecDissect
+    class Listener # :nodoc:
+      include Logging
+      using FloatDuration
+      using StringTruncate
+
+      NOTIFICATIONS = %i[
+        example_finished
+        example_group_finished
+      ].freeze
+
+      def initialize
+        @before_results = Utils::SizedOrderedSet.new(
+          top_count, sort_by: :before
+        )
+        @memo_results = Utils::SizedOrderedSet.new(
+          top_count, sort_by: :memo
+        )
+        @examples_count = 0
+        @examples_time = 0.0
+        @total_examples_time = 0.0
+      end
+
+      def example_finished(notification)
+        @examples_count += 1
+        @examples_time += notification.example.execution_result.run_time
+      end
+
+      def example_group_finished(notification)
+        return unless notification.group.top_level?
+
+        data = {}
+        data[:total] = @examples_time
+        data[:count] = @examples_count
+        data[:before] = RSpecDissect.before_time
+        data[:memo] = RSpecDissect.memo_time
+        data[:desc] = notification.group.top_level_description
+        data[:loc] = notification.group.metadata[:location]
+
+        @before_results << data
+        @memo_results << data
+
+        @total_examples_time += @examples_time
+        @examples_count = 0
+        @examples_time = 0.0
+
+        RSpecDissect.reset!
+      end
+
+      def print
+        msgs = []
+
+        msgs <<
+          <<~MSG
+            RSpecDissect report
+
+            Total time: #{@total_examples_time.duration}
+            Total `before(:each)` time: #{RSpecDissect.total_before_time.duration}
+            Total `let` time: #{RSpecDissect.total_memo_time.duration}
+
+          MSG
+
+        msgs <<
+          <<~MSG
+            Top #{top_count} slowest suites (by `before(:each)` time):
+
+          MSG
+
+        @before_results.each do |group|
+          msgs <<
+            <<~GROUP
+              #{group[:desc].truncate} (#{group[:loc]}) – #{group[:before].duration} of #{group[:total].duration} (#{group[:count]})
+            GROUP
+        end
+
+        msgs <<
+          <<~MSG
+            Top #{top_count} slowest suites (by `let` time):
+
+          MSG
+
+        @memo_results.each do |group|
+          msgs <<
+            <<~GROUP
+              #{group[:desc].truncate} (#{group[:loc]}) – #{group[:memo].duration} of #{group[:total].duration} (#{group[:count]})
+            GROUP
+        end
+
+        log :info, msgs.join
+      end
+
+      private
+
+      def top_count
+        RSpecDissect.config.top_count
+      end
+    end
+  end
+end
+
+# Register RSpecDissect listener
+TestProf.activate('RD') do
+  RSpec.configure do |config|
+    listener = TestProf::RSpecDissect::Listener.new
+
+    config.before(:suite) do
+      config.reporter.register_listener(
+        listener, *TestProf::RSpecDissect::Listener::NOTIFICATIONS
+      )
+    end
+
+    config.after(:suite) { listener.print }
+  end
+end

data/lib/test_prof/rspec_stamp/parser.rb

@@ -2,6 +2,8 @@
 
 require "ripper"
 
+# rubocop: disable Metrics/CyclomaticComplexity
+
 module TestProf
   module RSpecStamp
     # Parse examples headers
@@ -22,13 +24,12 @@ module TestProf
         end
 
         def remove_tag(tag)
-          @tags.delete(tag) if @tags
-          @htags.delete_if { |(k, _v)| k == tag } if @htags
+          @tags&.delete(tag)
+          @htags&.delete_if { |(k, _v)| k == tag }
         end
       end
 
       class << self
-        # rubocop: disable Metrics/CyclomaticComplexity
         def parse(code)
           sexp = Ripper.sexp(code)
           return unless sexp
@@ -76,7 +77,6 @@ module TestProf
 
           res
         end
-        # rubocop: enable Metrics/CyclomaticComplexity
 
         private
 

data/lib/test_prof/rspec_stamp/rspec.rb

@@ -1,12 +1,9 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
 module TestProf
   module RSpecStamp
     class RSpecListener # :nodoc:
       include Logging
-      using StringStripHeredoc
 
       NOTIFICATIONS = %i[
         example_failed
@@ -37,7 +34,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             RSpec Stamp results
 
             Total patches: #{@total}

data/lib/test_prof/ruby_prof.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
 module TestProf
   # RubyProf wrapper.
   #
@@ -121,7 +119,6 @@ module TestProf
 
     class << self
       include Logging
-      using StringStripHeredoc
 
       def config
         @config ||= Configuration.new
@@ -175,7 +172,7 @@ module TestProf
         ENV["RUBY_PROF_MEASURE_MODE"] = config.mode.to_s
         @initialized = TestProf.require(
           'ruby-prof',
-          <<-MSG.strip_heredoc
+          <<~MSG
             Please, install 'ruby-prof' first:
                # Gemfile
               gem 'ruby-prof', '>= 0.16.0', require: false
@@ -187,7 +184,7 @@ module TestProf
         if Utils.verify_gem_version('ruby-prof', at_least: '0.16.0')
           true
         else
-          log :error, <<-MGS.strip_heredoc
+          log :error, <<~MGS
             Please, upgrade 'ruby-prof' to version >= 0.16.0.
           MGS
           false

data/lib/test_prof/stack_prof.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/ext/string_strip_heredoc"
-
 module TestProf
   # StackProf wrapper.
   #
@@ -34,7 +32,6 @@ module TestProf
 
     class << self
       include Logging
-      using StringStripHeredoc
 
       def config
         @config ||= Configuration.new
@@ -91,7 +88,7 @@ module TestProf
 
         html_path = path.gsub(/\.dump$/, '.html')
 
-        log :info, <<-MSG.strip_heredoc
+        log :info, <<~MSG
           Run the following command to generate a flame graph report:
 
           stackprof --flamegraph #{path} > #{html_path} && stackprof --flamegraph-viewer=#{html_path}
@@ -114,7 +111,7 @@ module TestProf
         return @initialized if instance_variable_defined?(:@initialized)
         @initialized = TestProf.require(
           'stackprof',
-          <<-MSG.strip_heredoc
+          <<~MSG
             Please, install 'stackprof' first:
                # Gemfile
               gem 'stackprof', '>= 0.2.9', require: false
@@ -126,9 +123,9 @@ module TestProf
         if Utils.verify_gem_version('stackprof', at_least: '0.2.9')
           true
         else
-          log :error, <<-MSG.strip_heredoc
+          log :error, <<~MGS
             Please, upgrade 'stackprof' to version >= 0.2.9.
-          MSG
+          MGS
           false
         end
       end

data/lib/test_prof/tag_prof/rspec.rb

@@ -1,14 +1,12 @@
 # frozen_string_literal: true
 
 require "test_prof/ext/float_duration"
-require "test_prof/ext/string_strip_heredoc"
 
 module TestProf
   module TagProf
     class RSpecListener # :nodoc:
       include Logging
       using FloatDuration
-      using StringStripHeredoc
 
       NOTIFICATIONS = %i[
         example_started
@@ -39,7 +37,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             TagProf report for #{@tag}
           MSG
 

data/lib/test_prof/utils/sized_ordered_set.rb

@@ -4,11 +4,6 @@ module TestProf
   module Utils
     # Ordered set with capacity
     class SizedOrderedSet
-      unless [].respond_to?(:bsearch_index)
-        require "test_prof/ext/array_bsearch_index"
-        using ArrayBSearchIndex
-      end
-
       include Enumerable
 
       def initialize(max_size, sort_by: nil)
@@ -42,7 +37,11 @@ module TestProf
       end
 
       def each
-        data.each
+        if block_given?
+          data.each(&Proc.new)
+        else
+          data.each
+        end
       end
 
       def size

data/lib/test_prof/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TestProf
-  VERSION = "0.2.5".freeze
+  VERSION = "0.3.0.beta"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.3.0.beta
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-30 00:00:00.000000000 Z
+date: 2017-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.49'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: "\n    Ruby applications tests profiling tools.\n\n    Contains tools
   to anylyze factories usage, integrate with Ruby profilers,\n    profile your examples
   using ActiveSupport notifications (if any) and\n    statically analyze your code
@@ -108,6 +122,8 @@ files:
 - guides/factory_default.md
 - guides/factory_doctor.md
 - guides/factory_prof.md
+- guides/let_it_be.md
+- guides/rspec_dissect.md
 - guides/rspec_stamp.md
 - guides/rubocop.md
 - guides/ruby_prof.md
@@ -126,9 +142,7 @@ files:
 - lib/test_prof/event_prof/instrumentations/active_support.rb
 - lib/test_prof/event_prof/minitest.rb
 - lib/test_prof/event_prof/rspec.rb
-- lib/test_prof/ext/array_bsearch_index.rb
 - lib/test_prof/ext/float_duration.rb
-- lib/test_prof/ext/string_strip_heredoc.rb
 - lib/test_prof/ext/string_truncate.rb
 - lib/test_prof/factory_default.rb
 - lib/test_prof/factory_default/factory_girl_patch.rb
@@ -145,7 +159,10 @@ files:
 - lib/test_prof/recipes/rspec/any_fixture.rb
 - lib/test_prof/recipes/rspec/before_all.rb
 - lib/test_prof/recipes/rspec/factory_default.rb
+- lib/test_prof/recipes/rspec/let_it_be.rb
 - lib/test_prof/recipes/rspec/sample.rb
+- lib/test_prof/rspec_dissect.rb
+- lib/test_prof/rspec_dissect/rspec.rb
 - lib/test_prof/rspec_stamp.rb
 - lib/test_prof/rspec_stamp/parser.rb
 - lib/test_prof/rspec_stamp/rspec.rb
@@ -171,15 +188,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.2.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Ruby applications tests profiling tools

data/lib/test_prof/ext/array_bsearch_index.rb

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-module TestProf
-  # Ruby 2.3 #bsearch_index method (for usage with older Rubies)
-  # Straighforward and non-optimal implementation,
-  # just for compatiblity
-  module ArrayBSearchIndex
-    refine Array do
-      def bsearch_index(&block)
-        el = bsearch(&block)
-        index(el)
-      end
-    end
-  end
-end

data/lib/test_prof/ext/string_strip_heredoc.rb

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-module TestProf
-  # Add #strip_heredoc method to use instead of
-  # squiggly docs (to support older Rubies)
-  module StringStripHeredoc
-    refine String do
-      def strip_heredoc
-        min = scan(/^[ \t]*(?=\S)/).min
-        indent = min ? min.size : 0
-        gsub(/^[ \t]{#{indent}}/, '')
-      end
-    end
-  end
-end