Add die tests for extra-opts

diff --git a/.gitignore b/.gitignore
index d0941939280e7fef87450497e81db216841893fc..780461f952768624f9b20ab75bc7f1e995dd192c 100644 (file)
--- a/.gitignore
+++ b/.gitignore
@@ -96,6 +96,7 @@ NP-VERSION-FILE
 /lib/tests/test_ini
 /lib/tests/test_opts1
 /lib/tests/test_opts2
+/lib/tests/test_opts3
 /lib/tests/extra_opts.Po
 
 # /m4/

diff --git a/configure.in b/configure.in
index 107cfd954abbf9fd7f18e57389ea37aad6678986..57e438c5821d7fd49e1af22d5b08727494e9cc4b 100644 (file)
--- 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"
+               EXTRA_TEST="$EXTRA_TEST test_ini 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 f36c0be2a892ad4af1538bf1bb6b34b28e8177b1..4bab2f2a3027564f327458120fdfa8cadae2b0da 100644 (file)
--- 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
+EXTRA_PROGRAMS = test_utils test_disk test_tcp test_cmd test_base64 test_ini 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_tcp.t test_utils.t
+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_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
+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
 
 test: ${noinst_PROGRAMS}
        perl -MTest::Harness -e '$$Test::Harness::switches=""; runtests(map {$$_ .= ".t"} @ARGV)' $(EXTRA_PROGRAMS)

diff --git a/lib/tests/test_opts1.c b/lib/tests/test_opts1.c
index ea9840328f743a4627c602203f0bdcb545c5c1fd..b73e1a1f62d6f1555bf08563545fd77eaeb8139a 100644 (file)
--- a/lib/tests/test_opts1.c
+++ b/lib/tests/test_opts1.c
@@ -1,18 +1,18 @@
 /*****************************************************************************
-* 
+*
 * 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 "common.h"

diff --git a/lib/tests/test_opts2.c b/lib/tests/test_opts2.c
index 2646b2fc07cdcb0c6423679bd6be489685112549..e50f23e46d4e2984e7a07d1abfd4f5dcb9c59016 100644 (file)
--- a/lib/tests/test_opts2.c
+++ b/lib/tests/test_opts2.c
@@ -1,15 +1,15 @@
 /*****************************************************************************
-* 
+*
 * 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/>.
 * 

diff --git a/lib/tests/test_opts3.c b/lib/tests/test_opts3.c
new file mode 100644 (file)
index 0000000..b64270d
--- /dev/null
+++ b/lib/tests/test_opts3.c
@@ -0,0 +1,31 @@
+/*****************************************************************************
+*
+* 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 "extra_opts.h"
+
+int
+main (int argc, char **argv)
+{
+
+       /*
+        * This is for testing arguments expected to die.
+        */
+       argv=np_extra_opts(&argc, argv, argv[0]);
+
+       return 0;
+}
+

diff --git a/lib/tests/test_opts3.t b/lib/tests/test_opts3.t
new file mode 100755 (executable)
index 0000000..47ec26b
--- /dev/null
+++ b/lib/tests/test_opts3.t
@@ -0,0 +1,31 @@
+#!/usr/bin/perl
+use Test::More;
+use strict;
+use warnings;
+
+if (! -e "./test_opts3") {
+       plan skip_all => "./test_opts3 not compiled - please install tap library and/or enable parse-ini to test";
+}
+
+# array of argument arrays
+#   - first value is the NAGIOS_CONFIG_PATH
+#   - 2nd value is the plugin name
+#   - 3rc and up are arguments
+my @TESTS = (
+       ['/nonexistent', 'prog_name', 'arg1', '--extra-opts', '--arg3', 'val2'],
+       ['.', 'prog_name', 'arg1', '--extra-opts=missing@./config-opts.ini', '--arg3', 'val2'],
+       ['.', 'prog_name', 'arg1', '--extra-opts', 'missing@./config-opts.ini', '--arg3', 'val2'],
+       ['.', 'check_missing', 'arg1', '--extra-opts=@./config-opts.ini', '--arg3', 'val2'],
+       ['.', 'check_missing', 'arg1', '--extra-opts', '--arg3', 'val2'],
+);
+
+plan tests => scalar(@TESTS);
+
+my $count=1;
+
+foreach my $args (@TESTS) {
+  $ENV{"NAGIOS_CONFIG_PATH"} = shift(@$args);
+       system {'./test_opts3'} @$args;
+       cmp_ok($?>>8, '==', 3, "Extra-opts die " . $count++);
+}
+