You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by is...@apache.org on 2023/05/26 04:58:03 UTC
[tomcat] branch 9.0.x updated: Consolidated code to remove duplication
This is an automated email from the ASF dual-hosted git repository.
isapir pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new ba7b5f4a24 Consolidated code to remove duplication
ba7b5f4a24 is described below
commit ba7b5f4a2490f94a06e1bf01c61e94b20d14af1c
Author: Igal Sapir <is...@apache.org>
AuthorDate: Thu May 25 21:57:52 2023 -0700
Consolidated code to remove duplication
---
.../catalina/filters/TestRateLimitFilter.java | 38 +-------------------
.../catalina/filters/TestRemoteCIDRFilter.java | 40 +---------------------
.../catalina/filters/TesterFilterConfigs.java | 35 +++++++++++++++++++
3 files changed, 37 insertions(+), 76 deletions(-)
diff --git a/test/org/apache/catalina/filters/TestRateLimitFilter.java b/test/org/apache/catalina/filters/TestRateLimitFilter.java
index 19281ac4ae..5205d97b61 100644
--- a/test/org/apache/catalina/filters/TestRateLimitFilter.java
+++ b/test/org/apache/catalina/filters/TestRateLimitFilter.java
@@ -19,12 +19,9 @@ package org.apache.catalina.filters;
import java.io.IOException;
import java.time.Instant;
-import java.util.Enumeration;
-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 +33,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 +100,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 +165,4 @@ public class TestRateLimitFilter extends TomcatBaseTest {
}
}
- private static FilterConfig generateFilterConfig(FilterDef filterDef) {
-
- TesterServletContext mockServletContext = new TesterServletContext();
- 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 a10d5b31ed..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(FilterDef filterDef) {
-
- TesterServletContext mockServletContext = new TesterServletContext();
- 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..733b366c96 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(FilterDef filterDef) {
+
+ TesterServletContext mockServletContext = new TesterServletContext();
+ 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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org