Skip to content

Commit 77e8dcc

Browse files
committed
Adjust logging to be more meaningful for test-npm-packages
1 parent 5fd1a1e commit 77e8dcc

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

scripts/test-npm-packages.js

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,10 @@ async function main() {
291291
const failed = results.filter(r => !r.passed && r.reason !== 'Skipped')
292292
const skipped = results.filter(r => r.reason === 'Skipped')
293293

294-
logger.success(`Passed: ${passed.length}/${results.length}`)
294+
// Calculate total tested (excluding skipped).
295+
const totalTested = results.length - skipped.length
296+
297+
logger.success(`Passed: ${passed.length}/${totalTested} (${results.length} total)`)
295298
passed.forEach(r => logger.log(` ${r.package}`))
296299

297300
if (skipped.length > 0) {
@@ -300,10 +303,14 @@ async function main() {
300303
}
301304

302305
if (failed.length > 0) {
303-
logger.fail(`Failed: ${failed.length}/${results.length}`)
306+
logger.fail(`Failed: ${failed.length}/${totalTested} (${results.length} total)`)
304307
failed.forEach(r =>
305308
logger.log(` ${r.package}: ${r.reason?.substring(0, 50)}...`)
306309
)
310+
} else if (totalTested > 0) {
311+
// All non-skipped tests passed!
312+
logger.log('')
313+
logger.success('🎉 All tests passed! (excluding skipped packages)')
307314
}
308315

309316
// eslint-disable-next-line n/no-process-exit

0 commit comments

Comments
 (0)