extract IfKlass

Author: markburns

lib/interactify/dsl/each_chain.rb

@@ -26,7 +26,7 @@ def klass
         this = self
 
         Class.new do                                                                    # class SomeNamespace::EachPackage
-          include Interactify                                          #   include Interactify
+          include Interactify                                                           #   include Interactify
 
           expects do                                                                    #   expects do
             required(this.plural_resource_name)                                         #     required(:packages)

lib/interactify/dsl/if_interactor.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "interactify/dsl/unique_klass_name"
+require "interactify/dsl/if_klass"
 
 module Interactify
   module Dsl
@@ -30,33 +31,9 @@ def failure_interactor
       # allows us to dynamically create an interactor chain
       # that iterates over the packages and
       # uses the passed in each_loop_klasses
-      # rubocop:disable all
       def klass
-        this = self
-
-        klass_basis.tap do |k|
-          k.instance_eval do
-            expects do
-              required(this.condition) unless this.condition.is_a?(Proc)
-            end
-
-            define_singleton_method(:source_location) do
-              const_source_location this.evaluating_receiver.to_s                                     #     [file, line]
-            end
-
-            define_method(:run!) do
-              result = this.condition.is_a?(Proc) ? this.condition.call(context) : context.send(this.condition)
-              interactor = result ? this.success_interactor : this.failure_interactor
-              interactor&.respond_to?(:call!) ? interactor.call!(context) : interactor&.call(context)
-            end
-
-            define_method(:inspect) do
-              "<#{this.namespace}::#{this.if_klass_name} #{this.condition} ? #{this.success_interactor} : #{this.failure_interactor}>"
-            end
-          end
-        end
+        IfKlass.new(self).klass
       end
-      # rubocop:enable all
 
       # so we have something to attach subclasses to during building
       # of the outer class, before we finalize the outer If class
@@ -93,7 +70,7 @@ def build_chain(arg, truthiness)
 
         case arg
         when Array
-          name = "If#{condition.to_s.camelize}#{truthiness ? "IsTruthy" : "IsFalsey"}"
+          name = "If#{condition.to_s.camelize}#{truthiness ? 'IsTruthy' : 'IsFalsey'}"
           klass_basis.chain(name, *arg)
         else
           arg

lib/interactify/dsl/if_klass.rb

@@ -0,0 +1,80 @@
+# frozen_string_literal: true
+
+module Interactify
+  module Dsl
+    class IfKlass
+      attr_reader :if_builder
+
+      def initialize(if_builder)
+        @if_builder = if_builder
+      end
+
+      def klass
+        attach_expectations
+        attach_source_location
+        attach_run!
+        attach_inspect
+
+        if_builder.klass_basis
+      end
+
+      def run!(context)
+        result = condition.is_a?(Proc) ? condition.call(context) : context.send(condition)
+
+        interactor = result ? success_interactor : failure_interactor
+        interactor.respond_to?(:call!) ? interactor.call!(context) : interactor&.call(context)
+      end
+
+      private
+
+      def attach_source_location
+        attach do |_klass, this|
+          define_singleton_method(:source_location) do            #   def self.source_location
+            const_source_location this.evaluating_receiver.to_s   #     [file, line]
+          end
+        end
+      end
+
+      def attach_expectations
+        attach do |klass, this|
+          klass.expects do
+            required(this.condition) unless this.condition.is_a?(Proc)
+          end
+        end
+      end
+
+      def attach_run!
+        this = self
+
+        attach_method(:run!) do
+          this.run!(context)
+        end
+      end
+
+      delegate :condition, :success_interactor, :failure_interactor, to: :if_builder
+
+      def attach_inspect
+        this = if_builder
+
+        attach_method(:inspect) do
+          name = "#{this.namespace}::#{this.if_klass_name}"
+          "<#{name} #{this.condition} ? #{this.success_interactor} : #{this.failure_interactor}>"
+        end
+      end
+
+      def attach_method(name, &)
+        attach do |klass, _this|
+          klass.define_method(name, &)
+        end
+      end
+
+      def attach
+        this = if_builder
+
+        this.klass_basis.instance_eval do
+          yield self, this
+        end
+      end
+    end
+  end
+end

spec/lib/interactify/dsl_spec.rb

@@ -24,9 +24,11 @@
         )
       end
 
-      let(:on_success1) { ->(ctx) { 
-        ctx.success1 = true } 
-      }
+      let(:on_success1) do
+        lambda { |ctx|
+          ctx.success1 = true
+        }
+      end
       let(:on_success2) { ->(ctx) { ctx.success2 = true } }
 
       let(:on_failure1) { ->(ctx) { ctx.success1 = false } }
@@ -73,7 +75,6 @@
           expect(result.success2).to eq(false)
         end
       end
-
     end
   end
 end

spec/lib/interactify_spec.rb

@@ -7,7 +7,7 @@
 
   describe ".validate_app" do
     before do
-      wiring = instance_double(Interactify::Wiring, validate_app: 'ok')
+      wiring = instance_double(Interactify::Wiring, validate_app: "ok")
 
       expect(Interactify::Wiring)
         .to receive(:new)