Skip to content

Commit

Permalink
Bug 1666565 - Carry suite value for shouldAlert when there is only on…
Browse files Browse the repository at this point in the history
…e subtest r=perftest-reviewers,sparky

Differential Revision: https://phabricator.services.mozilla.com/D91038
  • Loading branch information
davehunt committed Sep 23, 2020
1 parent cb0f9ac commit 9fcc66b
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion testing/talos/talos/output.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,11 @@ def __call__(self):
if test.test_config.get('unit'):
subtest['unit'] = test.test_config['unit']

# if there is only one subtest, carry alerting setting from the suite
if len(subtests) == 1:
subtests[0]['shouldAlert'] = suite['shouldAlert']
# if there is more than one subtest, calculate a summary result
if len(subtests) > 1:
elif len(subtests) > 1:
suite['value'] = self.construct_results(
vals, testname=test.name())
if test.test_config.get('lower_is_better') is not None:
Expand Down

0 comments on commit 9fcc66b

Please sign in to comment.