-
- Downloads
Optimize character class escape tests
Previously, these tests simply compared against an equivalent regular expression pattern by generating a large string containing either all code points up to U+FFFF or U+10FFFF, depending on the `u` flag, and then performing:
str.replace(re, '') === str.replace(otherRe, '');
These two `String#replace` calls can be optimized into a single `RegExp#test` call by following the pattern used in the property escapes tests at https://github.com/mathiasbynens/unicode-property-escapes-tests.
Showing
- test/built-ins/RegExp/CharacterClassEscapes/character-class-digit-class-escape-flags-u.js 8 additions, 8 deletions...lassEscapes/character-class-digit-class-escape-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-digit-class-escape-plus-quantifier-flags-u.js 8 additions, 8 deletions...acter-class-digit-class-escape-plus-quantifier-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-digit-class-escape-plus-quantifier.js 8 additions, 8 deletions...pes/character-class-digit-class-escape-plus-quantifier.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-digit-class-escape.js 8 additions, 8 deletions...aracterClassEscapes/character-class-digit-class-escape.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-digit-class-escape-flags-u.js 9 additions, 8 deletions...Escapes/character-class-non-digit-class-escape-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-digit-class-escape-plus-quantifier-flags-u.js 9 additions, 8 deletions...r-class-non-digit-class-escape-plus-quantifier-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-digit-class-escape-plus-quantifier.js 9 additions, 8 deletions...character-class-non-digit-class-escape-plus-quantifier.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-digit-class-escape.js 9 additions, 8 deletions...terClassEscapes/character-class-non-digit-class-escape.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-whitespace-class-escape-flags-u.js 20 additions, 8 deletions...es/character-class-non-whitespace-class-escape-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-whitespace-class-escape-plus-quantifier-flags-u.js 20 additions, 8 deletions...ss-non-whitespace-class-escape-plus-quantifier-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-whitespace-class-escape-plus-quantifier.js 20 additions, 8 deletions...cter-class-non-whitespace-class-escape-plus-quantifier.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-whitespace-class-escape.js 20 additions, 8 deletions...assEscapes/character-class-non-whitespace-class-escape.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-word-class-escape-flags-u.js 13 additions, 8 deletions...sEscapes/character-class-non-word-class-escape-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-word-class-escape-plus-quantifier-flags-u.js 13 additions, 8 deletions...er-class-non-word-class-escape-plus-quantifier-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-word-class-escape-plus-quantifier.js 13 additions, 8 deletions.../character-class-non-word-class-escape-plus-quantifier.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-non-word-class-escape.js 13 additions, 8 deletions...cterClassEscapes/character-class-non-word-class-escape.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-whitespace-class-escape-flags-u.js 18 additions, 8 deletions...scapes/character-class-whitespace-class-escape-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-whitespace-class-escape-plus-quantifier-flags-u.js 18 additions, 8 deletions...-class-whitespace-class-escape-plus-quantifier-flags-u.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-whitespace-class-escape-plus-quantifier.js 18 additions, 8 deletions...haracter-class-whitespace-class-escape-plus-quantifier.js
- test/built-ins/RegExp/CharacterClassEscapes/character-class-whitespace-class-escape.js 18 additions, 8 deletions...erClassEscapes/character-class-whitespace-class-escape.js
Loading
Please register or sign in to comment