allow null as Iterator.range optionOrStep

Author: zloirock

CHANGELOG.md

@@ -1,7 +1,9 @@
 # Changelog
 ### Unreleased
 - Improved performance of `atob` and `btoa`, [#1503](https://github.com/zloirock/core-js/issues/1503), [#1464](https://github.com/zloirock/core-js/issues/1464)
-- [`Iterator.range`](https://github.com/tc39/proposal-iterator.range) updated following the actual spec version to throw a `RangeError` on `NaN` `start` / `end` / `step`
+- [`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`
 - `Math.atanh` has become slightly more correct with small values
 - Fixed one more case (`Iterator.prototype.take`) of a V8 ~ Chromium < 126 [bug](https://issues.chromium.org/issues/336839115)
 - Forced replacement of `Iterator.{ concat, zip, zipKeyed }` in the pure version for ensuring proper wrapped `Iterator` instances as the result

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

@@ -1,4 +1,5 @@
 'use strict';
+// https://github.com/tc39/proposal-iterator.range
 var InternalStateModule = require('../internals/internal-state');
 var createIteratorConstructor = require('../internals/iterator-create-constructor');
 var createIterResultObject = require('../internals/create-iter-result-object');
@@ -31,7 +32,7 @@ var $RangeIterator = createIteratorConstructor(function NumericRangeIterator(sta
   var ifIncrease = end > start;
   var inclusiveEnd = false;
   var step;
-  if (option === undefined) {
+  if (isNullOrUndefined(option)) {
     step = undefined;
   } else if (isObject(option)) {
     step = option.step;

packages/core-js/modules/esnext.bigint.range.js

@@ -4,7 +4,7 @@ var $ = require('../internals/export');
 var NumericRangeIterator = require('../internals/numeric-range-iterator');
 
 // `BigInt.range` method
-// https://github.com/tc39/proposal-Number.range
+// https://github.com/tc39/proposal-iterator.range
 // TODO: Remove from `core-js@4`
 if (typeof BigInt == 'function') {
   $({ target: 'BigInt', stat: true, forced: true }, {

packages/core-js/modules/esnext.iterator.range.js

@@ -6,7 +6,7 @@ var NumericRangeIterator = require('../internals/numeric-range-iterator');
 var $TypeError = TypeError;
 
 // `Iterator.range` method
-// https://github.com/tc39/proposal-Number.range
+// https://github.com/tc39/proposal-iterator.range
 $({ target: 'Iterator', stat: true, forced: true }, {
   range: function range(start, end, option) {
     if (typeof start == 'number') return new NumericRangeIterator(start, end, option, 'number', 0, 1);

packages/core-js/modules/esnext.number.range.js

@@ -3,7 +3,7 @@ var $ = require('../internals/export');
 var NumericRangeIterator = require('../internals/numeric-range-iterator');
 
 // `Number.range` method
-// https://github.com/tc39/proposal-Number.range
+// https://github.com/tc39/proposal-iterator.range
 // TODO: Remove from `core-js@4`
 $({ target: 'Number', stat: true, forced: true }, {
   range: function range(start, end, option) {

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

@@ -53,6 +53,12 @@ QUnit.test('Iterator.range', assert => {
   assert.same(iterator.step, -1);
   assert.true(iterator.inclusive);
 
+  iterator = range(0, 5, null);
+  assert.same(iterator.start, 0, 'null option: start');
+  assert.same(iterator.end, 5, 'null option: end');
+  assert.same(iterator.step, 1, 'null option: step defaults to 1');
+  assert.false(iterator.inclusive, 'null option: inclusive defaults to false');
+
   iterator = range(-1, -3, { step: 4, inclusive() { /* empty */ } });
   assert.same(iterator.start, -1);
   assert.same(iterator.end, -3);

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

@@ -52,6 +52,12 @@ QUnit.test('Iterator.range', assert => {
   assert.same(iterator.step, -1);
   assert.true(iterator.inclusive);
 
+  iterator = range(0, 5, null);
+  assert.same(iterator.start, 0, 'null option: start');
+  assert.same(iterator.end, 5, 'null option: end');
+  assert.same(iterator.step, 1, 'null option: step defaults to 1');
+  assert.false(iterator.inclusive, 'null option: inclusive defaults to false');
+
   iterator = range(-1, -3, { step: 4, inclusive() { /* empty */ } });
   assert.same(iterator.start, -1);
   assert.same(iterator.end, -3);