gnodet opened a new pull request, #24603: URL: https://github.com/apache/camel/pull/24603
_Claude Code on behalf of @gnodet_ ## Summary Replaces all `Thread.sleep()` calls in three `camel-mina-sftp` integration test classes with proper condition-based waiting, per project guidelines. - **`MinaSftpAdvancedFileOperationsIT`**: 3x `Thread.sleep(500)` → `Awaitility.await().untilAsserted(...)` for filesystem move verification after `mock.assertIsSatisfied()` - **`MinaSftpProtocolIT`**: 2x `Thread.sleep(1000)` → `Awaitility.await().untilAsserted(...)` for filesystem delete/move; 1x `Thread.sleep(2000)` → explicit `file.setLastModified()` (no delay needed) - **`MinaSftpConcurrencyIT`**: 2x `Thread.sleep(50/100)` removed — these were unnecessary pacing delays in concurrent threads that don't affect correctness **Important**: The Awaitility calls wrap only the **filesystem assertions** (file exists, file deleted, file moved) — they do NOT wrap `MockEndpoint.assertIsSatisfied()`, which already waits internally via `CountDownLatch`. ## Changed files | File | Change | |---|---| | `MinaSftpAdvancedFileOperationsIT.java` | 3x sleep → Awaitility for post-move checks | | `MinaSftpProtocolIT.java` | 2x sleep → Awaitility for post-delete/move; 1x sleep → `setLastModified()` | | `MinaSftpConcurrencyIT.java` | 2x pacing sleeps removed | ## Test plan - [x] Compilation passes - [x] Formatting validation passes (`formatter:validate`, `impsort:check`) - [ ] CI build passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
