some tweaks

Author: zloirock

packages/core-js/modules/esnext.promise.all-keyed.js

@@ -49,8 +49,8 @@ $({ target: 'Promise', stat: true, forced: true }, {
             --remaining;
             if (remaining === 0) {
               var res = create(null);
-              forEach(keys, function (k, idx) {
-                createProperty(res, k, values[idx]);
+              forEach(keys, function (k, i) {
+                createProperty(res, k, values[i]);
               });
               resolve(res);
             }

packages/core-js/modules/esnext.promise.all-settled-keyed.js

@@ -28,10 +28,10 @@ $({ target: 'Promise', stat: true, forced: true }, {
     var resolve = capability.resolve;
     var reject = capability.reject;
     var result = perform(function () {
-      var wrapResolve = function () {
+      var createResultObjectAndResolve = function () {
         var res = create(null);
-        forEach(keys, function (k, idx) {
-          createProperty(res, k, values[idx]);
+        forEach(keys, function (key, index) {
+          createProperty(res, key, values[index]);
         });
         resolve(res);
       };
@@ -53,17 +53,17 @@ $({ target: 'Promise', stat: true, forced: true }, {
             if (alreadyCalled) return;
             alreadyCalled = true;
             values[index] = { status: 'fulfilled', value: value };
-            --remaining || wrapResolve();
+            --remaining || createResultObjectAndResolve();
           }, function (error) {
             if (alreadyCalled) return;
             alreadyCalled = true;
             values[index] = { status: 'rejected', reason: error };
-            --remaining || wrapResolve();
+            --remaining || createResultObjectAndResolve();
           });
           counter++;
         }
       });
-      --remaining || wrapResolve();
+      --remaining || createResultObjectAndResolve();
     });
     if (result.error) reject(result.value);
     return capability.promise;

tests/unit-pure/esnext.promise.all-settled-keyed.js

@@ -38,7 +38,6 @@ QUnit.test('Promise.allSettledKeyed, resolved with rejection', assert => {
 });
 
 QUnit.test('Promise.allSettledKeyed, rejected', assert => {
-  // eslint-disable-next-line promise/valid-params -- required for testing
   return Promise.allSettledKeyed().then(() => {
     assert.avoid();
   }, () => {