Closed
Bug 1513023
Opened 5 years ago
Closed 5 years ago
Reloading single test quits the browser instead of rerunning it
Categories
(Testing :: web-platform-tests, defect)
Tracking
(firefox-esr60 unaffected, firefox64 wontfix, firefox65 fixed, firefox66 fixed)
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox64 | --- | wontfix |
firefox65 | --- | fixed |
firefox66 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: jgraham)
Details
(Keywords: regression)
Attachments
(1 file)
STEPS TO REPRODUCE: 1) Build Firefox. 2) mach wpt testing/web-platform/tests/css/selectors/attribute-selectors/attribute-case/semantics.html 3) Wait for test to finish and be sitting there showing results. 4) Edit the test file. 5) Reload the test. EXPECTED RESULTS: Reruns the test with the modified file. ACTUAL RESULTS: Quits the browser. The actual test doesn't matter much; the above is what I happened to be running today.
Assignee | ||
Comment 1•5 years ago
|
||
When wpt is used by developers they often want to reload a test that's running to allow iterative development (or running under devtools, etc.). The recent switch to target marionette commands at the test window rather than the opener window broke this as waiting is implemented as an async script that never returns (i.e. a promise that never resolves) and when this is in the test window navigating that window causes the script to return. A simple, but not particuarly elegant, solution here is to simply rerun the script in the new window.
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/autoland/rev/0a36f92c457a Wait for wpt tests to finish across reloads, r=ato
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/14441 for changes under testing/web-platform/tests
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0a36f92c457a
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Updated•5 years ago
|
Assignee: nobody → james
status-firefox64:
--- → wontfix
status-firefox65:
--- → affected
status-firefox-esr60:
--- → unaffected
Whiteboard: [checkin-needed-beta]
Can't merge web-platform-tests PR due to failing upstream checks: Github PR https://github.com/web-platform-tests/wpt/pull/14441 * continuous-integration/travis-ci/pr (https://travis-ci.org/web-platform-tests/wpt/builds/466204382?utm_source=github_status&utm_medium=notification)
Upstream PR merged
Comment 7•5 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/66f0ce8d3d64
Whiteboard: [checkin-needed-beta]
You need to log in
before you can comment on or make changes to this bug.
Description
•