Merge pull request #1191 from timcraft/regexp-quote-dollar-signs

Expand Regexp.quote spec to cover dollar sign characters
ruby · Sep 19, 2024 · e7233f3 · e7233f3
2 parents 95f602a + 9fecb19
commit e7233f3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/regexp/shared/quote.rb b/core/regexp/shared/quote.rb
@@ -2,8 +2,8 @@
 
 describe :regexp_quote, shared: true do
   it "escapes any characters with special meaning in a regular expression" do
-    Regexp.send(@method, '\*?{}.+^[]()- ').should == '\\\\\*\?\{\}\.\+\^\[\]\(\)\-\\ '
-    Regexp.send(@method, "\*?{}.+^[]()- ").should == '\\*\\?\\{\\}\\.\\+\\^\\[\\]\\(\\)\\-\\ '
+    Regexp.send(@method, '\*?{}.+^$[]()- ').should == '\\\\\*\?\{\}\.\+\^\$\[\]\(\)\-\\ '
+    Regexp.send(@method, "\*?{}.+^$[]()- ").should == '\\*\\?\\{\\}\\.\\+\\^\\$\\[\\]\\(\\)\\-\\ '
     Regexp.send(@method, '\n\r\f\t').should == '\\\\n\\\\r\\\\f\\\\t'
     Regexp.send(@method, "\n\r\f\t").should == '\\n\\r\\f\\t'
   end