summaryrefslogtreecommitdiff
path: root/spec/ruby/core/module/set_temporary_name_spec.rb
diff options
context:
space:
mode:
authorBenoit Daloze <eregontp@gmail.com>2023-07-06 13:25:21 +0200
committerBenoit Daloze <eregontp@gmail.com>2023-07-06 18:27:13 +0200
commit9ee1877e4ab535253b4f61302b9102d20e11db5a (patch)
tree94679e283f1dcac08f1e6a72cde81b71b49bd055 /spec/ruby/core/module/set_temporary_name_spec.rb
parente76022f41cd74d297189d85d74a18d9e7821fd94 (diff)
Ensure the name given to Module#set_temporary_name is not a valid constant path
Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
Notes
Notes: Merged: https://github.com/ruby/ruby/pull/8035
Diffstat (limited to 'spec/ruby/core/module/set_temporary_name_spec.rb')
-rw-r--r--spec/ruby/core/module/set_temporary_name_spec.rb19
1 files changed, 17 insertions, 2 deletions
diff --git a/spec/ruby/core/module/set_temporary_name_spec.rb b/spec/ruby/core/module/set_temporary_name_spec.rb
index 891a363ea2..93a02a4143 100644
--- a/spec/ruby/core/module/set_temporary_name_spec.rb
+++ b/spec/ruby/core/module/set_temporary_name_spec.rb
@@ -1,5 +1,4 @@
require_relative '../../spec_helper'
-require_relative 'fixtures/module'
ruby_version_is "3.3" do
describe "Module#set_temporary_name" do
@@ -14,6 +13,15 @@ ruby_version_is "3.3" do
m.name.should be_nil
end
+ it "can assign a temporary name which is not a valid constant path" do
+ m = Module.new
+ m.set_temporary_name("a::B")
+ m.name.should == "a::B"
+
+ m.set_temporary_name("Template['foo.rb']")
+ m.name.should == "Template['foo.rb']"
+ end
+
it "can't assign empty string as name" do
m = Module.new
-> { m.set_temporary_name("") }.should raise_error(ArgumentError, "empty class/module name")
@@ -21,7 +29,14 @@ ruby_version_is "3.3" do
it "can't assign a constant name as a temporary name" do
m = Module.new
- -> { m.set_temporary_name("Object") }.should raise_error(ArgumentError, "name must not be valid constant name")
+ -> { m.set_temporary_name("Object") }.should raise_error(ArgumentError, "name must not be valid constant path")
+ end
+
+ it "can't assign a constant path as a temporary name" do
+ m = Module.new
+ -> { m.set_temporary_name("A::B") }.should raise_error(ArgumentError, "name must not be valid constant path")
+ -> { m.set_temporary_name("::A") }.should raise_error(ArgumentError, "name must not be valid constant path")
+ -> { m.set_temporary_name("::A::B") }.should raise_error(ArgumentError, "name must not be valid constant path")
end
it "can't assign name to permanent module" do