This is an archive of the discontinued LLVM Phabricator instance.

dotest.py: Fail if we detect multiple tests with the same name
ClosedPublic

Authored by labath on Oct 9 2015, 5:35 AM.

Details

Summary

Log files produced by dotest have names derived from the test name, and this produces errors in
case we have multiple tests with the same name. Additionally, it's good if the test name explains
what the test is testing (which it clearly doesn't do well if there are multiple tests with
identical names). This commit makes the presence of such tests a hard error.

Diff Detail

Repository
rL LLVM

Event Timeline

labath updated this revision to Diff 36939.Oct 9 2015, 5:35 AM
labath retitled this revision from to dotest.py: Fail if we detect multiple tests with the same name.
labath updated this object.
labath added reviewers: tberghammer, zturner.
labath added subscribers: lldb-commits, iancottrell.
tberghammer accepted this revision.Oct 9 2015, 5:42 AM
tberghammer edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 9 2015, 5:42 AM
This revision was automatically updated to reflect the committed changes.