@trenskow/parallel 0.1.112 → 0.1.114

package/index.js

@@ -1,4 +1,4 @@
-// Created 2022 by Kristian Trenskow
+// Created 2022-2026 by Kristian Trenskow
 
 export default (promises) => {
 
@@ -7,50 +7,41 @@ export default (promises) => {
 
 	if (promises.length === 0) return Promise.resolve([]);
 
-	promises.forEach((promise) => {
-		if (typeof (promise || {}).then !== 'function') throw new Error('Promise is not thenable.');
-	});
-
-	let results = Array(promises.length).fill({});
+	let results = Array(promises.length);
+	let rejection;
+	let completed = 0;
 
 	return new Promise((resolve, reject) => {
 
 		const evaluate = () => {
 
-			if (!results) return;
-
-			if (results.filter(({ status }) => status).length === promises.length) {
-				
-				try {
-					resolve(results.map((result) => {
-						if (result.status === 'rejected') throw result.reason;
-						return result.value;
-					}));
-				} catch (error) {
-					reject(error);
+			if (completed === promises.length) {
+
+				if (typeof rejection !== 'undefined') {
+					return reject(rejection);
 				}
 
-				results = undefined;
+				resolve(results);
 
 			}
 
 		};
 
-		promises.forEach((promise, idx) => {
-			promise
-				.then((value) => {
-					results[idx] = {
-						status: 'resolved',
-						value
-					};
-				}, (reason) => {
-					results[idx] = {
-						status: 'rejected',
-						reason
-					};
-				})
-				.then(evaluate);
-		});
+		promises
+			.forEach((promise, idx) => {
+				Promise.resolve(promise)
+					.then((value) => {
+						results[idx] = value;
+					})
+					.catch((reason) => {
+						results[idx] = undefined;
+						rejection = rejection || reason;
+					})
+					.finally(() => {
+						completed++;
+					})
+					.then(evaluate);
+			});
 
 	});
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@trenskow/parallel",
-	"version": "0.1.112",
+	"version": "0.1.114",
 	"description": "A small library for doing async tasks in parallel.",
 	"main": "index.js",
 	"type": "module",
@@ -26,11 +26,11 @@
 	"devDependencies": {
 		"@eslint/eslintrc": "^3.3.5",
 		"@eslint/js": "^10.0.1",
-		"@trenskow/wait": "^1.3.102",
+		"@trenskow/wait": "^1.3.103",
 		"chai": "^6.2.2",
 		"chai-as-promised": "^8.0.2",
 		"eslint": "^10.4.0",
 		"globals": "^17.6.0",
-		"mocha": "^11.7.5"
+		"mocha": "^11.7.6"
 	}
 }

package/test.js

@@ -10,10 +10,6 @@ import parallel from './index.js';
 
 describe('parallel', () => {
 
-	it ('should throw an error if promises are not provided.', () => {
-		expect(() => parallel(123)).to.throw('Promise is not thenable.');
-	});
-
 	it ('should throw an error if no promises are provided.', () => {
 		expect(() => parallel()).to.throw('No promises was provided.');
 	});
@@ -47,4 +43,8 @@ describe('parallel', () => {
 		})();
 	});
 
+	it ('should eventually come back with [1, 2, 3] when non-promises', async () => {
+		await expect(parallel([0, 1, 2])).to.eventually.eql([0, 1, 2]);
+	});
+
 });