Closed Bug 1799131 Opened 2 years ago Closed 2 years ago

test-info-all report has manifest, we should include ancestor manifest

Categories

(Testing :: General, enhancement)

Default
enhancement

Tracking

(firefox108 fixed)

RESOLVED FIXED
108 Branch
Tracking Status
firefox108 --- fixed

People

(Reporter: jmaher, Assigned: jmaher)

Details

Attachments

(1 file)

currently there are manifests which include other manifests, resulting in tests that can be run multiple times despite being defined in a single .ini manifest.

I am thinking about mochitest-common.ini, and you can see that for a given test case in there, we run it in two different scenarios:

  • dom/serviceworkers/test/mochitest-dFPI.ini
  • dom/serviceworkers/test/mochitest.ini

so every test in mochitest-common.ini will be run twice. Right now we have the test mapping to indicate a single manifest mochitest-common.ini.

this bug I would like to make it map with what our tests output, specifically:
dom/serviceworkers/test/mochitest-dFPI.ini:dom/serviceworkers/test/mochitest-common.ini

this will result in some tests having >1 manifest listed in the test-info section. it will also result in some of the manifest fields being in the form of <ancestor>:<manifest>, so tools that parse that data (searchfox) will need to be aware and consider how to use the info.

Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f429bf3f5db4
Adjust test-info manifests to retain ancestor. r=gbrown,asuth
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: