Bug 1407000 - Fix error for string WebDriver capabilities. r=automatedtester
authorAndreas Tolfsen <ato@sny.no>
Mon, 09 Oct 2017 19:05:59 +0100
changeset 385311 3bcd6572f47dcd2960b7046a143dd22bfa58e9d7
parent 385310 529092fc92477be7accc8b604ea9c8812dafa969
child 385312 5caa7a4686ab573a554c5d3ed503782cb05b7d0d
push id32652
push userarchaeopteryx@coole-files.de
push dateTue, 10 Oct 2017 21:49:31 +0000
treeherdermozilla-central@f1ecd5c26948 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersautomatedtester
bugs1407000
milestone58.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1407000 - Fix error for string WebDriver capabilities. r=automatedtester MozReview-Commit-ID: I4Il8CoUBiz
testing/webdriver/src/capabilities.rs
--- a/testing/webdriver/src/capabilities.rs
+++ b/testing/webdriver/src/capabilities.rs
@@ -88,17 +88,17 @@ impl SpecNewSessionParameters {
                 "acceptInsecureCerts" => if !value.is_boolean() {
                         return Err(WebDriverError::new(ErrorStatus::InvalidArgument,
                                                        "acceptInsecureCerts was not a boolean"))
                     },
                 x @ "browserName" |
                 x @ "browserVersion" |
                 x @ "platformName" => if !value.is_string() {
                         return Err(WebDriverError::new(ErrorStatus::InvalidArgument,
-                                                       format!("{} was not a boolean", x)))
+                                                       format!("{} is not a string", x)))
                     },
                 "pageLoadStrategy" => {
                     try!(SpecNewSessionParameters::validate_page_load_strategy(value))
                 }
                 "proxy" => {
                     try!(SpecNewSessionParameters::validate_proxy(value))
                 },
                 "timeouts" => {