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 <[email protected]> --- 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 cbc2058..e839b84 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.1.1 _______________________________________________ lttng-dev mailing list [email protected] http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
