Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/getregistrations.sub.html| Actual Result: Harness status: Error ReferenceError: sequential_promise_test is not defined Found 0 tests Traces: https://pastebin.mozilla.org/8838608

Bug 1189671 - Fix getregistrations test. r?jgraham Moves the test to .https so it actually works. Switches to using promise_test like the current blink test since sequential promise is not implemented and not needed. compares registrations by scope since getRegistrations() produces new objects every time.

Comment on attachment 8651311 [details] MozReview Request: Bug 1189671 - Fix getregistrations test. r?jgraham https://reviewboard.mozilla.org/r/16899/#review15201 ::: testing/web-platform/mozilla/tests/service-workers/service-worker/getregistrations.https.html:96 (Diff revision 1) > // Top-level window's origin is http://{{domains[www]}}:{{ports[http][0]}} Since you removed the .sub. remove these comments here and elsewhere. ::: testing/web-platform/mozilla/tests/service-workers/service-worker/getregistrations.https.html:100 (Diff revision 1) > - '/service-worker/resources/frame-for-getregistrations.sub.html'; > + '/_mozilla/service-workers/service-worker/resources/frame-for-getregistrations.html'; You can't put _mozilla in paths if we want to upstream these tests. I guess the right option is to add some substitution for the path prefix, which might require an upstream change in wptserve. If that's the route we decide to go I'm OK with landing this change temporarily as long as you file a followup issue to remove the _mozilla once I land the upstream change (or, for bonus points, make the upstream change yourself :)

I changed the test to calculate the URL based on the current URL to obviate the second issue.

https://hg.mozilla.org/integration/mozilla-inbound/rev/d09c2c24624fe52c46b9059aa93a446360ee4d62 Bug 1189671 - Fix getregistrations test. r=jgraham

Hi, |./mach web-platform-tests _mozilla/service-workers/service-worker/getregistrations.https.html| just checked on m-c (001942e4617b revision) and the test successfully runs. Thanks for fixing it! Summary Harness status: OK Found 6 tests 6 Pass Details Result Test Name Pass Purge the existing registrations. Pass getRegistrations Pass Register then getRegistrations Pass Register multiple times then getRegistrations Pass Register then Unregister then getRegistrations Pass getRegistrations promise resolves only with same origin registrations.