tests: replace BT_ASSERT with exit in test_bin_info
authorMichael Jeanson <mjeanson@efficios.com>
Wed, 5 Jun 2019 15:36:19 +0000 (11:36 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Thu, 13 Jun 2019 21:24:21 +0000 (17:24 -0400)
Since this is a test executable and not a library, it's cleaner to exit
with an error code than use an assertion which could be disabled.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Change-Id: I1684a21d510ea444106b84f5e06ece4a7ff96ef4
Reviewed-on: https://review.lttng.org/c/babeltrace/+/1382
Tested-by: jenkins
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>
Reviewed-by: Francis Deslauriers <francis.deslauriers@efficios.com>
tests/plugins/test_bin_info.c

index 2f9a09a0b278a68938a67d44f159f1433ae15835..339f91a4a58d17158ef6e42693afe4f48ee0b0da 100644 (file)
@@ -224,7 +224,10 @@ void test_bin_info_build_id(const char *bin_info_dir)
        }
 
        ret = bt_fd_cache_init(&fdc);
-       BT_ASSERT(ret == 0);
+       if (ret != 0) {
+               diag("Failed to initialize FD cache");
+               exit(EXIT_FAILURE);
+       }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
                              data_dir, NULL);
@@ -275,7 +278,10 @@ void test_bin_info_debug_link(const char *bin_info_dir)
        }
 
        ret = bt_fd_cache_init(&fdc);
-       BT_ASSERT(ret == 0);
+       if (ret != 0) {
+               diag("Failed to initialize FD cache");
+               exit(EXIT_FAILURE);
+       }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
                              data_dir, NULL);
@@ -322,7 +328,10 @@ void test_bin_info_elf(const char *bin_info_dir)
        }
 
        ret = bt_fd_cache_init(&fdc);
-       BT_ASSERT(ret == 0);
+       if (ret != 0) {
+               diag("Failed to initialize FD cache");
+               exit(EXIT_FAILURE);
+       }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
                              data_dir, NULL);
@@ -367,7 +376,10 @@ void test_bin_info_bundled(const char *bin_info_dir)
        }
 
        ret = bt_fd_cache_init(&fdc);
-       BT_ASSERT(ret == 0);
+       if (ret != 0) {
+               diag("Failed to initialize FD cache");
+               exit(EXIT_FAILURE);
+       }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
                              data_dir, NULL);
This page took 0.025474 seconds and 4 git commands to generate.