Bug 1596599 - Minor update for 'mach test-info report --show-manifests'; r=jmaher
Puppeteer tests have t['manifest'] = None; allowing for this avoids downstream pain when reporting manifests. (Also removed a stray debugging print statement.) Differential Revision: https://phabricator.services.mozilla.com/D53102
This commit is contained in:
@@ -1265,7 +1265,8 @@ class TestInfoCommand(MachCommandBase):
|
|||||||
|
|
||||||
manifest_paths = set()
|
manifest_paths = set()
|
||||||
for t in tests:
|
for t in tests:
|
||||||
manifest_paths.add(t['manifest'])
|
if 'manifest' in t and t['manifest'] is not None:
|
||||||
|
manifest_paths.add(t['manifest'])
|
||||||
manifest_count = len(manifest_paths)
|
manifest_count = len(manifest_paths)
|
||||||
print("Resolver found {} tests, {} manifests".format(len(tests), manifest_count))
|
print("Resolver found {} tests, {} manifests".format(len(tests), manifest_count))
|
||||||
|
|
||||||
@@ -1285,7 +1286,6 @@ class TestInfoCommand(MachCommandBase):
|
|||||||
relpath = mozpath.relpath(manifest_path, self.topsrcdir)
|
relpath = mozpath.relpath(manifest_path, self.topsrcdir)
|
||||||
if mozpath.commonprefix((manifest_path, self.topsrcdir)) != self.topsrcdir:
|
if mozpath.commonprefix((manifest_path, self.topsrcdir)) != self.topsrcdir:
|
||||||
continue
|
continue
|
||||||
print(" {}".format(relpath))
|
|
||||||
manifest_info = None
|
manifest_info = None
|
||||||
if relpath in files_info:
|
if relpath in files_info:
|
||||||
bug_component = files_info[relpath].get('BUG_COMPONENT')
|
bug_component = files_info[relpath].get('BUG_COMPONENT')
|
||||||
|
|||||||
Reference in New Issue
Block a user