Better error message in tests about how to select a browser to use.

shs96c · shs96c · commit bb07a48543bb · 2017-03-20T12:31:58.000Z
Does not change production code.
diff --git a/java/client/test/org/openqa/selenium/testing/drivers/TestIgnorance.java b/java/client/test/org/openqa/selenium/testing/drivers/TestIgnorance.java
@@ -167,7 +167,9 @@ private boolean isIgnoredDueToEnvironmentVariables(Description method) {
   }
 
   public void setBrowser(Browser browser) {
-    this.browser = checkNotNull(browser, "Browser to use must be set");
+    this.browser = checkNotNull(
+        browser,
+        "Browser to use must be set. Do this by setting the 'selenium.browser' system property");
     addIgnoresForBrowser(browser, ignoreComparator);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,9 @@ private boolean isIgnoredDueToEnvironmentVariables(Description method) {`
`167`	`167`	`}`
`168`	`168`
`169`	`169`	`public void setBrowser(Browser browser) {`
`170`		`- this.browser = checkNotNull(browser, "Browser to use must be set");`
	`170`	`+ this.browser = checkNotNull(`
	`171`	`+ browser,`
	`172`	`+ "Browser to use must be set. Do this by setting the 'selenium.browser' system property");`
`171`	`173`	`addIgnoresForBrowser(browser, ignoreComparator);`
`172`	`174`	`}`
`173`	`175`