refactor: #5236 test code

larry4xie · larry4xie · commit ff75da32289c · 2024-10-06T22:18:55.000+08:00
diff --git a/apollo-configservice/src/test/java/com/ctrip/framework/apollo/configservice/filter/ClientAuthenticationFilterTest.java b/apollo-configservice/src/test/java/com/ctrip/framework/apollo/configservice/filter/ClientAuthenticationFilterTest.java
@@ -144,10 +144,7 @@ public void testAuthorizedSuccessfully() throws Exception {
 
     clientAuthenticationFilter.doFilter(request, response, filterChain);
 
-    verify(response, never()).sendError(HttpServletResponse.SC_BAD_REQUEST, "InvalidAppId");
-    verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "RequestTimeTooSkewed");
-    verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "Unauthorized");
-    verify(filterChain, times(1)).doFilter(request, response);
+    verifySuccessAndDoFilter();
   }
 
   @Test
@@ -168,10 +165,7 @@ public void testPreCheckInvalid() throws Exception {
 
     clientAuthenticationFilter.doFilter(request, response, filterChain);
 
-    verify(response, never()).sendError(HttpServletResponse.SC_BAD_REQUEST, "InvalidAppId");
-    verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "RequestTimeTooSkewed");
-    verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "Unauthorized");
-    verify(filterChain, times(1)).doFilter(request, response);
+    verifySuccessAndDoFilter();
     verify(clientAuthenticationFilter, times(2)).preCheckInvalidLogging(anyString());
   }
 
@@ -193,10 +187,14 @@ public void testPreCheckSuccessfully() throws Exception {
 
     clientAuthenticationFilter.doFilter(request, response, filterChain);
 
+    verifySuccessAndDoFilter();
+    verify(clientAuthenticationFilter, never()).preCheckInvalidLogging(anyString());
+  }
+
+  private void verifySuccessAndDoFilter() throws Exception {
     verify(response, never()).sendError(HttpServletResponse.SC_BAD_REQUEST, "InvalidAppId");
     verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "RequestTimeTooSkewed");
     verify(response, never()).sendError(HttpServletResponse.SC_UNAUTHORIZED, "Unauthorized");
     verify(filterChain, times(1)).doFilter(request, response);
-    verify(clientAuthenticationFilter, never()).preCheckInvalidLogging(anyString());
   }
 }
