Open Bug 1305795 Opened 8 years ago Updated 2 years ago

Use fewer files from in web-platform-tests


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

Version 3


(Not tracked)


(Reporter: gps, Unassigned)


(Depends on 1 open bug, 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+

The bug assignee didn't login in Bugzilla in the last 7 months.
:jgraham, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: gps → nobody
Flags: needinfo?(james)

I'm not aware of any current plans to move to using the source checkouts for tests directly (but I still think we would benefit from doing that!).

Flags: needinfo?(james)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.