unit-tests: Invoke all registered thread cleanup handlers on test failure

If a test fails in a timeout or a test failure, longjmp() is used to restore
the thread context and handle test failure. However, there might be unreleased
resources, namely locks, which prevent the library to clean up properly after
finishing the test.

By using thread cleanup handlers, we can release any test subject internal or
test specific external resources on test failure. We do so by calling all
registered cleanup handlers.

diff --git a/src/libstrongswan/tests/test_runner.c b/src/libstrongswan/tests/test_runner.c
index b77302820bf9fcfd0cdc4b8227cd5c7b155b0ac4..83f4e8faffe6c3ef2702de720a495283d9dff836 100644 (file)
--- a/src/libstrongswan/tests/test_runner.c
+++ b/src/libstrongswan/tests/test_runner.c
@@ -185,6 +185,7 @@ static bool run_test(test_function_t *tfun, int i)
                tfun->cb(i);
                return TRUE;
        }
+       thread_cleanup_popall();
        return FALSE;
 }
 
@@ -219,6 +220,7 @@ static bool call_fixture(test_case_t *tcase, bool up)
                }
                else
                {
+                       thread_cleanup_popall();
                        failure = TRUE;
                        break;
                }
@@ -336,6 +338,7 @@ static bool post_test(test_runner_init_t init, bool check_leaks,
                }
                else
                {
+                       thread_cleanup_popall();
                        library_deinit();
                        return FALSE;
                }