From 9fc2d9fcd2a5d3b6718d67307367da0f030307ca Mon Sep 17 00:00:00 2001 From: Mathieu Desnoyers Date: Sun, 4 Jan 2015 21:25:07 -0500 Subject: [PATCH] Fix: ust snapshot: cleanup after error MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The "break" statement on error skips the rest of the functions, thus leaving test applications running after the end of the test, which is a side-effect on the following tests. Signed-off-by: Mathieu Desnoyers Signed-off-by: Jérémie Galarneau --- tests/regression/tools/snapshots/ust_test | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/tests/regression/tools/snapshots/ust_test b/tests/regression/tools/snapshots/ust_test index cbc205828..e839b84bf 100755 --- a/tests/regression/tools/snapshots/ust_test +++ b/tests/regression/tools/snapshots/ust_test @@ -174,8 +174,6 @@ function test_ust_local_snapshot () if [ $? -eq 0 ]; then # Only delete if successful rm -rf $TRACE_PATH - else - break fi stop_test_apps @@ -253,8 +251,6 @@ function test_ust_local_snapshot_large_metadata () if [ $? -eq 0 ]; then # Only delete if successful rm -rf $TRACE_PATH - else - break fi } @@ -288,8 +284,6 @@ function test_ust_per_uid_local_snapshot () if [ $? -eq 0 ]; then # Only delete if successful rm -rf $TRACE_PATH - else - break fi stop_test_apps @@ -317,8 +311,6 @@ function test_ust_per_uid_local_snapshot_post_mortem () if [ $? -eq 0 ]; then # Only delete if successful rm -rf $TRACE_PATH - else - break fi } @@ -343,8 +335,6 @@ function test_ust_local_snapshots () if [ $? -eq 0 ]; then # Only delete if successful rm -rf $TRACE_PATH - else - break fi done stop_lttng_tracing $SESSION_NAME -- 2.34.1