If want successful test output, include messages (CleverRaven#51441)

The output for successful tests, even if such is requested of via Catch2's `-s` flag, now includes messages; it didn't before. Co-authored-by: actual-nh <[email protected]>
actual-nh · Sep 6, 2021 · cc98274 · cc98274
1 parent eede07d
commit cc98274
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/tests/test_main.cpp b/tests/test_main.cpp
@@ -249,11 +249,16 @@ struct CataListener : Catch::TestEventListenerBase {
 
  void sectionEnded( Catch::SectionStats const &sectionStats ) override {
  TestEventListenerBase::sectionEnded( sectionStats );
- if( !sectionStats.assertions.allPassed() ) {
+ if( !sectionStats.assertions.allPassed() ||
+ m_config->includeSuccessfulResults() ) {
  std::vector<std::pair<std::string, std::string>> messages =
  Messages::recent_messages( 0 );
  if( !messages.empty() ) {
- stream << "Log messages during failed test:\n";
+ if( !sectionStats.assertions.allPassed() ) {
+ stream << "Log messages during failed test:\n";
+ } else {
+ stream << "Log messages during successful test:\n";
+ }
  }
  for( const std::pair<std::string, std::string> &message : messages ) {
  stream << message.first << ": " << message.second << '\n';