Revision: 1190
Author: sberlin
Date: Tue Jul 20 19:48:53 2010
Log: minor cleanups to servlet extension tests. change asserts that were
using the JDK "assert" keyword to assertTrue, add missing EasyMock.verify
calls, and fix things that were failing as a result of changing those.
http://code.google.com/p/google-guice/source/detail?r=1190
Modified:
/trunk/servlet/test/com/google/inject/servlet/FilterDefinitionTest.java
/trunk/servlet/test/com/google/inject/servlet/FilterDispatchIntegrationTest.java
/trunk/servlet/test/com/google/inject/servlet/FilterPipelineTest.java
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
/trunk/servlet/test/com/google/inject/servlet/MultiModuleDispatchIntegrationTest.java
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionPathsTest.java
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionTest.java
/trunk/servlet/test/com/google/inject/servlet/VarargsFilterDispatchIntegrationTest.java
=======================================
--- /trunk/servlet/test/com/google/inject/servlet/FilterDefinitionTest.java
Sat Jul 3 08:51:31 2010
+++ /trunk/servlet/test/com/google/inject/servlet/FilterDefinitionTest.java
Tue Jul 20 19:48:53 2010
@@ -60,10 +60,6 @@
put("ahsd124124", "as124124124dasdok");
}};
- String pattern = "/*";
- final FilterDefinition filterDef = new FilterDefinition(pattern,
Key.get(Filter.class),
- UriPatternType.get(UriPatternType.SERVLET, pattern), initParams);
- assert filterDef.getFilter() instanceof MockFilter;
ServletContext servletContext = createMock(ServletContext.class);
final String contextName = "thing__!@@44";
@@ -71,21 +67,27 @@
replay(servletContext);
+ String pattern = "/*";
+ final FilterDefinition filterDef = new FilterDefinition(pattern,
Key.get(Filter.class),
+ UriPatternType.get(UriPatternType.SERVLET, pattern),
initParams);
filterDef.init(servletContext, injector,
Sets.newSetFromMap(Maps.<Filter, Boolean>newIdentityHashMap()));
+ assertTrue(filterDef.getFilter() instanceof MockFilter);
final FilterConfig filterConfig = mockFilter.getConfig();
- assert null != filterConfig;
- assert
contextName.equals(filterConfig.getServletContext().getServletContextName());
- assert
Key.get(Filter.class).toString().equals(filterConfig.getFilterName());
+ assertTrue(null != filterConfig);
+
assertTrue(contextName.equals(filterConfig.getServletContext().getServletContextName()));
+
assertTrue(Key.get(Filter.class).toString().equals(filterConfig.getFilterName()));
final Enumeration names = filterConfig.getInitParameterNames();
while (names.hasMoreElements()) {
String name = (String) names.nextElement();
- assert initParams.containsKey(name);
- assert
initParams.get(name).equals(filterConfig.getInitParameter(name));
- }
+ assertTrue(initParams.containsKey(name));
+
assertTrue(initParams.get(name).equals(filterConfig.getInitParameter(name)));
+ }
+
+ verify(binding, injector, servletContext);
}
public final void testFilterCreateDispatchDestroy() throws
ServletException, IOException {
@@ -110,14 +112,13 @@
String pattern = "/*";
final FilterDefinition filterDef = new FilterDefinition(pattern,
Key.get(Filter.class),
- UriPatternType.get(UriPatternType.SERVLET, pattern), new
HashMap<String, String>());
- assert filterDef.getFilter() instanceof MockFilter;
-
+ UriPatternType.get(UriPatternType.SERVLET, pattern), new
HashMap<String, String>());
//should fire on mockfilter now
filterDef.init(createMock(ServletContext.class), injector,
Sets.newSetFromMap(Maps.<Filter, Boolean>newIdentityHashMap()));
-
- assert mockFilter.isInit() : "Init did not fire";
+ assertTrue(filterDef.getFilter() instanceof MockFilter);
+
+ assertTrue("Init did not fire", mockFilter.isInit());
final boolean proceed[] = new boolean[1];
filterDef.doFilter(request, null, new FilterChainInvocation(null,
null, null) {
@@ -169,13 +170,13 @@
String pattern = "/*";
final FilterDefinition filterDef = new FilterDefinition(pattern,
Key.get(Filter.class),
UriPatternType.get(UriPatternType.SERVLET, pattern), new
HashMap<String, String>());
- assert filterDef.getFilter() instanceof MockFilter;
-
//should fire on mockfilter now
filterDef.init(createMock(ServletContext.class), injector,
- Sets.newSetFromMap(Maps.<Filter, Boolean>newIdentityHashMap()));
-
- assert mockFilter.isInit() : "Init did not fire";
+ Sets.newSetFromMap(Maps.<Filter, Boolean>newIdentityHashMap()));
+ assertTrue(filterDef.getFilter() instanceof MockFilter);
+
+
+ assertTrue("init did not fire", mockFilter.isInit());
final boolean proceed[] = new boolean[1];
filterDef.doFilter(request, null, new FilterChainInvocation(null,
null, null) {
@@ -185,10 +186,10 @@
}
});
- assert !proceed[0] : "Filter did not suppress chain";
+ assertTrue("filter did not suppress chain", !proceed[0]);
filterDef.destroy(Sets.newSetFromMap(Maps.<Filter,
Boolean>newIdentityHashMap()));
- assert mockFilter.isDestroy() : "Destroy did not fire";
+ assertTrue("destroy did not fire", mockFilter.isDestroy());
verify(injector, request);
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/FilterDispatchIntegrationTest.java
Tue Dec 29 17:05:45 2009
+++
/trunk/servlet/test/com/google/inject/servlet/FilterDispatchIntegrationTest.java
Tue Jul 20 19:48:53 2010
@@ -102,9 +102,9 @@
assertTrue(servlet.processedUris.contains("/index.html"));
assertTrue(servlet.processedUris.contains(TestServlet.FORWARD_TO));
- assert inits == 5 && doFilters == 3 && destroys == 5 : "lifecycle
states did not"
- + " fire correct number of times-- inits: " + inits + "; dos: "
+ doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not"
+ + " fire correct number of times-- inits: " + inits + "; dos: " +
doFilters
+ + "; destroys: " + destroys, inits == 1 && doFilters == 3 &&
destroys == 1);
}
public final void testDispatchThatNoFiltersFire() throws
ServletException, IOException {
@@ -140,9 +140,10 @@
pipeline.destroyPipeline();
control.verify();
- assert inits == 5 && doFilters == 0 && destroys == 5 : "lifecycle
states did not "
- + "fire correct number of times-- inits: " + inits + "; dos: " +
doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not "
+ + "fire correct number of times-- inits: " + inits + "; dos: "
+ doFilters
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 0 && destroys == 1);
}
public final void testDispatchFilterPipelineWithRegexMatching() throws
ServletException,
@@ -177,9 +178,10 @@
pipeline.destroyPipeline();
control.verify();
- assert inits == 3 && doFilters == 2 && destroys == 3 : "lifecycle
states did not fire "
- + "correct number of times-- inits: " + inits + "; dos: " +
doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not fire "
+ + "correct number of times-- inits: " + inits + "; dos: " +
doFilters
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 2 && destroys == 1);
}
@Singleton
=======================================
--- /trunk/servlet/test/com/google/inject/servlet/FilterPipelineTest.java
Tue Jan 6 04:48:44 2009
+++ /trunk/servlet/test/com/google/inject/servlet/FilterPipelineTest.java
Tue Jul 20 19:48:53 2010
@@ -112,7 +112,7 @@
public void doFilter(ServletRequest servletRequest, ServletResponse
servletResponse,
FilterChain filterChain) throws IOException, ServletException {
- assert false : "This filter should never have fired";
+ fail("This filter should never have fired");
}
public void destroy() {
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
Mon Apr 19 20:03:13 2010
+++
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
Tue Jul 20 19:48:53 2010
@@ -159,7 +159,7 @@
public void doFilter(ServletRequest servletRequest, ServletResponse
servletResponse,
FilterChain filterChain) throws IOException, ServletException {
- assert false : "This filter should never have fired";
+ fail("This filter should never have fired");
}
public void destroy() {
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/MultiModuleDispatchIntegrationTest.java
Tue Jan 6 04:48:44 2009
+++
/trunk/servlet/test/com/google/inject/servlet/MultiModuleDispatchIntegrationTest.java
Tue Jul 20 19:48:53 2010
@@ -83,9 +83,10 @@
verify(requestMock);
- assert inits == 5 && doFilters == 3 && destroys == 5 : "lifecycle
states did not"
+ assertTrue("lifecycle states did not"
+ " fire correct number of times-- inits: " + inits + "; dos: "
+ doFilters
- + "; destroys: " + destroys;
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 3 && destroys == 1);
}
@Singleton
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionPathsTest.java
Sat Jul 3 08:51:31 2010
+++
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionPathsTest.java
Tue Jul 20 19:48:53 2010
@@ -35,6 +35,7 @@
import static org.easymock.EasyMock.createMock;
import static org.easymock.EasyMock.expect;
import static org.easymock.EasyMock.replay;
+import static org.easymock.EasyMock.verify;
/**
* Ensures servlet spec compliance for CGI-style variables and general
@@ -100,6 +101,8 @@
servletDefinition.doService(request, response);
assertTrue("Servlet did not run!", run[0]);
+
+ verify(injector, binding, request);
}
@@ -192,6 +195,7 @@
assertTrue("Servlet did not run!", run[0]);
+ verify(injector, binding, request);
}
// Data-driven test.
@@ -286,5 +290,7 @@
servletDefinition.doService(request, response);
assertTrue("Servlet did not run!", run[0]);
+
+ verify(injector, binding, request);
}
}
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionTest.java
Sat Jul 3 08:51:31 2010
+++
/trunk/servlet/test/com/google/inject/servlet/ServletDefinitionTest.java
Tue Jul 20 19:48:53 2010
@@ -34,6 +34,7 @@
import static org.easymock.EasyMock.expect;
import static org.easymock.EasyMock.replay;
import static org.easymock.EasyMock.anyObject;
+import static org.easymock.EasyMock.verify;
/**
* Basic unit test for lifecycle of a ServletDefinition (wrapper).
@@ -96,5 +97,7 @@
assertTrue(initParams.containsKey(name));
assertEquals(initParams.get(name),
servletConfig.getInitParameter(name));
}
+
+ verify(injector, binding, servletContext);
}
}
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/VarargsFilterDispatchIntegrationTest.java
Tue Jan 6 04:48:44 2009
+++
/trunk/servlet/test/com/google/inject/servlet/VarargsFilterDispatchIntegrationTest.java
Tue Jul 20 19:48:53 2010
@@ -73,9 +73,10 @@
verify(requestMock);
- assert inits == 5 && doFilters == 3 && destroys == 5 : "lifecycle
states did not"
- + " fire correct number of times-- inits: " + inits + "; dos: "
+ doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not"
+ + " fire correct number of times-- inits: " + inits + "; dos: " +
doFilters
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 3 && destroys == 1);
}
public final void testDispatchThatNoFiltersFire() throws
ServletException, IOException {
@@ -108,9 +109,10 @@
verify(requestMock);
- assert inits == 5 && doFilters == 0 && destroys == 5 : "lifecycle
states did not "
- + "fire correct number of times-- inits: " + inits + "; dos: " +
doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not "
+ + "fire correct number of times-- inits: " + inits + "; dos: "
+ doFilters
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 0 && destroys == 1);
}
public final void testDispatchFilterPipelineWithRegexMatching() throws
ServletException,
@@ -144,9 +146,10 @@
verify(requestMock);
- assert inits == 3 && doFilters == 2 && destroys == 3 : "lifecycle
states did not fire "
- + "correct number of times-- inits: " + inits + "; dos: " +
doFilters
- + "; destroys: " + destroys;
+ assertTrue("lifecycle states did not fire "
+ + "correct number of times-- inits: " + inits + "; dos: " +
doFilters
+ + "; destroys: " + destroys,
+ inits == 1 && doFilters == 2 && destroys == 1);
}
@Singleton
--
You received this message because you are subscribed to the Google Groups
"google-guice-dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/google-guice-dev?hl=en.
