Skip to content

Commit b272bee

Browse files
committed
Comment out example in TestM17N#test_regexp_usascii MRI test
* Prism detects an issue with /\u1234\x80/ itself and provides SyntaxError but CRuby analysis regexps in runtime and raises RegexpError. Exclude the test until either Prism stops detecting such errors or CRuby starts detecting them.
1 parent 26af5fb commit b272bee

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/mri/tests/ruby/test_m17n.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1444,7 +1444,7 @@ def test_regexp_usascii
14441444
# assert_regexp_usascii_literal('/\u1234#{%q"\x80"}/', nil, SyntaxError) # edge case failing since Prism translator
14451445
assert_regexp_usascii_literal('/\u1234#{"\x80"}/', nil, SyntaxError)
14461446
assert_regexp_usascii_literal('/\u1234\x80/', nil, SyntaxError)
1447-
assert_regexp_usascii_literal('/\u1234#{ }\x80/', nil, RegexpError)
1447+
# assert_regexp_usascii_literal('/\u1234#{ }\x80/', nil, RegexpError) # Prism detects this at parse time and a SyntaxError, vs CRuby a RegexpError at runtime
14481448
end
14491449

14501450
def test_gbk

0 commit comments

Comments
 (0)