Require stride to be a positive integer

kgryte · kgryte · commit cd79399f0de7 · 2019-08-26T00:31:53.000-07:00
diff --git a/lib/node_modules/@stdlib/iter/strided/lib/main.js b/lib/node_modules/@stdlib/iter/strided/lib/main.js
@@ -21,7 +21,8 @@
 // MODULES //
 
 var setReadOnly = require( '@stdlib/utils/define-nonenumerable-read-only-property' );
-var isNonNegativeInteger = require( '@stdlib/assert/is-nonnegative-integer' );
+var isNonNegativeInteger = require( '@stdlib/assert/is-nonnegative-integer' ).isPrimitive;
+var isPositiveInteger = require( '@stdlib/assert/is-positive-integer' );
 var isBoolean = require( '@stdlib/assert/is-boolean' ).isPrimitive;
 var isFunction = require( '@stdlib/assert/is-function' );
 var isIteratorLike = require( '@stdlib/assert/is-iterator-like' );
@@ -35,11 +36,11 @@ var iteratorSymbol = require( '@stdlib/symbol/iterator' );
 * Returns an iterator which steps by a specified amount.
 *
 * @param {Iterator} iterator - input iterator
-* @param {NonNegativeInteger} stride - stride
+* @param {PositiveInteger} stride - stride
 * @param {NonNegativeInteger} [offset=0] - offset
 * @param {boolean} [eager=false] - boolean indicating whether to eagerly advance the input iterator when provided a non-zero offset.
 * @throws {TypeError} first argument must be an iterator protocol-compliant object
-* @throws {TypeError} second argument must be a nonnegative integer
+* @throws {TypeError} second argument must be a positive integer
 * @throws {TypeError} third argument must be a nonnegative integer
 * @throws {TypeError} fourth argument must be a boolean
 * @returns {Iterator} iterator
@@ -73,8 +74,8 @@ function iterStrided( iterator, stride, offset, eager ) {
 	if ( !isIteratorLike( iterator ) ) {
 		throw new TypeError( 'invalid argument. First argument must be an iterator protocol-compliant object. Value: `' + iterator + '`.' );
 	}
-	if ( !isNonNegativeInteger( stride ) ) {
-		throw new TypeError( 'invalid argument. Second argument must be a nonnegative integer. Value: `' + stride + '`.' );
+	if ( !isPositiveInteger( stride ) ) {
+		throw new TypeError( 'invalid argument. Second argument must be a positive integer. Value: `' + stride + '`.' );
 	}
 	if ( arguments.length === 3 ) {
 		if ( !isNonNegativeInteger( offset ) ) {
diff --git a/lib/node_modules/@stdlib/iter/strided/test/test.js b/lib/node_modules/@stdlib/iter/strided/test/test.js
@@ -154,13 +154,14 @@ tape( 'the function throws an error if provided an iterator argument which is no
 	}
 });
 
-tape( 'the function throws an error if provided a second argument which is not a nonnegative integer', function test( t ) {
+tape( 'the function throws an error if provided a second argument which is not a positive integer', function test( t ) {
 	var values;
 	var i;
 
 	values = [
 		'5',
 		-5,
+		0,
 		3.14,
 		NaN,
 		true,
@@ -184,13 +185,14 @@ tape( 'the function throws an error if provided a second argument which is not a
 	}
 });
 
-tape( 'the function throws an error if provided a second argument which is not a nonnegative integer (three arguments)', function test( t ) {
+tape( 'the function throws an error if provided a second argument which is not a positive integer (three arguments)', function test( t ) {
 	var values;
 	var i;
 
 	values = [
 		'5',
 		-5,
+		0,
 		3.14,
 		NaN,
 		true,
@@ -214,13 +216,14 @@ tape( 'the function throws an error if provided a second argument which is not a
 	}
 });
 
-tape( 'the function throws an error if provided a second argument which is not a nonnegative integer (four arguments)', function test( t ) {
+tape( 'the function throws an error if provided a second argument which is not a positive integer (four arguments)', function test( t ) {
 	var values;
 	var i;
 
 	values = [
 		'5',
 		-5,
+		0,
 		3.14,
 		NaN,
 		true,
