Move early return catches from #call to #run!

This should avoid a couple problems:
- In tests you might want to instantiate the interaktor directly with
  `.new`, and then stub methods on it, and then manually call `#run`.
  If you did this then calls to `#fail!` would fail to catch thrown
  early returns, breaking tests.
- Early returning will prevent `#run!` from calling `#called!` on the
  context object, breaking organizers.

Author: taylorthurlow

interaktor.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name = "interaktor"
-  spec.version = "0.1.4"
+  spec.version = "0.1.5"
 
   spec.author = "Taylor Thurlow"
   spec.email = "[email protected]"

lib/interaktor.rb

@@ -141,9 +141,7 @@ def call(context = {})
       apply_default_optional_attributes(context)
       verify_attribute_presence(context)
 
-      catch(:early_return) do
-        new(context).tap(&:run).instance_variable_get(:@context)
-      end
+      new(context).tap(&:run).instance_variable_get(:@context)
     end
 
     # Invoke an Interaktor. This method behaves identically to `#call`, with
@@ -160,9 +158,7 @@ def call!(context = {})
       apply_default_optional_attributes(context)
       verify_attribute_presence(context)
 
-      catch(:early_return) do
-        new(context).tap(&:run!).instance_variable_get(:@context)
-      end
+      new(context).tap(&:run!).instance_variable_get(:@context)
     end
 
     private
@@ -284,7 +280,10 @@ def run
   # @return [void]
   def run!
     with_hooks do
-      call
+      catch(:early_return) do
+        call
+      end
+
       @context.called!(self)
     end
   rescue StandardError

spec/integration_spec.rb

@@ -8,6 +8,8 @@ def build_interaktor(&block)
     interaktor = Class.new.send(:include, Interaktor)
     interaktor.class_eval(&block) if block
     interaktor.class_eval do
+      optional :steps
+
       def unexpected_error!
         raise "foo"
       end
@@ -20,6 +22,8 @@ def build_organizer(options = {}, &block)
     organizer.organize(options[:organize]) if options[:organize]
     organizer.class_eval(&block) if block
     organizer.class_eval do
+      optional :steps
+
       def unexpected_error!
         raise "foo"
       end

spec/support/lint.rb

@@ -1,5 +1,12 @@
 shared_examples "lint" do
-  let(:interaktor) { Class.new.include(described_class) }
+  let(:interaktor) do
+    klass = Class.new.include(described_class)
+    klass.class_eval do
+      optional :foo
+    end
+
+    klass
+  end
 
   describe ".call" do
     let(:context) { instance_double(Interaktor::Context) }