If a test run does not contain failures, when the user requests uncollapsed failures, we still include references to test runs with no failures.
Created attachment 383785 [details] Patch
Looking to avoid something like this: https://results.webkit.org/api/failures/webkitpy-tests?id=252557&collapsed=False
Comment on attachment 383785 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383785&action=review rs=me > Tools/resultsdbpy/resultsdbpy/model/failure_context.py:190 > + if len(unpacked) > 2: why 2? should we add a comment here indicating significance of 2 here.
Created attachment 383801 [details] Patch for landing
Comment on attachment 383785 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383785&action=review >> Tools/resultsdbpy/resultsdbpy/model/failure_context.py:190 >> + if len(unpacked) > 2: > > why 2? should we add a comment here indicating significance of 2 here. Because our unpacked objects will always contain the keys uuid and start_time, even if there are no tests.
Comment on attachment 383801 [details] Patch for landing Clearing flags on attachment: 383801 Committed r252606: <https://trac.webkit.org/changeset/252606>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57304858>