diff options
| author | Martin Hořeňovský | 2017-06-06 15:12:03 +0200 |
|---|---|---|
| committer | Martin Hořeňovský | 2017-06-06 15:12:03 +0200 |
| commit | 6583284731e961065f007782eb89be41e6ef4e6a (patch) | |
| tree | 638cd020a5e1c80e687b1eed7e84cbdd9105bda0 /include | |
| parent | 07ef028483dca3afad353655a3341be95bfd70be (diff) | |
| download | catch3-6583284731e961065f007782eb89be41e6ef4e6a.tar.gz | |
Fix erroneous result disposition for {REQUIRE,CHECK}_THAT
This fixes result disposition being ContinueOnFailure |
ContinueOnFailure for CHECK_THAT (obviously an error) and Normal |
ContinueOnFailure for REQUIRE_THAT (less obviously an error, but worse,
as that signals to the pipeline that assertion failure should both abort
and continue the test with ???? happening).
Diffstat (limited to 'include')
| -rw-r--r-- | include/internal/catch_capture.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/internal/catch_capture.hpp b/include/internal/catch_capture.hpp index 7816b50c..b25f13e8 100644 --- a/include/internal/catch_capture.hpp +++ b/include/internal/catch_capture.hpp @@ -162,7 +162,7 @@ try { \ __catchResult.captureMatch( arg, matcher, #matcher ); \ } catch( ... ) { \ - __catchResult.useActiveException( resultDisposition | Catch::ResultDisposition::ContinueOnFailure ); \ + __catchResult.useActiveException( resultDisposition ); \ } \ INTERNAL_CATCH_REACT( __catchResult ) \ } while( Catch::alwaysFalse() ) |