Skip to content

Commit e46daec

Browse files
authored
test: adjust total test count to exclude todo tests in summary output (#15943)
The runTestsWithSummary.ts script was counting .todo tests in the total but not in passed, so they showed up as failures. `.todo` tests are now excluded from the total count. Co-authored-by: German Jablonski <GermanJablo@users.noreply.github.com>
1 parent a5d9388 commit e46daec

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/runTestsWithSummary.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ function parseTestResults(output: string): { passed: number; total: number } {
145145
if (typeof data.numPassedTests === 'number' && typeof data.numTotalTests === 'number') {
146146
return {
147147
passed: data.numPassedTests,
148-
total: data.numTotalTests,
148+
total: data.numTotalTests - (data.numTodoTests || 0),
149149
}
150150
}
151151
} catch (e) {

0 commit comments

Comments
 (0)