1453084 - Provide string representations for webdriver.Session and webdriver.Element

Status: RESOLVED FIXED

(Testing :: web-platform-tests, enhancement, P3)

Description

[Andreas Tolfsen ❲:ato❳]

There are a couple of important classes in the webdriver Python
package used in WPT that are missing string representations.  Since
pytest logs their representations when a test fails, it would help
debugging if we provided some useful representations.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1453084 - Add webdriver.Session string representation.

Review commit: https://reviewboard.mozilla.org/r/235408/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/235408/

Comment 2

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1453084 - Add webdriver.Element string representation.

Review commit: https://reviewboard.mozilla.org/r/235410/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/235410/

Comment 3

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

Comment on attachment 8966704 [details]
Bug 1453084 - Add webdriver.Element string representation.

https://reviewboard.mozilla.org/r/235410/#review241280

Comment 4

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

Comment on attachment 8966703 [details]
Bug 1453084 - Add webdriver.Session string representation.

https://reviewboard.mozilla.org/r/235408/#review241282

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/097b57bbeb46
Add webdriver.Session string representation. r=whimboo
https://hg.mozilla.org/integration/autoland/rev/bdb78cb3d6d3
Add webdriver.Element string representation. r=whimboo

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/097b57bbeb46
https://hg.mozilla.org/mozilla-central/rev/bdb78cb3d6d3

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/w3c/web-platform-tests/pull/10432 for changes under testing/web-platform/tests

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged