1477978 - Make WPT session.end() swallow SessionNotCreatedException

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, enhancement)

Description

[Andreas Tolfsen ❲:ato❳]

See discussion in https://github.com/web-platform-tests/wpt/pull/12004
for more details.

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Swallow SessionNotCreatedException on session.end().

Subsequent calls to session.end() should not result in
SessionNotCreatedException being raised.

Comment 2

[Andreas Tolfsen ❲:ato❳]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2761e07798f1f677b45b949b62d10ba82471d99a

Comment 3

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

Will there be a way to stop the sync bug 1476131 from marking this test as expected fail? I mean when this patch lands before the expectation will be wrong again.

Comment 4

[James Graham [:jgraham]]

We redo the metadata asap before landing, so if something changed in the meantime that marks the test as pass, we will get that right (unless we are unlucky and the change happens between the metadata update and the patch landing, but that's unavoidable).

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/59ab6acdee53
Swallow SessionNotCreatedException on session.end(). r=whimboo

Comment 6

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

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

Comment 7

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

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/59ab6acdee53

Comment 9

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

Upstream PR merged