Author: kkolinko
Date: Sun Jan 17 21:02:02 2016
New Revision: 1725127
URL: http://svn.apache.org/viewvc?rev=1725127&view=rev
Log:
Simplify code by importing a more popular class.
javax.naming.Context vs. org.apache.catalina.Context
The first is used only twice, the second is used a dozen of times.
Modified:
tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java
Modified: tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java?rev=1725127&r1=1725126&r2=1725127&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java (original)
+++ tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java Sun Jan 17
21:02:02 2016
@@ -25,7 +25,6 @@ import java.net.URL;
import java.net.URLConnection;
import java.util.concurrent.atomic.AtomicInteger;
-import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
import javax.servlet.ServletException;
@@ -42,6 +41,7 @@ import static org.junit.Assert.fail;
import org.junit.Test;
+import org.apache.catalina.Context;
import org.apache.catalina.Host;
import org.apache.catalina.core.StandardContext;
import org.apache.catalina.core.StandardHost;
@@ -99,8 +99,9 @@ public class TestTomcat extends TomcatBa
String name = null;
try {
- Context initCtx = new InitialContext();
- Context envCtx = (Context) initCtx.lookup("java:comp/env");
+ javax.naming.Context initCtx = new InitialContext();
+ javax.naming.Context envCtx =
+ (javax.naming.Context) initCtx.lookup("java:comp/env");
name = (String) envCtx.lookup(JNDI_ENV_NAME);
} catch (NamingException e) {
throw new IOException(e);
@@ -186,7 +187,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext("", null);
+ Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "myServlet", new HelloWorld());
ctx.addServletMapping("/", "myServlet");
@@ -203,7 +204,7 @@ public class TestTomcat extends TomcatBa
File appDir = new File(getBuildDirectory(), "webapps/examples");
// app dir is relative to server home
- org.apache.catalina.Context ctxt = tomcat.addWebapp(
+ Context ctxt = tomcat.addWebapp(
null, "/examples", appDir.getAbsolutePath());
ctxt.addApplicationListener(WsContextListener.class.getName());
tomcat.start();
@@ -220,7 +221,7 @@ public class TestTomcat extends TomcatBa
File appDir = new File(getBuildDirectory(), "webapps/examples");
// app dir is relative to server home
- org.apache.catalina.Context ctxt = tomcat.addWebapp(
+ Context ctxt = tomcat.addWebapp(
null, "/examples", appDir.getAbsolutePath());
ctxt.addApplicationListener(WsContextListener.class.getName());
@@ -237,7 +238,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext("", null);
+ Context ctx = tomcat.addContext("", null);
Tomcat.addServlet(ctx, "myServlet", new HelloWorldSession());
ctx.addServletMapping("/", "myServlet");
@@ -267,7 +268,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext("", null);
+ Context ctx = tomcat.addContext("", null);
// Enable JNDI - it is disabled by default
tomcat.enableNaming();
@@ -295,7 +296,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext("", null);
+ Context ctx = tomcat.addContext("", null);
// Enable JNDI - it is disabled by default
tomcat.enableNaming();
@@ -332,7 +333,7 @@ public class TestTomcat extends TomcatBa
File appDir = new File(getBuildDirectory(), "webapps" + contextPath);
// app dir is relative to server home
- org.apache.catalina.Context ctx =
+ Context ctx =
tomcat.addWebapp(null, "/examples", appDir.getAbsolutePath());
ctx.addApplicationListener(WsContextListener.class.getName());
@@ -373,7 +374,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext("", null);
+ Context ctx = tomcat.addContext("", null);
InitCount initCount = new InitCount();
Tomcat.addServlet(ctx, "initCount", initCount);
@@ -429,7 +430,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
File appFile = new File("test/deployment/context.war");
- org.apache.catalina.Context context = tomcat.addWebapp(null,
+ Context context = tomcat.addWebapp(null,
"/test", appFile.getAbsolutePath());
assertEquals(StandardContext.class.getName(), context.getClass()
@@ -463,7 +464,7 @@ public class TestTomcat extends TomcatBa
}
File appFile = new File("test/deployment/context.war");
- org.apache.catalina.Context context = tomcat.addWebapp(null, "/test",
+ Context context = tomcat.addWebapp(null, "/test",
appFile.getAbsolutePath());
assertEquals(ReplicatedContext.class.getName(), context.getClass()
@@ -480,7 +481,7 @@ public class TestTomcat extends TomcatBa
}
File appFile = new File("test/deployment/context.war");
- org.apache.catalina.Context context = tomcat.addWebapp(host, "/test",
+ Context context = tomcat.addWebapp(host, "/test",
appFile.getAbsolutePath());
assertEquals(ReplicatedContext.class.getName(), context.getClass()
@@ -492,7 +493,7 @@ public class TestTomcat extends TomcatBa
Tomcat tomcat = getTomcatInstance();
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext(null, "", null);
+ Context ctx = tomcat.addContext(null, "", null);
assertEquals(StandardContext.class.getName(),
ctx.getClass().getName());
}
@@ -523,7 +524,7 @@ public class TestTomcat extends TomcatBa
}
// No file system docBase required
- org.apache.catalina.Context ctx = tomcat.addContext(host, "", null);
+ Context ctx = tomcat.addContext(host, "", null);
assertEquals(ReplicatedContext.class.getName(), ctx.getClass()
.getName());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
