Update to ruby/spec@7de852d

4 files changed, 149 insertions, 157 deletions

diff --git a/spec/ruby/library/fiber/alive_spec.rb b/spec/ruby/library/fiber/alive_spec.rb
index 72663dd173..0c20a1c6b8 100644
--- a/spec/ruby/library/fiber/alive_spec.rb
+++ b/spec/ruby/library/fiber/alive_spec.rb
@@ -1,48 +1,46 @@
 require_relative '../../spec_helper'
 
-with_feature :fiber_library do
-  require 'fiber'
+require 'fiber'
 
-  describe "Fiber#alive?" do
-    it "returns true for a Fiber that hasn't had #resume called" do
-      fiber = Fiber.new { true }
-      fiber.alive?.should be_true
-    end
+describe "Fiber#alive?" do
+  it "returns true for a Fiber that hasn't had #resume called" do
+    fiber = Fiber.new { true }
+    fiber.alive?.should be_true
+  end
 
-    # FIXME: Better description?
-    it "returns true for a Fiber that's yielded to the caller" do
-      fiber = Fiber.new { Fiber.yield }
-      fiber.resume
-      fiber.alive?.should be_true
-    end
+  # FIXME: Better description?
+  it "returns true for a Fiber that's yielded to the caller" do
+    fiber = Fiber.new { Fiber.yield }
+    fiber.resume
+    fiber.alive?.should be_true
+  end
 
-    it "returns true when called from its Fiber" do
-      fiber = Fiber.new { fiber.alive?.should be_true }
-      fiber.resume
-    end
+  it "returns true when called from its Fiber" do
+    fiber = Fiber.new { fiber.alive?.should be_true }
+    fiber.resume
+  end
 
-    it "doesn't invoke the block associated with the Fiber" do
-      offthehook = mock('do not call')
-      offthehook.should_not_receive(:ring)
-      fiber = Fiber.new { offthehook.ring }
-      fiber.alive?
-    end
+  it "doesn't invoke the block associated with the Fiber" do
+    offthehook = mock('do not call')
+    offthehook.should_not_receive(:ring)
+    fiber = Fiber.new { offthehook.ring }
+    fiber.alive?
+  end
 
-    it "returns false for a Fiber that's dead" do
-      fiber = Fiber.new { true }
-      fiber.resume
-      lambda { fiber.resume }.should raise_error(FiberError)
-      fiber.alive?.should be_false
-    end
+  it "returns false for a Fiber that's dead" do
+    fiber = Fiber.new { true }
+    fiber.resume
+    lambda { fiber.resume }.should raise_error(FiberError)
+    fiber.alive?.should be_false
+  end
 
-    it "always returns false for a dead Fiber" do
-      fiber = Fiber.new { true }
-      fiber.resume
-      lambda { fiber.resume }.should raise_error(FiberError)
-      fiber.alive?.should be_false
-      lambda { fiber.resume }.should raise_error(FiberError)
-      fiber.alive?.should be_false
-      fiber.alive?.should be_false
-    end
+  it "always returns false for a dead Fiber" do
+    fiber = Fiber.new { true }
+    fiber.resume
+    lambda { fiber.resume }.should raise_error(FiberError)
+    fiber.alive?.should be_false
+    lambda { fiber.resume }.should raise_error(FiberError)
+    fiber.alive?.should be_false
+    fiber.alive?.should be_false
   end
 end
diff --git a/spec/ruby/library/fiber/current_spec.rb b/spec/ruby/library/fiber/current_spec.rb
index 8b7fa7c4ca..52dff3dea1 100644
--- a/spec/ruby/library/fiber/current_spec.rb
+++ b/spec/ruby/library/fiber/current_spec.rb
@@ -1,53 +1,51 @@
 require_relative '../../spec_helper'
 
-with_feature :fiber_library do
-  require 'fiber'
+require 'fiber'
 
-  describe "Fiber.current" do
-    it "returns the root Fiber when called outside of a Fiber" do
-      root = Fiber.current
-      root.should be_an_instance_of(Fiber)
-      # We can always transfer to the root Fiber; it will never die
-      5.times do
-        root.transfer.should be_nil
-        root.alive?.should be_true
-      end
+describe "Fiber.current" do
+  it "returns the root Fiber when called outside of a Fiber" do
+    root = Fiber.current
+    root.should be_an_instance_of(Fiber)
+    # We can always transfer to the root Fiber; it will never die
+    5.times do
+      root.transfer.should be_nil
+      root.alive?.should be_true
     end
+  end
 
-    it "returns the current Fiber when called from a Fiber" do
-      fiber = Fiber.new do
-        this = Fiber.current
-        this.should be_an_instance_of(Fiber)
-        this.should == fiber
-        this.alive?.should be_true
-      end
-      fiber.resume
+  it "returns the current Fiber when called from a Fiber" do
+    fiber = Fiber.new do
+      this = Fiber.current
+      this.should be_an_instance_of(Fiber)
+      this.should == fiber
+      this.alive?.should be_true
     end
+    fiber.resume
+  end
 
-    it "returns the current Fiber when called from a Fiber that transferred to another" do
-      states = []
-      fiber = Fiber.new do
-        states << :fiber
-        this = Fiber.current
-        this.should be_an_instance_of(Fiber)
-        this.should == fiber
-        this.alive?.should be_true
-      end
-
-      fiber2 = Fiber.new do
-        states << :fiber2
-        fiber.transfer
-        flunk
-      end
+  it "returns the current Fiber when called from a Fiber that transferred to another" do
+    states = []
+    fiber = Fiber.new do
+      states << :fiber
+      this = Fiber.current
+      this.should be_an_instance_of(Fiber)
+      this.should == fiber
+      this.alive?.should be_true
+    end
 
-      fiber3 = Fiber.new do
-        states << :fiber3
-        fiber2.transfer
-        flunk
-      end
+    fiber2 = Fiber.new do
+      states << :fiber2
+      fiber.transfer
+      flunk
+    end
 
-      fiber3.resume
-      states.should == [:fiber3, :fiber2, :fiber]
+    fiber3 = Fiber.new do
+      states << :fiber3
+      fiber2.transfer
+      flunk
     end
+
+    fiber3.resume
+    states.should == [:fiber3, :fiber2, :fiber]
   end
 end
diff --git a/spec/ruby/library/fiber/resume_spec.rb b/spec/ruby/library/fiber/resume_spec.rb
index 9789cf5b7c..3d86aed94e 100644
--- a/spec/ruby/library/fiber/resume_spec.rb
+++ b/spec/ruby/library/fiber/resume_spec.rb
@@ -1,14 +1,12 @@
 require_relative '../../spec_helper'
 
-with_feature :fiber_library do
-  require 'fiber'
+require 'fiber'
 
-  describe "Fiber#resume" do
-    it "raises a FiberError if the Fiber has transferred control to another Fiber" do
-      fiber1 = Fiber.new { true }
-      fiber2 = Fiber.new { fiber1.transfer; Fiber.yield }
-      fiber2.resume
-      lambda { fiber2.resume }.should raise_error(FiberError)
-    end
+describe "Fiber#resume" do
+  it "raises a FiberError if the Fiber has transferred control to another Fiber" do
+    fiber1 = Fiber.new { true }
+    fiber2 = Fiber.new { fiber1.transfer; Fiber.yield }
+    fiber2.resume
+    lambda { fiber2.resume }.should raise_error(FiberError)
   end
 end
diff --git a/spec/ruby/library/fiber/transfer_spec.rb b/spec/ruby/library/fiber/transfer_spec.rb
index 22bb568840..1063bebeda 100644
--- a/spec/ruby/library/fiber/transfer_spec.rb
+++ b/spec/ruby/library/fiber/transfer_spec.rb
@@ -1,88 +1,86 @@
 require_relative '../../spec_helper'
 require_relative '../../shared/fiber/resume'
 
-with_feature :fiber_library do
-  require 'fiber'
+require 'fiber'
 
-  describe "Fiber#transfer" do
-    it_behaves_like :fiber_resume, :transfer
+describe "Fiber#transfer" do
+  it_behaves_like :fiber_resume, :transfer
+end
+
+describe "Fiber#transfer" do
+  it "transfers control from one Fiber to another when called from a Fiber" do
+    fiber1 = Fiber.new { :fiber1 }
+    fiber2 = Fiber.new { fiber1.transfer; :fiber2 }
+    fiber2.resume.should == :fiber1
   end
 
-  describe "Fiber#transfer" do
-    it "transfers control from one Fiber to another when called from a Fiber" do
-      fiber1 = Fiber.new { :fiber1 }
-      fiber2 = Fiber.new { fiber1.transfer; :fiber2 }
-      fiber2.resume.should == :fiber1
-    end
+  it "returns to the root Fiber when finished" do
+    f1 = Fiber.new { :fiber_1 }
+    f2 = Fiber.new { f1.transfer; :fiber_2 }
 
-    it "returns to the root Fiber when finished" do
-      f1 = Fiber.new { :fiber_1 }
-      f2 = Fiber.new { f1.transfer; :fiber_2 }
+    f2.transfer.should == :fiber_1
+    f2.transfer.should == :fiber_2
+  end
 
-      f2.transfer.should == :fiber_1
-      f2.transfer.should == :fiber_2
-    end
+  it "can be invoked from the same Fiber it transfers control to" do
+    states = []
+    fiber = Fiber.new { states << :start; fiber.transfer; states << :end }
+    fiber.transfer
+    states.should == [:start, :end]
 
-    it "can be invoked from the same Fiber it transfers control to" do
-      states = []
-      fiber = Fiber.new { states << :start; fiber.transfer; states << :end }
-      fiber.transfer
-      states.should == [:start, :end]
+    states = []
+    fiber = Fiber.new { states << :start; fiber.transfer; states << :end }
+    fiber.resume
+    states.should == [:start, :end]
+  end
 
-      states = []
-      fiber = Fiber.new { states << :start; fiber.transfer; states << :end }
-      fiber.resume
-      states.should == [:start, :end]
-    end
+  it "can transfer control to a Fiber that has transferred to another Fiber" do
+    states = []
+    fiber1 = Fiber.new { states << :fiber1 }
+    fiber2 = Fiber.new { states << :fiber2_start; fiber1.transfer; states << :fiber2_end}
+    fiber2.resume.should == [:fiber2_start, :fiber1]
+    fiber2.transfer.should == [:fiber2_start, :fiber1, :fiber2_end]
+  end
 
-    it "can transfer control to a Fiber that has transferred to another Fiber" do
-      states = []
-      fiber1 = Fiber.new { states << :fiber1 }
-      fiber2 = Fiber.new { states << :fiber2_start; fiber1.transfer; states << :fiber2_end}
-      fiber2.resume.should == [:fiber2_start, :fiber1]
-      fiber2.transfer.should == [:fiber2_start, :fiber1, :fiber2_end]
-    end
+  it "raises a FiberError when transferring to a Fiber which resumes itself" do
+    fiber = Fiber.new { fiber.resume }
+    lambda { fiber.transfer }.should raise_error(FiberError)
+  end
 
-    it "raises a FiberError when transferring to a Fiber which resumes itself" do
-      fiber = Fiber.new { fiber.resume }
-      lambda { fiber.transfer }.should raise_error(FiberError)
+  it "works if Fibers in different Threads each transfer to a Fiber in the same Thread" do
+    # This catches a bug where Fibers are running on a thread-pool
+    # and Fibers from a different Ruby Thread reuse the same native thread.
+    # Caching the Ruby Thread based on the native thread is not correct in that case,
+    # and the check for "fiber called across threads" in Fiber#transfer
+    # might be incorrect based on that.
+    2.times do
+      Thread.new do
+        io_fiber = Fiber.new do |calling_fiber|
+          calling_fiber.transfer
+        end
+        io_fiber.transfer(Fiber.current)
+        value = Object.new
+        io_fiber.transfer(value).should equal value
+      end.join
     end
+  end
 
-    it "works if Fibers in different Threads each transfer to a Fiber in the same Thread" do
-      # This catches a bug where Fibers are running on a thread-pool
-      # and Fibers from a different Ruby Thread reuse the same native thread.
-      # Caching the Ruby Thread based on the native thread is not correct in that case,
-      # and the check for "fiber called across threads" in Fiber#transfer
-      # might be incorrect based on that.
-      2.times do
-        Thread.new do
-          io_fiber = Fiber.new do |calling_fiber|
-            calling_fiber.transfer
-          end
-          io_fiber.transfer(Fiber.current)
-          value = Object.new
-          io_fiber.transfer(value).should equal value
-        end.join
+  it "transfers control between a non-main thread's root fiber to a child fiber and back again" do
+    states = []
+    thread = Thread.new do
+      f1 = Fiber.new do |f0|
+        states << 0
+        value2 = f0.transfer(1)
+        states << value2
+        3
       end
-    end
-
-    it "transfers control between a non-main thread's root fiber to a child fiber and back again" do
-      states = []
-      thread = Thread.new do
-        f1 = Fiber.new do |f0|
-          states << 0
-          value2 = f0.transfer(1)
-          states << value2
-          3
-        end
 
-        value1 = f1.transfer(Fiber.current)
-        states << value1
-        value3 = f1.transfer(2)
-        states << value3
-      end
-      thread.join
-      states.should == [0, 1, 2, 3]
+      value1 = f1.transfer(Fiber.current)
+      states << value1
+      value3 = f1.transfer(2)
+      states << value3
     end
+    thread.join
+    states.should == [0, 1, 2, 3]
   end
 end