Skip to content

Commit

Permalink
tests: disable automake serial_tests
Browse files Browse the repository at this point in the history
Serial mode is the old one and offers much less options for
running the tests. Generally our tests seem to work fine
with the newer parallel mode. The only reason we stuck with
serial_tests seems to be that we didn't like that it doesn't
output the test output by default. We could fix that with a
custom test driver. But will put that into a separate commit.

Change-Id: Ic7265d89142637b0963a6847c6beb06d9163bbb1
Signed-off-by: Frank Lichtenheld <[email protected]>
Acked-by: Arne Schwabe <[email protected]>
Message-Id: <[email protected]>
URL: https://www.mail-archive.com/[email protected]/msg27812.html
Signed-off-by: Gert Doering <[email protected]>
  • Loading branch information
flichtenheld authored and cron2 committed Dec 18, 2023
1 parent 8656b85 commit 55d7395
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 15 deletions.
16 changes: 1 addition & 15 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -41,20 +41,6 @@ AC_CONFIG_HEADERS([config.h include/openvpn-plugin.h])
AC_CONFIG_SRCDIR([src/openvpn/syshead.h])
AC_CONFIG_MACRO_DIR([m4])

dnl Initialize automake. automake < 1.12 didn't have serial-tests and
dnl gives an error if it sees this, but for automake >= 1.13
dnl serial-tests is required so we have to include it. Solution is to
dnl test for the version of automake (by running an external command)
dnl and provide it if necessary. Note we have to do this entirely using
dnl m4 macros since automake queries this macro by running
dnl 'autoconf --trace ...'.
m4_define([serial_tests], [
m4_esyscmd([automake --version |
head -1 |
awk '{split ($NF,a,"."); if (a[1] == 1 && a[2] >= 12) { print "serial-tests" }}'
])
])

dnl Automake 1.14+ warns if sources are in sub-directories but subdir-objects
dnl options is not enabled. However, automake before 1.15a has a bug that causes
dnl variable expansion to fail in foo_SOURCES when this option is used.
Expand All @@ -69,7 +55,7 @@ m4_define([subdir_objects], [

# This foreign option prevents autoreconf from overriding our COPYING and
# INSTALL targets:
AM_INIT_AUTOMAKE(foreign serial_tests subdir_objects 1.9) dnl NB: Do not [quote] this parameter.
AM_INIT_AUTOMAKE(foreign subdir_objects 1.9) dnl NB: Do not [quote] this parameter.
AC_CANONICAL_HOST
AC_USE_SYSTEM_EXTENSIONS

Expand Down
2 changes: 2 additions & 0 deletions tests/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ MAINTAINERCLEANFILES = \

SUBDIRS = unit_tests

AM_TESTSUITE_SUMMARY_HEADER = ' for $(PACKAGE_STRING) System Tests'

if !WIN32
test_scripts = t_client.sh t_lpback.sh t_cltsrv.sh
if HAVE_SITNL
Expand Down
2 changes: 2 additions & 0 deletions tests/unit_tests/example_test/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
AUTOMAKE_OPTIONS = foreign

AM_TESTSUITE_SUMMARY_HEADER = ' for $(PACKAGE_STRING) example Unit-Tests'

check_PROGRAMS = example_testdriver example2_testdriver

if !CROSS_COMPILING
Expand Down
2 changes: 2 additions & 0 deletions tests/unit_tests/openvpn/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
AUTOMAKE_OPTIONS = foreign

AM_TESTSUITE_SUMMARY_HEADER = ' for $(PACKAGE_STRING) Unit-Tests'

test_binaries=

if HAVE_LD_WRAP_SUPPORT
Expand Down
2 changes: 2 additions & 0 deletions tests/unit_tests/plugins/auth-pam/Makefile.am
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
AUTOMAKE_OPTIONS = foreign

AM_TESTSUITE_SUMMARY_HEADER = ' for $(PACKAGE_STRING) auth_pam Plugin Unit-Tests'

if ENABLE_PLUGIN_AUTH_PAM
check_PROGRAMS = auth_pam_testdriver
TESTS = $(check_PROGRAMS)
Expand Down

0 comments on commit 55d7395

Please sign in to comment.