@matheusbsbs / matheus-bagliano / commits / 13ca8fb79e
Fork 0

t5150: skip request-pull test if Perl is disabled

The git-request-pull.sh script invokes Perl, so it requires Perl to be available, but the associated test t5150 does not skip itself when Perl has been disabled, which then makes subtest 4 through 10 fail. Subtest 3 still passes, but for the wrong reasons (it expects git-request-pull to fail, and it does fail when Perl is not available). The initial two subtests that do pass are only doing setup. To prevent t5150 from failing the build when NO_PERL=1, add a check that sets skip_all when "! test_have_prereq PERL", just like how for example t3701-add-interactive skips itself when Perl has been disabled. Signed-off-by: Ruud van Asseldonk <dev@veniogames.com> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Ruud van Asseldonk committed Nov 26, 2019 at 01:02 UTC 13ca8fb79e37f66e31b20c4444fa23ccd671c03a
1 file changed +6
t/t5150-request-pull.sh
+6
@@ -4,6 +4,12 @@ test_description='Test workflows involving pull request.'
4
5 . ./test-lib.sh
6
7 +if ! test_have_prereq PERL
8 +then
9 + skip_all='skipping request-pull tests, perl not available'
10 + test_done
11 +fi
12 +
13 test_expect_success 'setup' '
14
15 git init --bare upstream.git &&