activeinteractor 1.0.5 → 1.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 491bd06fcc2f9baf372f817dab6e40c69a07d5ff15721f5136815bcdeb04656b
-  data.tar.gz: 80e520f97369ab0f9465842eb31f9b0c0e638135cdc73ee27ded9cdc12272cf1
+  metadata.gz: 6326a8924b24caa906991b69c80eebeb735183386bf0914b6efb04881910ff03
+  data.tar.gz: aa2c69acce205ff007a54ec141b7eef0a917d9af68805ce7bd68b91686882139
 SHA512:
-  metadata.gz: 3409d7c2a1c81fcfa6482493569a015356f6239635b4ea539947687c2e2ba44947825b8914ee264df7280e7f612184b35ae67cc10010a6df793087f249aacc81
-  data.tar.gz: dd892f0304bd572a8074c56cad88a231a9ca328b6365ffb96cf87ea7c60c5121f11ed32c3de70a550e3d9ea7fdeb5489667b653041e41c64e90dd661ea35d5e7
+  metadata.gz: 145052b08089c27b93ef87cfe7a9e29a6ffca822d3a190e8b9925117331a262a2179c2275365fe14c6e4d6997f81fbd96a4a581d8412797743c941a34caed8ec
+  data.tar.gz: 8eafb80d3c0fab16fe28a9b5669b237f00ca738e37729869488c4a32ebe36b0afd39b82384a9c66e758e1c8f10159f68a71e38914154361484fedc8c9407a910

data/CHANGELOG.md

@@ -7,11 +7,48 @@ and this project adheres to [Semantic Versioning].
 
 ## [Unreleased]
 
+## [v1.1.3] - 2022-02-16
+
+### Changed
+
+- [\#338] Bump activemodel and activesupport to 6.1.4.4
+
+## [v1.1.2] - 2020-11-10
+
+### Changed
+
+- [\#300](https://github.com/aaronmallen/activeinteractor/pull/300) `ActiveModel` and `ActiveSupport` v6.1 will be the
+  highest supported version for `ActiveInteractor` v1.1.x
+
+### Fixed
+
+- [\#300](https://github.com/aaronmallen/activeinteractor/pull/300) `ActiveInteractor::Context::Error` compatibility
+  with `ActiveModel` v6.1
+
+## [v1.1.1] - 2020-10-21
+
+### Fixed
+
+- [\#267](https://github.com/aaronmallen/activeinteractor/pull/267) Allow default attributes to propagate to
+  sibling/child interactors
+
+## [v1.1.0] - 2020-10-04
+
+### Added
+
+- [\#247](https://github.com/aaronmallen/activeinteractor/issues/247) Support in place callbacks
+
+### Fixed
+
+- [\#242](https://github.com/aaronmallen/activeinteractor/issues/242) Optional attributes are always null
+- [\#243](https://github.com/aaronmallen/activeinteractor/issues/243) Nested Organizers do not rollback parent context
+
 ## [v1.0.5] - 2020-09-15
 
 ### Fixed
 
-- [\#200](https://github.com/aaronmallen/activeinteractor/issues/200) Context attributes assigned in interactor not accessible as element within interactor
+- [\#200](https://github.com/aaronmallen/activeinteractor/issues/200) Context attributes assigned in interactor not
+  accessible as element within interactor
 
 ## [v1.0.4] - 2020-02-11
 
@@ -200,7 +237,12 @@ and this project adheres to [Semantic Versioning].
 
 <!-- versions -->
 
-[Unreleased]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.4...HEAD
+[Unreleased]: https://github.com/aaronmallen/activeinteractor/compare/v1.1.3...HEAD
+[v1.1.3]: https://github.com/aaronmallen/activeinteractor/compare/v1.1.2...v1.1.3
+[v1.1.2]: https://github.com/aaronmallen/activeinteractor/compare/v1.1.1...v1.1.2
+[v1.1.1]: https://github.com/aaronmallen/activeinteractor/compare/v1.1.0...v1.1.1
+[v1.1.0]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.5...v1.1.0
+[v1.0.5]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.4...v1.0.5
 [v1.0.4]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.3...v1.0.4
 [v1.0.3]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.2...v1.0.3
 [v1.0.2]: https://github.com/aaronmallen/activeinteractor/compare/v1.0.1...v1.0.2

data/README.md

@@ -22,7 +22,13 @@ course on how to use ActiveInteractors. Read the [wiki] for detailed usage infor
 * Thread safe performance calls
 * Organize multiple interactors [conditionally][wiki_organizers_conditionally] or in [parallel][wiki_organizers_parallel]
 
-## Getting Started
+## Documentation
+
+Be sure to read the [wiki] for detailed information on how to use ActiveInteractor.
+
+For technical documentation please see the gem's [ruby docs].
+
+## Install
 
 Add this line to your application's Gemfile:
 
@@ -30,54 +36,29 @@ Add this line to your application's Gemfile:
 gem 'activeinteractor', require: 'active_interactor'
 ```
 
-And then execute:
-
-```bash
-bundle
-```
-
 Or install it yourself as:
 
-```bash
+```sh
 gem install activeinteractor
 ```
 
-## Usage
-
-Be sure to read the [wiki] for detailed information on how to use ActiveInteractor.
-
-For technical documentation please see the gem's [ruby docs].
-
-## Development
-
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests.
-You can also run `bin/console` for an interactive prompt that will allow you to experiment.
-
-To install this gem onto your local machine, run `bundle exec rake install`.
-
 ## Contributing
 
 Read our guidelines for [Contributing](CONTRIBUTING.md).
 
 ## Acknowledgements
 
-* Special thanks to [@collectiveidea] for their amazing foundational work on
-  the [interactor][collective_idea_interactors] gem.
-* Special thanks to the [@rails] team for their work on [ActiveModel][active_model_git]
-  and [ActiveSupport][active_support_git] gems.
+ActiveInteractor is made possible by wonderful [humans].
 
 ## License
 
 The gem is available as open source under the terms of the [MIT License][mit_license].
 
-[@collectiveidea]: https://github.com/collectiveidea
-[@rails]: https://github.com/rails
-[active_model_git]: https://github.com/rails/rails/tree/master/activemodel
-[active_support_git]: https://github.com/rails/rails/tree/master/activesupport
 [ActiveModel::Validations]: https://api.rubyonrails.org/classes/ActiveModel/Validations.html
 [business_logic_wikipedia]: https://en.wikipedia.org/wiki/Business_logic
 [collective_idea_interactors]: https://github.com/collectiveidea/interactor
 [command pattern]: https://en.wikipedia.org/wiki/Command_pattern
+[humans]: https://github.com/aaronmallen/activeinteractor/tree/main/HUMANS.md
 [Medium article]: https://medium.com/@aaronmallen/activeinteractor-8557c0dc78db
 [mit_license]: https://opensource.org/licenses/MIT
 [ruby docs]: https://www.rubydoc.info/gems/activeinteractor

data/lib/active_interactor/context/attributes.rb

@@ -69,6 +69,19 @@ module ActiveInteractor
         @table[name.to_sym] || attributes[name.to_sym]
       end
 
+      # Sets value of a Hash attribute in context.attributes
+      #
+      # @since 1.1.0
+      #
+      # @param name [String, Symbol] the key name of the attribute
+      # @param value [*] the value to be given attribute name
+      # @returns [*] the attribute value
+      def []=(name, value)
+        public_send("#{name}=", value)
+
+        super unless @table.nil?
+      end
+
       # Get values defined on the instance of {Base context} whose keys are defined on the {Base context} class'
       # {ClassMethods#attributes .attributes}
       #
@@ -117,8 +130,9 @@ module ActiveInteractor
       def merge!(context)
         merge_errors!(context) if context.respond_to?(:errors)
         copy_flags!(context)
-        context.each_pair do |key, value|
-          public_send("#{key}=", value) unless value.nil?
+
+        merged_context_attributes(context).each_pair do |key, value|
+          self[key] = value unless value.nil?
         end
         self
       end
@@ -129,6 +143,13 @@ module ActiveInteractor
         @_called ||= []
       end
 
+      def merged_context_attributes(context)
+        attrs = {}
+        attrs.merge!(context.to_h) if context.respond_to?(:to_h)
+        attrs.merge!(context.attributes.to_h) if context.respond_to?(:attributes)
+        attrs
+      end
+
       def context_attributes_as_hash(context)
         return context.to_h if context&.respond_to?(:to_h)
         return context.attributes.to_h if context.respond_to?(:attributes)
@@ -149,8 +170,8 @@ module ActiveInteractor
       def merge_attribute_values(context)
         return unless context
 
-        context.each_pair do |key, value|
-          public_send("#{key}=", value)
+        attributes.compact.merge(context).each_pair do |key, value|
+          self[key] = value
         end
       end
     end

data/lib/active_interactor/context/errors.rb

@@ -18,6 +18,16 @@ module ActiveInteractor
 
       private
 
+      def add_errors(errors)
+        errors.each do |error|
+          if self.errors.respond_to?(:import)
+            self.errors.import(error)
+          else
+            self.errors.add(error[0], error[1])
+          end
+        end
+      end
+
       def clear_all_errors
         errors.clear
         failure_errors.clear
@@ -40,7 +50,7 @@ module ActiveInteractor
       def resolve_errors
         all_errors = (failure_errors.uniq + errors.uniq).compact.uniq
         clear_all_errors
-        all_errors.each { |error| errors.add(error[0], error[1]) }
+        add_errors(all_errors)
       end
     end
   end

data/lib/active_interactor/organizer/interactor_interface.rb

@@ -14,6 +14,13 @@ module ActiveInteractor
     #
     #  @return [Hash{Symbol=>Proc, Symbol}] conditional options for the {ActiveInteractor::Base interactor} class
     #
+    # @!attribute [r] callbacks
+    #   Callbacks for the interactor_class
+    #
+    #   @since 1.1.0
+    #
+    #   @return [Hash{Symbol=>*}] the interactor callbacks
+    #
     # @!attribute [r] interactor_class
     #  An {ActiveInteractor::Base interactor} class
     #
@@ -27,11 +34,12 @@ module ActiveInteractor
     #  @return [Hash{Symbol=>*}] {Interactor::Perform::Options} for the {ActiveInteractor::Base interactor}
     #   {Interactor::Perform#perform #perform}
     class InteractorInterface
-      attr_reader :filters, :interactor_class, :perform_options
+      attr_reader :filters, :callbacks, :interactor_class, :perform_options
 
       # Keywords for conditional filters
       # @return [Array<Symbol>]
       CONDITIONAL_FILTERS = %i[if unless].freeze
+      CALLBACKS = %i[before after].freeze
 
       # Initialize a new instance of {InteractorInterface}
       #
@@ -42,7 +50,8 @@ module ActiveInteractor
       def initialize(interactor_class, options = {})
         @interactor_class = interactor_class.to_s.camelize.safe_constantize
         @filters = options.select { |key, _value| CONDITIONAL_FILTERS.include?(key) }
-        @perform_options = options.reject { |key, _value| CONDITIONAL_FILTERS.include?(key) }
+        @callbacks = options.select { |key, _value| CALLBACKS.include?(key) }
+        @perform_options = options.reject { |key, _value| CONDITIONAL_FILTERS.include?(key) || CALLBACKS.include?(key) }
       end
 
       # Call the {#interactor_class} {Interactor::Perform::ClassMethods#perform .perform} or
@@ -57,21 +66,29 @@ module ActiveInteractor
       #  {Context::Status#fail! fails} its {Context::Base context}.
       # @return [Class] an instance of {Context::Base context}
       def perform(target, context, fail_on_error = false, perform_options = {})
-        return if check_conditionals(target, filters[:if]) == false
-        return if check_conditionals(target, filters[:unless]) == true
+        return if check_conditionals(target, :if) == false
+        return if check_conditionals(target, :unless) == true
 
         method = fail_on_error ? :perform! : :perform
         options = self.perform_options.merge(perform_options)
         interactor_class.send(method, context, options)
       end
 
+      def execute_inplace_callback(target, callback)
+        resolve_option(target, callbacks[callback])
+      end
+
       private
 
       def check_conditionals(target, filter)
-        return unless filter
+        resolve_option(target, filters[filter])
+      end
+
+      def resolve_option(target, opt)
+        return unless opt
 
-        return target.send(filter) if filter.is_a?(Symbol)
-        return target.instance_exec(&filter) if filter.is_a?(Proc)
+        return target.send(opt) if opt.is_a?(Symbol)
+        return target.instance_exec(&opt) if opt.is_a?(Proc)
       end
     end
   end

data/lib/active_interactor/organizer/perform.rb

@@ -73,10 +73,19 @@ module ActiveInteractor
         context_fail! if contexts.any?(&:failure?)
       end
 
+      def execute_and_merge_contexts(interface)
+        interface.execute_inplace_callback(self, :before)
+        result = execute_interactor_with_callbacks(interface, true)
+        return if result.nil?
+
+        context.merge!(result)
+        context_fail! if result.failure?
+        interface.execute_inplace_callback(self, :after)
+      end
+
       def perform_in_order
         self.class.organized.each do |interface|
-          result = execute_interactor_with_callbacks(interface, true)
-          context.merge!(result) if result
+          execute_and_merge_contexts(interface)
         end
       rescue ActiveInteractor::Error::ContextFailure => e
         context.merge!(e.context)

data/lib/active_interactor/version.rb

@@ -1,7 +1,45 @@
 # frozen_string_literal: true
 
 module ActiveInteractor
-  # The ActiveInteractor version
-  # @return [String] the ActiveInteractor version
-  VERSION = '1.0.5'
+  # The ActiveInteractor version info
+  #
+  # @author Aaron Allen <hello@aaronmallen.me>
+  # @since unreleased
+  module Version
+    # The ActiveInterctor major version number
+    # @return [Integer] The ActiveInteractor major version number
+    MAJOR = 1
+
+    # The ActiveInterctor minor version number
+    # @return [Integer] The ActiveInteractor minor version number
+    MINOR = 1
+
+    # The ActiveInterctor patch version number
+    # @return [Integer] The ActiveInteractor patch version number
+    PATCH = 3
+
+    # The ActiveInterctor pre-release version
+    # @return [String | nil] The ActiveInteractor pre-release version
+    PRE = nil
+
+    # The ActiveInterctor meta version
+    # @return [String | nil] The ActiveInteractor meta version
+    META = nil
+
+    # The ActiveInterctor rubygems version
+    # @return [String] The ActiveInteractor rubygems version
+    def self.gem_version
+      pre_meta = PRE.nil? ? nil : [PRE, META].compact.join('.').freeze
+      [MAJOR, MINOR, PATCH, pre_meta].compact.join('.').freeze
+    end
+
+    # The ActiveInterctor semver version
+    # @return [String] The ActiveInteractor semver version
+    def self.semver
+      version = [MAJOR, MINOR, PATCH].join('.')
+      version = "#{version}-#{PRE}" if PRE
+      version = "#{version}+#{META}" if META
+      version.freeze
+    end
+  end
 end

data/spec/active_interactor/base_spec.rb

@@ -4,6 +4,7 @@ require 'spec_helper'
 
 RSpec.describe ActiveInteractor::Base do
   let(:interactor_class) { described_class }
+
   include_examples 'a class with interactor methods'
   include_examples 'a class with interactor callback methods'
   include_examples 'a class with interactor context methods'

data/spec/active_interactor/config_spec.rb

@@ -4,6 +4,7 @@ require 'spec_helper'
 
 RSpec.describe ActiveInteractor::Config do
   subject { described_class.new }
+
   it { is_expected.to respond_to :logger }
 
   describe '.defaults' do

data/spec/active_interactor/context/base_spec.rb

@@ -6,7 +6,9 @@ RSpec.describe ActiveInteractor::Context::Base do
   describe '.attributes' do
     context 'when no arguments are passed' do
       subject { context_class.attributes }
+
       let!(:context_class) { build_context }
+
       it { is_expected.to eq [] }
 
       context 'when an attribute :foo was previously defined' do
@@ -22,6 +24,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
     context 'when given arguments :foo and :bar' do
       subject { context_class.attributes(:foo, :bar) }
+
       let!(:context_class) { build_context }
 
       it { is_expected.to eq %i[bar foo] }
@@ -48,6 +51,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
       context 'with attribute equal to "foo"' do
         let(:attribute) { :foo }
+
         before { instance.foo = 'foo' }
 
         it { is_expected.to eq 'foo' }
@@ -70,6 +74,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
       context 'with attribute equal to "foo"' do
         let(:attribute) { :foo }
+
         before { instance.foo = 'foo' }
 
         it { is_expected.to eq 'foo' }
@@ -136,6 +141,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
         it { is_expected.to be_a Hash }
         it { is_expected.to eq(bar: 'bar', foo: 'foo') }
+
         it 'is expected to assign :baz' do
           expect(instance.baz).to eq 'baz'
         end
@@ -162,6 +168,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
   describe '#fail!' do
     subject { instance.fail!(errors) }
+
     let(:instance) { described_class.new }
 
     context 'with errors equal to nil' do
@@ -224,6 +231,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
   describe '#failure?' do
     subject { instance.failure? }
+
     let(:instance) { described_class.new }
 
     it { is_expected.to eq false }
@@ -275,6 +283,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
           it { is_expected.to be_a described_class }
           it { is_expected.to have_attributes(foo: 'foo') }
+
           it 'is expected to preserve @_failed instance variable' do
             expect(subject.instance_variable_get('@_failed')).to eq true
           end
@@ -285,6 +294,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
           it { is_expected.to be_a described_class }
           it { is_expected.to have_attributes(foo: 'foo') }
+
           it 'is expected to preserve @_rolled_back instance variable' do
             expect(subject.instance_variable_get('@_rolled_back')).to eq true
           end
@@ -347,6 +357,7 @@ RSpec.describe ActiveInteractor::Context::Base do
         before { attributes.errors.add(:foo, 'invalid') }
 
         it { is_expected.to be_a described_class }
+
         it 'is expected to have errors on :foo' do
           expect(subject.errors[:foo]).not_to be_nil
           expect(subject.errors[:foo]).to include 'invalid'
@@ -358,6 +369,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
         it { is_expected.to be_a described_class }
         it { is_expected.to have_attributes(foo: 'foo') }
+
         it 'is expected to preserve @_called instance variable' do
           expect(subject.instance_variable_get('@_called')).to eq %w[foo]
         end
@@ -368,6 +380,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
         it { is_expected.to be_a described_class }
         it { is_expected.to have_attributes(foo: 'foo') }
+
         it 'is expected to preserve @_failed instance variable' do
           expect(subject.instance_variable_get('@_failed')).to eq true
         end
@@ -378,6 +391,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
         it { is_expected.to be_a described_class }
         it { is_expected.to have_attributes(foo: 'foo') }
+
         it 'is expected to preserve @_rolled_back instance variable' do
           expect(subject.instance_variable_get('@_rolled_back')).to eq true
         end
@@ -387,6 +401,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
   describe '#rollback!' do
     subject { instance.rollback! }
+
     let(:instance) { described_class.new }
 
     context 'with #called! interactors' do
@@ -414,6 +429,7 @@ RSpec.describe ActiveInteractor::Context::Base do
 
   describe '#success?' do
     subject { instance.success? }
+
     let(:instance) { described_class.new }
 
     it { is_expected.to eq true }

data/spec/active_interactor/error_spec.rb

@@ -14,10 +14,12 @@ RSpec.describe ActiveInteractor::Error::ContextFailure do
   end
 
   context 'when context is an instance of "TestContext"' do
-    before { build_context }
     subject { described_class.new(TestContext.new) }
 
+    before { build_context }
+
     it { is_expected.to have_attributes(message: 'TestContext failed!') }
+
     it 'is expected to have an instance of TestContext' do
       expect(subject.context).to be_a TestContext
     end

data/spec/active_interactor/interactor/worker_spec.rb

@@ -5,6 +5,7 @@ require 'spec_helper'
 RSpec.describe ActiveInteractor::Interactor::Worker do
   context 'with interactor class TestInteractor' do
     before { build_interactor }
+
     let(:interactor) { TestInteractor.new }
 
     RSpec.shared_examples 'an interactor with options' do
@@ -118,6 +119,7 @@ RSpec.describe ActiveInteractor::Interactor::Worker do
         end
 
         it { expect { subject }.to raise_error(ActiveInteractor::Error::ContextFailure) }
+
         it 'is expected to rollback the interactor context' do
           expect_any_instance_of(TestInteractor).to receive(:context_rollback!)
           expect { subject }.to raise_error(ActiveInteractor::Error::ContextFailure)
@@ -135,6 +137,7 @@ RSpec.describe ActiveInteractor::Interactor::Worker do
         end
 
         it { expect { subject }.to raise_error(ActiveInteractor::Error::ContextFailure) }
+
         it 'is expected to rollback the interactor context' do
           expect_any_instance_of(TestInteractor).to receive(:context_rollback!)
           expect { subject }.to raise_error(ActiveInteractor::Error::ContextFailure)

data/spec/active_interactor/organizer/base_spec.rb

@@ -4,6 +4,7 @@ require 'spec_helper'
 
 RSpec.describe ActiveInteractor::Organizer::Base do
   let(:interactor_class) { described_class }
+
   include_examples 'a class with interactor methods'
   include_examples 'a class with interactor callback methods'
   include_examples 'a class with interactor context methods'
@@ -66,6 +67,7 @@ RSpec.describe ActiveInteractor::Organizer::Base do
           subject { organizer.organized }
 
           it { expect(subject.collection).to all(be_a ActiveInteractor::Organizer::InteractorInterface) }
+
           it 'is expected to organize the approriate interactors' do
             expect(subject.collection.first.interactor_class).to eq TestInteractor1
             expect(subject.collection.last.interactor_class).to eq TestInteractor2
@@ -87,6 +89,7 @@ RSpec.describe ActiveInteractor::Organizer::Base do
           subject { organizer.organized }
 
           it { expect(subject.collection).to all(be_a ActiveInteractor::Organizer::InteractorInterface) }
+
           it 'is expected to organize the approriate interactors' do
             expect(subject.collection.first.interactor_class).to eq TestInteractor1
             expect(subject.collection.last.interactor_class).to eq TestInteractor2
@@ -103,11 +106,12 @@ RSpec.describe ActiveInteractor::Organizer::Base do
       end
     end
 
-    it { should have_attributes(parallel: true) }
+    it { is_expected.to have_attributes(parallel: true) }
   end
 
   describe '#perform' do
     subject { interactor_class.perform }
+
     context 'with two existing interactors' do
       let!(:interactor1) { build_interactor('TestInteractor1') }
       let!(:interactor2) { build_interactor('TestInteractor2') }
@@ -118,6 +122,7 @@ RSpec.describe ActiveInteractor::Organizer::Base do
       end
 
       it { is_expected.to be_a interactor_class.context_class }
+
       it 'is expected to receive #perform on both interactors' do
         expect_any_instance_of(interactor1).to receive(:perform)
         expect_any_instance_of(interactor2).to receive(:perform)
@@ -128,11 +133,13 @@ RSpec.describe ActiveInteractor::Organizer::Base do
         subject { interactor_class.perform({}, skip_each_perform_callbacks: true) }
 
         it { is_expected.to be_a interactor_class.context_class }
+
         it 'is expected to receive #perform on both interactors' do
           expect_any_instance_of(interactor1).to receive(:perform)
           expect_any_instance_of(interactor2).to receive(:perform)
           subject
         end
+
         it 'is expected not to receive #run_callbacks with :each_perform' do
           expect_any_instance_of(interactor_class).not_to receive(:run_callbacks)
             .with(:each_perform)
@@ -152,14 +159,17 @@ RSpec.describe ActiveInteractor::Organizer::Base do
         it { expect { subject }.not_to raise_error }
         it { is_expected.to be_failure }
         it { is_expected.to be_a interactor_class.context_class }
+
         it 'is expected to receive #perform on the first interactor' do
           expect_any_instance_of(interactor1).to receive(:perform)
           subject
         end
+
         it 'is expected not to receive #perform on the second interactor' do
           expect_any_instance_of(interactor2).not_to receive(:perform)
           subject
         end
+
         it 'is expected to receive #rollback on the first interactor' do
           expect_any_instance_of(interactor1).to receive(:rollback)
           subject
@@ -178,11 +188,13 @@ RSpec.describe ActiveInteractor::Organizer::Base do
         it { expect { subject }.not_to raise_error }
         it { is_expected.to be_failure }
         it { is_expected.to be_a interactor_class.context_class }
+
         it 'is expected to receive #perform on both interactors' do
           expect_any_instance_of(interactor1).to receive(:perform)
           expect_any_instance_of(interactor2).to receive(:perform)
           subject
         end
+
         it 'is expected to receive #rollback on both interactors' do
           expect_any_instance_of(interactor1).to receive(:rollback)
           expect_any_instance_of(interactor2).to receive(:rollback)
@@ -200,6 +212,7 @@ RSpec.describe ActiveInteractor::Organizer::Base do
         end
 
         it { is_expected.to be_a interactor_class.context_class }
+
         it 'is expected to receive #perform on both interactors' do
           expect_any_instance_of(interactor1).to receive(:perform)
           expect_any_instance_of(interactor2).to receive(:perform)
@@ -218,11 +231,13 @@ RSpec.describe ActiveInteractor::Organizer::Base do
           it { expect { subject }.not_to raise_error }
           it { is_expected.to be_failure }
           it { is_expected.to be_a interactor_class.context_class }
+
           it 'is expected to receive #perform on both interactors' do
             expect_any_instance_of(interactor1).to receive(:perform)
             expect_any_instance_of(interactor2).to receive(:perform)
             subject
           end
+
           it 'is expected to receive #rollback both interactors' do
             expect_any_instance_of(interactor1).to receive(:rollback)
             expect_any_instance_of(interactor2).to receive(:rollback)
@@ -242,11 +257,13 @@ RSpec.describe ActiveInteractor::Organizer::Base do
           it { expect { subject }.not_to raise_error }
           it { is_expected.to be_failure }
           it { is_expected.to be_a interactor_class.context_class }
+
           it 'is expected to receive #perform on both interactors' do
             expect_any_instance_of(interactor1).to receive(:perform)
             expect_any_instance_of(interactor2).to receive(:perform)
             subject
           end
+
           it 'is expected to receive #rollback on both interactors' do
             expect_any_instance_of(interactor1).to receive(:rollback)
             expect_any_instance_of(interactor2).to receive(:rollback)