Open Bug 1305795 Opened 5 years ago Updated 3 years ago

Use fewer files from in web-platform-tests


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

Version 3
Not set


(Not tracked)



(Reporter: gps, Assigned: gps)


(Depends on 2 open bugs, Blocks 1 open bug)



(10 files)

Bug 1286900 started using most WPT files from a source checkout (namely tests and meta files). But we still rely on several files from This bug is about reducing that dependency, possibly even eliminating it.
chmanchester: you may care about the patch that introduced
Gah - mozinfo.json isn't working right on TC with this series. I'll look into it tomorrow. Buildbot is fine though. for the curious.
Blocks: 1305205
Comment on attachment 8795536 [details]
Bug 1305795 - Produce a with binary files;

::: python/mozbuild/mozbuild/
(Diff revision 5)
> +    # TODO reconsider doing this.
> +    # When we introduced test artifact processing, the point was to extract
> +    # bin/* files from into a smaller zip. Now that we have
> +    #, this extract step may not be necessary and we can perform
> +    # any filtering (if we need to do it at all) during archive extraction
> +    # time.

I don't think this comment is adding much. It would apply nearly equally to the packaged build we handle, so implies we should re-write that too. I don't agree with the premise, but if you'd like to discuss this I think it would be better off tracked in a follow up bug.
Attachment #8795489 - Flags: review?(ted)
Attachment #8795491 - Flags: review?(ted)
Attachment #8795490 - Flags: review?(ted)
Depends on: 1306098
Comment on attachment 8795490 [details]
Bug 1305795 - Set mozinfo_json_url property;

::: testing/mozharness/mozharness/mozilla/
(Diff revision 3)
> -            p['packageUrl'] if p.get('packageUrl') else None,
> -            p['testPackagesUrl'] if p.get('testPackagesUrl') else None,
> -            p['symbolsUrl'] if p.get('symbolsUrl') else None
> +            p.get('packageUrl') or None,
> +            p.get('testPackagesUrl') or None,
> +            p.get('symbolsUrl') or None,
> +            p.get('mozinfoJsonUrl') or None,

Is the `or None` part even necessary?
Comment on attachment 8795485 [details]
Bug 1305795 - Stop extracting web-platform directory;
Attachment #8795485 - Flags: review?(ted) → review+
Comment on attachment 8795538 [details]
Bug 1305795 - Remove pull action from WPT script;
Attachment #8795538 - Flags: review?(ted) → review+
You need to log in before you can comment on or make changes to this bug.