Add die tests for parse_ini

8 files changed, 79 insertions, 7 deletions

diff --git a/.gitignore b/.gitignore
index 780461f..be40c08 100644
--- a/.gitignore
+++ b/.gitignore
@@ -93,7 +93,8 @@ NP-VERSION-FILE
 /lib/tests/utils_disk.Po
 /lib/tests/utils_tcp.Po
 /lib/tests/parse_ini.Po
-/lib/tests/test_ini
+/lib/tests/test_ini1
+/lib/tests/test_ini3
 /lib/tests/test_opts1
 /lib/tests/test_opts2
 /lib/tests/test_opts3
diff --git a/configure.in b/configure.in
index 57e438c..68ac0a0 100644
--- a/configure.in
+++ b/configure.in
@@ -188,7 +188,7 @@ AM_CONDITIONAL([USE_PARSE_INI],[test "$enable_extra_opts" = "yes"])
 if test "$enable_extra_opts" = "yes" ; then
         AC_DEFINE(NP_EXTRA_OPTS,[1],[Enable INI file parsing.])
         if test "$enable_libtap" = "yes"; then
-                EXTRA_TEST="$EXTRA_TEST test_ini test_opts1 test_opts2 test_opts3"
+                EXTRA_TEST="$EXTRA_TEST test_ini1 test_ini3 test_opts1 test_opts2 test_opts3"
                 AC_SUBST(EXTRA_TEST)
         fi
 fi
diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am
index 4bab2f2..694baf0 100644
--- a/lib/tests/Makefile.am
+++ b/lib/tests/Makefile.am
@@ -7,9 +7,9 @@ check_PROGRAMS = @EXTRA_TEST@
 
 INCLUDES = -I$(top_srcdir)/lib -I$(top_srcdir)/gl -I$(top_srcdir)/intl -I$(top_srcdir)/plugins
 
-EXTRA_PROGRAMS = test_utils test_disk test_tcp test_cmd test_base64 test_ini test_opts1 test_opts2 test_opts3
+EXTRA_PROGRAMS = test_utils test_disk test_tcp test_cmd test_base64 test_ini1 test_ini3 test_opts1 test_opts2 test_opts3
 
-np_test_scripts = test_base64.t test_cmd.t test_disk.t test_ini.t test_opts1.t test_opts2.t test_opts3.t test_tcp.t test_utils.t
+np_test_scripts = test_base64.t test_cmd.t test_disk.t test_ini1.t test_ini3.t test_opts1.t test_opts2.t test_opts3.t test_tcp.t test_utils.t
 np_test_files = config-dos.ini config-opts.ini config-tiny.ini plugin.ini plugins.ini
 EXTRA_DIST = $(np_test_scripts) $(np_test_files)
 
@@ -28,7 +28,7 @@ AM_CFLAGS = -g -I$(top_srcdir)/lib -I$(top_srcdir)/gl $(tap_cflags)
 AM_LDFLAGS = $(tap_ldflags) -ltap
 LDADD = $(top_srcdir)/lib/libnagiosplug.a $(top_srcdir)/gl/libgnu.a
 
-SOURCES = test_utils.c test_disk.c test_tcp.c test_cmd.c test_base64.c test_ini.c test_opts1.c test_opts2.c test_opts3.c
+SOURCES = test_utils.c test_disk.c test_tcp.c test_cmd.c test_base64.c test_ini1.c test_ini3.c test_opts1.c test_opts2.c test_opts3.c
 
 test: ${noinst_PROGRAMS}
         perl -MTest::Harness -e '$$Test::Harness::switches=""; runtests(map {$$_ .= ".t"} @ARGV)' $(EXTRA_PROGRAMS)
diff --git a/lib/tests/test_ini.c b/lib/tests/test_ini1.c
index c0853d7..c0853d7 100644
--- a/lib/tests/test_ini.c
+++ b/lib/tests/test_ini1.c
diff --git a/lib/tests/test_ini.t b/lib/tests/test_ini1.t
index 0487098..0487098 100755
--- a/lib/tests/test_ini.t
+++ b/lib/tests/test_ini1.t
diff --git a/lib/tests/test_ini3.c b/lib/tests/test_ini3.c
new file mode 100644
index 0000000..814b3ec
--- /dev/null
+++ b/lib/tests/test_ini3.c
@@ -0,0 +1,33 @@
+/*****************************************************************************
+* 
+* This program is free software: you can redistribute it and/or modify
+* it under the terms of the GNU General Public License as published by
+* the Free Software Foundation, either version 3 of the License, or
+* (at your option) any later version.
+* 
+* This program is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+* GNU General Public License for more details.
+* 
+* You should have received a copy of the GNU General Public License
+* along with this program.  If not, see <http://www.gnu.org/licenses/>.
+* 
+* 
+*****************************************************************************/
+
+#include "parse_ini.h"
+
+int
+main (int argc, char **argv)
+{
+
+        /*
+         * This is for testing arguments expected to die.
+         */
+
+        np_get_defaults(argv[1], argv[0]);
+
+        return 0;
+}
+
diff --git a/lib/tests/test_ini3.t b/lib/tests/test_ini3.t
new file mode 100755
index 0000000..57388dd
--- /dev/null
+++ b/lib/tests/test_ini3.t
@@ -0,0 +1,34 @@
+#!/usr/bin/perl
+use Test::More;
+use strict;
+use warnings;
+
+if (! -e "./test_ini") {
+        plan skip_all => "./test_ini not compiled - please install tap library and/or enable parse-ini to test";
+}
+
+# array of argument arrays
+#   - First value is the expected return code
+#   - 2nd value is the NAGIOS_CONFIG_PATH
+#     TODO: looks like we look in default path after looking trough this variable - shall we?
+#   - 3rd value is the plugin name
+#   - 4th is the ini locator
+my @TESTS = (
+        [3, undef, "section", "section_unknown@./config-tiny.ini"],
+);
+
+plan tests => scalar(@TESTS);
+
+my $count=1;
+
+foreach my $args (@TESTS) {
+        my $rc = shift(@$args);
+        if (my $env = shift(@$args)) {
+                $ENV{"NAGIOS_CONFIG_PATH"} = $env;
+        } else {
+                delete($ENV{"NAGIOS_CONFIG_PATH"});
+        }
+        system {'./test_ini2'} @$args;
+        cmp_ok($?>>8, '==', $rc, "Parse-ini die " . $count++);
+}
+
diff --git a/lib/tests/test_opts3.t b/lib/tests/test_opts3.t
index 35b4468..fa7876d 100755
--- a/lib/tests/test_opts3.t
+++ b/lib/tests/test_opts3.t
@@ -28,8 +28,12 @@ plan tests => scalar(@TESTS);
 my $count=1;
 
 foreach my $args (@TESTS) {
-  my $rc = shift(@$args);
-  $ENV{"NAGIOS_CONFIG_PATH"} = shift(@$args);
+        my $rc = shift(@$args);
+        if (my $env = shift(@$args)) {
+                $ENV{"NAGIOS_CONFIG_PATH"} = $env;
+        } else {
+                delete($ENV{"NAGIOS_CONFIG_PATH"});
+        }
         system {'./test_opts3'} @$args;
         cmp_ok($?>>8, '==', $rc, "Extra-opts die " . $count++);
 }