This is an automated email from the ASF dual-hosted git repository.

apupier pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new c3907db391c chore: stabilize FtpRecursiveDepth2IT
c3907db391c is described below

commit c3907db391c8b687dddecb06480624a4cc189d03
Author: Aurélien Pupier <apup...@redhat.com>
AuthorDate: Tue Oct 22 10:01:28 2024 +0200

    chore: stabilize FtpRecursiveDepth2IT
    
    this test failed several times in a row on Jenkins CI with slightly
    different errors.
    ```
    mock://result Message with body a2 was expected but not found in [b2,
    b3, b2, a3]
    ```
    ```
    mock://result Message with body a3 was expected but not found in [b3,
    b2]
    ```
    
    When playing locally with @RepeatedTest(100), I do not reproduce the
    error but noticed that 4 or 5 tests took more than 5 seconds although
    others took 1 second. When I tried to set a low timeout locally, it is
    failing the test with similar kind of errors.
    So trying with an higher timeout, even though it seems strange that
    there can be that much variation in route execution.
    
    Signed-off-by: Aurélien Pupier <apup...@redhat.com>
---
 .../camel/component/file/remote/integration/FtpRecursiveDepth2IT.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/integration/FtpRecursiveDepth2IT.java
 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/integration/FtpRecursiveDepth2IT.java
index b4e68284103..91685807392 100644
--- 
a/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/integration/FtpRecursiveDepth2IT.java
+++ 
b/components/camel-ftp/src/test/java/org/apache/camel/component/file/remote/integration/FtpRecursiveDepth2IT.java
@@ -16,6 +16,8 @@
  */
 package org.apache.camel.component.file.remote.integration;
 
+import java.util.concurrent.TimeUnit;
+
 import org.apache.camel.Exchange;
 import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.component.mock.MockEndpoint;
@@ -45,7 +47,7 @@ public class FtpRecursiveDepth2IT extends 
FtpServerTestSupport {
         
template.sendBodyAndHeader("ftp://admin@localhost:{{ftp.server.port}}/depth2/bar/foo?password=admin";,
 "b3",
                 Exchange.FILE_NAME, "b3.txt");
 
-        MockEndpoint.assertIsSatisfied(context);
+        MockEndpoint.assertIsSatisfied(context, 20, TimeUnit.SECONDS);
     }
 
     @Override

Reply via email to