Skip to content

Commit f83b444

Browse files
committed
Add todo count to total test count
1 parent 3435179 commit f83b444

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

__tests__/buildJsonResults.test.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -447,6 +447,13 @@ describe('buildJsonResults', () => {
447447
expect(jsonResults.testsuites[1].testsuite[0]._attr.skipped).toBe(1);
448448
});
449449

450+
it('should include number of todo tests in testSuite total count', () => {
451+
const noFailingTestsWithTodoReport = require('../__mocks__/no-failing-tests-with-todo.json');
452+
jsonResults = buildJsonResults(noFailingTestsWithTodoReport, '/', constants.DEFAULT_OPTIONS);
453+
454+
expect(jsonResults.testsuites[1].testsuite[0]._attr.tests).toBe(2);
455+
});
456+
450457
it('should include a skipped tag when outputting todo tests', () => {
451458
const noFailingTestsWithTodoReport = require('../__mocks__/no-failing-tests-with-todo.json');
452459
jsonResults = buildJsonResults(noFailingTestsWithTodoReport, '/', constants.DEFAULT_OPTIONS);

utils/buildJsonResults.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ module.exports = function (report, appDirectory, options, rootDir = null) {
226226
suiteNameVariables[constants.DISPLAY_NAME_VAR] = displayName;
227227

228228
// Add <testsuite /> properties
229-
const suiteNumTests = suite.numFailingTests + suite.numPassingTests + suite.numPendingTests;
229+
const suiteNumTests = suite.numFailingTests + suite.numPassingTests + suite.numPendingTests + (suite.numTodoTests ? suite.numTodoTests : 0);
230230
const suiteExecutionTime = executionTime(suite.perfStats.start, suite.perfStats.end);
231231

232232
const suiteErrors = noResults ? 1 : 0;

0 commit comments

Comments
 (0)