From 5369d33ae2b29798d63c4d44b4a4efbb3b0f99cc Mon Sep 17 00:00:00 2001 From: Przemek Wirkus Date: Thu, 11 Feb 2016 19:18:46 +0000 Subject: [PATCH] Expect FAIL test cases. Merge and squash changes from Niklas From @niklas-arm: * Report success to greentea for expected failures. * Use the greentea setup handlers for correct reporting. * Remove GREENTEA_START() from all tests. * Reorder greentea_send_kv to be more readable. --- "frameworks\\utest/source/greentea_handlers.cpp" | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git "a/frameworks\\utest/source/greentea_handlers.cpp" "b/frameworks\\utest/source/greentea_handlers.cpp" index 3dbd2001c1..3424ae75ea 100644 --- "a/frameworks\\utest/source/greentea_handlers.cpp" +++ "b/frameworks\\utest/source/greentea_handlers.cpp" @@ -90,8 +90,8 @@ status_t utest::v1::greentea_test_setup_handler(const size_t number_of_cases) void utest::v1::greentea_test_teardown_handler(const size_t passed, const size_t failed, const failure_t failure) { - greentea_send_kv(TEST_ENV_TESTCASE_SUMMARY, passed, failed); verbose_test_teardown_handler(passed, failed, failure); + greentea_send_kv(TEST_ENV_TESTCASE_SUMMARY, passed, failed); int result = !(failed || (failure.reason && !(failure.reason & REASON_IGNORE))); GREENTEA_TESTSUITE_RESULT(result); } @@ -104,8 +104,9 @@ void utest::v1::greentea_test_failure_handler(const failure_t failure) // --- GREENTEA CASE HANDLERS --- status_t utest::v1::greentea_case_setup_handler(const Case *const source, const size_t index_of_case) { + status_t status = verbose_case_setup_handler(source, index_of_case); greentea_send_kv(TEST_ENV_TESTCASE_START, source->get_description()); - return verbose_case_setup_handler(source, index_of_case); + return status; } status_t utest::v1::greentea_case_teardown_handler(const Case *const source, const size_t passed, const size_t failed, const failure_t failure)