(incubator-uniffle) branch master updated: [MINOR] fix(test): remove useless code from ServletTest.testRequestWithWrongCredentials (#1976)

[xianjingfeng]

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

xianjingfeng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 276c57b22 [MINOR] fix(test): remove useless code from 
ServletTest.testRequestWithWrongCredentials (#1976)
276c57b22 is described below

commit 276c57b22e3d80a728044bb09bac4aee2a81ce55
Author: xianjingfeng <xianjingfeng...@gmail.com>
AuthorDate: Mon Jul 29 10:21:02 2024 +0800

    [MINOR] fix(test): remove useless code from 
ServletTest.testRequestWithWrongCredentials (#1976)
    
    ### What changes were proposed in this pull request?
    Remove useless code from ServletTest.testRequestWithWrongCredentials.
    
    ### Why are the changes needed?
    The logic of sleep is useless. This is a mistake.
    
    ### Does this PR introduce any user-facing change?
    No.
    
    ### How was this patch tested?
    CI
---
 .../common/src/test/java/org/apache/uniffle/test/ServletTest.java      | 3 ---
 1 file changed, 3 deletions(-)

diff --git 
a/integration-test/common/src/test/java/org/apache/uniffle/test/ServletTest.java
 
b/integration-test/common/src/test/java/org/apache/uniffle/test/ServletTest.java
index 639ec11ab..aeaa58d97 100644
--- 
a/integration-test/common/src/test/java/org/apache/uniffle/test/ServletTest.java
+++ 
b/integration-test/common/src/test/java/org/apache/uniffle/test/ServletTest.java
@@ -376,9 +376,6 @@ public class ServletTest extends IntegrationTestBase {
             CANCEL_DECOMMISSION_URL,
             objectMapper.writeValueAsString(decommissionRequest),
             ImmutableMap.of("Authorization", "Basic " + wrongCredentials));
-    for (int i = 0; i < 1000; i++) {
-      Thread.sleep(1000);
-    }
     assertEquals("Authentication Failed", content);
   }
 }