1140791 - Run fetch tests on main thread too

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Right now most tests are only run on workers, they should be run in both threads. This should be pretty easy with some refactoring.

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Attachment: Run fetch tests on main thread and workers

Unified all tests into dom/tests/mochitest/fetch
Moved test runner into fetch_test_framework. The common header tests also use this framework, while the main thread tests now have their own html file.

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8576277 [details] [diff] [review]
Run fetch tests on main thread and workers

Redirecting to Ehsan as we need a DOM peer to sign off anyways and he has been looking at this kind of test framework stuff recently.

Comment 3

[(no longer active)]

What features does your framework provide, Nikhil?  I have been developing my own little framework for cache too (it's https://dxr.mozilla.org/mozilla-central/source/dom/cache/test/mochitest/driver.js) that knows how to run a test in a worker, a service worker and a window, and I'm extending it to do exciting things such as interleave the tests run in the three threads in bug 1142772.  Maybe we should consider unifying them?

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

My 'framework' is no framework. It just loads a script in 1) main thread 2) a worker, calls a function on them, and waits for the promise to resolve. Nothing to do with serviceworkers. I guess I could use some part of it, but this is a 20 line test runner and I'd rather not spend the time right now figuring out how to integrate with the cache driver. I particularly do not want interleaving since the fetch tests test for cookies which need ordering.

Comment 5

[(no longer active)]

OK fair enough!

Comment 6

[(no longer active)]

Comment on attachment 8576277 [details] [diff] [review]
Run fetch tests on main thread and workers

Review of attachment 8576277 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/tests/mochitest/fetch/fetch_test_framework.js
@@ +5,5 @@
> +      worker.onmessage = function(event) {
> +        if (event.data.type == 'finish') {
> +          resolve();
> +        } else if (event.data.type == 'status') {
> +          ok(event.data.status, event.data.msg);

Nit: please prepend "Worker: " or some such to the message here.

@@ +9,5 @@
> +          ok(event.data.status, event.data.msg);
> +        }
> +      }
> +      worker.onerror = function(event) {
> +      info("Got worker error!");

Nit: indentation.

Comment 7

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e50a44b381ff

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e50a44b381ff