Add a test case to allow the Core tests to run

This exercises the HTMLLauncher.

Author: shs96c

java/client/test/com/thoughtworks/selenium/BUCK

@@ -54,4 +54,7 @@ java_library(
     '//third_party/java/mockito:mockito',
     '//third_party/java/servlet:servlet-api',
   ],
+  visibility = [
+    '//java/server/test/org/openqa/selenium/server/htmlrunner:htmlrunner',
+  ],
 )

java/server/src/org/openqa/selenium/server/htmlrunner/BUCK

@@ -24,5 +24,6 @@ java_library(
   ],
   visibility = [
     '//java/server/src/com/thoughtworks/selenium:leg-rc',
+    '//java/server/test/org/openqa/selenium/...',
   ],
 )

java/server/src/org/openqa/selenium/server/htmlrunner/CoreTestCase.java

@@ -47,9 +47,12 @@ public void run(Results results, WebDriver driver, Selenium selenium) {
     for (CoreTestStep step : steps) {
       try {
         step.run(results, driver, selenium);
+        Thread.sleep(2000);
       } catch (SeleniumException e) {
         results.addTestFailure();
         return;
+      } catch (InterruptedException e) {
+        e.printStackTrace();
       }
     }
   }

java/server/src/org/openqa/selenium/server/htmlrunner/HTMLLauncher.java

@@ -158,7 +158,9 @@ private String runHTMLSuite(String browser, String browserURL, String suiteURL,
         }
       }
 
-      driver.quit();
+      if (driver != null) {
+        driver.quit();
+      }
     }
 
 //    if (results == null) {

java/server/test/org/openqa/selenium/server/htmlrunner/BUCK

@@ -0,0 +1,12 @@
+java_test(
+  name = 'htmlrunner',
+  srcs = glob(['*.java']),
+  deps = [
+    '//java/client/src/org/openqa/selenium:selenium',
+    '//java/client/test/com/thoughtworks/selenium:tests',
+    '//java/client/test/org/openqa/selenium/environment:environment',
+    '//java/server/src/org/openqa/selenium/server/htmlrunner:htmlrunner',
+    '//third_party/java/guava:guava',
+    '//third_party/java/junit:junit',
+  ],
+)

java/server/test/org/openqa/selenium/server/htmlrunner/CoreSelfTest.java

@@ -0,0 +1,97 @@
+// Licensed to the Software Freedom Conservancy (SFC) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The SFC licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+package org.openqa.selenium.server.htmlrunner;
+
+import static org.junit.Assert.assertTrue;
+
+import com.google.common.collect.ImmutableSortedSet;
+
+import com.thoughtworks.selenium.testing.SeleniumAppServer;
+
+import org.junit.AfterClass;
+import org.junit.BeforeClass;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+import org.openqa.selenium.Platform;
+import org.openqa.selenium.environment.webserver.AppServer;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.concurrent.TimeUnit;
+
+@RunWith(Parameterized.class)
+public class CoreSelfTest {
+
+  private final String browser;
+  private static AppServer server;
+
+  public CoreSelfTest(String browser) {
+    this.browser = browser;
+  }
+
+  @BeforeClass
+  public static void startTestServer() {
+    server = new SeleniumAppServer();
+    server.start();
+  }
+
+  @AfterClass
+  public static void stopTestServer() {
+    server.stop();
+  }
+
+  @Test
+  public void executeTests() throws IOException {
+    String testBase = server.whereIs("/selenium-server/tests");
+    File outputFile = File.createTempFile("core-test-suite", browser.replace('*', '-') + ".txt");
+    assertTrue(outputFile.delete());
+
+    new HTMLLauncher().runHTMLSuite(browser,
+                    // We need to do this because the path relativizing code in java.net.URL is
+                    // clearly having a bad day. "/selenium-server/tests" appended to "../tests/"
+                    // ends up as "/tests" rather than "/selenium-server/tests" as you'd expect.
+                    testBase + "/TestSuite.html",
+                    testBase + "/TestSuite.html",
+                    outputFile,
+                    TimeUnit.MINUTES.toMillis(5),
+                    true);
+  }
+
+  @Parameterized.Parameters
+  public static Iterable<String> parameters() {
+    ImmutableSortedSet.Builder<String> browsers = ImmutableSortedSet.naturalOrder();
+
+    //    if (CommandLine.find("wires") != null) {
+    browsers.add("*firefox");
+    //    }
+
+    switch (Platform.getCurrent().family()) {
+      case MAC:
+        //        browsers.add("*safari");
+        break;
+
+      case WINDOWS:
+        browsers.add("*iexplore");
+        break;
+    }
+
+    System.out.println("browsers.build() = " + browsers.build());
+    return browsers.build();
+  }
+}