From 6678d7dd072caf7fc2b9aec8998c565a6326d359 Mon Sep 17 00:00:00 2001 From: Yannick Brosseau Date: Wed, 23 Feb 2011 18:46:22 +0100 Subject: [PATCH] Fix libustctl_function_tests After discussions, we concluded that the enable a non existing marker is a valid case, so we move it to the working case section. While being there, check that the re-enable a marker set the right errno Signed-off-by: Yannick Brosseau Signed-off-by: Nils Carlson --- .../libustctl_function_tests.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/libustctl_function_tests/libustctl_function_tests.c b/tests/libustctl_function_tests/libustctl_function_tests.c index 947028f..7c12695 100644 --- a/tests/libustctl_function_tests/libustctl_function_tests.c +++ b/tests/libustctl_function_tests/libustctl_function_tests.c @@ -21,6 +21,7 @@ #include #include #include +#include #include #include @@ -136,15 +137,16 @@ static void ustctl_function_tests(pid_t pid) tap_ok(!ustctl_destroy_trace(trace, pid), "ustctl_destroy_trace - without ever starting"); + tap_ok(ustctl_set_marker_state(trace, "ustl", "blar", 1, pid) == 0, + "Enable non-existent marker ustl blar"); printf("##### Tests that definetly should work are completed #####\n"); printf("############## Start expected failure cases ##############\n"); tap_ok(ustctl_set_marker_state(trace, "ust","bar", 1, pid), "Enable already enabled marker ust/bar"); - - tap_ok(ustctl_set_marker_state(trace, "ustl", "blar", 1, pid), - "Enable non-existent marker ustl blar"); + tap_ok(EEXIST == errno, + "Right error code for enabling an already enabled marker"); tap_ok(ustctl_start_trace(trace, pid), "Start a non-existent trace"); @@ -161,7 +163,7 @@ int main(int argc, char **argv) int i, status; pid_t parent_pid, child_pid; - tap_plan(27); + tap_plan(28); printf("Function tests for ustctl\n"); -- 2.34.1