This is an automated email from the ASF dual-hosted git repository.
isapir pushed a commit to branch 8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/8.5.x by this push:
new 746373b0bc Consolidated code to remove duplication
746373b0bc is described below
commit 746373b0bc28bd6cf73e67dfe3eb8ee04a4cd812
Author: Igal Sapir <isa...@apache.org>
AuthorDate: Thu May 25 22:15:54 2023 -0700
Consolidated code to remove duplication
---
.../catalina/filters/TestRateLimitFilter.java | 36 +------------------
.../catalina/filters/TestRemoteCIDRFilter.java | 40 +---------------------
.../catalina/filters/TesterFilterConfigs.java | 35 +++++++++++++++++++
3 files changed, 37 insertions(+), 74 deletions(-)
diff --git a/test/org/apache/catalina/filters/TestRateLimitFilter.java
b/test/org/apache/catalina/filters/TestRateLimitFilter.java
index 6f780b4c88..49e785d7e6 100644
--- a/test/org/apache/catalina/filters/TestRateLimitFilter.java
+++ b/test/org/apache/catalina/filters/TestRateLimitFilter.java
@@ -24,7 +24,6 @@ import java.util.Map;
import javax.servlet.FilterChain;
import javax.servlet.FilterConfig;
-import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import org.junit.Assert;
@@ -36,7 +35,6 @@ import
org.apache.catalina.filters.TestRemoteIpFilter.MockHttpServletRequest;
import org.apache.catalina.startup.Tomcat;
import org.apache.catalina.startup.TomcatBaseTest;
import org.apache.tomcat.unittest.TesterResponse;
-import org.apache.tomcat.unittest.TesterServletContext;
import org.apache.tomcat.util.descriptor.web.FilterDef;
import org.apache.tomcat.util.descriptor.web.FilterMap;
@@ -104,7 +102,7 @@ public class TestRateLimitFilter extends TomcatBaseTest {
filterMap.addURLPatternDecoded("*");
root.addFilterMap(filterMap);
- FilterConfig filterConfig = generateFilterConfig(filterDef);
+ FilterConfig filterConfig =
TesterFilterConfigs.generateFilterConfig(filterDef);
rateLimitFilter.init(filterConfig);
@@ -169,36 +167,4 @@ public class TestRateLimitFilter extends TomcatBaseTest {
}
}
- private static FilterConfig generateFilterConfig(FilterDef filterDef) {
-
- final TesterServletContext mockServletContext = new
TesterServletContext();
- final Map<String,String> parameters = filterDef.getParameterMap();
-
- FilterConfig filterConfig = new FilterConfig() {
-
- @Override
- public String getFilterName() {
- return "rate-limit-filter";
- }
-
- @Override
- public ServletContext getServletContext() {
- return mockServletContext;
- }
-
- @Override
- public String getInitParameter(String name) {
-
- return parameters.get(name);
- }
-
- @Override
- public Enumeration<String> getInitParameterNames() {
- return null;
- }
- };
-
- return filterConfig;
- }
-
}
diff --git a/test/org/apache/catalina/filters/TestRemoteCIDRFilter.java
b/test/org/apache/catalina/filters/TestRemoteCIDRFilter.java
index c7a2854c3f..87f9c3b0d0 100644
--- a/test/org/apache/catalina/filters/TestRemoteCIDRFilter.java
+++ b/test/org/apache/catalina/filters/TestRemoteCIDRFilter.java
@@ -17,13 +17,8 @@
package org.apache.catalina.filters;
-import java.util.Collections;
-import java.util.Enumeration;
-import java.util.Map;
-
import javax.servlet.Filter;
import javax.servlet.FilterConfig;
-import javax.servlet.ServletContext;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletResponse;
@@ -35,7 +30,6 @@ import org.apache.catalina.connector.Request;
import org.apache.catalina.startup.Tomcat;
import org.apache.catalina.startup.TomcatBaseTest;
import org.apache.tomcat.unittest.TesterResponse;
-import org.apache.tomcat.unittest.TesterServletContext;
import org.apache.tomcat.util.descriptor.web.FilterDef;
import org.apache.tomcat.util.descriptor.web.FilterMap;
@@ -146,43 +140,11 @@ public class TestRemoteCIDRFilter extends TomcatBaseTest {
filterMap.addURLPatternDecoded(urlPattern);
root.addFilterMap(filterMap);
- FilterConfig filterConfig = generateFilterConfig(filterDef);
+ FilterConfig filterConfig =
TesterFilterConfigs.generateFilterConfig(filterDef);
remoteCIDRFilter.init(filterConfig);
return remoteCIDRFilter;
}
- private static FilterConfig generateFilterConfig(final FilterDef
filterDef) {
-
- final TesterServletContext mockServletContext = new
TesterServletContext();
- final Map<String,String> parameters = filterDef.getParameterMap();
-
- FilterConfig filterConfig = new FilterConfig() {
-
- @Override
- public String getFilterName() {
- return filterDef.getFilterName();
- }
-
- @Override
- public ServletContext getServletContext() {
- return mockServletContext;
- }
-
- @Override
- public String getInitParameter(String name) {
-
- return parameters.get(name);
- }
-
- @Override
- public Enumeration<String> getInitParameterNames() {
- return Collections.enumeration(parameters.keySet());
- }
- };
-
- return filterConfig;
- }
-
}
diff --git a/test/org/apache/catalina/filters/TesterFilterConfigs.java
b/test/org/apache/catalina/filters/TesterFilterConfigs.java
index b6c5e9c030..25f105805b 100644
--- a/test/org/apache/catalina/filters/TesterFilterConfigs.java
+++ b/test/org/apache/catalina/filters/TesterFilterConfigs.java
@@ -16,12 +16,15 @@
*/
package org.apache.catalina.filters;
+import java.util.Collections;
import java.util.Enumeration;
+import java.util.Map;
import javax.servlet.FilterConfig;
import javax.servlet.ServletContext;
import org.apache.tomcat.unittest.TesterServletContext;
+import org.apache.tomcat.util.descriptor.web.FilterDef;
public class TesterFilterConfigs {
public static final String HTTPS_WWW_APACHE_ORG = "https://www.apache.org";;
@@ -197,6 +200,38 @@ public class TesterFilterConfigs {
supportCredentials, preflightMaxAge, decorateRequest);
}
+ public static FilterConfig generateFilterConfig(final FilterDef filterDef)
{
+
+ final TesterServletContext mockServletContext = new
TesterServletContext();
+ final Map<String,String> parameters = filterDef.getParameterMap();
+
+ FilterConfig filterConfig = new FilterConfig() {
+
+ @Override
+ public String getFilterName() {
+ return filterDef.getFilterName();
+ }
+
+ @Override
+ public ServletContext getServletContext() {
+ return mockServletContext;
+ }
+
+ @Override
+ public String getInitParameter(String name) {
+
+ return parameters.get(name);
+ }
+
+ @Override
+ public Enumeration<String> getInitParameterNames() {
+ return Collections.enumeration(parameters.keySet());
+ }
+ };
+
+ return filterConfig;
+ }
+
private static FilterConfig generateFilterConfig(final String
allowedHttpHeaders, final String allowedHttpMethods,
final String allowedOrigins, final String exposedHeaders, final
String supportCredentials,
final String preflightMaxAge, final String decorateRequest) {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
