Closed
Bug 758192
Opened 13 years ago
Closed 13 years ago
Update Marionette exceptions to mirror Selenium Wire Exceptions
Categories
(Remote Protocol :: Marionette, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: automatedtester, Assigned: automatedtester)
Details
Attachments
(1 file)
|
7.20 KB,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
There are a few exceptions that get caught by the catch all that should have their own exceptions like in Selenium
http://code.google.com/p/selenium/source/browse/trunk/py/selenium/webdriver/remote/errorhandler.py#34
| Assignee | ||
Updated•13 years ago
|
Assignee: nobody → dburns
| Assignee | ||
Comment 1•13 years ago
|
||
Attachment #628486 -
Flags: review?(jgriffin)
Comment 2•13 years ago
|
||
Comment on attachment 628486 [details] [diff] [review]
Updating expections
Looks good, thanks!
Attachment #628486 -
Flags: review?(jgriffin) → review+
| Assignee | ||
Comment 3•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•13 years ago
|
||
Just remembered I forgot to update github
landed in https://github.com/mozilla/marionette_client/commit/43923c7ffe08ab343ce484cb9bc35d5668f0c67e
Updated•3 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•