1150522 - Support string based statuses in the Marionette Python client

Status: RESOLVED FIXED

(Testing :: Marionette Client and Harness, defect)

Description

[Andreas Tolfsen ❲:ato❳]

In preparation for replacing number-based status codes in the Marionette server (bug 945729) we need to add support for the new string-based errors as described in the W3C WebDriver protocol.

This is the full list of errors: https://w3c.github.io/webdriver/webdriver-spec.html#dfn-error-code

The plan is to add in support for looking up exception classes by string as well as by number, then land the change to the remote end that removes number codes, then remove number codes completely from the local end.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: MozReview Request: bz://1150522/ato

/r/6555 - Bug 1150522: Add WebDriver string statuses to Marionette client

Pull down this commit:

hg pull -r 230060c0f237afbe86dd04159e6138e0a69b1b9f https://reviewboard-hg.mozilla.org/gecko/

Comment 2

[Andreas Tolfsen ❲:ato❳]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=66a588b1c7c9

Comment 3

[David Burns :automatedtester]

Comment on attachment 8587585 [details]
MozReview Request: bz://1150522/ato

https://reviewboard.mozilla.org/r/6553/#review5443

Ship It!

::: testing/marionette/driver/marionette_driver/errors.py
(Diff revision 1)
> +    MarionetteException,

nit: have these line up all together and should be 4 spaces

Comment 4

[Andreas Tolfsen ❲:ato❳]

Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/853bf8d3a270

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/853bf8d3a270

Comment 7

[Andreas Tolfsen ❲:ato❳]

Attachment: MozReview Request: Bug 1150522: Add WebDriver string statuses to Marionette client

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Moving bugs for Marionette client due to component changes.