Skip to content

OAK-12022: upgrade Pax Exam and Sling test support - #3267

Merged
mbaedke merged 2 commits into
trunkfrom
OAK-12022-alt
Nov 6, 2026
Merged

mbaedke merged 2 commits into
trunkfrom
OAK-12022-alt

Conversation

@mbaedke

@mbaedke mbaedke commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Enable Pax Exam 4.14.0 in oak-store-composite and upgrade the parent-managed Sling Testing PaxExam dependency to 4.1.2. The upgraded test support uses the TinyBundles API required by Pax Exam, so the composite tests can keep inheriting Sling's TestSupport.

Use SPI Fly 1.3.8 for Java 25 compatibility. Its newer ASM avoids the Unsupported class file major version 69 failure during framework startup. The composite tests override Sling's framework-extension version; oak-it-osgi retains the normal dynamic bundle with ASM 9.10.1 so its existing bundle-state assertions remain intact.

Validation:

  • Actual Java 17, 21 and 25: 55 composite unit tests, both composite integration tests, and all 14 OSGi integration test results pass per JDK.
  • The Elasticsearch unit test is skipped without Docker; the two Elasticsearch-dependent integration scenarios are not exercised without an Elasticsearch connection.
  • RAT and the composite OSGi baseline checks pass. Existing assertions are unchanged.

Tracking: OAK-12022 includes the upgrade and Java verification scope consolidated from OAK-12492, now closed as a duplicate. Supersedes #3247.

Co-Authored-By: OpenAI Codex noreply@openai.com

@rishabhdaim rishabhdaim changed the title OAK-12022: oak-store-composite: Update pax-exam test dependency to 4.14.0 OAK-12022: upgrade Pax Exam and Sling test support Nov 5, 2026
@sonarqubecloud

sonarqubecloud Bot commented Nov 5, 2026

Copy link
Copy Markdown

@mbaedke
mbaedke merged commit 8ad1a2c into trunk Nov 6, 2026
4 checks passed
@mbaedke
mbaedke deleted the OAK-12022-alt branch November 6, 2026 14:19
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