359994 - load Tp2 pageset list dynamically

Status: RESOLVED WONTFIX

(Release Engineering :: General, enhancement)

Description

[Robert Helmer [:rhelmer]]

It should be possible for Tp2 to load a list of pages to test dynamically, instead of having to hardcode one pageset list into cycler.html

Comment 1

[Robert Helmer [:rhelmer]]

Attachment: load pageset list from file

If the "pageset" HTTP parameter is passed to cycler.html, it will attempt to load the indicated file from the local webserver via XHR and eval it. The intent is to use this file as an override for the "pages" and "pages_i18n" arrays, to specify alternate pagesets to use.

Comment 2

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 245022 [details] [diff] [review]
load pageset list from file

I think this will fail with file:// URIs, but as the pageset is served from a local server, it should be good enough for now to fix the immediate problem.

Comment 3

[Robert Helmer [:rhelmer]]

(In reply to comment #2)
> (From update of attachment 245022 [details] [diff] [review] [edit])
> I think this will fail with file:// URIs, but as the pageset is served from a
> local server, it should be good enough for now to fix the immediate problem.

Just tested, as long as the test was loaded with file:// then this should work for that case as well.

Comment 4

[Robert Helmer [:rhelmer]]

Alice, would this patch be useful to you? I'll go ahead and land if so. This lets you specify the file to load URLs from instead of hardcoding into the test itself (it can be overridden by hardcoding, so it should still be backwards compatible).

Comment 5

[alice nodelman [:alice] [:anode]]

This is no longer needed as we've gone over to using the pageloader with loads urls from a given file - so they are no longer hard coded into the cycling code.