Closed Bug 1181037 Opened 10 years ago Closed 10 years ago

"Harness status: OK" + failing tests when running "registration-iframe.https.html" test

Categories

(Testing :: web-platform-tests, defect)

Product:
Testing
Component:
web-platform-tests
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox42 affected, firefox43 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla43
Tracking Flags:
Tracking Status
firefox42 --- affected
firefox43 --- fixed

People

(Reporter: noemi, Assigned: nsm)

References

Details

Attachments

(1 file)

MozReview Request: Bug 1181037 - Use entry settings object's base URL. r?ehsan
40 bytes, text/x-review-board-request
ehsan.akhgari
: review+
Details
Checked with 7/6 master build Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/registration-iframe.https.html| Result: * Harness status: OK * Found 3 tests * 3 Fail: ** Subframe's container's register method should use calling frame's document's url as a base url for parsing its script url and scope url - normal case *** assert_unreached: unexpected rejection: The expression cannot be converted to return the specified type. Reached unreachable code unreached_rejection/<@https://web-platform.test:8443/_mozilla/service-workers/service-worker/resources/test-helpers.sub.js:42:7 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 Test.prototype.step_func/<@https://web-platform.test:8443/resources/testharness.js:1387:1 Promise*@https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:20:3 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 async_test@https://web-platform.test:8443/resources/testharness.js:513:13 @https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:11:1 ** Subframe's container's register method should use calling frame's document's url as a base url for parsing its script url and scope url - error case *** assert_unreached: unexpected rejection: assert_unreached: register() should reject Reached unreachable code Reached unreachable code unreached_rejection/<@https://web-platform.test:8443/_mozilla/service-workers/service-worker/resources/test-helpers.sub.js:42:7 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 Test.prototype.step_func/<@https://web-platform.test:8443/resources/testharness.js:1387:1 Promise*@https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:58:3 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 async_test@https://web-platform.test:8443/resources/testharness.js:513:13 @https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:51:1 ** A scope url should start with the given script url *** assert_unreached: unexpected rejection: assert_equals: expected "SecurityError" but got "TypeError" Reached unreachable code unreached_rejection/<@https://web-platform.test:8443/_mozilla/service-workers/service-worker/resources/test-helpers.sub.js:42:7 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 Test.prototype.step_func/<@https://web-platform.test:8443/resources/testharness.js:1387:1 Promise*@https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:89:3 Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1363:20 async_test@https://web-platform.test:8443/resources/testharness.js:513:13 @https://web-platform.test:8443/_mozilla/service-workers/service-worker/registration-iframe.https.html:82:1 * Traces: https://pastebin.mozilla.org/8838628
Summary: "Harness status: OK" + failing tests when running wpt "registration-iframe.https.html" test → "Harness status: OK" + failing tests when running "registration-iframe.https.html" test
Component: DOM: Service Workers → web-platform-tests
Product: Core → Testing
Bug 1181037 - Use entry settings object's base URL. r?ehsan The second test, which checks for NetworkError is incorrect. [[Update]] step 12 states: "If response is a network error or response's status is not in the range 200 to 299, then: 1. Reject p with a TypeError." which specifically converts the NetworkError (due to 404) to a TypeError. Update web-platform-tests expected data
Comment on attachment 8650018 [details] MozReview Request: Bug 1181037 - Use entry settings object's base URL. r?ehsan Bug 1181037 - Use entry settings object's base URL. r?ehsan The second test, which checks for NetworkError is incorrect. [[Update]] step 12 states: "If response is a network error or response's status is not in the range 200 to 299, then: 1. Reject p with a TypeError." which specifically converts the NetworkError (due to 404) to a TypeError. Update web-platform-tests expected data
Attachment #8650018 - Flags: review?(nsm.nikhil)
Attachment #8650018 - Flags: review?(nsm.nikhil) → review?(ehsan)
Assignee: nobody → nsm.nikhil
Attachment #8650018 - Flags: review?(ehsan) → review+
Comment on attachment 8650018 [details] MozReview Request: Bug 1181037 - Use entry settings object's base URL. r?ehsan https://reviewboard.mozilla.org/r/16527/#review14809 Ship It!
Status: NEW → ASSIGNED
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/ec022b4e9d0b70e6c636ced98f0c5d99a9606eb3 changeset: ec022b4e9d0b70e6c636ced98f0c5d99a9606eb3 user: Nikhil Marathe <nsm.nikhil@gmail.com> date: Thu Aug 20 13:56:37 2015 -0700 description: Bug 1181037 - Use entry settings object's base URL. r=ehsan The second test, which checks for NetworkError is incorrect. [[Update]] step 12 states: "If response is a network error or response's status is not in the range 200 to 299, then: 1. Reject p with a TypeError." which specifically converts the NetworkError (due to 404) to a TypeError. Also fixes controller-on-reload.https.html Update web-platform-tests expected data
https://hg.mozilla.org/mozilla-central/rev/ec022b4e9d0b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox43: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Hi, just checked on m-c (ba43a48d3c52 revision) and the tests successfully run. Thanks for fixing it!. Summary Harness status: OK Found 3 tests 3 Pass Details Result Test Name Pass Subframe's container's register method should use calling frame's document's url as a base url for parsing its script url and scope url - normal case Pass Subframe's container's register method should use calling frame's document's url as a base url for parsing its script url and scope url - error case Pass A scope url should start with the given script url
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: