diff options
author | Benoit Daloze <eregontp@gmail.com> | 2023-07-06 13:25:21 +0200 |
---|---|---|
committer | Benoit Daloze <eregontp@gmail.com> | 2023-07-06 18:27:13 +0200 |
commit | 9ee1877e4ab535253b4f61302b9102d20e11db5a (patch) | |
tree | 94679e283f1dcac08f1e6a72cde81b71b49bd055 /spec | |
parent | e76022f41cd74d297189d85d74a18d9e7821fd94 (diff) | |
download | ruby-9ee1877e4ab535253b4f61302b9102d20e11db5a.tar.gz |
Ensure the name given to Module#set_temporary_name is not a valid constant path
Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
Diffstat (limited to 'spec')
-rw-r--r-- | spec/ruby/core/module/set_temporary_name_spec.rb | 19 |
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 |