diff --git a/lib/common/tests/options/Makefile.am b/lib/common/tests/options/Makefile.am index 204b21a809..bda3b1127e 100644 --- a/lib/common/tests/options/Makefile.am +++ b/lib/common/tests/options/Makefile.am @@ -1,23 +1,25 @@ # # Copyright 2022 the Pacemaker project contributors # # The version control history for this file may have further details. # # This source code is licensed under the GNU General Public License version 2 # or later (GPLv2+) WITHOUT ANY WARRANTY. # AM_CPPFLAGS = -I$(top_srcdir)/include \ -I$(top_builddir)/include \ -I$(top_srcdir)/lib/common LDADD = $(top_builddir)/lib/common/libcrmcommon_test.la \ -lcmocka AM_CFLAGS = -DPCMK__UNIT_TESTING AM_LDFLAGS = $(LDFLAGS_WRAP) include $(top_srcdir)/mk/tap.mk # Add "_test" to the end of all test program names to simplify .gitignore. -check_PROGRAMS = pcmk__env_option_test +check_PROGRAMS = \ + pcmk__env_option_test \ + pcmk__set_env_option_test TESTS = $(check_PROGRAMS) diff --git a/lib/common/tests/options/pcmk__set_env_option_test.c b/lib/common/tests/options/pcmk__set_env_option_test.c new file mode 100644 index 0000000000..69ee84bde5 --- /dev/null +++ b/lib/common/tests/options/pcmk__set_env_option_test.c @@ -0,0 +1,162 @@ +/* + * Copyright 2022 the Pacemaker project contributors + * + * The version control history for this file may have further details. + * + * This source code is licensed under the GNU Lesser General Public License + * version 2.1 or later (LGPLv2.1+) WITHOUT ANY WARRANTY. + */ + +#include +#include + +#include "mock_private.h" + +static void +bad_input_string(void **state) +{ + // Bad setenv()/unsetenv() input: NULL, empty, or containing '=' + + // Never call setenv() + pcmk__mock_setenv = true; + + pcmk__set_env_option(NULL, "new_value"); + pcmk__set_env_option("", "new_value"); + pcmk__set_env_option("name=val", "new_value"); + + pcmk__mock_setenv = false; + + // Never call unsetenv() + pcmk__mock_unsetenv = true; + + pcmk__set_env_option(NULL, NULL); + pcmk__set_env_option("", NULL); + pcmk__set_env_option("name=val", NULL); + + pcmk__mock_unsetenv = false; +} + +static void +input_too_long_for_both(void **state) +{ + /* pcmk__set_env_option() wants to set "PCMK_