Skip to content

Extend guides tests to check for main as well as the default version#2777

Merged
gsmet merged 1 commit into
quarkusio:mainfrom
holly-cummins:extend-guides-tests
Jun 25, 2026
Merged

Extend guides tests to check for main as well as the default version#2777
gsmet merged 1 commit into
quarkusio:mainfrom
holly-cummins:extend-guides-tests

Conversation

@holly-cummins

@holly-cummins holly-cummins commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I realised the guides tests check for the latest stable, but not main or some older versions. That would be an easy thing to regress (and we can't even check it in the previews). Sadly, we also can't check anything but main and latest in CI, because the _only_latest_guides_config.yml is being used.

I had a lovely version of this which avoided hardcoding version strings and used the data in #2750, but it fails in CI because the pages aren't built. So I've scaled down my ambitions and only check main.

@holly-cummins
holly-cummins marked this pull request as draft June 23, 2026 12:21
@holly-cummins
holly-cummins force-pushed the extend-guides-tests branch 3 times, most recently from 4e768f6 to e1b24e5 Compare June 23, 2026 13:20
…oke test

The CI build only includes latest and main guide versions, so test those
two rather than dynamically reading releases.yaml for LTS versions that
aren't present in the CI site. Add tests for specific important guide
pages (getting-started, rest, hibernate-orm, cdi, security-overview) to
catch deep link regressions. Remove /extensions/ from the smoke test
since the extensions page is generated separately.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@holly-cummins holly-cummins changed the title Extend guides tests to check for a few non-recent-LTS versions Extend guides tests to check for main as well as the default version Jun 24, 2026
@holly-cummins
holly-cummins marked this pull request as ready for review June 24, 2026 17:18
@quarkus-bot

quarkus-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

Status for workflow Build

This is the status report for running Build on commit 2cb746e.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

🙈 The PR is closed and the preview is expired.

@gsmet
gsmet merged commit e0bc1fa into quarkusio:main Jun 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants