test-prof 0.3.0.beta3 → 0.3.0.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6f2516ff090c97feae6be50fe1edc765aa495337
-  data.tar.gz: d6c4aba8cd0ace60d4c80898a8cfc1b22015a32c
+  metadata.gz: b81e0cb37ac0120a12a8497dc9de6fe7d14c4c53
+  data.tar.gz: b5f060c7d0ce985e5e4508be8cc2f161efe9389b
 SHA512:
-  metadata.gz: ddf6a60d44432bc56b253b7c667f40efedee36f7f37a4710c826ef5625dcb5c6768416e2c8fea681843894dfd8e39d78ba53ea48fb7ff6ee382eb24b807e8afa
-  data.tar.gz: 5fadec4649c715e2fcd93ab5ec6fc4b3e55c6b2cfeba877659431cb6894d3dcc9ed9a5b6bef5b46b36f28cfd34d928a2618b89f195808cc4ef95d9b0f6935f50
+  metadata.gz: c12c578c836c7553655925489fce0bdc1509b930f6f9170976e9b1d8eb4ba75fd60644e1f84eba92162cb9b365030d9a978b559635ce9b665c162fedf696b074
+  data.tar.gz: f81248b95f3320095878ad45288766d5646dd6f3dab53ecae844cc3fe12fa6e7f31ce3ec1502105aa00e02fee04fa64fc458e14e43e78dca0eff7bac5f7ff79a

data/CHANGELOG.md

@@ -4,20 +4,6 @@
 
 Features:
 
-- Combine RSpecStamp with FactoryDoctor. ([@palkan][])
-
-Automatically mark _bad_ examples with custom tags.
-
-- [#17](https://github.com/palkan/test-prof/pull/17) Combine RSpecStamp with EventProf and RSpecDissect. ([@palkan][])
-
-It is possible now to automatically mark _slow_ examples and groups with custom tags. For example:
-
-```sh
-EVENT_PROF="sql.active_record" EVENT_PROF_STAMP="slow:sql" rspec ...
-```
-
-After running the command above the top 5 slowest example groups would be marked with `slow: :sql` tag.
-
 - [#14](https://github.com/palkan/test-prof/pull/14) RSpecDissect profiler. ([@palkan][])
 
 RSpecDissect tracks how much time do you spend in `before` hooks 
@@ -27,16 +13,6 @@ and memoization helpers (i.e. `let`) in your tests.
 
 Just like `let`, but persist the result for the whole group (i.e. `let` + `before_all`).
 
-## 0.2.5
-
-- [#16](https://github.com/palkan/test-prof/pull/16) Support Ruby >= 2.2.0 (was >= 2.3.0). ([@palkan][])
-
-## 0.2.4
-
-- EventProf: Fix regression bug with examples profiling. ([@palkan][])
-
-There was a bug when an event occurs before the example has started (e.g. in `before(:context)` hook).
-
 ## 0.2.3
 
 - Minor improvements. ([@palkan][])

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>
 

data/guides/event_prof.md

@@ -66,16 +66,6 @@ EVENT_PROF_RANK=count EVENT_PROF='instantiation.active_record' be rspec
 
 See [event_prof.rb](https://github.com/palkan/test-prof/tree/master/lib/test_prof/event_prof.rb) for all available configuration options and their usage.
 
-## Using with RSpecStamp
-
-EventProf can be used with [RSpec Stamp](https://github.com/palkan/test-prof/tree/master/guides/rspec_stamp.md) to automatically mark _slow_ examples with custom tags. For example:
-
-```sh
-EVENT_PROF="sql.active_record" EVENT_PROF_STAMP="slow:sql" rspec ...
-```
-
-After running the command above the slowest example groups (and examples if configured) would be marked with the `slow: :sql` tag.
-
 ## Custom Instrumentation
 
 To use EventProf with your instrumentation engine just complete the two following steps:

data/guides/factory_doctor.md

@@ -62,13 +62,3 @@ To activate FactoryDoctor use `FDOC` environment variable:
 ```sh
 FDOC=1 rspec ...
 ```
-
-## Using with RSpecStamp
-
-FactoryDoctor can be used with [RSpec Stamp](https://github.com/palkan/test-prof/tree/master/guides/rspec_stamp.md) to automatically mark _bad_ examples with custom tags. For example:
-
-```sh
-FDOC=1 FDOC_STAMP="fdoc:consider" rspec ...
-```
-
-After running the command above all _potentially_ bad examples would be marked with the `fdoc: :consider` tag.

data/guides/rspec_dissect.md

@@ -47,12 +47,3 @@ You can also specify the number of top slow groups through `RD_TOP` variable:
 RD=1 RD_TOP=10 rspec ...
 ```
 
-## Using with RSpecStamp
-
-RSpecDissect can be used with [RSpec Stamp](https://github.com/palkan/test-prof/tree/master/guides/rspec_stamp.md) to automatically mark _slow_ examples with custom tags. For example:
-
-```sh
-RD=1 RD_STAMP="slow" rspec ...
-```
-
-After running the command above the slowest example groups would be marked with the `:slow` tag.

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

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/rspec_stamp"
 require "test_prof/event_prof/instrumentations/active_support"
 require "test_prof/utils/sized_ordered_set"
 
@@ -40,13 +39,6 @@ module TestProf
         @top_count = (ENV['EVENT_PROF_TOP'] || 5).to_i
         @per_example = ENV['EVENT_PROF_EXAMPLES'] == '1'
         @rank_by = (ENV['EVENT_PROF_RANK'] || :time).to_sym
-        @stamp = ENV['EVENT_PROF_STAMP']
-
-        RSpecStamp.config.tags = @stamp if stamp?
-      end
-
-      def stamp?
-        !@stamp.nil?
       end
 
       def per_example?
@@ -113,7 +105,7 @@ module TestProf
         @time += time
         @count += 1
 
-        return if @current_example.nil?
+        return unless config.per_example?
 
         @example_time += time
         @example_count += 1
@@ -132,10 +124,8 @@ module TestProf
         @current_group = nil
       end
 
-      def example_started(id)
-        return unless config.per_example?
-        reset_example!
-        @current_example = id
+      def example_started(_id)
+        reset_example! if config.per_example?
       end
 
       def example_finished(id)
@@ -144,7 +134,6 @@ module TestProf
 
         data = { id: id, time: @example_time, count: @example_count }
         @examples << data unless data[rank_by].zero?
-        @current_example = nil
       end
 
       def results

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,48 +72,13 @@ 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
         end
 
         log :info, msgs.join
-
-        stamp! if EventProf.config.stamp?
-      end
-
-      def stamp!
-        result = @profiler.results
-
-        stamper = RSpecStamp::Stamper.new
-
-        examples = Hash.new { |h, k| h[k] = [] }
-
-        (result[:groups].to_a + result.fetch(:examples, []).to_a)
-          .map { |obj| obj[:id].metadata[:location] }.each do |location|
-          file, line = location.split(":")
-          examples[file] << line.to_i
-        end
-
-        examples.each do |file, lines|
-          stamper.stamp_file(file, lines.uniq)
-        end
-
-        msgs = []
-
-        msgs <<
-          <<-MSG.strip_heredoc
-            RSpec Stamp results
-
-            Total patches: #{stamper.total}
-            Total files: #{examples.keys.size}
-
-            Failed patches: #{stamper.failed}
-            Ignored files: #{stamper.ignored}
-          MSG
-
-        log :info, msgs.join
       end
     end
   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
 
@@ -54,14 +54,6 @@ module TestProf
           defined?(::FactoryGirl)
 
         subscribe!
-
-        @stamp = ENV['FDOC_STAMP']
-
-        RSpecStamp.config.tags = @stamp if stamp?
-      end
-
-      def stamp?
-        !@stamp.nil?
       end
 
       def start
@@ -119,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}
@@ -70,40 +68,6 @@ module TestProf
         end
 
         log :info, msgs.join
-
-        stamp! if FactoryDoctor.stamp?
-      end
-
-      def stamp!
-        stamper = RSpecStamp::Stamper.new
-
-        examples = Hash.new { |h, k| h[k] = [] }
-
-        @example_groups.each do |_group, bad_examples|
-          bad_examples.each do |example|
-            file, line = example.metadata[:location].split(":")
-            examples[file] << line.to_i
-          end
-        end
-
-        examples.each do |file, lines|
-          stamper.stamp_file(file, lines.uniq)
-        end
-
-        msgs = []
-
-        msgs <<
-          <<-MSG.strip_heredoc
-            RSpec Stamp results
-
-            Total patches: #{stamper.total}
-            Total files: #{examples.keys.size}
-
-            Failed patches: #{stamper.failed}
-            Ignored files: #{stamper.ignored}
-          MSG
-
-        log :info, msgs.join
       end
 
       private

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

@@ -6,27 +6,18 @@ 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 = RUBY_ENGINE == 'jruby' ? "@__jruby_is_not_cat_friendly__".freeze : "@😸".freeze
+    PREFIX = "@😸"
+
+    def let_it_be(identifier, reload: false, refind: false, &block)
+      raise ArgumentError, "Block is required!" unless block_given?
+
+      this = self
 
-    def let_it_be(identifier, **options, &block)
       initializer = proc do
-        instance_variable_set(:"#{PREFIX}#{identifier}", instance_exec(&block))
+        this.instance_variable_set(:"#{PREFIX}#{identifier}", this.instance_exec(&block))
       end
 
       if within_before_all?
@@ -35,41 +26,20 @@ module TestProf
         before_all(&initializer)
       end
 
-      define_let_it_be_methods(identifier, **options)
-    end
+      let_accessor = accessor = -> { this.instance_variable_get(:"#{PREFIX}#{identifier}") }
 
-    def define_let_it_be_methods(identifier, reload: false, refind: false)
-      let_accessor = -> { instance_variable_get(:"#{PREFIX}#{identifier}") }
-
-      if reload
-        let_accessor = lambda do
-          record = instance_variable_get(:"#{PREFIX}#{identifier}")
-          next unless record.is_a?(::ActiveRecord::Base)
-          record.reload
-        end
-      end
+      let_accessor = -> { this.instance_variable_get(:"#{PREFIX}#{identifier}")&.reload } if reload
 
       if refind
         let_accessor = lambda do
-          record = instance_variable_get(:"#{PREFIX}#{identifier}")
+          record = this.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
-
+      define_singleton_method(identifier, &accessor)
       let(identifier, &let_accessor)
     end
   end

data/lib/test_prof/rspec_dissect.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "test_prof/rspec_stamp"
 require "test_prof/logging"
 
 module TestProf
@@ -37,13 +36,6 @@ module TestProf
 
       def initialize
         @top_count = (ENV['RD_TOP'] || 5).to_i
-        @stamp = ENV['RD_STAMP']
-
-        RSpecStamp.config.tags = @stamp if stamp?
-      end
-
-      def stamp?
-        !@stamp.nil?
       end
     end
 

data/lib/test_prof/rspec_dissect/rspec.rb

@@ -3,7 +3,6 @@
 require "test_prof/ext/float_duration"
 require "test_prof/ext/string_truncate"
 require "test_prof/utils/sized_ordered_set"
-require "test_prof/ext/string_strip_heredoc"
 
 module TestProf
   module RSpecDissect
@@ -11,7 +10,6 @@ module TestProf
       include Logging
       using FloatDuration
       using StringTruncate
-      using StringStripHeredoc
 
       NOTIFICATIONS = %i[
         example_finished
@@ -60,7 +58,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             RSpecDissect report
 
             Total time: #{@total_examples_time.duration}
@@ -70,65 +68,32 @@ module TestProf
           MSG
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             Top #{top_count} slowest suites (by `before(:each)` time):
 
           MSG
 
         @before_results.each do |group|
           msgs <<
-            <<-GROUP.strip_heredoc
+            <<~GROUP
               #{group[:desc].truncate} (#{group[:loc]}) – #{group[:before].duration} of #{group[:total].duration} (#{group[:count]})
             GROUP
         end
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             Top #{top_count} slowest suites (by `let` time):
 
           MSG
 
         @memo_results.each do |group|
           msgs <<
-            <<-GROUP.strip_heredoc
+            <<~GROUP
               #{group[:desc].truncate} (#{group[:loc]}) – #{group[:memo].duration} of #{group[:total].duration} (#{group[:count]})
             GROUP
         end
 
         log :info, msgs.join
-
-        stamp! if RSpecDissect.config.stamp?
-      end
-
-      def stamp!
-        stamper = RSpecStamp::Stamper.new
-
-        examples = Hash.new { |h, k| h[k] = [] }
-
-        (@before_results.to_a + @memo_results.to_a)
-          .map { |obj| obj[:loc] }.each do |location|
-          file, line = location.split(":")
-          examples[file] << line.to_i
-        end
-
-        examples.each do |file, lines|
-          stamper.stamp_file(file, lines.uniq)
-        end
-
-        msgs = []
-
-        msgs <<
-          <<-MSG.strip_heredoc
-            RSpec Stamp results
-
-            Total patches: #{stamper.total}
-            Total files: #{examples.keys.size}
-
-            Failed patches: #{stamper.failed}
-            Ignored files: #{stamper.ignored}
-          MSG
-
-        log :info, msgs.join
       end
 
       private

data/lib/test_prof/rspec_stamp.rb

@@ -44,49 +44,6 @@ module TestProf
       end
     end
 
-    # Stamper collects statistics about applying tags
-    # to examples.
-    class Stamper
-      include TestProf::Logging
-
-      attr_reader :total, :failed, :ignored
-
-      def initialize
-        @total = 0
-        @failed = 0
-        @ignored = 0
-      end
-
-      def stamp_file(file, lines)
-        @total += lines.size
-        return if ignored?(file)
-
-        log :info, "(dry-run) Patching #{file}" if dry_run?
-
-        code = File.readlines(file)
-
-        @failed += RSpecStamp.apply_tags(code, lines, RSpecStamp.config.tags)
-
-        File.write(file, code.join) unless dry_run?
-      end
-
-      private
-
-      def ignored?(file)
-        ignored = RSpecStamp.config.ignore_files.find do |pattern|
-          file =~ pattern
-        end
-
-        return unless ignored
-        log :warn, "Ignore stamping file: #{file}"
-        @ignored += 1
-      end
-
-      def dry_run?
-        RSpecStamp.config.dry_run?
-      end
-    end
-
     class << self
       include TestProf::Logging
 
@@ -127,7 +84,7 @@ module TestProf
         parsed = Parser.parse(code)
         return false unless parsed
 
-        desc = parsed.desc_const || quote(parsed.desc || 'works')
+        parsed.desc ||= 'works'
 
         tags.each do |t|
           if t.is_a?(Hash)
@@ -144,9 +101,9 @@ module TestProf
         need_parens = block == "{"
 
         tags_str = parsed.tags.map { |t| t.is_a?(Symbol) ? ":#{t}" : t }.join(", ") unless
-          parsed.tags.nil? || parsed.tags.empty?
+          parsed.tags.nil?
 
-        unless parsed.htags.nil? || parsed.htags.empty?
+        unless parsed.htags.nil?
           htags_str = parsed.htags.map do |(k, v)|
             vstr = v.is_a?(Symbol) ? ":#{v}" : quote(v)
 
@@ -155,7 +112,7 @@ module TestProf
         end
 
         replacement = "\\1#{parsed.fname}#{need_parens ? '(' : ' '}"\
-                      "#{[desc, tags_str, htags_str].compact.join(', ')}"\
+                      "#{[quote(parsed.desc), tags_str, htags_str].compact.join(', ')}"\
                       "#{need_parens ? ') ' : ' '}\\3"
 
         if config.dry_run?
@@ -169,7 +126,6 @@ module TestProf
       # rubocop: enable Metrics/PerceivedComplexity
 
       def quote(str)
-        return str unless str.is_a?(String)
         if str.include?("'")
           "\"#{str}\""
         else

data/lib/test_prof/rspec_stamp/parser.rb

@@ -2,13 +2,15 @@
 
 require "ripper"
 
+# rubocop: disable Metrics/CyclomaticComplexity
+
 module TestProf
   module RSpecStamp
     # Parse examples headers
     module Parser
       # Contains the result of parsing
       class Result
-        attr_accessor :fname, :desc, :desc_const
+        attr_accessor :fname, :desc
         attr_reader :tags, :htags
 
         def add_tag(v)
@@ -22,14 +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
-        # rubocop: disable Metrics/PerceivedComplexity
         def parse(code)
           sexp = Ripper.sexp(code)
           return unless sexp
@@ -58,16 +58,10 @@ module TestProf
           res = Result.new
 
           fcall = sexp[1][0][1]
-          args_block = sexp[1][0][2]
-
-          if fcall.first == :fcall
-            fcall = fcall[1]
-          elsif fcall.first == :var_ref
-            res.fname = [parse_const(fcall), sexp[1][0][3][1]].join(".")
-            args_block = sexp[1][0][4]
-          end
+          fcall = fcall[1] if fcall.first == :fcall
+          res.fname = fcall[1]
 
-          res.fname ||= fcall[1]
+          args_block = sexp[1][0][2]
 
           return res if args_block.nil?
 
@@ -77,16 +71,12 @@ module TestProf
 
           if args.first.first == :string_literal
             res.desc = parse_literal(args.shift)
-          elsif args.first.first == :var_ref || args.first.first == :const_path_ref
-            res.desc_const = parse_const(args.shift)
           end
 
           parse_arg(res, args.shift) until args.empty?
 
           res
         end
-        # rubocop: enable Metrics/CyclomaticComplexity
-        # rubocop: enable Metrics/PerceivedComplexity
 
         private
 
@@ -100,24 +90,7 @@ module TestProf
 
         def parse_hash(res, hash_arg)
           hash_arg.each do |(_, label, val)|
-            res.add_htag label[1][0..-2].to_sym, parse_value(val)
-          end
-        end
-
-        # Expr of the form:
-        #  bool - [:var_ref, [:@kw, "true", [1, 24]]]
-        #  string - [:string_literal, [:string_content, [...]]]
-        #  int - [:@int, "3", [1, 52]]]]
-        def parse_value(expr)
-          case expr.first
-          when :var_ref
-            expr[1][1] == "true"
-          when :@int
-            expr[1].to_i
-          when :@float
-            expr[1].to_f
-          else
-            parse_literal(expr)
+            res.add_htag label[1][0..-2].to_sym, parse_literal(val)
           end
         end
 
@@ -129,25 +102,6 @@ module TestProf
                               expr[0] == :assoc_new
           val
         end
-
-        # Expr of the form:
-        #  [:var_ref, [:@const, "User", [1, 9]]]
-        #
-        #  or
-        #
-        #  [:const_path_ref, [:const_path_ref, [:var_ref,
-        #    [:@const, "User", [1, 17]]],
-        #    [:@const, "Guest", [1, 23]]],
-        #    [:@const, "Collection", [1, 30]]
-        def parse_const(expr)
-          if expr.first == :var_ref
-            expr[1][1]
-          elsif expr.first == :@const
-            expr[1]
-          elsif expr.first == :const_path_ref
-            expr[1..-1].map(&method(:parse_const)).join("::")
-          end
-        end
       end
     end
   end

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
@@ -30,27 +27,54 @@ module TestProf
       end
 
       def stamp!
-        stamper = Stamper.new
-
         @examples.each do |file, lines|
-          stamper.stamp_file(file, lines.uniq)
+          stamp_file(file, lines.uniq)
         end
 
         msgs = []
 
         msgs <<
-          <<-MSG.strip_heredoc
+          <<~MSG
             RSpec Stamp results
 
-            Total patches: #{stamper.total}
+            Total patches: #{@total}
             Total files: #{@examples.keys.size}
 
-            Failed patches: #{stamper.failed}
-            Ignored files: #{stamper.ignored}
+            Failed patches: #{@failed}
+            Ignored files: #{@ignored}
           MSG
 
         log :info, msgs.join
       end
+
+      private
+
+      def stamp_file(file, lines)
+        @total += lines.size
+        return if ignored?(file)
+
+        log :info, "(dry-run) Patching #{file}" if dry_run?
+
+        code = File.readlines(file)
+
+        @failed += RSpecStamp.apply_tags(code, lines, RSpecStamp.config.tags)
+
+        File.write(file, code.join) unless dry_run?
+      end
+
+      def ignored?(file)
+        ignored = RSpecStamp.config.ignore_files.find do |pattern|
+          file =~ pattern
+        end
+
+        return unless ignored
+        log :warn, "Ignore stamping file: #{file}"
+        @ignored += 1
+      end
+
+      def dry_run?
+        RSpecStamp.config.dry_run?
+      end
     end
   end
 end

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)
@@ -54,7 +49,7 @@ module TestProf
       end
 
       def to_a
-        data.dup
+        data
       end
 
       private

data/lib/test_prof/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.3.0.beta3
+  version: 0.3.0.pre
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-05 00:00:00.000000000 Z
+date: 2017-08-28 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
@@ -128,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
@@ -176,7 +188,7 @@ 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:
   - - ">"

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