Merge pull request #192 from connectedbits/dont-swallow-nested-failures

Don't rescue Failures from other contexts

lib/interactor.rb

@@ -113,7 +113,10 @@ def initialize(context = {})
   # Returns nothing.
   def run
     run!
-  rescue Failure
+  rescue Failure => e
+    if context.object_id != e.context.object_id
+      raise
+    end
   end
 
   # Internal: Invoke an Interactor instance along with all defined hooks. The

spec/support/lint.rb

@@ -70,14 +70,22 @@
       instance.run
     end
 
-    it "rescues failure" do
-      expect(instance).to receive(:run!).and_raise(Interactor::Failure)
+    it "rescues failure with the same context" do
+      expect(instance).to receive(:run!).and_raise(Interactor::Failure.new(instance.context))
 
       expect {
         instance.run
       }.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")