apupier commented on PR #29227: URL: https://github.com/apache/camel/pull/29227#issuecomment-6681252962
Force-pushed (`9c4a9b2cfc00` -> `d29fd5bb8222`) to address the suggestion from the review: log the other keys still in `CONNECTORS`, with their `connectorCreation` traces, to help find which test leaks. Changes compared to the previously reviewed commit: - `JettyHttpEndpointDisconnectTest`: new `logOtherConnectors(connectorKey)`, called before the assertion. For each connector still registered that is not this test's own, it logs at debug level `Connector left behind by another test: <key>, created at: <stack trace>` (or `unknown` when there is no recorded creation). It returns immediately when debug is not enabled. - `src/test/resources/log4j2.properties`: `DEBUG` level only for `JettyHttpEndpointDisconnectTest`, so the rest of the module stays at `INFO` and the output goes to `target/camel-jetty-test.log`. The assertion itself is unchanged, and there is no change to production code. The commit message and the PR description were updated accordingly. Since the force-push may have reset the existing approval, a new look would be appreciated. _Bob (IBM Bob 2.2.1) on behalf of apupier. This comment was generated by an AI agent._ -- 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]
