modernize: add exclusionPatterns

apache · Sep 7, 2022 · 1ba1ba0 · 1ba1ba0
1 parent 957bea5
commit 1ba1ba0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/pulsar-broker/pom.xml b/pulsar-broker/pom.xml
@@ -438,6 +438,9 @@
             <ignorePackage>org.apache.pulsar.broker.stats</ignorePackage>
             <ignorePackage>org.apache.pulsar.client</ignorePackage>
           </ignorePackages>
+          <exclusionPatterns>
+            <exclusionPattern>java/lang/StringBuffer."&lt;init&gt;":.*</exclusionPattern>
+          </exclusionPatterns>
         </configuration>
         <executions>
           <execution>

diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/intercept/InterceptFilterOutTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/intercept/InterceptFilterOutTest.java
@@ -71,7 +71,7 @@ public void testFilterOutForPreInterceptFilter() throws Exception {
         Mockito.doNothing().when(chain).doFilter(Mockito.any(), Mockito.any());
         HttpServletRequestWrapper mockInputStream = new MockRequestWrapper(request);
         Mockito.doReturn(mockInputStream.getInputStream()).when(request).getInputStream();
-        Mockito.doReturn(new StringBuilder("http://127.0.0.1:8080")).when(request).getRequestURL();
+        Mockito.doReturn(new StringBuffer("http://127.0.0.1:8080")).when(request).getRequestURL();
 
         // "application/json" should be intercepted
         Mockito.doReturn("application/json").when(request).getContentType();
@@ -103,7 +103,7 @@ public void testOnFilter() throws Exception {
         Mockito.doNothing().when(chain).doFilter(Mockito.any(), Mockito.any());
         HttpServletRequestWrapper mockInputStream = new MockRequestWrapper(request);
         Mockito.doReturn(mockInputStream.getInputStream()).when(request).getInputStream();
-        Mockito.doReturn(new StringBuilder("http://127.0.0.1:8080")).when(request).getRequestURL();
+        Mockito.doReturn(new StringBuffer("http://127.0.0.1:8080")).when(request).getRequestURL();
         // "application/json" should be intercepted
         Mockito.doReturn("application/json").when(request).getContentType();
 
@@ -129,7 +129,7 @@ public void testFilterOutForResponseInterceptFilter() throws Exception {
         Mockito.doNothing().when(chain).doFilter(Mockito.any(), Mockito.any());
         HttpServletRequestWrapper mockInputStream = new MockRequestWrapper(request);
         Mockito.doReturn(mockInputStream.getInputStream()).when(request).getInputStream();
-        Mockito.doReturn(new StringBuilder("http://127.0.0.1:8080")).when(request).getRequestURL();
+        Mockito.doReturn(new StringBuffer("http://127.0.0.1:8080")).when(request).getRequestURL();
 
         // "application/json" should be intercepted
         Mockito.doReturn("application/json").when(request).getContentType();
@@ -161,7 +161,7 @@ public void testShouldNotInterceptWhenInterceptorDisabled() throws Exception {
         Mockito.doNothing().when(chain).doFilter(Mockito.any(), Mockito.any());
         HttpServletRequestWrapper mockInputStream = new MockRequestWrapper(request);
         Mockito.doReturn(mockInputStream.getInputStream()).when(request).getInputStream();
-        Mockito.doReturn(new StringBuilder("http://127.0.0.1:8080")).when(request).getRequestURL();
+        Mockito.doReturn(new StringBuffer("http://127.0.0.1:8080")).when(request).getRequestURL();
 
         // Should not be intercepted since the broker interceptor disabled.
         Mockito.doReturn("application/json").when(request).getContentType();