You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by jb...@apache.org on 2013/08/04 20:55:04 UTC
svn commit: r1510310 -
/tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java
Author: jboynes
Date: Sun Aug 4 18:55:04 2013
New Revision: 1510310
URL: http://svn.apache.org/r1510310
Log:
Fix checkstyle
Modified:
tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java
Modified: tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java?rev=1510310&r1=1510309&r2=1510310&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java (original)
+++ tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java Sun Aug 4 18:55:04 2013
@@ -32,16 +32,9 @@ import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;
+import org.easymock.EasyMock;
import org.easymock.IMocksControl;
-import static org.easymock.EasyMock.createMockBuilder;
-import static org.easymock.EasyMock.createStrictControl;
-import static org.easymock.EasyMock.eq;
-import static org.easymock.EasyMock.expect;
-import static org.easymock.EasyMock.expectLastCall;
-import static org.easymock.EasyMock.isA;
-import static org.easymock.EasyMock.same;
-
public class TestWebappServiceLoader {
private static final String CONFIG_FILE =
"META-INF/services/javax.servlet.ServletContainerInitializer";
@@ -53,21 +46,23 @@ public class TestWebappServiceLoader {
@Before
public void init() {
- control = createStrictControl();
+ control = EasyMock.createStrictControl();
parent = control.createMock(ClassLoader.class);
- cl = createMockBuilder(ClassLoader.class)
+ cl = EasyMock.createMockBuilder(ClassLoader.class)
.withConstructor(parent)
.addMockedMethod("loadClass", String.class)
.createMock(control);
context = control.createMock(ServletContext.class);
- expect(context.getClassLoader()).andStubReturn(cl);
+ EasyMock.expect(context.getClassLoader()).andStubReturn(cl);
}
@Test
public void testNoInitializersFound() throws IOException {
loader = new WebappServiceLoader<>(context);
- expect(context.getAttribute(ServletContext.ORDERED_LIBS)).andReturn(null);
- expect(cl.getResources(CONFIG_FILE)).andReturn(Collections.<URL>emptyEnumeration());
+ EasyMock.expect(context.getAttribute(ServletContext.ORDERED_LIBS))
+ .andReturn(null);
+ EasyMock.expect(cl.getResources(CONFIG_FILE))
+ .andReturn(Collections.<URL>emptyEnumeration());
control.replay();
Assert.assertTrue(loader.load(ServletContainerInitializer.class).isEmpty());
control.verify();
@@ -77,13 +72,14 @@ public class TestWebappServiceLoader {
@SuppressWarnings("unchecked")
public void testInitializerFromClasspath() throws IOException {
URL url = new URL("file://test");
- loader = createMockBuilder(WebappServiceLoader.class)
+ loader = EasyMock.createMockBuilder(WebappServiceLoader.class)
.addMockedMethod("parseConfigFile", Set.class, URL.class)
.withConstructor(context).createMock(control);
- expect(context.getAttribute(ServletContext.ORDERED_LIBS)).andReturn(null);
- expect(cl.getResources(CONFIG_FILE))
+ EasyMock.expect(context.getAttribute(ServletContext.ORDERED_LIBS))
+ .andReturn(null);
+ EasyMock.expect(cl.getResources(CONFIG_FILE))
.andReturn(Collections.enumeration(Collections.singleton(url)));
- loader.parseConfigFile(isA(Set.class), same(url));
+ loader.parseConfigFile(EasyMock.isA(Set.class), EasyMock.same(url));
control.replay();
Assert.assertTrue(loader.load(ServletContainerInitializer.class).isEmpty());
control.verify();
@@ -96,16 +92,20 @@ public class TestWebappServiceLoader {
URL sci1 = new URL("jar:file://jar1.jar!/" + CONFIG_FILE);
URL url2 = new URL("file://dir/");
URL sci2 = new URL("file://dir/" + CONFIG_FILE);
- loader = createMockBuilder(WebappServiceLoader.class)
+ loader = EasyMock.createMockBuilder(WebappServiceLoader.class)
.addMockedMethod("parseConfigFile", Set.class, URL.class)
.withConstructor(context).createMock(control);
List<String> jars = Arrays.asList("jar1.jar", "dir/");
- expect(context.getAttribute(ServletContext.ORDERED_LIBS)).andReturn(jars);
- expect(context.getResource("/WEB-INF/lib/jar1.jar")).andReturn(url1);
- loader.parseConfigFile(isA(Set.class), eq(sci1));
- expect(context.getResource("/WEB-INF/lib/dir/")).andReturn(url2);
- loader.parseConfigFile(isA(Set.class), eq(sci2));
- expect(parent.getResources(CONFIG_FILE)).andReturn(Collections.<URL>emptyEnumeration());
+ EasyMock.expect(context.getAttribute(ServletContext.ORDERED_LIBS))
+ .andReturn(jars);
+ EasyMock.expect(context.getResource("/WEB-INF/lib/jar1.jar"))
+ .andReturn(url1);
+ loader.parseConfigFile(EasyMock.isA(Set.class), EasyMock.eq(sci1));
+ EasyMock.expect(context.getResource("/WEB-INF/lib/dir/"))
+ .andReturn(url2);
+ loader.parseConfigFile(EasyMock.isA(Set.class), EasyMock.eq(sci2));
+ EasyMock.expect(parent.getResources(CONFIG_FILE))
+ .andReturn(Collections.<URL>emptyEnumeration());
control.replay();
Assert.assertTrue(loader.load(ServletContainerInitializer.class).isEmpty());
@@ -125,7 +125,8 @@ public class TestWebappServiceLoader {
Class<?> sci = TesterServletContainerInitializer1.class;
loader = new WebappServiceLoader<>(context);
cl.loadClass(sci.getName());
- expectLastCall().andReturn(sci);
+ EasyMock.expectLastCall()
+ .andReturn(sci);
Set<String> names = Collections.singleton(sci.getName());
control.replay();
Collection<ServletContainerInitializer> initializers =
@@ -140,7 +141,8 @@ public class TestWebappServiceLoader {
Class<?> sci = Object.class;
loader = new WebappServiceLoader<>(context);
cl.loadClass(sci.getName());
- expectLastCall().andReturn(sci);
+ EasyMock.expectLastCall()
+ .andReturn(sci);
Set<String> names = Collections.singleton(sci.getName());
control.replay();
try {
@@ -157,7 +159,8 @@ public class TestWebappServiceLoader {
Class<?> sci = Integer.class;
loader = new WebappServiceLoader<>(context);
cl.loadClass(sci.getName());
- expectLastCall().andReturn(sci);
+ EasyMock.expectLastCall()
+ .andReturn(sci);
Set<String> names = Collections.singleton(sci.getName());
control.replay();
try {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
Re: svn commit: r1510310 - /tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java
Posted by Jeremy Boynes <jb...@apache.org>.
On Aug 4, 2013, at 11:55 AM, jboynes@apache.org wrote:
> Author: jboynes
> Date: Sun Aug 4 18:55:04 2013
> New Revision: 1510310
>
> URL: http://svn.apache.org/r1510310
This may also have broken a clean build which I didn't catch as my IDE had copied service-config.txt to the output for me.
I've got a patch for build.xml to copy it and am running a verification test for it at the moment before committing it.