Skip to content

Commit

Permalink
update charrange tests for RT #128550
Browse files Browse the repository at this point in the history
Removed todo on base case; also added test for :i case, which had also been
doing it wrong.
  • Loading branch information
TimToady committed Aug 13, 2016
1 parent 922ee0d commit 172f694
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions S15-nfg/regex.t
@@ -1,7 +1,7 @@
use v6;
use Test;

plan 12;
plan 13;

# Tests with 6 codepoints (4 in NFC), which are 2 NFG characters.
# LATIN CAPITAL LETTER D, COMBINING DOT BELOW, COMBINING DOT ABOVE,
Expand All @@ -23,8 +23,8 @@ plan 12;
nok $x ~~ /<[Dd]>/, 'Do not have accidents involving enumerated char class and base char';
nok $x ~~ /<[\x1E0C\x1E0D]>/, 'Do not have accidents involving enumerated char class and NFC';
nok $x ~~ /<[D..d]>/, 'Do not have accidents involving range char class and base char';
#?rakudo todo 'charrange bugginess with synthetics'
nok $x ~~ /<[\x1E0C..\x1E0D]>/, 'Do not have accidents involving range char class and NFC';
nok $x ~~ / <[\x1E0C..\x1E0D]>/, 'Do not have accidents involving range char class and NFC';
nok $x ~~ /:i <[\x1E0C..\x1E0D]>/, 'Do not have accidents involving range char class and NFC under :i';

# Character properties work on the base character.
ok $x ~~ /^<:Lu><:Ll>$/, 'Can use Unicode properties on grapheme';
Expand Down

0 comments on commit 172f694

Please sign in to comment.