test-prof 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5c0da7ccf787bf83ee5ed43d069256d1183682ae
-  data.tar.gz: 44fd06ab01a79c6b499e4f9c36caaf56623df65c
+  metadata.gz: 0df1119b6d47cf2641984fe6bcc7387c228353ff
+  data.tar.gz: d472861344964d895fc964afb099f3d3a35dbfae
 SHA512:
-  metadata.gz: 87ad75af611a473079d57cbe82d30a4c1a54eeb8d61a94190c72761c87ebad47bca6169b98f2fd84eda549c63dfce4d2dd9d11214a86a26ccebbec83895f53d4
-  data.tar.gz: 1cf076dc20db11b302d646ca8caf7379916b536ab24e42500cdb4378cbbc5cc74ba22411c8a15f024227023cc3bb8830c67d15e4e012ebc3099905470af17a5c
+  metadata.gz: ccde20ed06a18ff23b85ee1d2b42269214d8ea8dbaf88fda55a78093bd76c99cafef050b3db70e76f98570709b7adfec46d45e9cbb1140233f688c80bc158115
+  data.tar.gz: 2bda30e78808f8edfa279b88abafa617d416fa6f1bc9f235c883ab106a18d9619c8a5f4db3c412ede368307eade325db0d3559788e2015f2a7aeac6445cdeaed

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change log
 
+## 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][])

data/README.md

@@ -24,6 +24,12 @@ 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/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&.begin_type?
+          return unless body && 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&.block_type? &&
+          node && node.block_type? &&
             (node.source.lines.size == 1) &&
             example_node?(node)
         end

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

@@ -1,5 +1,9 @@
 # frozen_string_literal: true
 
+require "test_prof/ext/string_strip_heredoc"
+
+using TestProf::StringStripHeredoc
+
 module TestProf::EventProf::CustomEvents
   module FactoryCreate # :nodoc: all
     module RunnerPatch
@@ -42,7 +46,7 @@ end
 TestProf.activate('EVENT_PROF', 'factory.create') do
   if TestProf.require(
     'factory_girl',
-    <<~MSG
+    <<-MSG.strip_heredoc
       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,5 +1,9 @@
 # frozen_string_literal: true
 
+require "test_prof/ext/string_strip_heredoc"
+
+using TestProf::StringStripHeredoc
+
 module TestProf::EventProf::CustomEvents
   module SidekiqInline # :nodoc: all
     module ClientPatch
@@ -39,7 +43,7 @@ end
 TestProf.activate('EVENT_PROF', 'sidekiq.inline') do
   if TestProf.require(
     'sidekiq/testing',
-    <<~MSG
+    <<-MSG.strip_heredoc
       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,5 +1,9 @@
 # frozen_string_literal: true
 
+require "test_prof/ext/string_strip_heredoc"
+
+using TestProf::StringStripHeredoc
+
 module TestProf::EventProf::CustomEvents
   module SidekiqJobs # :nodoc: all
     module ClientPatch
@@ -26,7 +30,7 @@ end
 TestProf.activate('EVENT_PROF', 'sidekiq.jobs') do
   if TestProf.require(
     'sidekiq/testing',
-    <<~MSG
+    <<-MSG.strip_heredoc
       Failed to load Sidekiq.
 
       Make sure that "sidekiq" gem is in your Gemfile.

data/lib/test_prof/event_prof/rspec.rb

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

data/lib/test_prof/ext/array_bsearch_index.rb

@@ -0,0 +1,15 @@
+# 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

@@ -0,0 +1,15 @@
+# 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

data/lib/test_prof/factory_doctor.rb

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

data/lib/test_prof/factory_doctor/rspec.rb

@@ -1,14 +1,16 @@
 # 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!"'
+      SUCCESS_MESSAGE = 'FactoryDoctor says: "Looks good to me!"'.freeze
 
       NOTIFICATIONS = %i[
         example_started
@@ -49,7 +51,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<~MSG
+          <<-MSG.strip_heredoc
             FactoryDoctor report
 
             Total (potentially) bad examples: #{@count}

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

@@ -1,16 +1,19 @@
 # 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
+            <<-MSG.strip_heredoc
               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.positive?
+    pool.active_connection? && pool.connection.open_transactions > 0
   end
 end
 

data/lib/test_prof/rspec_stamp/parser.rb

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

data/lib/test_prof/rspec_stamp/rspec.rb

@@ -1,9 +1,12 @@
 # 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
@@ -34,7 +37,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<~MSG
+          <<-MSG.strip_heredoc
             RSpec Stamp results
 
             Total patches: #{@total}

data/lib/test_prof/ruby_prof.rb

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

data/lib/test_prof/stack_prof.rb

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

data/lib/test_prof/tag_prof/rspec.rb

@@ -1,12 +1,14 @@
 # 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
@@ -37,7 +39,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<~MSG
+          <<-MSG.strip_heredoc
             TagProf report for #{@tag}
           MSG
 

data/lib/test_prof/utils/sized_ordered_set.rb

@@ -4,6 +4,11 @@ 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)

data/lib/test_prof/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-29 00:00:00.000000000 Z
+date: 2017-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -80,20 +80,6 @@ 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
@@ -140,7 +126,9 @@ 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
@@ -183,7 +171,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -191,7 +179,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: Ruby applications tests profiling tools