1 files changed, 38 insertions, 15 deletions
diff --git a/spec/ruby/core/module/remove_const_spec.rb b/spec/ruby/core/module/remove_const_spec.rb
index 20eb2dc3ed..4e4b9fcb45 100644
--- a/spec/ruby/core/module/remove_const_spec.rb
+++ b/spec/ruby/core/module/remove_const_spec.rb
@@ -7,13 +7,13 @@ describe "Module#remove_const" do
     ConstantSpecs::ModuleM::CS_CONST252.should == :const252
 
     ConstantSpecs::ModuleM.send :remove_const, :CS_CONST252
-    -> { ConstantSpecs::ModuleM::CS_CONST252 }.should raise_error(NameError)
+    -> { ConstantSpecs::ModuleM::CS_CONST252 }.should.raise(NameError)
 
     ConstantSpecs::ModuleM::CS_CONST253 = :const253
     ConstantSpecs::ModuleM::CS_CONST253.should == :const253
 
     ConstantSpecs::ModuleM.send :remove_const, "CS_CONST253"
-    -> { ConstantSpecs::ModuleM::CS_CONST253 }.should raise_error(NameError)
+    -> { ConstantSpecs::ModuleM::CS_CONST253 }.should.raise(NameError)
   end
 
   it "returns the value of the removed constant" do
@@ -23,7 +23,7 @@ describe "Module#remove_const" do
 
   it "raises a NameError and does not call #const_missing if the constant is not defined" do
     ConstantSpecs.should_not_receive(:const_missing)
-    -> { ConstantSpecs.send(:remove_const, :Nonexistent) }.should raise_error(NameError)
+    -> { ConstantSpecs.send(:remove_const, :Nonexistent) }.should.raise(NameError)
   end
 
   it "raises a NameError and does not call #const_missing if the constant is not defined directly in the module" do
@@ -34,28 +34,28 @@ describe "Module#remove_const" do
 
       -> do
         ConstantSpecs::ContainerA.send :remove_const, :CS_CONST255
-      end.should raise_error(NameError)
+      end.should.raise(NameError)
     ensure
       ConstantSpecs::ModuleM.send :remove_const, "CS_CONST255"
     end
   end
 
   it "raises a NameError if the name does not start with a capital letter" do
-    -> { ConstantSpecs.send :remove_const, "name" }.should raise_error(NameError)
+    -> { ConstantSpecs.send :remove_const, "name" }.should.raise(NameError)
   end
 
   it "raises a NameError if the name starts with a non-alphabetic character" do
-    -> { ConstantSpecs.send :remove_const, "__CONSTX__" }.should raise_error(NameError)
-    -> { ConstantSpecs.send :remove_const, "@Name" }.should raise_error(NameError)
-    -> { ConstantSpecs.send :remove_const, "!Name" }.should raise_error(NameError)
-    -> { ConstantSpecs.send :remove_const, "::Name" }.should raise_error(NameError)
+    -> { ConstantSpecs.send :remove_const, "__CONSTX__" }.should.raise(NameError)
+    -> { ConstantSpecs.send :remove_const, "@Name" }.should.raise(NameError)
+    -> { ConstantSpecs.send :remove_const, "!Name" }.should.raise(NameError)
+    -> { ConstantSpecs.send :remove_const, "::Name" }.should.raise(NameError)
   end
 
   it "raises a NameError if the name contains non-alphabetic characters except '_'" do
     ConstantSpecs::ModuleM::CS_CONST256 = :const256
     ConstantSpecs::ModuleM.send :remove_const, "CS_CONST256"
-    -> { ConstantSpecs.send :remove_const, "Name=" }.should raise_error(NameError)
-    -> { ConstantSpecs.send :remove_const, "Name?" }.should raise_error(NameError)
+    -> { ConstantSpecs.send :remove_const, "Name=" }.should.raise(NameError)
+    -> { ConstantSpecs.send :remove_const, "Name?" }.should.raise(NameError)
   end
 
   it "calls #to_str to convert the given name to a String" do
@@ -67,18 +67,41 @@ describe "Module#remove_const" do
 
   it "raises a TypeError if conversion to a String by calling #to_str fails" do
     name = mock('123')
-    -> { ConstantSpecs.send :remove_const, name }.should raise_error(TypeError)
+    -> { ConstantSpecs.send :remove_const, name }.should.raise(TypeError)
 
     name.should_receive(:to_str).and_return(123)
-    -> { ConstantSpecs.send :remove_const, name }.should raise_error(TypeError)
+    -> { ConstantSpecs.send :remove_const, name }.should.raise(TypeError)
   end
 
   it "is a private method" do
-    Module.private_methods.should include(:remove_const)
+    Module.private_methods.should.include?(:remove_const)
   end
 
   it "returns nil when removing autoloaded constant" do
     ConstantSpecs.autoload :AutoloadedConstant, 'a_file'
-    ConstantSpecs.send(:remove_const, :AutoloadedConstant).should be_nil
+    ConstantSpecs.send(:remove_const, :AutoloadedConstant).should == nil
+  end
+
+  it "updates the constant value" do
+    module ConstantSpecs::RemovedConstantUpdate
+      module M
+        FOO = 'm'
+      end
+
+      module A
+        include M
+        FOO = 'a'
+        def self.foo
+          FOO
+        end
+      end
+
+      A.foo.should == 'a'
+
+      A.send(:remove_const,:FOO)
+      A.foo.should == 'm'
+    end
+  ensure
+    ConstantSpecs.send(:remove_const, :RemovedConstantUpdate)
   end
 end