Closed Bug 883865 Opened 11 years ago Closed 6 years ago

manifestdestiny needs a writemanifest option


(Testing :: Mozbase, defect)

Not set


(Not tracked)



(Reporter: jmaher, Unassigned)


After we read manifests and filter them, it would be a much needed feature to have the ability to write the resulting annotated manifest out.  Sometimes we have to adjust paths of the tests in the manifest, this should be optional as well.  Also we should allow this to be written out to a specific location.

Currently I am doing this in a patch for mochitest manifests.
there is a need to output the json manifest data structure to either .ini or .json.
write() does .ini; is this adequate Joel?
Flags: needinfo?(jmaher)

A missing piece of the puzzle:

    8.61 +      # Bug 883865 - add this functionality into manifestDestiny
    8.62 +      with open('tests.json', 'w') as manifestFile:
    8.63 +        manifestFile.write(json.dumps({'tests': paths}))
    8.64 +      options.manifestFile = 'tests.json'

It'd probably be nice to split out the de/serialization from ManifestParser class itself.
I think the JSON serialization was the main point of this bug, since that's what Joel's Mochitest patch is doing.
what ted said, if we can somehow output .json, it would solve my original request.
Flags: needinfo?(jmaher)
Mass closing bugs with no activity in 2+ years. If this bug is important to you, please re-open.
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.