David Blaikie 8c2dc92bd4 ProfData: Fix some unchecked Errors in unit tests ...

The 'NoError' function was meant to be used as the input to
ASSERT/EXPECT_TRUE, but it is easy to forget this (it could be annotated
with nodiscard to help this) so many sites that look like they're checked
are not (& silently discard the failure). Only one site actually has an
Error sneaking out this way and I've replaced that one with a
FIXME+consumeError.

The rest of the code has been modified to use the EXPECT_THAT_ERROR
macros Zach introduced a while back. Between the options available this
seems OK/good/something to standardize on - though it's difficult to
build a matcher that could handle checking for a specific llvm::Error
result, so those remain using the custom ErrorEquals (& the nodiscard
added to ensure it is not misused as it was previous to this patch). It
could still be generalized a bit further (even not as far as a matcher,
but at least support multiple kinds of Error, etc) & added to the
general Error utility header.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@307440 91177308-0d34-0410-b5e6-96231b3b80d8

2017-07-07 21:02:59 +00:00

..

CMakeLists.txt

ProfData: Fix some unchecked Errors in unit tests

2017-07-07 21:02:59 +00:00

CoverageMappingTest.cpp

ProfData: Fix some unchecked Errors in unit tests

2017-07-07 21:02:59 +00:00

InstrProfTest.cpp

ProfData: Fix some unchecked Errors in unit tests

2017-07-07 21:02:59 +00:00

SampleProfTest.cpp

Re-sort #include lines for unittests. This uses a slightly modified

2017-06-06 11:06:56 +00:00