Commit dcb4d23

authored

ci: Fix running of only changed E2E tests (#17551)

Noticed here: https://github.com/getsentry/sentry-javascript/actions/runs/17542916888/job/49818463117 that this was not actually working 🤔 I played around a bit with this locally, and this change made it work for me. Not sure why the `path` part is not working as expected, but we filter for the correct changes anyhow below, so this should be fine IMHO.

1 parent 4745177 commit dcb4d23Copy full SHA for dcb4d23

File tree

1 file changed

-2

lines changed

dev-packages/e2e-tests/lib
- getTestMatrix.ts

1 file changed

-2

lines changed

`‎dev-packages/e2e-tests/lib/getTestMatrix.ts‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ function run(): void {`
`46`	`46`	`},`
`47`	`47`	`});`
`48`	`48`
`49`		`- const { base, head, optional } = values;`
	`49`	`+ const { base, head='HEAD', optional } = values;`
`50`	`50`
`51`	`51`	`const testApplications = globSync('*/package.json', {`
`52`	`52`	cwd: `${__dirname}/../test-applications`,
`@@ -174,7 +174,7 @@ function getAffectedTestApplications(`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`function getChangedTestApps(base: string, head?: string): false \| Set<string> {`
`177`		- const changedFiles = execSync(`git diff --name-only ${base}${head ? `..${head}` : ''} -- dev-packages/e2e-tests/`, {
	`177`	+ const changedFiles = execSync(`git diff --name-only ${base}${head ? `..${head}` : ''} -- .`, {
`178`	`178`	`encoding: 'utf-8',`
`179`	`179`	`})`
`180`	`180`	`.toString()`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit dcb4d23

File tree

1 file changed

1 file changed

`‎dev-packages/e2e-tests/lib/getTestMatrix.ts‎`

0 commit comments