From c1b8ee4222cebe09588aaeee63fbdfc3c5c32c0d Mon Sep 17 00:00:00 2001 From: Darshit Shah Date: Sun, 11 Oct 2015 00:37:01 +0530 Subject: [PATCH 2/2] Fix Test-ftp-pasv-not-supported.px * tests/Test-ftp-pasv-not-supported.px: We do *NOT* expect any downloaded files. Also, do not negate the Test response. The test originally expected a downloaded file, but this is not true. As a result, the test would fail and return exit code 1. This was presumably the reason why the test result was negated before returning to the shell. Fix this issue, so that the test runs correctly without any hacks. --- tests/Test-ftp-pasv-not-supported.px | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/tests/Test-ftp-pasv-not-supported.px b/tests/Test-ftp-pasv-not-supported.px index 97d0610..1a473ad 100755 --- a/tests/Test-ftp-pasv-not-supported.px +++ b/tests/Test-ftp-pasv-not-supported.px @@ -41,11 +41,7 @@ my $cmdline = $WgetTest::WGETPATH . " -S ftp://localhost:{{port}}/afile.txt"; my $expected_error_code = 8; -my %expected_downloaded_files = ( - 'afile.txt' => { - content => $afile, - }, -); +my %expected_downloaded_files = (); ############################################################################### @@ -55,6 +51,6 @@ my $the_test = FTPTest->new ( cmdline => $cmdline, errcode => $expected_error_code, output => \%expected_downloaded_files); -exit !$the_test->run(); +exit $the_test->run(); # vim: et ts=4 sw=4 -- 2.6.1