Refactor to ignore iterator return values

Author: kgryte

lib/node_modules/@stdlib/iter/dedupe/lib/main.js

@@ -24,7 +24,6 @@ var setReadOnly = require( '@stdlib/utils/define-nonenumerable-read-only-propert
 var isFunction = require( '@stdlib/assert/is-function' );
 var isPositiveInteger = require( '@stdlib/assert/is-positive-integer' ).isPrimitive;
 var isIteratorLike = require( '@stdlib/assert/is-iterator-like' );
-var hasOwnProp = require( '@stdlib/assert/has-own-property' );
 var iteratorSymbol = require( '@stdlib/symbol/iterator' );
 
 
@@ -94,7 +93,6 @@ function iterDedupe( iterator, limit ) {
 	* @returns {Object} iterator protocol-compliant object
 	*/
 	function next() {
-		var out;
 		var v;
 		if ( FLG ) {
 			return {
@@ -104,12 +102,7 @@ function iterDedupe( iterator, limit ) {
 		v = iterator.next();
 		if ( v.done ) {
 			FLG = true;
-			out = {};
-			if ( hasOwnProp( v, 'value' ) && v.value !== prev ) {
-				out.value = v.value;
-			}
-			out.done = true;
-			return out;
+			return v;
 		}
 		if ( v.value === prev ) {
 			count += 1;
@@ -119,12 +112,7 @@ function iterDedupe( iterator, limit ) {
 					v = iterator.next();
 					if ( v.done ) {
 						FLG = true;
-						out = {};
-						if ( hasOwnProp( v, 'value' ) && v.value !== prev ) {
-							out.value = v.value;
-						}
-						out.done = true;
-						return out;
+						return v;
 					}
 				}
 				prev = v.value;

lib/node_modules/@stdlib/iter/dedupe/test/test.js

@@ -29,37 +29,6 @@ var iterCounter = require( '@stdlib/iter/counter' );
 var iterDedupe = require( './../lib' );
 
 
-// FUNCTIONS //
-
-function createIterator( arr ) {
-	var len;
-	var it;
-	var i;
-
-	len = arr.length;
-	i = -1;
-
-	it = {};
-	it.next = next;
-
-	return it;
-
-	function next() {
-		var out;
-		i += 1;
-		if ( i < len ) {
-			out = {};
-			out.value = arr[ i ];
-			out.done = ( i === len-1 );
-			return out;
-		}
-		return {
-			'done': true
-		};
-	}
-}
-
-
 // TESTS //
 
 tape( 'main export is a function', function test( t ) {
@@ -379,226 +348,6 @@ tape( 'the function supports specifying the number of allowed consecutive duplic
 	t.end();
 });
 
-tape( 'the function returns an iterator protocol-compliant object which removes duplicated values (value+done)', function test( t ) {
-	var expected;
-	var values;
-	var actual;
-	var it;
-	var i;
-
-	values = [ 1, 1, 2, 3, 3, 4, 4, 4 ];
-	expected = [
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 2,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': false
-		},
-		{
-			'done': true
-		}
-	];
-
-	it = iterDedupe( createIterator( values ) );
-	t.equal( it.next.length, 0, 'has zero arity' );
-
-	actual = [];
-	for ( i = 0; i < expected.length; i++ ) {
-		actual.push( it.next() );
-	}
-	t.deepEqual( actual, expected, 'returns expected values' );
-	t.end();
-});
-
-tape( 'the function returns an iterator protocol-compliant object which removes duplicated values (value+done; last unique)', function test( t ) {
-	var expected;
-	var values;
-	var actual;
-	var it;
-	var i;
-
-	values = [ 1, 1, 2, 3, 3, 4 ];
-	expected = [
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 2,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': true
-		}
-	];
-
-	it = iterDedupe( createIterator( values ) );
-	t.equal( it.next.length, 0, 'has zero arity' );
-
-	actual = [];
-	for ( i = 0; i < expected.length; i++ ) {
-		actual.push( it.next() );
-	}
-	t.deepEqual( actual, expected, 'returns expected values' );
-	t.end();
-});
-
-tape( 'the function returns an iterator protocol-compliant object which removes duplicated values (value+done; last two unique)', function test( t ) {
-	var expected;
-	var values;
-	var actual;
-	var it;
-	var i;
-
-	values = [ 1, 1, 2, 3, 3, 4, 5 ];
-	expected = [
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 2,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': false
-		},
-		{
-			'value': 5,
-			'done': true
-		}
-	];
-
-	it = iterDedupe( createIterator( values ) );
-	t.equal( it.next.length, 0, 'has zero arity' );
-
-	actual = [];
-	for ( i = 0; i < expected.length; i++ ) {
-		actual.push( it.next() );
-	}
-	t.deepEqual( actual, expected, 'returns expected values' );
-	t.end();
-});
-
-tape( 'the function supports specifying the number of allowed consecutive duplicated values (value+done)', function test( t ) {
-	var expected;
-	var values;
-	var actual;
-	var it;
-	var i;
-
-	values = [ 1, 1, 2, 3, 3, 3, 3, 4, 4, 4 ];
-	expected = [
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 2,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': false
-		},
-		{
-			'done': true
-		}
-	];
-
-	it = iterDedupe( createIterator( values ), 2 );
-	t.equal( it.next.length, 0, 'has zero arity' );
-
-	actual = [];
-	for ( i = 0; i < expected.length; i++ ) {
-		actual.push( it.next() );
-	}
-	t.deepEqual( actual, expected, 'returns expected values' );
-	t.end();
-});
-
-tape( 'the function supports specifying the number of allowed consecutive duplicated values (value+done; last unique)', function test( t ) {
-	var expected;
-	var values;
-	var actual;
-	var it;
-	var i;
-
-	values = [ 1, 1, 2, 3, 3, 3, 3, 4 ];
-	expected = [
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 1,
-			'done': false
-		},
-		{
-			'value': 2,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 3,
-			'done': false
-		},
-		{
-			'value': 4,
-			'done': true
-		}
-	];
-
-	it = iterDedupe( createIterator( values ), 2 );
-	t.equal( it.next.length, 0, 'has zero arity' );
-
-	actual = [];
-	for ( i = 0; i < expected.length; i++ ) {
-		actual.push( it.next() );
-	}
-	t.deepEqual( actual, expected, 'returns expected values' );
-	t.end();
-});
-
 tape( 'the returned iterator has a `return` method for closing an iterator (no argument)', function test( t ) {
 	var it;
 	var r;