From: Martin Schwenke <martin@meltin.net>
Date: Mon, 20 Aug 2018 20:55:33 +0000 (+1000)
Subject: ctdb-tests: Add an extra conf loading test case
X-Git-Tag: tdb-1.3.17~2017
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=55893bf8d2cc7e01b3a93d8e1fde16408244cb65;p=thirdparty%2Fsamba.git

ctdb-tests: Add an extra conf loading test case

This shows that config file loading continues in spite of unknown keys
if ignore_unknown is true.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13589

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
---

diff --git a/ctdb/tests/cunit/config_test_001.sh b/ctdb/tests/cunit/config_test_001.sh
index c0a6e89aeb2..bf1e589c318 100755
--- a/ctdb/tests/cunit/config_test_001.sh
+++ b/ctdb/tests/cunit/config_test_001.sh
@@ -90,3 +90,23 @@ required_result 2 <<EOF
 Configuration option [section] -> "key" not defined
 EOF
 unit_test ctdb-config get section key
+
+# Confirm that an unknown key doesn't stop the rest of the file from
+# loading
+cat > "$conffile" <<EOF
+[database]
+	unknown key = 123
+
+[logging]
+	log level = debug
+EOF
+
+required_result 2 <<EOF
+Failed to load config file $conffile
+EOF
+unit_test ctdb-config validate
+
+ok <<EOF
+debug
+EOF
+unit_test ctdb-config get "logging" "log level"