Closed Bug 1150522 Opened 9 years ago Closed 9 years ago

Support string based statuses in the Marionette Python client

Categories

(Testing :: Marionette Client and Harness, defect)

defect
Not set
normal

Tracking

(firefox40 fixed)

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: ato, Assigned: ato)

References

Details

(Keywords: pi-marionette-client)

Attachments

(1 file, 1 obsolete file)

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.
Blocks: 945729
Assignee: nobody → ato
Status: NEW → ASSIGNED
Blocks: 1150527
Attached file MozReview Request: bz://1150522/ato (obsolete) —
/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/
Attachment #8587585 - Flags: review?(dburns)
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
Attachment #8587585 - Flags: review?(dburns) → review+
https://hg.mozilla.org/mozilla-central/rev/853bf8d3a270
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Attachment #8587585 - Attachment is obsolete: true
Attachment #8619955 - Flags: review+
Product: Testing → Remote Protocol

Moving bugs for Marionette client due to component changes.

Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: