Add manifest support to cpp unittest harness

RESOLVED FIXED in mozilla27

Status

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dminor, Assigned: dminor)

Tracking

Trunk
mozilla27
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

6 years ago
When running the cpp unittests on Android, there are a small number of consistent failures. 

Adding a simple manifest file to the harness would allow these to be skipped until they are resolved. A list of filenames to be skipped would be sufficient for this purpose and I don't seen an immediate need for anything more sophisticated.
Assignee

Comment 1

6 years ago
Try runs here:
https://tbpl.mozilla.org/?tree=Try&rev=e936924251e0
https://tbpl.mozilla.org/?tree=Try&rev=15081ccb3648

I'll file bugs for the skipped tests, but I usually hold off until things are running on Cedar just to make sure results there match what I see locally.
Attachment #803733 - Flags: review?(ted)
Comment on attachment 803733 [details] [diff] [review]
Patch to add manifest support and stage manifest file in test package.

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

::: testing/runcppunittests.py
@@ +131,5 @@
>                          action = "store", type = "string", dest = "symbols_path",
>                          default = None,
>                          help = "absolute path to directory containing breakpad symbols, or the URL of a zip file containing symbols")
> +        self.add_option("--manifest-file",
> +                        action = "store", type = "string", dest = "manifest_file",

Maybe we should make this more specific and call it --skip-manifest or something like that?

@@ +142,2 @@
>      progs = []
> +    skipped_progs = []

I'd use a set() here, but for the size of the skipped list it's not going to make a huge difference.
Attachment #803733 - Flags: review?(ted) → review+
https://hg.mozilla.org/mozilla-central/rev/3934868da81e
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.