Refactoring GridLauncher build process

odedniv · Oct 15, 2015 · 392c8bb · 392c8bb
1 parent 4c41e06
commit 392c8bb
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/java/server/src/org/openqa/grid/build.desc b/java/server/src/org/openqa/grid/build.desc
@@ -3,12 +3,14 @@ java_library(name = "grid",
   srcs = [
     "internal/**/*.java",
     "web/**/*.java",
+    "selenium/proxy/**/*.java",
   ],
   embedded = [
     "images/",
   ],
   deps = [
     "//java/client/src/org/openqa/selenium:webdriver-api",
+    "//java/client/src/org/openqa/selenium/chrome",
     "//java/server/src/org/openqa/grid/common",
     "//java/server/src/org/openqa/selenium/remote/server",
     "//third_party/java/httpcomponents:httpclient",

diff --git a/java/server/src/org/openqa/grid/selenium/build.desc b/java/server/src/org/openqa/grid/selenium/build.desc
@@ -1,10 +1,8 @@
 
 java_binary(name = "selenium",
   main_class = "org.openqa.grid.selenium.GridLauncher",
-  srcs = [ "**/*.java" ],
+  srcs = [ "GridLauncher.java" ],
   deps = [
-    "//java/client/src/com/thoughtworks/selenium/webdriven:webdriven",
-    "//java/client/src/org/openqa/selenium/firefox",
     "//java/server/src/org/openqa/selenium/remote/server",
     "//java/server/src/org/openqa/selenium/server",
     "//java/server/src/org/openqa/grid",

diff --git a/java/server/src/org/openqa/grid/selenium/proxy/DefaultRemoteProxy.java b/java/server/src/org/openqa/grid/selenium/proxy/DefaultRemoteProxy.java
@@ -30,7 +30,6 @@
 import org.openqa.grid.internal.listeners.TestSessionListener;
 import org.openqa.grid.internal.listeners.TimeoutListener;
 import org.openqa.grid.internal.utils.HtmlRenderer;
-import org.openqa.grid.selenium.utils.WebProxyHtmlRenderer;
 import org.openqa.selenium.chrome.ChromeOptions;
 import org.openqa.selenium.firefox.FirefoxDriver;
 import org.openqa.selenium.remote.BrowserType;

diff --git a/.../selenium/utils/WebProxyHtmlRenderer.java → .../selenium/proxy/WebProxyHtmlRenderer.java b/.../selenium/utils/WebProxyHtmlRenderer.java → .../selenium/proxy/WebProxyHtmlRenderer.java
@@ -15,7 +15,7 @@
 // specific language governing permissions and limitations
 // under the License.
 
-package org.openqa.grid.selenium.utils;
+package org.openqa.grid.selenium.proxy;
 
 import org.openqa.grid.internal.RemoteProxy;
 import org.openqa.grid.internal.TestSession;