diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-09-03 12:58:30 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-09-10 00:46:44 +0200 |
commit | 610618ff61a87ae652116d2e0b58fa556b3c66ba (patch) | |
tree | 130176b0acaf3e910ad5ba54bc316def83a0b35e /src/test/test-socket-netlink.c | |
parent | 38c30b35f38298c2fbdcee46bb39aed3baa3019c (diff) | |
download | systemd-610618ff61a87ae652116d2e0b58fa556b3c66ba.tar.gz |
test-socket-netlink: print the proper expected string
We would use the return value from the tested function to decide
what to print as "expected", which is confusing when something is wrong
with the tested function.
Diffstat (limited to 'src/test/test-socket-netlink.c')
-rw-r--r-- | src/test/test-socket-netlink.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/src/test/test-socket-netlink.c b/src/test/test-socket-netlink.c index f86f123782..f03e455d0e 100644 --- a/src/test/test-socket-netlink.c +++ b/src/test/test-socket-netlink.c @@ -12,11 +12,19 @@ static void test_socket_address_parse_one(const char *in, int ret, int family, c int r; r = socket_address_parse(&a, in); - if (r >= 0) - assert_se(socket_address_print(&a, &out) >= 0); + if (r >= 0) { + r = socket_address_print(&a, &out); + if (r < 0) + log_error_errno(r, "Printing failed for \"%s\": %m", in); + assert(r >= 0); + } - log_info("\"%s\" → %s → \"%s\" (expect \"%s\")", in, - r >= 0 ? "✓" : "✗", empty_to_dash(out), r >= 0 ? expected ?: in : "-"); + log_info("\"%s\" → %s %d → \"%s\" (expect %d / \"%s\")", + in, + r >= 0 ? "✓" : "✗", r, + empty_to_dash(out), + ret, + ret >= 0 ? expected ?: in : "-"); assert_se(r == ret); if (r >= 0) { assert_se(a.sockaddr.sa.sa_family == family); |