This is an automated email from the ASF dual-hosted git repository. chrisdutz pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/plc4x.git
commit 8c0d7d914a93bb472c3bd655a88d9385c0419aaa Author: Christofer Dutz <[email protected]> AuthorDate: Tue Jul 14 12:01:37 2026 +0200 fix: Hopefully hardened a flaky test. --- .../plc4x/java/utils/cache/CachedPlcConnectionManagerTest.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/plc4j/tools/connection-cache/src/test/java/org/apache/plc4x/java/utils/cache/CachedPlcConnectionManagerTest.java b/plc4j/tools/connection-cache/src/test/java/org/apache/plc4x/java/utils/cache/CachedPlcConnectionManagerTest.java index f1416f5708..25827d83bc 100644 --- a/plc4j/tools/connection-cache/src/test/java/org/apache/plc4x/java/utils/cache/CachedPlcConnectionManagerTest.java +++ b/plc4j/tools/connection-cache/src/test/java/org/apache/plc4x/java/utils/cache/CachedPlcConnectionManagerTest.java @@ -593,8 +593,12 @@ class CachedPlcConnectionManagerTest { PlcConnection connection1 = manager.getConnection("test:tcp://localhost"); connection1.close(); - // Wait for connection to become idle beyond threshold - Thread.sleep(600); + // Wait for connection to become idle beyond threshold. + // Use a generous margin over the 500ms threshold: the connection is only marked + // "requires validation" by a task on the (single-thread) scheduler that fires at the + // threshold, and on a loaded CI runner that task can be delayed. A small margin + // (e.g. 100ms) makes this test flaky; 1500ms gives the scheduler ample room. + Thread.sleep(1500); // Reset ping mock to track subsequent calls reset(mockConnection);
