Add spec for GH-1270.

Note that there's an unfixable (in 1.7) edge case described and disabled in the spec. I will apply the unmodified case to master, which handles it ok.
jruby · Mar 2, 2015 · ab2d782 · ab2d782
1 parent e61b737
commit ab2d782
Showing 1 changed file with 54 additions and 0 deletions.
diff --git a/spec/regression/GH-1270_break_propagates_through_escaped_proc_spec.rb b/spec/regression/GH-1270_break_propagates_through_escaped_proc_spec.rb
@@ -0,0 +1,54 @@
+describe "A break passing through an intermediate escaped closure" do
+  it "does not turn into a LocalJumpError" do
+    class Enumerator1270
+      def initialize(&b)
+        @g = Generator1270.new(&b)
+      end
+
+      def each
+        @g.each do |x|
+          yield x
+        end
+      end
+    end
+
+    class Generator1270
+      def initialize(&b)
+        @b = b
+      end
+
+      def each(&b)
+        y = Yielder1270.new(&b)
+        @b.call(y)
+      end
+    end
+
+    class Yielder1270
+      def initialize(&b)
+        @b = b
+      end
+
+      def yield(x)
+        @b.call(x)
+      end
+    end
+
+    # This is hacked a bit because the single-method case is not currently fixable in 1.7:
+    #   Enumerator1270.new {|y| y.yield 1}.each {break 1}
+    # This case shares a frame -- and therefore a jumpTarget -- which causes the escaped
+    # Generator proc to prematurely turn the break into a LocalJumpError. We only have the
+    # one identifier for non-local jumps, so there's no easy way to fix this at the
+    # moment.
+    #
+    # JRuby 9k avoids this by associating jumps with the originating closure rather than
+    # the target method. The single-line version works correctly in JRuby 9k.
+    obj = Object.new
+    def obj.make_enum
+      Enumerator1270.new {|y| y.yield 1}
+    end
+    def obj.main
+      make_enum.each {break 1}
+    end
+    obj.main.should == 1
+  end
+end