From 454d4e22aaab3a7d4be3c1fe2f291630a952b1a0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Ekl=C3=B6f?= Date: Sat, 18 Dec 2021 20:42:37 +0100 Subject: [PATCH] test: config: verify setting an empty key/mouse binding fails --- tests/test-config.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/tests/test-config.c b/tests/test-config.c index 1e99b3c4..6a2edcb8 100644 --- a/tests/test-config.c +++ b/tests/test-config.c @@ -296,7 +296,16 @@ test_key_binding(struct context *ctx, bool (*parse_fun)(struct context *ctx), const int click_count = action % 3 + 1; /* Finally, generate the ‘value’ (e.g. “Control+shift+x”) */ - char value[128]; + char value[128] = {0}; + + ctx->key = key; + ctx->value = value; + + /* First, try setting the empty string */ + if (parse_fun(ctx)) { + BUG("[%s].%s=: did not fail to parse as expected", + ctx->section, ctx->key); + } switch (type) { case KEY_BINDING: { @@ -319,9 +328,6 @@ test_key_binding(struct context *ctx, bool (*parse_fun)(struct context *ctx), } } - ctx->key = key; - ctx->value = value; - if (!parse_fun(ctx)) { BUG("[%s].%s=%s failed to parse", ctx->section, ctx->key, ctx->value);