Andreas Tolfsen ❲:ato❳ Assignee
	Description • 7 years ago

As can be seen from https://github.com/mozilla/geckodriver/issues/633, the error messages from the Set Timeouts command are not obvious.  It first look as the type of the value, but a more useful error message would explain that it is the key, or field, that is not recognised.

We can rectify this in Marionette by reordering the assertions.

	Henrik Skupin [:whimboo][⌚️UTC+1]
	Comment 2 • 7 years ago
mozreview-review

Comment on attachment 8856975 [details]
Bug 1355471 - Test timeout field before value's typing;

https://reviewboard.mozilla.org/r/128886/#review131426

	Pulsebot
	Comment 3 • 7 years ago

Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/64fa27c7e727
Test timeout field before value's typing; r=whimboo

	Iris Hsiao [:ihsiao]
	Comment 4 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/64fa27c7e727

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 5 • 7 years ago

Sheriffs: Please uplift to Aurora and Beta as test-only.

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 7 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-aurora/rev/999732bc0d64

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 7 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/37076febf926
https://hg.mozilla.org/releases/mozilla-release/rev/37076febf926