GitHub user onderson opened a pull request:
https://github.com/apache/camel/pull/2550
CAMEL-12853 - disable SftpConsumerDisconnectTest
I think it will be better to mark this test case as ignore.
the reason for this is that the error case is simulated with processor to
disconnect all sessions and exchange often does not reach to an end and session
is closed and leaving with closed input stream as indicated in the log entry of
the issue. Because in
https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/component/file/GenericFileOnCompletion.java#L127
where the deletion is handle over GenericFileDeleteProcessStrategy which
runs after exchange is completed and ftp sessions are disconnected before it
really does moving or deleting.
Even though test almost fails on Linux VM (whereas it sometimes flakes and
passes), it almost run with no failure on windows. Apparently it also runs on
CI as well.
Please let me know your ideas if disabling makes sense or not.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/onderson/camel CAMEL-12853
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/2550.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2550
----
commit a0203ed92a7f1fcb0d17abfd39a97a64caaf84a3
Author: onders <onders@...>
Date: 2018-10-04T09:11:56Z
CAMEL-12853 - disable SftpConsumerDisconnectTest
----
---
