test-prof 0.7.5 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d382abde5291f54f1168e9403e3151aaafbe6e6e7409de394169169a0eca19fe
-  data.tar.gz: 68037c8c7d819c6e68d7e8d4a4291471101539a6239f452e630de5ea0a4648d6
+  metadata.gz: 6cf66f80707806f1378f3164e1376e6ee8fe5c3472d0704ef865de4939852283
+  data.tar.gz: 472ea3dde66363470b5e62995548760359f5ecd358b84099d12d9a622b36063d
 SHA512:
-  metadata.gz: fe73709e9e6d6e609f4a0f705d2de0dbab37daecffeb1bac91a4b1a7585a9f81bc656489e9b84d43ad5a9dd04be93a5b9b1020eecd370cbbfee2d2c2fbc862ce
-  data.tar.gz: 91ff9f2c02b86260c9d0d88b834775528007b05eefa3fb00ba28ce9464dbfc615ae7ddde33f2d7544bfc048602687da078b6d01bd38038609b1ca33bb83500fe
+  metadata.gz: 4c21d8cae35943af31f72617a20b11d653ade888ba264c510dbe1fc365052a3ed7c7e04a7516b967a4f47dcb2442754a0d4e60053c385ae859963b666d239f9e
+  data.tar.gz: 7615a8bdb3d087c254333c265ffd759d13b51e76c6913d8e38a6856bdc036303f096ef2f2fb83c1b867024a87fe4989635105361103e85d7e1f4b14c0ae584bd

data/CHANGELOG.md

@@ -1,6 +1,75 @@
 # Change log
 
-## master
+## master (unreleased)
+
+## 0.9.0 (2019-05-14)
+
+- Add threshold and custom event support to FactoryDoctor. ([@palkan][])
+
+  ```sh
+  $ FDOC=1 FDOC_EVENT="sql.rom" FDOC_THRESHOLD=0.1 rspec
+  ```
+
+- Add Fabrication support to FactoryDoctor. ([@palkan][])
+
+- Add `guard` and `top_level` options to `EventProf::Monitor`. ([@palkan][])
+
+  For example:
+
+  ```ruby
+  TestProf::EventProf.monitor(
+    Sidekiq::Client,
+    "sidekiq.inline",
+    :raw_push,
+    top_level: true,
+    guard: ->(*) { Sidekiq::Testing.inline? }
+  )
+  ```
+
+- Add global `before_all` hooks. ([@danielwaterworth][], [@palkan][])
+
+  Now you can run additional code before and after every `before_all` transaction
+  begins and rollbacks:
+
+  ```ruby
+  TestProf::BeforeAll.configure do |config|
+    config.before(:begin) do
+      # do something before transaction opens
+    end
+
+    config.after(:rollback) do
+      # do something after transaction closes
+    end
+  end
+  ```
+
+- Add ability to use `let_it_be` aliases with predefined options. ([@danielwaterworth][])
+
+  ```ruby
+  TestProf::LetItBe.configure do |config|
+    config.alias_to :let_it_be_with_refind, refind: true
+  end
+  ```
+
+- Made FactoryProf measure and report on timing ([@danielwaterworth][])
+
+## 0.8.0 (2019-04-12) 🚀
+
+- **Ruby 2.4+ is requiered** ([@palkan][])
+
+- **RSpec 3.5+ is requiered for RSpec features** ([@palkan][])
+
+- Make `before_all` compatible with [`isolator`](https://github.com/palkan/isolator). ([@palkan][])
+
+- Add `with_logging` and `with_ar_logging` helpers to logging recipe. ([@palkan][])
+
+- Make `before_all` for Active Record `lock_thread` aware. ([@palkan][])
+
+  `before_all` can went crazy if you open multiple connections within it
+  (since it tracks the number of open transactions).
+  Rails 5+ `lock_thread` feature only locks the connection thread in
+  `before`/`setup` hook thus making it possible to have multiple connections/transactions
+  in `before_all` (e.g. performing jobs with Active Job async adapter).
 
 ## 0.7.5 (2019-02-22)
 
@@ -20,10 +89,10 @@
 
 - Improve test sampling.([@mkldon][])
 
- ```bash
- SAMPLE=10 rake test # runs 10 random test examples
- SAMPLE_GROUPS=10 rake test # runs 10 random example groups
- ```
+  ```bash
+  $ SAMPLE=10 rake test # runs 10 random test examples
+  $ SAMPLE_GROUPS=10 rake test # runs 10 random example groups
+  ```
 
 - Extend Event Prof formatter to include the absolute run time and the percentage of the event tim [#100](https://github.com/palkan/test-prof/issues/100) ([@dmagro][])
 
@@ -35,32 +104,32 @@
 
 - Add ability to ignore connection configurations in shared connection.([@palkan][])
 
-Example:
+  Example:
 
-```ruby
-# Do not use shared connection for sqlite db
-TestProf::ActiveRecordSharedConnection.ignore { |config| config[:adapter] == "sqlite3" }
-```
+  ```ruby
+  # Do not use shared connection for sqlite db
+  TestProf::ActiveRecordSharedConnection.ignore { |config| config[:adapter] == "sqlite3" }
+  ```
 
 ## 0.7.0 (2018-08-12)
 
 - **Ruby 2.3+ is required**. ([@palkan][])
 
-Ruby 2.2 EOL was on 2018-03-31.
+  Ruby 2.2 EOL was on 2018-03-31.
 
 - Upgrade RubyProf integration to `ruby-prof >= 0.17`. ([@palkan][])
 
-Use `exclude_common_methods!` instead of the deprecated `eliminate_methods!`.
+  Use `exclude_common_methods!` instead of the deprecated `eliminate_methods!`.
 
-Add RSpec specific exclusions.
+  Add RSpec specific exclusions.
 
-Add ability to specify custom exclusions through `config.custom_exclusions`, e.g.:
+  Add ability to specify custom exclusions through `config.custom_exclusions`, e.g.:
 
-```ruby
-TestProf::RubyProf.configure do |config|
-  config.custom_exclusions = { User => %i[save save!] }
-end
-```
+  ```ruby
+  TestProf::RubyProf.configure do |config|
+    config.custom_exclusions = {User => %i[save save!]}
+  end
+  ```
 
 ## 0.6.0 (2018-06-29)
 
@@ -68,25 +137,24 @@ end
 
 - Add `EventProf.monitor` to instrument arbitrary methods. ([@palkan][])
 
-Add custom instrumetation easily:
+  Add custom instrumetation easily:
 
-```ruby
-class Work
-  def do
-    # ...
+  ```ruby
+  class Work
+    def do
+      # ...
+    end
   end
-end
 
-# Instrument Work#do calls with "my.work" event
-TestProf::EventProf.monitor(Work, "my.work", :do)
-```
+  # Instrument Work#do calls with "my.work" event
+  TestProf::EventProf.monitor(Work, "my.work", :do)
+  ```
 
 [📝 Docs](https://test-prof.evilmartians.io/#/event_prof?id=profile-arbitrary-methods)
 
 - Adapterize `before_all`. ([@palkan][])
 
-Now it's possible to write your own adapter for `before_all` to manage
-transactions.
+  Now it's possible to write your own adapter for `before_all` to manage transactions.
 
 [📝 Docs](https://test-prof.evilmartians.io/#/before_all?id=database-adapters)
 
@@ -98,32 +166,32 @@ transactions.
 
 - Show top `let` declarations per example group in RSpecDissect profiler. ([@palkan][])
 
-The output now includes the following information:
+  The output now includes the following information:
 
-```
-Top 5 slowest suites (by `let` time):
+  ```
+  Top 5 slowest suites (by `let` time):
 
-FunnelsController (./spec/controllers/funnels_controller_spec.rb:3) – 00:38.532 of 00:43.649 (133)
- ↳ user – 3
- ↳ funnel – 2
-ApplicantsController (./spec/controllers/applicants_controller_spec.rb:3) – 00:33.252 of 00:41.407 (222)
- ↳ user – 10
- ↳ funnel – 5
- ```
+  FunnelsController (./spec/controllers/funnels_controller_spec.rb:3) – 00:38.532 of 00:43.649 (133)
+  ↳ user – 3
+  ↳ funnel – 2
+  ApplicantsController (./spec/controllers/applicants_controller_spec.rb:3) – 00:33.252 of 00:41.407 (222)
+  ↳ user – 10
+  ↳ funnel – 5
+  ```
 
-Enabled by default. Disable it with:
+  Enabled by default. Disable it with:
 
-```ruby
-TestProf::RSpecDissect.configure do |config|
-  config.let_stats_enabled = false
-end
-```
+  ```ruby
+  TestProf::RSpecDissect.configure do |config|
+    config.let_stats_enabled = false
+  end
+  ```
 
 - [Fix [#80](https://github.com/palkan/test-prof/issues/80)] Added ability to preserve traits. ([@Vasfed][])
 
-Disabled by default for compatibility. Enable globally by `FactoryDefault.preserve_traits = true` or for single `create_default`: `create_default(:user, preserve_traits: true)`
+  Disabled by default for compatibility. Enable globally by `FactoryDefault.preserve_traits = true` or for single `create_default`: `create_default(:user, preserve_traits: true)`
 
-When enabled - default object will be used only when there's no [traits](https://github.com/thoughtbot/factory_bot/blob/master/GETTING_STARTED.md#traits) in association.
+  When enabled - default object will be used only when there's no [traits](https://github.com/thoughtbot/factory_bot/blob/master/GETTING_STARTED.md#traits) in association.
 
 - Add ability to run only `let` or `before` profiler with RSpecDissect. ([@palkan][])
 
@@ -133,8 +201,8 @@ When enabled - default object will be used only when there's no [traits](https:/
 
 - [Fix [#75](https://github.com/palkan/test-prof/issues/75)] Fix `RSpec/Aggregate` failures with non-regular examples. ([@palkan][])
 
-Do not take into account `xit`, `pending`, `its`, etc. examples,
-only consider regular `it`, `specify`, `scenario`, `example`.
+  Do not take into account `xit`, `pending`, `its`, etc. examples,
+  only consider regular `it`, `specify`, `scenario`, `example`.
 
 ## 0.5.0 (2018-04-25)
 
@@ -142,78 +210,78 @@ only consider regular `it`, `specify`, `scenario`, `example`.
 
 - Add events support to TagProf. ([@palkan][])
 
-Example usage:
+  Example usage:
 
-```sh
-TAG_PROF=type TAG_PROF_EVENT=sql.active_record rspec
-```
+  ```sh
+  TAG_PROF=type TAG_PROF_EVENT=sql.active_record rspec
+  ```
 
-[📝 Docs](https://test-prof.evilmartians.io/#/tag_prof?id=profiling-events)
+  [📝 Docs](https://test-prof.evilmartians.io/#/tag_prof?id=profiling-events)
 
 - Add logging helpers for Rails. ([@palkan][])
 
-Enalbe verbose logging globally:
+  Enable verbose logging globally:
 
-```sh
-LOG=all rspec
-```
+  ```sh
+  LOG=all rspec
+  ```
 
-Or per example (group):
+  Or per example (group):
 
-```ruby
-it 'does smth weird', :log do
-  # ...
-end
-```
+  ```ruby
+  it "does smth weird", :log do
+    # ...
+  end
+  ```
 
-[📝 Docs](https://test-prof.evilmartians.io/#/logging)
+  [📝 Docs](https://test-prof.evilmartians.io/#/logging)
 
 - Add HTML report for `TagProf`. ([@palkan][])
 
-Generate HTML report by setting `TAG_PROF_FORMAT` to `html`.
+  Generate HTML report by setting `TAG_PROF_FORMAT` to `html`.
 
 - Add ability to track multiple events at the same time with `EventProf`. ([@palkan][])
 
 - Add `AnyFixture` DSL. ([@palkan][])
 
-Example:
+  Example:
 
-```ruby
-# Enable DSL
-using TestProf::AnyFixture::DSL
+  ```ruby
+  # Enable DSL
+  using TestProf::AnyFixture::DSL
 
-# and then you can use `fixture` method (which is just an alias for `TestProf::AnyFixture.register`)
-before(:all) { fixture(:account) }
+  # and then you can use `fixture` method (which is just an alias for `TestProf::AnyFixture.register`)
+  before(:all) { fixture(:account) }
 
-# You can also use it to fetch the record (instead of storing it in instance variable)
-let(:account) { fixture(:account) }
-```
+  # You can also use it to fetch the record (instead of storing it in instance variable)
+  let(:account) { fixture(:account) }
+  ```
 
-[📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=dsl)
+  [📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=dsl)
 
 - Add `AnyFixture` usage report. ([@palkan][])
 
-Enable `AnyFixture` usage reporting with `ANYFIXTURE_REPORTING=1` or with:
+  Enable `AnyFixture` usage reporting with `ANYFIXTURE_REPORTING=1` or with:
 
-```ruby
-TestProf::AnyFixture.reporting_enabled = true
-```
+  ```ruby
+  TestProf::AnyFixture.reporting_enabled = true
+  ```
 
-[📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=usage-report)
+  [📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=usage-report)
 
 - Add `ActiveRecordSharedConnection` recipe. ([@palkan][])
 
-Force ActiveRecord to use the same connection between threads (to avoid database cleaning in browser tests).
+  Force ActiveRecord to use the same connection between threads (to avoid database cleaning in browser tests).
 
-[📝 Docs](https://test-prof.evilmartians.io/#/active_record_shared_connection)
+  [📝 Docs](https://test-prof.evilmartians.io/#/active_record_shared_connection)
 
 - [#70](https://github.com/palkan/test-prof/pull/70) Add `FactoryAllStub` recipe. ([@palkan][])
 
-[📝 Docs](https://test-prof.evilmartians.io/#/factory_all_stub)
+  [📝 Docs](https://test-prof.evilmartians.io/#/factory_all_stub)
 
 - Add `ActiveRecordRefind` refinement. ([@palkan][])
 
-[📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=activerecordrefind)
+  [📝 Docs](https://test-prof.evilmartians.io/#/any_fixture?id=activerecordrefind)
 
 ### Fixes & Improvements
 
@@ -277,15 +345,15 @@ Force ActiveRecord to use the same connection between threads (to avoid database
 
 - [#44](https://github.com/palkan/test-prof/pull/44) Support older versions of RSpec. ([@palkan][])
 
-Support RSpec 3.1.0+ in general.
+  Support RSpec 3.1.0+ in general.
 
-`let_it_be` supports only RSpec 3.3.0+.
+  `let_it_be` supports only RSpec 3.3.0+.
 
-RSpecDissect `let` tracking supports only RSpec 3.3.0+.
+  RSpecDissect `let` tracking supports only RSpec 3.3.0+.
 
 - [#38](https://github.com/palkan/test-prof/pull/38) Factory Doctor Minitest integration. ([@IDolgirev][])
 
-It is possible now to use Factory Doctor with Minitest
+  It is possible now to use Factory Doctor with Minitest
 
 ## 0.4.0 (2017-10-03)
 
@@ -293,7 +361,7 @@ It is possible now to use Factory Doctor with Minitest
 
 - [#29](https://github.com/palkan/test-prof/pull/29) EventProf Minitest integration. ([@IDolgirev][])
 
-It is possible now to use Event Prof with Minitest
+  It is possible now to use Event Prof with Minitest
 
 - [#30](https://github.com/palkan/test-prof/pull/30) Fabrication support for FactoryProf. ([@Shkrt][])
 
@@ -305,26 +373,25 @@ FactoryProf now also accounts objects created by Fabrication gem (in addition to
 
 - Combine RSpecStamp with FactoryDoctor. ([@palkan][])
 
-Automatically mark _bad_ examples with custom tags.
+  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:
+  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 ...
-```
+  ```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.
+  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.
+  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`).
+  Just like `let`, but persist the result for the whole group (i.e. `let` + `before_all`).
 
 ### Improvements:
 
@@ -344,7 +411,7 @@ Just like `let`, but persist the result for the whole group (i.e. `let` + `befor
 
 - 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).
+  There was a bug when an event occurs before the example has started (e.g. in `before(:context)` hook).
 
 ## 0.2.3 (2017-08-28)
 
@@ -354,8 +421,7 @@ There was a bug when an event occurs before the example has started (e.g. in `be
 
 - 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.
+  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).
 
@@ -363,17 +429,17 @@ Fixes [#10](https://github.com/palkan/test-prof/issues/10).
 
 - Detect `RSpec` by checking the presence of `RSpec::Core`. ([@palkan][])
 
-Fixes [#8](https://github.com/palkan/test-prof/issues/8).
+  Fixes [#8](https://github.com/palkan/test-prof/issues/8).
 
 ## 0.2.0 (2017-08-18)
 
 - Ensure output directory exists. ([@danielwestendorf][])
 
-**Change default output dir** to "tmp/test_prof".
+  **Change default output dir** to "tmp/test_prof".
 
-Rename `#artefact_path` to `#artifact_path` to be more US-like
+  Rename `#artefact_path` to `#artifact_path` to be more US-like
 
-Ensure output dir exists in `#artifact_path` method.
+  Ensure output dir exists in `#artifact_path` method.
 
 - FactoryDoctor: print success message when no bad examples found. ([@palkan][])
 
@@ -396,3 +462,4 @@ Ensure output dir exists in `#artifact_path` method.
 [@szemek]: https://github.com/szemek
 [@mkldon]: https://github.com/mkldon
 [@dmagro]: https://github.com/dmagro
+[@danielwaterworth]: https://github.com/danielwaterworth

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2017 palkan
+Copyright (c) 2017-2019 palkan
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -33,9 +33,11 @@ Of course, we have some [solutions](https://test-prof.evilmartians.io/#/?id=reci
 
 Supported Ruby versions:
 
-- Ruby (MRI) >= 2.3.0 (**NOTE:** for Ruby 2.2 use TestProf < 0.7.0)
+- Ruby (MRI) >= 2.4.0 (**NOTE:** for Ruby 2.2 use TestProf < 0.7.0 or Ruby 2.3 use TestProf ~> 0.7.0)
 
-- JRuby >= 9.1.0.0
+- JRuby >= 9.1.0.0 (**NOTE:** refinements-dependent features might require 9.2.7+)
+
+Supported RSpec version (for RSpec features only): >= 3.5.0 (for older RSpec versions use TestProf < 0.8.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>
@@ -62,7 +64,7 @@ Add `test-prof` gem to your application:
 
 ```ruby
 group :test do
-  gem 'test-prof'
+  gem "test-prof"
 end
 ```
 
@@ -83,3 +85,7 @@ Already using TestProf? [Share your story!](https://github.com/palkan/test-prof/
 The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).
 
 [docs]: https://test-prof.evilmartians.io
+
+## Security Contact
+
+To report a security vulnerability, please use the [Tidelift security contact](https://tidelift.com/security). Tidelift will coordinate the fix and disclosure.

data/lib/minitest/base_reporter.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'minitest'
-require 'test_prof/logging'
+require "minitest"
+require "test_prof/logging"
 
 module Minitest
   module TestProf
@@ -15,17 +15,23 @@ module Minitest
         inject_to_minitest_reporters if defined? Minitest::Reporters
       end
 
-      def start; end
+      def start
+      end
 
-      def prerecord(group, example); end
+      def prerecord(group, example)
+      end
 
-      def before_test(test); end
+      def before_test(test)
+      end
 
-      def record(*); end
+      def record(*)
+      end
 
-      def after_test(test); end
+      def after_test(test)
+      end
 
-      def report; end
+      def report
+      end
 
       private
 
@@ -33,22 +39,22 @@ module Minitest
         # Minitest::Result (>= 5.11) has `source_location` method
         return group.source_location if group.respond_to?(:source_location)
         if group.is_a? Class
-          suite = group.public_instance_methods.select { |mtd| mtd.to_s.match /^test_/ }
+          suite = group.public_instance_methods.select { |mtd| mtd.to_s.match(/^test_/) }
           name = suite.find { |mtd| mtd.to_s == example }
           group.instance_method(name).source_location
         else
-          suite = group.methods.select { |mtd| mtd.to_s.match /^test_/ }
+          suite = group.methods.select { |mtd| mtd.to_s.match(/^test_/) }
           name = suite.find { |mtd| mtd.to_s == group.name }
           group.method(name).source_location
         end
       end
 
       def location_with_line_number(group, example = nil)
-        File.expand_path(location(group, example).join(':')).gsub(Dir.getwd, '.')
+        File.expand_path(location(group, example).join(":")).gsub(Dir.getwd, ".")
       end
 
       def location_without_line_number(group, example = nil)
-        File.expand_path(location(group, example).first).gsub(Dir.getwd, '.')
+        File.expand_path(location(group, example).first).gsub(Dir.getwd, ".")
       end
 
       def inject_to_minitest_reporters

data/lib/minitest/test_prof_plugin.rb

@@ -1,17 +1,17 @@
 # frozen_string_literal: true
 
-require 'test_prof/event_prof/minitest'
-require 'test_prof/factory_doctor/minitest'
+require "test_prof/event_prof/minitest"
+require "test_prof/factory_doctor/minitest"
 
 module Minitest # :nodoc:
   module TestProf # :nodoc:
     def self.configure_options(options = {})
       options.tap do |opts|
-        opts[:event]        = ENV['EVENT_PROF'] if ENV['EVENT_PROF']
-        opts[:rank_by]      = ENV['EVENT_PROF_RANK'].to_sym if ENV['EVENT_PROF_RANK']
-        opts[:top_count]    = ENV['EVENT_PROF_TOP'].to_i if ENV['EVENT_PROF_TOP']
-        opts[:per_example]  = true if ENV['EVENT_PROF_EXAMPLES']
-        opts[:fdoc]         = true if ENV['FDOC']
+        opts[:event]        = ENV["EVENT_PROF"] if ENV["EVENT_PROF"]
+        opts[:rank_by]      = ENV["EVENT_PROF_RANK"].to_sym if ENV["EVENT_PROF_RANK"]
+        opts[:top_count]    = ENV["EVENT_PROF_TOP"].to_i if ENV["EVENT_PROF_TOP"]
+        opts[:per_example]  = true if ENV["EVENT_PROF_EXAMPLES"]
+        opts[:fdoc]         = true if ENV["FDOC"]
       end
     end
   end
@@ -29,7 +29,7 @@ module Minitest # :nodoc:
     opts.on "--event-prof-per-example", TrueClass, "Includes examples metrics to results" do |flag|
       options[:per_example] = flag
     end
-    opts.on "--factory-doctor", TrueClass, 'Enable Factory Doctor for your examples' do |flag|
+    opts.on "--factory-doctor", TrueClass, "Enable Factory Doctor for your examples" do |flag|
       options[:fdoc] = flag
     end
   end

data/lib/test_prof/any_fixture.rb

@@ -27,7 +27,7 @@ module TestProf
 
         ts = TestProf.now
         store[key] = yield
-        stats[key] = { time: TestProf.now - ts, hit: 0 }
+        stats[key] = {time: TestProf.now - ts, hit: 0}
         store[key]
       end
 
@@ -91,7 +91,7 @@ module TestProf
 
         msgs << format(
           "%#{first_column}s  %12s  %9s  %12s",
-          'key', 'build time', 'hit count', 'saved time'
+          "key", "build time", "hit count", "saved time"
         )
 
         msgs << ""
@@ -144,6 +144,6 @@ module TestProf
       end
     end
 
-    self.reporting_enabled = ENV['ANYFIXTURE_REPORT'] == '1'
+    self.reporting_enabled = ENV["ANYFIXTURE_REPORT"] == "1"
   end
 end

data/lib/test_prof/before_all/adapters/active_record.rb

@@ -26,5 +26,16 @@ module TestProf
         end
       end
     end
+
+    configure do |config|
+      # Make sure ActiveRecord uses locked thread.
+      # It only gets locked in `before` / `setup` hook,
+      # thus using thread in `before_all` (e.g. ActiveJob async adapter)
+      # might lead to leaking connections
+      config.before(:begin) do
+        next unless ::ActiveRecord::Base.connection.pool.respond_to?(:lock_thread=)
+        ::ActiveRecord::Base.connection.pool.lock_thread = true
+      end
+    end
   end
 end

data/lib/test_prof/before_all/isolator.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module TestProf
+  module BeforeAll
+    # Disable Isolator within before_all blocks
+    module Isolator
+      def begin_transaction(*)
+        ::Isolator.disable { super }
+      end
+
+      def within_transaction(*)
+        ::Isolator.disable { super }
+      end
+    end
+  end
+end
+
+TestProf::BeforeAll.singleton_class.prepend(TestProf::BeforeAll::Isolator)