From da023b2f9ac516b17928f8ae804416712e72d937 Mon Sep 17 00:00:00 2001 From: Pietro Cerutti Date: Wed, 1 Mar 2017 09:24:58 +0000 Subject: [PATCH] TAP Reporter: count success tests even if not printed This fixes a bug whereas running the TAP reporter without the -s switch causes the reporter to print 1..0. --- include/reporters/catch_reporter_tap.hpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/include/reporters/catch_reporter_tap.hpp b/include/reporters/catch_reporter_tap.hpp index 16e8a7fe..36e4f901 100644 --- a/include/reporters/catch_reporter_tap.hpp +++ b/include/reporters/catch_reporter_tap.hpp @@ -46,6 +46,7 @@ namespace Catch { virtual bool assertionEnded( AssertionStats const& _assertionStats ) { AssertionResult const& result = _assertionStats.assertionResult; + ++counter; bool printInfoMessages = true; @@ -56,7 +57,7 @@ namespace Catch { printInfoMessages = false; } - AssertionPrinter printer( stream, _assertionStats, printInfoMessages, ++counter ); + AssertionPrinter printer( stream, _assertionStats, printInfoMessages, counter ); printer.print(); stream << " # " << currentTestCaseInfo->name ; @@ -75,7 +76,7 @@ namespace Catch { class AssertionPrinter { void operator= ( AssertionPrinter const& ); public: - AssertionPrinter( std::ostream& _stream, AssertionStats const& _stats, bool _printInfoMessages, size_t counter = 0 ) + AssertionPrinter( std::ostream& _stream, AssertionStats const& _stats, bool _printInfoMessages, size_t counter ) : stream( _stream ) , stats( _stats ) , result( _stats.assertionResult )