From 29445adb7fb0ef41624c45e055d2228fda4836f6 Mon Sep 17 00:00:00 2001
From: Valery Kharseko <vharseko@3a-systems.ru>
Date: Sat, 18 Jul 2026 16:17:41 +0000
Subject: [PATCH] [#748] stabilize flaky GrizzlyLDAPConnectionFactoryTestCase.testResourceManagement (#749)
---
opendj-grizzly/src/test/java/org/forgerock/opendj/grizzly/GrizzlyLDAPConnectionFactoryTestCase.java | 13 +++++++++++--
1 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/opendj-grizzly/src/test/java/org/forgerock/opendj/grizzly/GrizzlyLDAPConnectionFactoryTestCase.java b/opendj-grizzly/src/test/java/org/forgerock/opendj/grizzly/GrizzlyLDAPConnectionFactoryTestCase.java
index 0196153..2c66613 100644
--- a/opendj-grizzly/src/test/java/org/forgerock/opendj/grizzly/GrizzlyLDAPConnectionFactoryTestCase.java
+++ b/opendj-grizzly/src/test/java/org/forgerock/opendj/grizzly/GrizzlyLDAPConnectionFactoryTestCase.java
@@ -12,6 +12,7 @@
* information: "Portions Copyright [year] [name of copyright owner]".
*
* Copyright 2013-2016 ForgeRock AS.
+ * Portions Copyrighted 2026 3A Systems, LLC
*/
package org.forgerock.opendj.grizzly;
@@ -86,8 +87,16 @@
*/
private static final int ITERATIONS = 100;
- /** Test timeout for tests which need to wait for network events. */
- private static final long TEST_TIMEOUT = 30L;
+ /**
+ * Test timeout for tests which need to wait for network events. This is a
+ * generous upper bound rather than an expected duration: in the normal case
+ * the awaited latch/promise is released within milliseconds. It only takes
+ * effect when a CI runner is slow enough to delay delivery of an async
+ * event (see issue #748, where {@code testResourceManagement} timed out on
+ * a loaded macOS/Java 11 runner), so a larger value trades a slower failure
+ * for far fewer false-negative builds.
+ */
+ private static final long TEST_TIMEOUT = 60L;
/*
* It is usually quite a bad code smell to share state between unit tests.
--
Gitblit v1.10.0