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

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


The following commit(s) were added to refs/heads/master by this push:
     new 3feaeff6 SUBMARINE-1305. Fix submarine-server-core test error
3feaeff6 is described below

commit 3feaeff6b160cfc314276caf01e66dc70c5b2ab9
Author: cdmikechen <[email protected]>
AuthorDate: Fri Aug 12 21:01:09 2022 +0800

    SUBMARINE-1305. Fix submarine-server-core test error
    
    ### What is this PR for?
    Currently, the test case class `SubmarineAuthSimpleTest` in 
`submarine-server-core` does not reset `submarine.auth.type` to `none` at the 
end (After), which will cause subsequent test cases to be blocked.
    
    ### What type of PR is it?
    Bug Fix
    
    ### Todos
    * [x] - Reset `submarine.auth.type` to `none` after test passed.
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/SUBMARINE-1305
    
    ### How should this be tested?
    Have fixed.
    
    ### Screenshots (if appropriate)
    No
    
    ### Questions:
    * Do the license files need updating? No
    * Are there breaking changes for older versions? No
    * Does this need new documentation? No
    
    Author: cdmikechen <[email protected]>
    
    Signed-off-by: Kevin <[email protected]>
    
    Closes #987 from cdmikechen/SUBMARINE-1305 and squashes the following 
commits:
    
    c1c055ff [cdmikechen] Rest submarine.auth.type to none at @After
---
 .../apache/submarine/server/security/SubmarineAuthSimpleTest.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/submarine-server/server-core/src/test/java/org/apache/submarine/server/security/SubmarineAuthSimpleTest.java
 
b/submarine-server/server-core/src/test/java/org/apache/submarine/server/security/SubmarineAuthSimpleTest.java
index b86b7b52..284362f4 100644
--- 
a/submarine-server/server-core/src/test/java/org/apache/submarine/server/security/SubmarineAuthSimpleTest.java
+++ 
b/submarine-server/server-core/src/test/java/org/apache/submarine/server/security/SubmarineAuthSimpleTest.java
@@ -32,6 +32,7 @@ import 
org.apache.submarine.server.security.simple.SimpleFilter;
 import org.apache.submarine.server.utils.gson.EnvironmentIdDeserializer;
 import org.apache.submarine.server.utils.gson.EnvironmentIdSerializer;
 import org.apache.submarine.server.utils.response.JsonResponse;
+import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
@@ -143,4 +144,9 @@ public class SubmarineAuthSimpleTest {
     assertNotNull(mockRequest.getAttribute(Pac4jConstants.USER_PROFILES));
   }
 
+  @After
+  public void after() {
+    conf.updateConfiguration("submarine.auth.type", "none");
+  }
+
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to