Skip to content

Commit dcb4d23

Browse files
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 dcb4d23

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ function run(): void {
4646
},
4747
});
4848

49-
const { base, head, optional } = values;
49+
const { base, head = 'HEAD', optional } = values;
5050

5151
const testApplications = globSync('*/package.json', {
5252
cwd: `${__dirname}/../test-applications`,
@@ -174,7 +174,7 @@ function getAffectedTestApplications(
174174
}
175175

176176
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}` : ''} -- .`, {
178178
encoding: 'utf-8',
179179
})
180180
.toString()

0 commit comments

Comments
 (0)