aldettinger commented on code in PR #4133:
URL: https://github.com/apache/camel-quarkus/pull/4133#discussion_r993099264
##########
integration-test-groups/foundation/scheduler/src/test/java/org/apache/camel/quarkus/component/scheduler/it/SchedulerTest.java:
##########
@@ -28,12 +28,42 @@
class SchedulerTest {
@Test
- public void test() throws Exception {
- // wait until the scheduler has run and return a counter that is > 0
+ public void testInitialDelay() throws Exception {
await().atMost(5, TimeUnit.SECONDS).until(() -> {
String body =
RestAssured.get("/scheduler/get").then().statusCode(200).extract().body().asString();
return !body.equals("0");
});
}
+ @Test
+ public void testDelay() throws Exception {
+
+ await().atMost(2, TimeUnit.SECONDS).until(() -> {
Review Comment:
The main idea is that, from my point of view, the current implementation of
this test could pass even when the `withDelay` route is never called.
From there, one solution could be to have two distinct AtomicInteger, let's
say `withDelayCounter` and `useFixedDelayCounter`. And then, it's possible to
check that each route has been invoked separately.
--
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: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
