sync Iterator.range edge case with the proposal

Author: zloirock

CHANGELOG.md

@@ -4,7 +4,6 @@
 - [`Iterator.range`](https://github.com/tc39/proposal-iterator.range) updated following the actual spec version
   - Throw a `RangeError` on `NaN` `start` / `end` / `step`
   - Allow `null` as `optionOrStep`
-  - Fixed some other edge cases
 - Improved accuracy of `Math.{ asinh, atanh }` polyfills with big and small values
 - Wrap `Symbol.for` in `Symbol.prototype.description` polyfill for correct handling of empty string descriptions
 - Fixed one more case (`Iterator.prototype.take`) of a V8 ~ Chromium < 126 [bug](https://issues.chromium.org/issues/336839115)

packages/core-js/internals/numeric-range-iterator.js

@@ -52,7 +52,7 @@ var $RangeIterator = createIteratorConstructor(function NumericRangeIterator(sta
   if (step !== step || step === Infinity || step === -Infinity || (step === zero && start !== end)) {
     throw new $RangeError(INCORRECT_RANGE);
   }
-  var hitsEnd = start === end || (end > start !== step > zero);
+    var hitsEnd = end > start !== step > zero;
   setInternalState(this, {
     type: NUMERIC_RANGE_ITERATOR,
     start: start,

tests/unit-global/esnext.iterator.range.js

@@ -35,7 +35,7 @@ QUnit.test('Iterator.range', assert => {
   assert.deepEqual(from(range(0, 0)), []);
   assert.deepEqual(from(range(0, 0, { step: 1, inclusive: true })), []);
   assert.deepEqual(from(range(0, 0, -1)), [], 'start === end with negative step yields nothing');
-  assert.deepEqual(from(range(0, 0, { step: -1, inclusive: true })), [], 'start === end with negative step inclusive yields nothing');
+  assert.deepEqual(from(range(0, 0, { step: -1, inclusive: true })), [0], 'start === end with negative step inclusive yields start');
   assert.deepEqual(from(range(0, -5, 1)), []);
 
   assert.throws(() => range(NaN, 0), RangeError, 'NaN as start');
@@ -104,7 +104,7 @@ QUnit.test('Iterator.range', assert => {
     assert.deepEqual(from(range(BigInt(0), BigInt(0))), []);
     assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(1), inclusive: true })), []);
     assert.deepEqual(from(range(BigInt(0), BigInt(0), BigInt(-1))), [], 'BigInt: start === end with negative step yields nothing');
-    assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(-1), inclusive: true })), [], 'BigInt: start === end with negative step inclusive yields nothing');
+    assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(-1), inclusive: true })), [BigInt(0)], 'BigInt: start === end with negative step inclusive yields start');
     assert.deepEqual(from(range(BigInt(0), BigInt(-5), BigInt(1))), []);
 
     iterator = range(BigInt(1), BigInt(3));

tests/unit-pure/esnext.iterator.range.js

@@ -34,7 +34,7 @@ QUnit.test('Iterator.range', assert => {
   assert.deepEqual(from(range(0, 0)), []);
   assert.deepEqual(from(range(0, 0, { step: 1, inclusive: true })), []);
   assert.deepEqual(from(range(0, 0, -1)), [], 'start === end with negative step yields nothing');
-  assert.deepEqual(from(range(0, 0, { step: -1, inclusive: true })), [], 'start === end with negative step inclusive yields nothing');
+  assert.deepEqual(from(range(0, 0, { step: -1, inclusive: true })), [0], 'start === end with negative step inclusive yields start');
   assert.deepEqual(from(range(0, -5, 1)), []);
 
   assert.throws(() => range(NaN, 0), RangeError, 'NaN as start');
@@ -104,7 +104,7 @@ QUnit.test('Iterator.range', assert => {
     assert.deepEqual(from(range(BigInt(0), BigInt(0))), []);
     assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(1), inclusive: true })), []);
     assert.deepEqual(from(range(BigInt(0), BigInt(0), BigInt(-1))), [], 'BigInt: start === end with negative step yields nothing');
-    assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(-1), inclusive: true })), [], 'BigInt: start === end with negative step inclusive yields nothing');
+    assert.deepEqual(from(range(BigInt(0), BigInt(0), { step: BigInt(-1), inclusive: true })), [BigInt(0)], 'BigInt: start === end with negative step inclusive yields start');
     assert.deepEqual(from(range(BigInt(0), BigInt(-5), BigInt(1))), []);
 
     iterator = range(BigInt(1), BigInt(3));