fix some cases of chars case in escape polyfill

Author: zloirock

CHANGELOG.md

@@ -16,6 +16,7 @@
 - Fixed (updated following the final spec) one more case `Set.prototype.difference` polyfill with updating `this`
 - Fixed `DataView.prototype.setFloat16` polyfill in (0, 1) range
 - Fixed order of arguments validation in `String.prototype.{ startsWith, endsWith }` polyfills
+- Fixed some cases of chars case in `escape` polyfill
 - Fixed named backreferences in `RegExp` NCG polyfill
 - Fixed some cases of `RegExp` NCG polyfill in combination with other types of groups
 - Fixed some cases of `RegExp` NCG polyfill in combination with `dotAll`

packages/core-js/modules/es.escape.js

@@ -33,7 +33,7 @@ $({ global: true }, {
       } else {
         code = charCodeAt(chr, 0);
         if (code < 256) {
-          result += '%' + hex(code, 2);
+          result += '%' + toUpperCase(hex(code, 2));
         } else {
           result += '%u' + toUpperCase(hex(code, 4));
         }

tests/unit-global/es.escape.js

@@ -4,6 +4,9 @@ QUnit.test('escape', assert => {
   assert.arity(escape, 1);
   assert.looksNative(escape);
   assert.same(escape('!q2ф'), '%21q2%u0444');
+  assert.same(escape('\n'), '%0A', 'percent encoding uses uppercase hex digits');
+  assert.same(escape('\u0001'), '%01', 'low code points use uppercase hex');
+  assert.same(escape('\u00FF'), '%FF', 'code < 256 uses uppercase hex');
   assert.same(escape(null), 'null');
   assert.same(escape(undefined), 'undefined');
 

tests/unit-pure/es.escape.js

@@ -4,6 +4,9 @@ QUnit.test('escape', assert => {
   assert.isFunction(escape);
   assert.arity(escape, 1);
   assert.same(escape('!q2ф'), '%21q2%u0444');
+  assert.same(escape('\n'), '%0A', 'percent encoding uses uppercase hex digits');
+  assert.same(escape('\u0001'), '%01', 'low code points use uppercase hex');
+  assert.same(escape('\u00FF'), '%FF', 'code < 256 uses uppercase hex');
   assert.same(escape(null), 'null');
   assert.same(escape(undefined), 'undefined');