diff options
author | Moritz Pflanzer <moritz.pflanzer@arm.com> | 2017-09-12 15:51:33 +0100 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:35:24 +0000 |
commit | 5b61fd3fbaf41031232296abde56258d12ba3340 (patch) | |
tree | a44cc3071d4d7b91480cb672c3ed4536857bb4e6 /tests/framework/Framework.cpp | |
parent | a3adb3a3bdce1f2ef764c5d5098e99695323f0a3 (diff) | |
download | ComputeLibrary-5b61fd3fbaf41031232296abde56258d12ba3340.tar.gz |
COMPMID-417: Fix validation
Change-Id: I7a745037136bc6e02d177f65fe4f4cd43873b98e
Reviewed-on: http://mpd-gerrit.cambridge.arm.com/87406
Tested-by: Kaizen <jeremy.johnson+kaizengerrit@arm.com>
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Diffstat (limited to 'tests/framework/Framework.cpp')
-rw-r--r-- | tests/framework/Framework.cpp | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/tests/framework/Framework.cpp b/tests/framework/Framework.cpp index 343b7a8561..31e524338b 100644 --- a/tests/framework/Framework.cpp +++ b/tests/framework/Framework.cpp @@ -164,7 +164,8 @@ void Framework::log_failed_expectation(const TestError &error) { if(_log_level >= error.level() && _printer != nullptr) { - _printer->print_error(error); + constexpr bool expected_error = true; + _printer->print_error(error, expected_error); } if(_current_test_result != nullptr) @@ -232,6 +233,8 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) _printer->print_errors_header(); } + const bool is_expected_failure = test_factory.status() == TestCaseFactory::Status::EXPECTED_FAILURE; + try { std::unique_ptr<TestCase> test_case = test_factory.make(); @@ -265,7 +268,7 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { if(_log_level >= error.level() && _printer != nullptr) { - _printer->print_error(error); + _printer->print_error(error, is_expected_failure); } result.status = TestResult::Status::FAILED; @@ -282,7 +285,8 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { std::stringstream stream; stream << "Error code: " << error.err(); - _printer->print_error(TestError(error.what(), LogLevel::ERRORS, stream.str())); + TestError test_error(error.what(), LogLevel::ERRORS, stream.str()); + _printer->print_error(test_error, is_expected_failure); } result.status = TestResult::Status::FAILED; @@ -297,7 +301,7 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { if(_log_level >= LogLevel::ERRORS && _printer != nullptr) { - _printer->print_error(error); + _printer->print_error(error, is_expected_failure); } result.status = TestResult::Status::CRASHED; @@ -311,7 +315,7 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { if(_log_level >= LogLevel::ERRORS && _printer != nullptr) { - _printer->print_error(TestError("Received unknown exception")); + _printer->print_error(TestError("Received unknown exception"), is_expected_failure); } result.status = TestResult::Status::CRASHED; @@ -326,7 +330,7 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { if(_log_level >= LogLevel::ERRORS && _printer != nullptr) { - _printer->print_error(error); + _printer->print_error(error, is_expected_failure); } result.status = TestResult::Status::CRASHED; @@ -340,7 +344,7 @@ void Framework::run_test(const TestInfo &info, TestCaseFactory &test_factory) { if(_log_level >= LogLevel::ERRORS && _printer != nullptr) { - _printer->print_error(TestError("Received unknown exception")); + _printer->print_error(TestError("Received unknown exception"), is_expected_failure); } result.status = TestResult::Status::CRASHED; |