fix(jest-circus): Send test case results for todo type of tests (#13915)

jestjs · Feb 23, 2023 · 9d4efd1 · 9d4efd1
1 parent 407da29
commit 9d4efd1
Show file tree

Hide file tree

Showing 5 changed files with 36 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,8 @@
 
 ### Fixes
 
+- `[jest-circus]` Send test case results for `todo` tests ([#13915](https://github.com/facebook/jest/pull/13915))
+
 ### Chore & Maintenance
 
 ### Performance

diff --git a/e2e/__tests__/__snapshots__/customReportersOnCircus.test.ts.snap b/e2e/__tests__/__snapshots__/customReportersOnCircus.test.ts.snap
@@ -9,3 +9,8 @@ exports[`Custom Reporters Integration on jest-circus valid passing assertion cou
 "onTestCaseResult: adds ok, status: passed, numExpectations: 3
 onTestFileResult testCaseResult 0: adds ok, status: passed, numExpectations: 3"
 `;
+
+exports[`Custom Reporters Integration on jest-circus push test case results for todo tests 1`] = `
+"onTestCaseResult: sample, status: todo, numExpectations: 0
+onTestFileResult testCaseResult 0: sample, status: todo, numExpectations: 0"
+`;
diff --git a/e2e/__tests__/customReportersOnCircus.test.ts b/e2e/__tests__/customReportersOnCircus.test.ts
@@ -39,4 +39,19 @@ describe('Custom Reporters Integration on jest-circus', () => {
 
     expect(stdout).toMatchSnapshot();
   });
+
+  test('push test case results for todo tests', () => {
+    const {stdout} = runJest('custom-reporters', [
+      '--config',
+      JSON.stringify({
+        reporters: [
+          'default',
+          '<rootDir>/reporters/AssertionCountsReporter.js',
+        ],
+      }),
+      'todo.test.js',
+    ]);
+
+    expect(stdout).toMatchSnapshot();
+  });
 });
diff --git a/e2e/custom-reporters/__tests__/todo.test.js b/e2e/custom-reporters/__tests__/todo.test.js
@@ -0,0 +1,13 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+'use strict';
+
+describe('Custom Reporters', () => {
+  it.todo('sample');
+});
diff --git a/packages/jest-circus/src/testCaseReportHandler.ts b/packages/jest-circus/src/testCaseReportHandler.ts
@@ -13,6 +13,7 @@ const testCaseReportHandler =
   (testPath: string, sendMessageToJest: TestFileEvent) =>
   (event: Circus.Event): void => {
     switch (event.name) {
+      case 'test_todo':
       case 'test_done': {
         const testResult = makeSingleTestResult(event.test);
         const testCaseResult = parseSingleTestResult(testResult);