Fix unstable test: BrokerOuterAPITest.test_needRegister_timeout (#9250)

DivyanshIITB · web-flow · commit 94dd2e7dda2c · 2025-03-17T14:21:11.000+08:00
diff --git a/broker/src/test/java/org/apache/rocketmq/broker/BrokerOuterAPITest.java b/broker/src/test/java/org/apache/rocketmq/broker/BrokerOuterAPITest.java
@@ -154,7 +154,7 @@ public RemotingCommand answer(InvocationOnMock invocation) throws Throwable {
                 } else if (invocation.getArgument(0) == nameserver2) {
                     return buildResponse(Boolean.FALSE);
                 } else if (invocation.getArgument(0) == nameserver3) {
-                    TimeUnit.MILLISECONDS.sleep(timeOut + 20);
+                    TimeUnit.MILLISECONDS.sleep(timeOut + 100);  // Increase sleep time to force timeout
                     return buildResponse(Boolean.TRUE);
                 }
                 return buildResponse(Boolean.TRUE);

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,7 @@ public RemotingCommand answer(InvocationOnMock invocation) throws Throwable {`
`154`	`154`	`} else if (invocation.getArgument(0) == nameserver2) {`
`155`	`155`	`return buildResponse(Boolean.FALSE);`
`156`	`156`	`} else if (invocation.getArgument(0) == nameserver3) {`
`157`		`- TimeUnit.MILLISECONDS.sleep(timeOut + 20);`
	`157`	`+ TimeUnit.MILLISECONDS.sleep(timeOut + 100); // Increase sleep time to force timeout`
`158`	`158`	`return buildResponse(Boolean.TRUE);`
`159`	`159`	`}`
`160`	`160`	`return buildResponse(Boolean.TRUE);`