Page MenuHomePhabricator

[DebugInfo][test][NFC] Remove case numbers
ClosedPublic

Authored by jhenderson on Tue, Feb 11, 6:17 AM.

Details

Summary

This reduces the noise caused by adding cases earlier in the sequence.

Diff Detail

Event Timeline

jhenderson created this revision.Tue, Feb 11, 6:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Feb 11, 6:17 AM
dblaikie accepted this revision.Tue, Feb 11, 9:52 AM

Looks good (should these tests be testing the diagnostic text of the recoverable errors, in addition to testing the fact that they are recovered from/dumped?)

This revision is now accepted and ready to land.Tue, Feb 11, 9:52 AM

Looks good (should these tests be testing the diagnostic text of the recoverable errors, in addition to testing the fact that they are recovered from/dumped?)

They do - see the block at the bottom of the test with the ALL/OTHER prefixes. They aren't inline with the body, as I don't think the exact interleaving of the output is necessarily that important.

This revision was automatically updated to reflect the committed changes.