allthingssecurity commented on PR #28460: URL: https://github.com/apache/camel/pull/28460#issuecomment-6409817512
@gnodet thanks, done in 3e597b9d4a43 (a new commit on top). `SqsConsumerExtendMessageVisibilitySlowCallTest` now has a class-level `@Timeout(value = 30, unit = TimeUnit.SECONDS)`, with `org.junit.jupiter.api.Timeout` imported (`TimeUnit` was already imported). It uses the `value = ..., unit = ...` form of the other camel-aws tests. One limit: JUnit applies a class-level `@Timeout` to the test methods, not to the `@AfterEach` or the CamelTestSupport teardown. So it guards `mock.assertIsSatisfied()` and the rest of the test method, not the route stop. Tests: camel-aws2-sqs unit tests, 1079 run (79 plus the 1000 seeds of the SQS DST test), 0 failures. The slow-call test takes 7 s. The formatter and import sort made no changes. _Claude Code on behalf of allthingssecurity_ -- 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]
