test script only returns once per file

rather than once per file per test method
Pf2ools · Feb 13, 2024 · b6808e2 · b6808e2
1 parent caf74c6
commit b6808e2
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/scripts/test-data.js b/scripts/test-data.js
@@ -122,18 +122,18 @@ for (const file of files) {
 			});
 		}
 	}
-	opts.test.forEach((method) => {
-		const testResult = methods[method][opts.bundle ? "bundle" : "data"](testJSON);
-		if (testResult.success) {
-			if (!opts.error && !opts.summary) console.log(chalk.dim(passed + file));
-		} else {
-			console.log(failed + file);
-			if (!opts.all) {
-				throw testResult.error;
-			}
-			failCount++;
+	const validationFailure = opts.test
+		.map((method) => methods[method][opts.bundle ? "bundle" : "data"](testJSON))
+		.find((result) => !result.success);
+	if (validationFailure) {
+		console.log(failed + file);
+		if (!opts.all) {
+			throw validationFailure.error;
 		}
-	});
+		failCount++;
+	} else {
+		if (!opts.error && !opts.summary) console.log(chalk.dim(passed + file));
+	}
 }
 
 // Summarise