Revision: 1193
Author: sberlin
Date: Sat Jul 31 15:13:21 2010
Log: fix InjectedFilterPipelineTest -- it's been failing since r1150
(introduction of using injected pipelines over static ones) if assertions
were turned, and failing since r1190 (change to use assertTrue instead of
assert) always.
http://code.google.com/p/google-guice/source/detail?r=1193
Modified:
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
=======================================
---
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
Tue Jul 20 19:48:53 2010
+++
/trunk/servlet/test/com/google/inject/servlet/InjectedFilterPipelineTest.java
Sat Jul 31 15:13:21 2010
@@ -86,7 +86,7 @@
.once();
expect(request.getServletPath())
- .andReturn("/public/login.jsp")
+ .andReturn("/non-jsp/login.html") // use a path that will fail in
injector2
.anyTimes();
//at the end, proceed down webapp's normal filter chain
@@ -122,9 +122,13 @@
.andReturn(servletContext)
.once();
expect(request.getServletPath())
- .andReturn("/non-jsp/thing.html")
+ .andReturn("/public/login/login.jsp") // use a path that will
fail in injector1
.anyTimes();
+ //at the end, proceed down webapp's normal filter chain
+ proceedingFilterChain2.doFilter(isA(HttpServletRequest.class),
(ServletResponse) isNull());
+ expectLastCall().once();
+
// Never fire on this pipeline
replay(filterConfig, servletContext, request, proceedingFilterChain2,
proceedingFilterChain);
--
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.
