Closed
Bug 1181056
Opened 10 years ago
Closed 10 years ago
"Harness status: OK" + failing test when running "extendable-event-async-waituntil.https.html" test
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox42 affected, firefox43 fixed)
RESOLVED
FIXED
mozilla43
People
(Reporter: noemi, Assigned: nsm)
References
Details
Attachments
(1 file)
|
3.01 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
Checked with 7/6 master build
Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/extendable-event-async-waituntil.https.html|
Result:
* Harness status: OK
* Found 1 tests
* 1 Fail:
** Calling waitUntil asynchronously throws an exception
*** assert_equals: expected "PASS" but got "FAIL: did not throw."
@https://web-platform.test:8443/_mozilla/service-workers/service-worker/extendable-event-async-waituntil.https.html:26:11 Promise*@https://web-platform.test:8443/_mozilla/service-workers/service-worker/extendable-event-async-waituntil.https.html:12:12 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 promise_test@https://web-platform.test:8443/resources/testharness.js:520:25 @https://web-platform.test:8443/_mozilla/service-workers/service-worker/extendable-event-async-waituntil.https.html:7:1
* Traces: https://pastebin.mozilla.org/8838639
|
Reporter | |
Updated•10 years ago
|
Summary: "Harness status: OK" + failing test when running wpt "extendable-event-async-waituntil.https.html" test → "Harness status: OK" + failing test when running "extendable-event-async-waituntil.https.html" test
|
|
Reporter | |
Updated•10 years ago
|
Component: DOM: Service Workers → web-platform-tests
Product: Core → Testing
|
Assignee | |
Comment 1•10 years ago
|
||
Olli, This should match "If the dispatch flag is unset..." at http://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#wait-until-method
Attachment #8649578 -
Flags: review?(bugs)
|
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → nsm.nikhil
|
|
Reporter | |
Updated•10 years ago
|
Status: NEW → ASSIGNED
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8649578 [details] [diff] [review]
waitUntil() should throw if called when event is not dispatching.
nsIDOMEvent::NONE or some such might be a bit more clear.
Attachment #8649578 -
Flags: review?(bugs) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
|
|
Assignee | |
Comment 4•10 years ago
|
||
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/a9ae8b90c678e8ed81bbc7b71c8c7489066f3a22
changeset: a9ae8b90c678e8ed81bbc7b71c8c7489066f3a22
user: Nikhil Marathe <nsm.nikhil@gmail.com>
date: Tue Aug 18 16:28:04 2015 -0700
description:
Bug 1181056 - waitUntil() should throw if called when event is not dispatching. r=smaug
|
||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
|
|
Reporter | |
Comment 6•10 years ago
|
||
Hi,
just checked on m-c (ba43a48d3c52 revision) and the test successfully runs. Thanks for fixing it!.
Summary
Harness status: OK
Found 1 tests
1 Pass
Details
Result Test Name
Pass Calling waitUntil asynchronously throws an exception
You need to log in
before you can comment on or make changes to this bug.
Description
•