Skip to content

Conversation

@stefanseifert
Copy link
Member

@stefanseifert stefanseifert commented Jan 9, 2026

we want to move the integration tests form the separate github repo
https://github.com/apache/sling-org-apache-sling-models-integration-tests
into the main code repo, to make sure the ITs are executed on each code change.

integration tests are migrated to Sling Starter 14 and feature launcher.

this PR introduces a structure of modules instead of the single "impl" bundle:

reactor
   +-- parent
   +-- impl
   +-- it/test-models
   +-- it/integration-tests

this also adds a new integration test case using jakarta serlvet API.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
76.8% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

stefanseifert added a commit to wcm-io-caravan/caravan-rhyme that referenced this pull request Jan 13, 2026
…more elegant solution using "prepare-agent-integration" goal from the maven plugin, but it needs additional work to get running with sling feature starter plugin - see apache/sling-org-apache-sling-models-impl#68 as example if we want to enable it again
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.

1 participant