test-prof 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2b61bd0dd4011eb8d77c50a343ba8e582393bd68
-  data.tar.gz: 3a233a89a57d261382dc253cb87156655a7356c6
+  metadata.gz: eb103d86ccc7a513de8b30ae8ddf536c9bbc6255
+  data.tar.gz: f0fe1c206faab5761b4419ccbf8745f4060d3e82
 SHA512:
-  metadata.gz: 335d1151f7fcf286f9bf0ba01fcadcbe09c042e9712d069c4c85a3acc5e28f548add3580292fbed6ec8d89da796330b81ac66809fd55ab952fef5a20caf4e200
-  data.tar.gz: b2fb7e978bb9247c9d53e082682bb5fae34d0f7179784295457077100abff1365481b564fa4bb62e1bb24f4a3edf9d8fff0ef6be199ddf188ee99813ee369573
+  metadata.gz: f6a4881cfc954438fa4184c2cd9be1784d10bfcd6d92931ea5132c8422aa08866b77e60ec91066ddd6afb620c524e99b4584818d158f396176f9649ef3a0e038
+  data.tar.gz: a699bd6e308b370aa4e53447025551a5b2f30249eb893356d30238eaed4dc135bb69335c780ea9d0b306ce569bca29117494ab91b6a70bbb7a22f714a65f827f

data/CHANGELOG.md

@@ -1,6 +1,64 @@
 # Change log
 
-## master
+## 0.3.0
+
+### 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 
+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`).
+
+### Improvements:
+
+- Add ability to specify RubyProf report through `TEST_RUBY_PROF` env variable. ([@palkan][])
+
+- Add ability to specify StackProf raw mode through `TEST_STACK_PROF` env variable. ([@palkan][])
+
+### Changes
+
+- Use RubyProf `FlatPrinter` by default (was `CallStackPrinter`). ([@palkan][])
+
+## 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][])
+
+## 0.2.2
+
+- Fix time calculation when Time class is monkey-patched. ([@palkan][])
+
+Add `TestProf.now` method which is just a copy of original `Time.now` and
+use it everywhere.
+
+Fixes [#10](https://github.com/palkan/test-prof/issues/10).
 
 ## 0.2.1
 

data/README.md

@@ -10,7 +10,7 @@ Simply speaking, slow tests waste your time making you less productive.
 
 TestProf toolbox aims to help you identify bottlenecks in your test suite. It contains:
 
-- Plug'n'Play integrations for general Ruby profilers ([`ruby-prof`](https://github.com/ruby-prof), [`stackprof`](https://github.com/tmm1/stackprof))
+- Plug'n'Play integrations for general Ruby profilers ([`ruby-prof`](https://github.com/ruby-prof/ruby-prof), [`stackprof`](https://github.com/tmm1/stackprof))
 
 - Factories usage analyzers and profilers
 
@@ -20,10 +20,16 @@ TestProf toolbox aims to help you identify bottlenecks in your test suite. It co
 
 - etc.
 
-Of course, we have some [solutions](#tips-and-tricks) for common performance issues too, bundled into the gem.
+Of course, we have some [solutions](#tips-and-tricks-or-recipes) for common performance issues too, bundled into the gem.
 
 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>
 
@@ -63,6 +69,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_)
@@ -71,6 +79,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/event_prof.md

@@ -55,6 +55,8 @@ TestProf::EventProf.configure do |config|
 end
 ```
 
+Or provide the `EVENT_PROF_EXAMPLES=1` env variable.
+
 Another useful configuration parameter – `rank_by`. It's responsible for sorting stats – 
 either by the time spent in the event or by the number of occurrences:
 
@@ -64,6 +66,16 @@ 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,3 +62,13 @@ 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/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,58 @@
+# 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_PROF` environment variable:
+
+```sh
+RD_PROF=1 rspec ...
+```
+
+You can also specify the number of top slow groups through `RD_PROF_TOP` variable:
+
+```sh
+RD_PROF=1 RD_PROF_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_PROF=1 RD_PROF_STAMP="slow" rspec ...
+```
+
+After running the command above the slowest example groups would be marked with the `:slow` tag.

data/guides/ruby_prof.md

@@ -9,7 +9,7 @@ Install `ruby-prof` gem (>= 0.16):
 ```ruby
 # Gemfile
 group :development, :test do
-  gem 'ruby-prof', require: false
+  gem 'ruby-prof', '>= 0.16.0', require: false
 end
 ```
 
@@ -42,21 +42,21 @@ end
 
 The most useful configuration option is `printer` – it allows you to specify a RubyProf [printer](https://github.com/ruby-prof/ruby-prof#printers).
 
-You can specify a printer through environment variable `TEST_RUBY_PROF_PRINTER`:
+You can specify a printer through environment variable `TEST_RUBY_PROF`:
 
 ```sh
-TEST_RUBY_PROF_PRINTER=flat bundle exec rake test
+TEST_RUBY_PROF=call_stack bundle exec rake test
 ```
 
 Or in your code:
 
 ```ruby
 TestProf::RubyProf.configure do |config|
-  config.printer = :flat
+  config.printer = :call_stack
 end
 ```
 
-By default, we use `CallStackPrinter`.
+By default, we use `FlatPrinter`.
 
 Also, you can specify RubyProf mode (`wall`, `cpu`, etc) through `TEST_RUBY_PROF_MODE` env variable.
 

data/guides/stack_prof.md

@@ -4,12 +4,12 @@
 
 ## Instructions
 
-Install 'stackprof' gem (>= 0.2.7):
+Install 'stackprof' gem (>= 0.2.9):
 
 ```ruby
 # Gemfile
 group :development, :test do
-  gem 'stackprof', require: false
+  gem 'stackprof', '>= 0.2.9', require: false
 end
 ```
 
@@ -42,6 +42,6 @@ end
 
 You can change StackProf mode (which is `wall` by default) through `TEST_STACK_PROF_MODE` env variable.
 
-If you want to generate flame graphs you should collect _raw_ data. Turn _raw_ collection on by passing `TEST_STACK_PROF_RAW=1`.
+If you want to generate flame graphs you should collect _raw_ data. Turn _raw_ collection on by passing `TEST_STACK_PROF=raw`.
 
 See [stack_prof.rb](https://github.com/palkan/test-prof/tree/master/lib/test_prof/stack_prof.rb) for all available configuration options and their usage.

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

@@ -1,12 +1,16 @@
 # frozen_string_literal: true
 
 require "test_prof/ext/float_duration"
+require "test_prof/ext/string_truncate"
+require "test_prof/ext/string_strip_heredoc"
 
 module TestProf
   module EventProf
     class RSpecListener # :nodoc:
       include Logging
       using FloatDuration
+      using StringTruncate
+      using StringStripHeredoc
 
       NOTIFICATIONS = %i[
         example_group_started
@@ -43,7 +47,7 @@ module TestProf
         msgs = []
 
         msgs <<
-          <<~MSG
+          <<-MSG.strip_heredoc
             EventProf results for #{@profiler.event}
 
             Total time: #{@profiler.total_time.duration}
@@ -58,8 +62,8 @@ module TestProf
           location = group[:id].metadata[:location]
 
           msgs <<
-            <<~GROUP
-              #{description} (#{location}) – #{group[:time].duration} (#{group[:count]} / #{group[:examples]})
+            <<-GROUP.strip_heredoc
+              #{description.truncate} (#{location}) – #{group[:time].duration} (#{group[:count]} / #{group[:examples]})
             GROUP
         end
 
@@ -70,13 +74,48 @@ module TestProf
             description = example[:id].description
             location = example[:id].metadata[:location]
             msgs <<
-              <<~GROUP
-                #{description} (#{location}) – #{example[:time].duration} (#{example[:count]})
+              <<-GROUP.strip_heredoc
+                #{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/event_prof.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
+require "test_prof/rspec_stamp"
 require "test_prof/event_prof/instrumentations/active_support"
+require "test_prof/utils/sized_ordered_set"
 
 module TestProf
   # EventProf profiles your tests and suites against custom events,
@@ -19,6 +21,8 @@ module TestProf
   #  TestProf::EventProf.configure do |config|
   #    config.per_example = true
   #  end
+  #
+  # Or provide the EVENT_PROF_EXAMPLES=1 env variable.
   module EventProf
     # EventProf configuration
     class Configuration
@@ -34,8 +38,15 @@ module TestProf
         @event = ENV['EVENT_PROF']
         @instrumenter = :active_support
         @top_count = (ENV['EVENT_PROF_TOP'] || 5).to_i
-        @per_example = false
+        @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?
@@ -73,7 +84,7 @@ module TestProf
     class Profiler # :nodoc:
       include TestProf::Logging
 
-      attr_reader :event, :top_count, :rank_by, :total_count, :total_time
+      attr_reader :event, :total_count, :total_time
 
       def initialize(event:, instrumenter:)
         @event = event
@@ -82,13 +93,16 @@ module TestProf
 
         instrumenter.subscribe(event) { |time| track(time) }
 
-        @groups = Hash.new { |h, k| h[k] = { id: k } }
-        @examples = Hash.new { |h, k| h[k] = { id: k } }
+        @groups = Utils::SizedOrderedSet.new(
+          top_count, sort_by: rank_by
+        )
+
+        @examples = Utils::SizedOrderedSet.new(
+          top_count, sort_by: rank_by
+        )
 
         @total_count = 0
         @total_time = 0.0
-
-        reset!
       end
 
       def track(time)
@@ -99,41 +113,47 @@ module TestProf
         @time += time
         @count += 1
 
-        @example_time += time if config.per_example?
-        @example_count += 1 if config.per_example?
+        return if @current_example.nil?
+
+        @example_time += time
+        @example_count += 1
       end
 
       def group_started(id)
-        reset!
+        reset_group!
         @current_group = id
       end
 
       def group_finished(id)
-        @groups[id][:time] = @time
-        @groups[id][:count] = @count
-        @groups[id][:examples] = @total_examples
+        data = { id: id, time: @time, count: @count, examples: @total_examples }
+
+        @groups << data unless data[rank_by].zero?
+
         @current_group = nil
       end
 
-      def example_started(_id)
+      def example_started(id)
+        return unless config.per_example?
         reset_example!
+        @current_example = id
       end
 
       def example_finished(id)
         @total_examples += 1
         return unless config.per_example?
 
-        @examples[id][:time] = @example_time
-        @examples[id][:count] = @example_count
+        data = { id: id, time: @example_time, count: @example_count }
+        @examples << data unless data[rank_by].zero?
+        @current_example = nil
       end
 
       def results
         {
-          groups: fetch_top(@groups.values)
+          groups: @groups.to_a
         }.tap do |data|
           next unless config.per_example?
 
-          data[:examples] = fetch_top(@examples.values)
+          data[:examples] = @examples.to_a
         end
       end
 
@@ -147,21 +167,14 @@ module TestProf
 
       private
 
-      def fetch_top(arr)
-        arr.reject { |el| el[rank_by].zero? }
-           .sort_by { |el| -el[rank_by] }
-           .take(top_count)
-      end
-
       def config
         EventProf.config
       end
 
-      def reset!
+      def reset_group!
         @time = 0.0
         @count = 0
         @total_examples = 0
-        reset_example!
       end
 
       def reset_example!