test-prof 1.0.0.rc1 → 1.0.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b43e971bbd8c55a38c657e3338dc625d5860bd067e96385f310885024a1c5cc5
-  data.tar.gz: 6d4b767d0ed366be1b27521e986785b6ddfa9f2008eb9640c76baae55084bb34
+  metadata.gz: ae1131d5828c1e2b52eab444c34c89bd82b294a7b3725f62c01ce30d7526f46b
+  data.tar.gz: 5f0bf2610fef5ad188999259aaca14918e3d96d9d5d2fdc9b6f60a961b02a20f
 SHA512:
-  metadata.gz: 1b490801a31def008b6e8c77dff834e67c4c27a96d9d09fbe626dbed0009cb541da48acc7308905f3f8b722e8fc6f6bde39ebce5362e928abb0c2d319f259248
-  data.tar.gz: 825b2ca0cd5f0c4ea235c5df11686a39799a70685a1ae0911facceb383facec815bed04322336ec67867b51e3a0152d150ed9fe7db9f01bb1acea520fd095258
+  metadata.gz: ac9329e707caa20870ec22035678585a9a344968ea98f5f210a6fb0c25558d85162145ff47d731b77f9b17a71bb0c8d1fb23d57b5a46eba2af4244b2c6f269f6
+  data.tar.gz: dc0795ccc8916f93ffb9303d703a81da20d3723401b55d2c1ddfef45f8fb2d5879f9988c8b78cb9efe9ea0279e5adb610055bbf56433ff4a01101b9d7c6344e3

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## master (unrealeased)
 
+## 1.0.0.rc2 (2021-01-06)
+
+- Make Rails fixtures accesible in `before_all`. ([@palkan][])
+
+You can load and access fixtures when explicitly enabling them via `before_all(setup_fixtures: true, &block)`.
+
+- Minitest's `before_all` is not longer experimental. ([@palkan][])
+
+- Add `after_all` to Minitest in addition to `before_all`. ([@palkan][])
+
 ## 1.0.0.rc1 (2020-12-30)
 
 - Remove deprecated `AggregateFailures` cop. ([@palkan][])

data/lib/test_prof/before_all.rb

@@ -32,6 +32,12 @@ module TestProf
         end
       end
 
+      def setup_fixtures(test_object)
+        raise ArgumentError, "Current adapter doesn't support #setup_fixtures" unless adapter.respond_to?(:setup_fixtures)
+
+        adapter.setup_fixtures(test_object)
+      end
+
       def config
         @config ||= Configuration.new
       end
@@ -60,8 +66,11 @@ module TestProf
     class Configuration
       HOOKS = %i[begin rollback].freeze
 
+      attr_accessor :setup_fixtures
+
       def initialize
         @hooks = Hash.new { |h, k| h[k] = HooksChain.new(k) }
+        @setup_fixtures = false
       end
 
       # Add `before` hook for `begin` or

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

@@ -18,6 +18,20 @@ module TestProf
             end
             ::ActiveRecord::Base.connection.rollback_transaction
           end
+
+          def setup_fixtures(test_object)
+            test_object.instance_eval do
+              @@already_loaded_fixtures ||= {}
+              @fixture_cache ||= {}
+
+              if @@already_loaded_fixtures[self.class]
+                @loaded_fixtures = @@already_loaded_fixtures[self.class]
+              else
+                @loaded_fixtures = load_fixtures(config)
+                @@already_loaded_fixtures[self.class] = @loaded_fixtures
+              end
+            end
+          end
         end
       end
     end

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

@@ -8,33 +8,45 @@ module TestProf
     # store instance variables
     module Minitest # :nodoc: all
       class Executor
-        attr_reader :active, :block, :captured_ivars
+        attr_reader :active, :block, :captured_ivars, :teardown_block, :current_test_object,
+          :setup_fixtures
 
         alias active? active
+        alias setup_fixtures? setup_fixtures
 
-        def initialize(&block)
+        def initialize(setup_fixtures: false, &block)
+          @setup_fixtures = setup_fixtures
           @block = block
           @captured_ivars = []
         end
 
+        def teardown(&block)
+          @teardown_block = block
+        end
+
         def activate!(test_object)
+          @current_test_object = test_object
+
           return restore_ivars(test_object) if active?
           @active = true
-          @examples_left = test_object.class.runnable_methods.size
+          BeforeAll.setup_fixtures(test_object) if setup_fixtures?
           BeforeAll.begin_transaction do
             capture!(test_object)
           end
         end
 
-        def try_deactivate!
-          @examples_left -= 1
-          return unless @examples_left.zero?
-
+        def deactivate!
           @active = false
+
+          current_test_object&.instance_eval(&teardown_block) if teardown_block
+
+          @current_test_object = nil
           BeforeAll.rollback_transaction
         end
 
         def capture!(test_object)
+          return unless block
+
           before_ivars = test_object.instance_variables
 
           test_object.instance_eval(&block)
@@ -63,21 +75,29 @@ module TestProf
       module ClassMethods
         attr_accessor :before_all_executor
 
-        def before_all(&block)
-          self.before_all_executor = Executor.new(&block)
+        def before_all(setup_fixtures: BeforeAll.config.setup_fixtures, &block)
+          self.before_all_executor = Executor.new(setup_fixtures: setup_fixtures, &block)
 
           prepend(Module.new do
-            def setup
+            def before_setup
               self.class.before_all_executor.activate!(self)
               super
             end
+          end)
 
-            def teardown
+          singleton_class.prepend(Module.new do
+            def run(*)
               super
-              self.class.before_all_executor.try_deactivate!
+            ensure
+              before_all_executor&.deactivate!
             end
           end)
         end
+
+        def after_all(&block)
+          self.before_all_executor ||= Executor.new
+          before_all_executor.teardown(&block)
+        end
       end
     end
   end

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

@@ -6,14 +6,22 @@ module TestProf
   module BeforeAll
     # Helper to wrap the whole example group into a transaction
     module RSpec
-      def before_all(&block)
+      def before_all(setup_fixtures: BeforeAll.config.setup_fixtures, &block)
         raise ArgumentError, "Block is required!" unless block_given?
 
-        return before(:all, &block) if within_before_all?
+        if within_before_all?
+          before(:all) do
+            @__inspect_output = "before_all hook"
+            instance_eval(&block)
+          end
+          return
+        end
 
         @__before_all_activated__ = true
 
         before(:all) do
+          @__inspect_output = "before_all hook"
+          BeforeAll.setup_fixtures(self) if setup_fixtures
           BeforeAll.begin_transaction do
             instance_eval(&block)
           end

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

@@ -108,8 +108,9 @@ module TestProf
 
       LetItBe.module_for(self).module_eval do
         define_method(identifier) do
-          # Trying to detect the context (couldn't find other way so far)
-          if /\(:context\)/.match?(@__inspect_output)
+          # Trying to detect the context
+          # Based on https://github.com/rspec/rspec-rails/commit/7cb796db064f58da7790a92e73ab906ef50b1f34
+          if @__inspect_output.include?("before(:context)") || @__inspect_output.include?("before_all")
             instance_variable_get(:"#{PREFIX}#{identifier}")
           else
             # Fallback to let definition

data/lib/test_prof/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TestProf
-  VERSION = "1.0.0.rc1"
+  VERSION = "1.0.0.rc2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-prof
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc1
+  version: 1.0.0.rc2
 platform: ruby
 authors:
 - Vladimir Dementyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-30 00:00:00.000000000 Z
+date: 2021-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler