davsclaus opened a new pull request, #29757: URL: https://github.com/apache/camel/pull/29757
Fixes [CAMEL-26517](https://issues.apache.org/jira/browse/CAMEL-26517) `LdapRouteManualIT` and `LdapRouteTwoManualIT` were gated with `@EnabledIfSystemProperty(named = "enable.ldap.itests")` ("complex ldap server environment"), and their `*IT` names kept them out of surefire as well. CAMEL-26198 (#29023) fixed the embedded ApacheDS setup of these tests, so they now run on their own. Changes: - removed the `enable.ldap.itests` gate - renamed them to `LdapRouteTest` and `LdapRouteTwoTest` (`git mv`, content otherwise unchanged; JUnit assertions kept as in the module) Nothing else in the repo referenced the old names or the property. Why `*Test` rather than `*IT`: the tests start an embedded ApacheDS (`@CreateLdapServer`) on a random port, with no container, so they also run on the ppc64le and s390x builds. Each run used different ports (e.g. 34093, 40391, 45581). Surefire runs the test classes one after another in the module's fork (`camel.surefire.parallel=false`), so they do not clash with `LdapProducerDirContextTest`. Local runs of `mvn -o formatter:format impsort:sort test` in `components/camel-ldap`, three times: 9 tests pass each time (LdapRouteTest 4, LdapRouteTwoTest 1, LdapProducerDirContextTest 4). The two route tests take 1.2-1.8 s and 0.6-0.9 s; the whole module run takes 8-11 s. _Claude Code on behalf of davsclaus_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
