Test that RegExpIdentifierName characters are allowed

Author: josephfrazier

tests/spec/s-xregexp.js

@@ -393,21 +393,20 @@ describe('XRegExp()', function() {
             // Named capture *functionality* is tested by the specs for named backreference syntax,
             // XRegExp.exec, XRegExp.replace, etc.
 
-            it('should allow the characters A-Z, a-z, 0-9, $, and _ to be used in capture names', function() {
+            it('should allow the characters A-Z, a-z, 0-9, $, _, and RegExpIdentifierName characters to be used in capture names', function() {
                 expect(XRegExp('(?<Az>x)').test('x')).toBe(true);
                 expect(XRegExp('(?<_09>x)').test('x')).toBe(true);
                 expect(XRegExp('(?<$>x)').test('x')).toBe(true);
+                expect(function() {XRegExp('(?<naïve>)');}).not.toThrow();
+                expect(function() {XRegExp('(?<Русский>)');}).not.toThrow();
+                expect(function() {XRegExp('(?<日本語>)');}).not.toThrow();
             });
 
             it('should throw an exception if characters other than A-Z, a-z, 0-9, $, and _ are used in capture names', function() {
                 expect(function() {XRegExp('(?<?>)');}).toThrowError(SyntaxError);
                 expect(function() {XRegExp('(?<.>)');}).toThrowError(SyntaxError);
                 expect(function() {XRegExp('(?<<>)');}).toThrowError(SyntaxError);
                 expect(function() {XRegExp('(?<->)');}).toThrowError(SyntaxError);
-                // Native named capture uses different allowed chars that XRegExp should be updated to handle
-                //expect(function() {XRegExp('(?<naïve>)');}).toThrowError(SyntaxError);
-                //expect(function() {XRegExp('(?<Русский>)');}).toThrowError(SyntaxError);
-                //expect(function() {XRegExp('(?<日本語>)');}).toThrowError(SyntaxError);
             });
 
             it('should not allow capture names to start with digits', function() {