diff options
| author | Moritz Pflanzer <moritz.pflanzer@arm.com> | 2017-07-28 13:57:53 +0100 |
|---|---|---|
| committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-09-17 14:16:42 +0100 |
| commit | 8df3fafde3dcf131def3471db8e8b1a1c34b354b (patch) | |
| tree | b729551bc111a83cb8dfc6b25d9948a14ac7aee8 | |
| parent | 6db73ce5222d4b27b06c4e4aa9e466ceb9a09ba2 (diff) | |
| download | ComputeLibrary-8df3fafde3dcf131def3471db8e8b1a1c34b354b.tar.gz | |
COMPMID-415: Small bug fixes in framework
- Fix wrong test case count
- Only print context if there is one
Change-Id: I788654833f73d8b1fabc3a99be8efe02cf4a2145
Reviewed-on: http://mpd-gerrit.cambridge.arm.com/81993
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Tested-by: Kaizen <jeremy.johnson+kaizengerrit@arm.com>
| -rw-r--r-- | framework/Framework.cpp | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/framework/Framework.cpp b/framework/Framework.cpp index bdf5319ebc..008c4c00eb 100644 --- a/framework/Framework.cpp +++ b/framework/Framework.cpp @@ -126,11 +126,14 @@ bool Framework::has_test_info() const void Framework::print_test_info(std::ostream &os) const { - os << "CONTEXT:\n"; - - for(const auto &str : _test_info) + if(!_test_info.empty()) { - os << " " << str << "\n"; + os << "CONTEXT:\n"; + + for(const auto &str : _test_info) + { + os << " " << str << "\n"; + } } } @@ -390,7 +393,7 @@ bool Framework::run() if(_log_level > LogLevel::NONE) { - std::cout << "Executed " << results.size() << " test(s) (" + std::cout << "Executed " << _test_results.size() << " test(s) (" << results[TestResult::Status::SUCCESS] << " passed, " << results[TestResult::Status::EXPECTED_FAILURE] << " expected failures, " << results[TestResult::Status::FAILED] << " failed, " |