diff options
| author | Luke Shumaker <lukeshu@lukeshu.com> | 2017-02-19 03:06:16 -0500 |
|---|---|---|
| committer | Luke Shumaker <lukeshu@lukeshu.com> | 2017-04-11 13:02:27 -0400 |
| commit | d71dbf8ee17fa8def38ed027fe8cd851fee701f1 (patch) | |
| tree | 539ff5994ef62c2eeee36cac430882e2eea69d89 /test | |
| parent | 7f6e7623c68113ee0fea05c0bdeb548de6a82e09 (diff) | |
libremakepkg: correctly exit if a hook fails
Diffstat (limited to 'test')
| -rw-r--r-- | test/libremakepkg-test.sh | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/test/libremakepkg-test.sh b/test/libremakepkg-test.sh index ddccab0..50d43f6 100644 --- a/test/libremakepkg-test.sh +++ b/test/libremakepkg-test.sh @@ -125,6 +125,7 @@ it_otherwise_fails_as_normal_user() { # I do this to give it a chance of passing cp libremakepkg.d/PKGBUILD-hello "$tmpdir/PKGBUILD" cd "$tmpdir" + local stat=0 libremakepkg >$tmpdir/stdout 2>$tmpdir/stderr || stat=$? @@ -132,3 +133,19 @@ it_otherwise_fails_as_normal_user() { empty $tmpdir/stdout not empty $tmpdir/stderr } + +it_fails_if_a_hook_fails() { + require network sudo || return 0 + cp libremakepkg.d/PKGBUILD-hello "$tmpdir/PKGBUILD" + cd "$tmpdir" + local stat=0 + + sed -i 's/^BLACKLIST=.*/&-bogus/' "$XDG_CONFIG_HOME"/libretools/libretools.conf + trap 'sed -i s/-bogus// "$XDG_CONFIG_HOME"/libretools/libretools.conf' RETURN + + libremessages msg 'Creating a chroot, may take a few minutes' &>/dev/tty + testsudo libremakepkg -l "$roundup_test_name" >$tmpdir/stdout 2>$tmpdir/stderr || stat=$? + + [[ $stat != 0 ]] + tail -n1 $tmpdir/stderr | grep -qF '==> ERROR: Failure(s) in check_pkgbuild: check_pkgbuild_nonfree' +} |