Skip to content

ci(test): run e2e tests only when back- or frontend changes - #13586

Merged
ChristophWurst merged 2 commits into
mainfrom
ci/test-e2e-path-filter
Aug 28, 2026
Merged

ci(test): run e2e tests only when back- or frontend changes#13586
ChristophWurst merged 2 commits into
mainfrom
ci/test-e2e-path-filter

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

Follow-up to #13584. The e2e suite builds the frontend and exercises it against the PHP backend, so gate it on the union of the backend and frontend path filters: it runs whenever either stack changes and is skipped only for PRs that touch neither (docs, l10n, images). The matrix helper job, which exists solely to feed e2e, is gated on the same condition.

Assisted-by: Claude:claude-opus-4-8

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Follow-up to #13584. The e2e suite builds the frontend and exercises it
against the PHP backend, so gate it on the union of the backend and
frontend path filters: it runs whenever either stack changes and is
skipped only for PRs that touch neither (docs, l10n, images). The matrix
helper job, which exists solely to feed e2e, is gated on the same
condition.

Assisted-by: Claude:claude-opus-4-8
Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@ChristophWurst ChristophWurst self-assigned this Aug 27, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 27, 2026
@odzhychko odzhychko mentioned this pull request Aug 28, 2026
1 task
Comment thread .github/workflows/test.yml
Comment thread .github/workflows/test.yml
The changes-job filters missed several tracked source paths, so edits
to them would wrongly skip the tests that depend on them:

- backend: composer/ (autoloader), resources/ (public suffix list),
  .patches/ + patches.json + patches.lock.json (composer-patches),
  img/ (ProxyController and Html read img/blocked-image.png)
- frontend: css/ (imported from src via scss/js), img/ (svgs imported
  in Vue components)

img/ feeds both the PHP image proxy and the Vue build, so it is listed
in both filters. Build output (js/) stays excluded.

Assisted-by: Claude:claude-opus-4-8
Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@ChristophWurst
ChristophWurst merged commit 275268c into main Aug 28, 2026
46 checks passed
@ChristophWurst
ChristophWurst deleted the ci/test-e2e-path-filter branch August 28, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants