(knox) branch master updated: KNOX-3345: Fix intermittent testRefreshGatewayConfig test issue (#1259)

[hanicz]

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

hanicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/knox.git


The following commit(s) were added to refs/heads/master by this push:
     new 7c94926b1 KNOX-3345: Fix intermittent testRefreshGatewayConfig test 
issue (#1259)
7c94926b1 is described below

commit 7c94926b1cc967212ed12beb8aa36054677548d8
Author: hanicz <han...@users.noreply.github.com>
AuthorDate: Thu Jun 11 12:11:31 2026 +0200

    KNOX-3345: Fix intermittent testRefreshGatewayConfig test issue (#1259)
---
 .../src/test/java/org/apache/knox/gateway/GatewayServerTest.java | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git 
a/gateway-server/src/test/java/org/apache/knox/gateway/GatewayServerTest.java 
b/gateway-server/src/test/java/org/apache/knox/gateway/GatewayServerTest.java
index c6402877b..4ad86d326 100644
--- 
a/gateway-server/src/test/java/org/apache/knox/gateway/GatewayServerTest.java
+++ 
b/gateway-server/src/test/java/org/apache/knox/gateway/GatewayServerTest.java
@@ -21,7 +21,6 @@ import 
org.apache.knox.gateway.config.GatewayConfigChangeListener;
 import org.apache.knox.gateway.config.impl.GatewayConfigImpl;
 import org.easymock.EasyMock;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
@@ -48,7 +47,6 @@ public class GatewayServerTest {
   }
 
   @Test
-  @Ignore
   public void testRefreshGatewayConfig() throws Exception {
     GatewayConfigImpl config = 
EasyMock.createNiceMock(GatewayConfigImpl.class);
 
@@ -65,6 +63,11 @@ public class GatewayServerTest {
     Method refreshMethod = 
GatewayServer.class.getDeclaredMethod("refreshGatewayConfig", 
GatewayConfigImpl.class, Path.class);
     refreshMethod.setAccessible(true);
 
+    // Reset static lastReloadTime to ensure test isolation
+    Field lastReloadTimeField = 
GatewayServer.class.getDeclaredField("lastReloadTime");
+    lastReloadTimeField.setAccessible(true);
+    lastReloadTimeField.set(null, null);
+
     // Initial load
     config.reloadConfiguration();
     EasyMock.expectLastCall().once();
@@ -75,8 +78,6 @@ public class GatewayServerTest {
     EasyMock.verify(config);
 
     // Check lastReloadTime is set
-    Field lastReloadTimeField = 
GatewayServer.class.getDeclaredField("lastReloadTime");
-    lastReloadTimeField.setAccessible(true);
     FileTime lastReloadTime = (FileTime) lastReloadTimeField.get(null);
     assertNotNull(lastReloadTime);