diff --git a/packages/playwright/src/runner/tasks.ts b/packages/playwright/src/runner/tasks.ts index d50bb91229..624b6e1d9f 100644 --- a/packages/playwright/src/runner/tasks.ts +++ b/packages/playwright/src/runner/tasks.ts @@ -31,7 +31,7 @@ import type { Matcher } from '../util'; import { Suite } from '../common/test'; import { buildDependentProjects, buildTeardownToSetupsMap, filterProjects } from './projectUtils'; import { FailureTracker } from './failureTracker'; -import { detectChangedFiles } from './vcs'; +import { detectChangedTests } from './vcs'; const readDirAsync = promisify(fs.readdir); @@ -234,7 +234,7 @@ function createLoadTask(mode: 'out-of-process' | 'in-process', options: { filter if (testRun.config.cliOnlyChanged && options.filterOnlyChanged) { for (const plugin of testRun.config.plugins) await plugin.instance?.populateDependencies?.(); - const changedFiles = await detectChangedFiles(testRun.config.cliOnlyChanged, testRun.config.configDir); + const changedFiles = await detectChangedTests(testRun.config.cliOnlyChanged, testRun.config.configDir); cliOnlyChangedMatcher = file => changedFiles.has(file); } diff --git a/packages/playwright/src/runner/vcs.ts b/packages/playwright/src/runner/vcs.ts index d0dd038c44..160ca60840 100644 --- a/packages/playwright/src/runner/vcs.ts +++ b/packages/playwright/src/runner/vcs.ts @@ -18,7 +18,7 @@ import childProcess from 'child_process'; import { affectedTestFiles } from '../transform/compilationCache'; import path from 'path'; -export async function detectChangedFiles(baseCommit: string, configDir: string): Promise> { +export async function detectChangedTests(baseCommit: string, configDir: string): Promise> { function gitFileList(command: string) { try { return childProcess.execSync( @@ -41,11 +41,5 @@ export async function detectChangedFiles(baseCommit: string, configDir: string): const [gitRoot] = gitFileList('rev-parse --show-toplevel'); const trackedFilesWithChanges = gitFileList(`diff ${baseCommit} --name-only`).map(file => path.join(gitRoot, file)); - const filesWithChanges = [...untrackedFiles, ...trackedFilesWithChanges]; - const affectedFiles = affectedTestFiles(filesWithChanges); - - return new Set([ - ...filesWithChanges, - ...affectedFiles, - ]); + return new Set(affectedTestFiles([...untrackedFiles, ...trackedFilesWithChanges])); } \ No newline at end of file