Commit 97b6f2eb authored by Andrew Dunstan's avatar Andrew Dunstan

Remove tests for pg_dumpall --exclude-database missing argument

It turns out that different getopt implementations spell the error for
missing arguments different ways. This test is of fairly marginal
value, so instead of trying to keep up  with the different error
messages just remove the test.
parent ad0bda5d
...@@ -4,7 +4,7 @@ use warnings; ...@@ -4,7 +4,7 @@ use warnings;
use Config; use Config;
use PostgresNode; use PostgresNode;
use TestLib; use TestLib;
use Test::More tests => 74; use Test::More tests => 72;
my $tempdir = TestLib::tempdir; my $tempdir = TestLib::tempdir;
my $tempdir_short = TestLib::tempdir_short; my $tempdir_short = TestLib::tempdir_short;
...@@ -151,11 +151,6 @@ command_fails_like( ...@@ -151,11 +151,6 @@ command_fails_like(
'pg_restore: options -C\/--create and -1\/--single-transaction cannot be used together' 'pg_restore: options -C\/--create and -1\/--single-transaction cannot be used together'
); );
command_fails_like(
[ 'pg_dumpall', '--exclude-database' ],
qr/\Qpg_dumpall: option '--exclude-database' requires an argument\E/,
'pg_dumpall: option --exclude-database requires an argument');
# also fails for -r and -t, but it seems pointless to add more tests for those. # also fails for -r and -t, but it seems pointless to add more tests for those.
command_fails_like( command_fails_like(
[ 'pg_dumpall', '--exclude-database=foo', '--globals-only' ], [ 'pg_dumpall', '--exclude-database=foo', '--globals-only' ],
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment