Fixes bug #13222

e2e/filter/my-filter.js

@@ -11,9 +11,7 @@ module.exports = function (tests) {
   return new Promise(resolve => {
     setTimeout(() => {
       resolve({
-        filtered: tests
-          .filter(t => t.indexOf('foo') !== -1)
-          .map(test => ({message: 'some message', test})),
+        filtered: tests.filter(t => t.indexOf('foo') !== -1),
       });
     }, 100);
   });

e2e/filter/my-secondary-filter.js

@@ -9,6 +9,6 @@
 
 module.exports = function (tests) {
   return {
-    filtered: tests.filter(t => t.indexOf('foo') !== -1).map(test => ({test})),
+    filtered: tests.filter(t => t.indexOf('foo') !== -1),
   };
 };

e2e/filter/my-setup-filter.js

@@ -13,9 +13,7 @@ const setupData = {
 
 module.exports = function (tests) {
   return {
-    filtered: tests
-      .filter(t => t.indexOf(setupData.filterText) !== -1)
-      .map(test => ({test})),
+    filtered: tests.filter(t => t.indexOf(setupData.filterText) !== -1),
   };
 };
 

packages/jest-core/src/SearchSource.ts

@@ -340,9 +340,7 @@ export default class SearchSource {
         );
       }
 
-      const filteredSet = new Set(
-        filterResult.filtered.map(result => result.test),
-      );
+      const filteredSet = new Set(filterResult.filtered);
 
       return {
         ...searchResult,

packages/jest-core/src/__tests__/SearchSource.test.ts

@@ -11,7 +11,8 @@ import type {Test} from '@jest/test-result';
 import type {Config} from '@jest/types';
 import {normalize} from 'jest-config';
 import Runtime from 'jest-runtime';
-import SearchSource, {SearchResult} from '../SearchSource';
+import SearchSource from '../SearchSource';
+import type {Filter} from '../types';
 
 jest.setTimeout(15000);
 
@@ -103,11 +104,18 @@ describe('SearchSource', () => {
   });
 
   describe('getTestPaths', () => {
-    const getTestPaths = async (initialOptions: Config.InitialOptions) => {
+    const getTestPaths = async (
+      initialOptions: Config.InitialOptions,
+      filter?: Filter,
+    ) => {
       const searchSource = await initSearchSource(initialOptions);
-      const {tests: paths} = await searchSource.getTestPaths({
-        testPathPattern: '',
-      });
+      const {tests: paths} = await searchSource.getTestPaths(
+        {
+          testPathPattern: '',
+        },
+        null,
+        filter,
+      );
       return paths.map(({path: p}) => path.relative(rootDir, p)).sort();
     };
 
@@ -288,6 +296,23 @@ describe('SearchSource', () => {
         path.normalize('__testtests__/test.jsx'),
       ]);
     });
+
+    it('filter tests based on an optional filter method', async () => {
+      const filterFunction = (testPaths: Array<string>) =>
+        Promise.resolve({
+          filtered: testPaths.filter(testPath => testPath.includes('test.jsx')),
+        });
+      const paths = await getTestPaths(
+        {
+          id,
+          rootDir,
+        },
+        filterFunction,
+      );
+
+      expect(paths).toHaveLength(1);
+      expect(paths[0]).toStrictEqual(path.normalize('__testtests__/test.jsx'));
+    });
   });
 
   describe('filterPathsWin32', () => {

packages/jest-core/src/types.ts

@@ -34,11 +34,6 @@ export type TestPathCasesWithPathPattern = TestPathCases & {
   testPathPattern: (path: string) => boolean;
 };
 
-export type FilterResult = {
-  test: string;
-  message: string;
-};
-
 export type Filter = (testPaths: Array<string>) => Promise<{
-  filtered: Array<FilterResult>;
+  filtered: Array<string>;
 }>;