fix(tests): the Camel ITs follow their move to the api package - #431
Merged
Merged
Conversation
Dirigible 14.27.0 converted six Camel route ITs from Selenide/Chrome tests to HTTP-level PredefinedProjectITs and moved them out of integration.tests.ui.tests.camel into integration.tests.api.camel. DirigibleCommonTestSuiteIT names them by reference, so the release build stopped at testCompile with six "cannot find symbol" errors. Import the api.camel package alongside ui.tests.camel, which still holds the two starter-template ITs that did not move. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The 14.27.0 release run (31936061338) failed in both
release-artifactsandrelease-docker-imagesat the same place —codbex-atlas-integration-testsdid not compile:Dirigible PR eclipse-dirigible#6735 ("the camel route ITs publish over HTTP, not through Chrome") converted six Camel route ITs from Selenide/Chrome tests to HTTP-level
PredefinedProjectITs and moved them:CamelDirigibleJavaScriptComponent{Cron,Http}RouteIT…ui.tests.camel…api.camelCamelDirigibleTwoStepsJSInvoker{Cron,Http}RouteIT…ui.tests.camel…api.camelCamelExtractTransformLoad{Jdbc,Typescript}IT…ui.tests.camel…api.camelCamel{Cron,Http}RouteStarterTemplateIT…ui.tests.camel…ui.tests.camel(unchanged)DirigibleCommonTestSuiteITimported onlyui.tests.camel.*, which is exactly why the two starter-template ITs still resolved and the other six did not.What
Import
…api.camel.*alongside the existing…ui.tests.camel.*. Both are needed — the two starter-template ITs stayed put. The@SelectClasseslist is unchanged; the same eight ITs run as before.Also recorded the trap in
CLAUDE.md's bump notes: this suite names Dirigible's ITs by reference, so any upstream IT rename/move/removal breaks it, and only at compile time.Rest of the 14.24.0 → 14.27.0 delta
Checked the other surfaces that drift on a bump; all clean, no changes needed:
SecurityIT,CmsSuiteIT,MonitoringShellIT,IntentBuilderShellIT, …) — all still at their declared packages inv14.27.0.components/ui/menu-help,components/ui/view-welcome) — upstream made no changes to either betweenv14.24.0andv14.27.0, so no fork resync.application/pom.xml— all 24dirigible-*artifacts it names still exist inv14.27.0.UserInterfaceIntegrationTest,HtmlElementType,IDE.openHomePage()/openIde()all still present; the Home taglineHomePageITasserts is unchanged inresources-home/index.html.Verification
mvn clean install -P quick-build+ a forced cleantest-compileofintegration-testsagainst the real 14.27.0 artifacts: BUILD SUCCESS.javapon the compiled suite confirms the wildcard resolved as intended — six fromapi/camel, two still fromui/tests/camel:formatter:validatepasses.🤖 Generated with Claude Code