RFC: Nested failures fail, not raise

lib/interactor.rb

@@ -115,8 +115,9 @@ def run
     run!
   rescue Failure => e
     if context.object_id != e.context.object_id
-      raise
+      @context = e.context
     end
+    @context.failure = true
   end
 
   # Internal: Invoke an Interactor instance along with all defined hooks. The

spec/integration_spec.rb

@@ -929,6 +929,30 @@ def rollback
     end
   end
 
+  context "when a nested call! fails in an interactor" do
+    let(:interactor3) {
+      build_interactor do
+        class NestedInteractor
+          include Interactor
+
+          def call
+            context.fail!(something: :went_wrong)
+          end
+        end
+
+        def call
+          NestedInteractor.call!
+        end
+      end
+    }
+
+    it 'sets the failure status on enclosing interactor' do
+      result = interactor3.call
+
+      expect(result.failure?).to eq(true)
+    end
+  end
+
   context "when a nested call errors" do
     let(:interactor3) {
       build_interactor do

spec/support/lint.rb

@@ -78,14 +78,6 @@
       }.not_to raise_error
     end
 
-    it "raises other failures" do
-      expect(instance).to receive(:run!).and_raise(Interactor::Failure.new(Interactor::Context.new))
-
-      expect {
-        instance.run
-      }.to raise_error(Interactor::Failure)
-    end
-
     it "raises other errors" do
       expect(instance).to receive(:run!).and_raise("foo")