From 52754725e185f1331f821d85ed2ef78fb92af1fe Mon Sep 17 00:00:00 2001 From: Filipe Brandenburger Date: Mon, 25 Aug 2014 22:05:02 -0700 Subject: [PATCH] test-compress: make sure asserts with side effects use assert_se() Otherwise the test fails when built with CPPFLAGS='-DNDEBUG' which disables assertions. Tested: - make check TESTS='test-compress' CPPFLAGS='-DNDEBUG' --- src/journal/test-compress.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/journal/test-compress.c b/src/journal/test-compress.c index f5f5f8df390..026d630ac2a 100644 --- a/src/journal/test-compress.c +++ b/src/journal/test-compress.c @@ -145,11 +145,11 @@ static void test_compress_stream(int compression, assert_se((dst = mkostemp_safe(pattern, O_RDWR|O_CLOEXEC)) >= 0); - assert(compress(src, dst, -1) == 0); + assert_se(compress(src, dst, -1) == 0); if (cat) { assert_se(asprintf(&cmd, "%s %s | diff %s -", cat, pattern, srcfile) > 0); - assert(system(cmd) == 0); + assert_se(system(cmd) == 0); } log_debug("/* test decompression */");