Skip to content

Commit 62c0fe6

Browse files
committed
Renaming the Jetty7AppServer to remove the version number.
It's enough to know it uses Jetty.
1 parent e0b55cf commit 62c0fe6

File tree

4 files changed

+11
-12
lines changed

4 files changed

+11
-12
lines changed

java/client/test/com/thoughtworks/selenium/testing/SeleniumAppServer.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@
1919

2020
import java.io.File;
2121

22-
import org.openqa.selenium.environment.webserver.Jetty7AppServer;
22+
import org.openqa.selenium.environment.webserver.JettyAppServer;
2323
import org.openqa.selenium.testing.InProject;
2424
import org.seleniumhq.jetty7.servlet.ServletContextHandler;
2525

26-
public class SeleniumAppServer extends Jetty7AppServer {
26+
public class SeleniumAppServer extends JettyAppServer {
2727

2828
private static final String RC_CONTEXT_PATH = "/selenium-server";
2929

@@ -45,7 +45,7 @@ protected String getMainContextPath(String relativeUrl) {
4545
}
4646

4747
public static void main(String[] args) {
48-
Jetty7AppServer server = new SeleniumAppServer();
48+
JettyAppServer server = new SeleniumAppServer();
4949

5050
server.listenOn(getHttpPortFromEnv());
5151
System.out.println(String.format("Starting server on port %d", getHttpPortFromEnv()));

java/client/test/org/openqa/selenium/environment/InProcessTestEnvironment.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@
1818
package org.openqa.selenium.environment;
1919

2020
import org.openqa.selenium.environment.webserver.AppServer;
21-
import org.openqa.selenium.environment.webserver.Jetty7AppServer;
21+
import org.openqa.selenium.environment.webserver.JettyAppServer;
2222

2323
public class InProcessTestEnvironment implements TestEnvironment {
2424

2525
private AppServer appServer;
2626

2727
public InProcessTestEnvironment() {
28-
appServer = new Jetty7AppServer();
28+
appServer = new JettyAppServer();
2929
appServer.start();
3030
}
3131

java/client/test/org/openqa/selenium/environment/webserver/Jetty7AppServer.java renamed to java/client/test/org/openqa/selenium/environment/webserver/JettyAppServer.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
import javax.servlet.Filter;
3838
import javax.servlet.Servlet;
3939

40-
public class Jetty7AppServer implements AppServer {
40+
public class JettyAppServer implements AppServer {
4141

4242
private static final String HOSTNAME_FOR_TEST_ENV_NAME = "HOSTNAME";
4343
private static final String ALTERNATIVE_HOSTNAME_FOR_TEST_ENV_NAME = "ALTERNATIVE_HOSTNAME";
@@ -60,7 +60,7 @@ public class Jetty7AppServer implements AppServer {
6060
private ContextHandlerCollection handlers;
6161
private final String hostName;
6262

63-
public Jetty7AppServer() {
63+
public JettyAppServer() {
6464
this(detectHostname());
6565
}
6666

@@ -69,7 +69,7 @@ public static String detectHostname() {
6969
return hostnameFromProperty == null ? "localhost" : hostnameFromProperty;
7070
}
7171

72-
public Jetty7AppServer(String hostName) {
72+
public JettyAppServer(String hostName) {
7373
this.hostName = hostName;
7474
// Be quiet. Unless we want things to be chatty
7575
if (!Boolean.getBoolean("webdriver.debug")) {
@@ -258,7 +258,7 @@ protected static int getHttpsPortFromEnv() {
258258
}
259259

260260
public static void main(String[] args) {
261-
Jetty7AppServer server = new Jetty7AppServer(detectHostname());
261+
JettyAppServer server = new JettyAppServer(detectHostname());
262262

263263
server.listenOn(getHttpPortFromEnv());
264264
System.out.println(String.format("Starting server on port %d", getHttpPortFromEnv()));

java/client/test/org/openqa/selenium/environment/webserver/build.desc

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,11 @@ java_library(
33
srcs = [
44
"AppServer.java",
55

6-
"Jetty7AppServer.java",
76
"BasicAuth.java",
87
"CookieServlet.java",
98
"EncodingServlet.java",
109
"GeneratedJsTestServlet.java",
11-
"Jetty7AppServer.java",
10+
"JettyAppServer.java",
1211
"KillSwitchServlet.java",
1312
"ManifestServlet.java",
1413
"Utf8Servlet.java",
@@ -44,7 +43,7 @@ java_library(
4443

4544
java_binary(name = "WebServer",
4645
deps = [":webserver"],
47-
main_class = "org.openqa.selenium.environment.webserver.Jetty7AppServer")
46+
main_class = "org.openqa.selenium.environment.webserver.JettyAppServer")
4847

4948
java_test(name = "webserver_test",
5049
srcs = [

0 commit comments

Comments
 (0)