pg_dump new test: Change order of arguments
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 25 Mar 2020 18:14:14 +0000 (15:14 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 25 Mar 2020 18:15:32 +0000 (15:15 -0300)
Some getopt_long implementations don't like to have a non-option
argument before option arguments, so put the database name as the
last switch.

Per buildfarm member hoverfly.

src/bin/pg_dump/t/003_pg_dump_with_server.pl

index 3573eb2fbf89945577ddf686de1807294bc7671c..8c8063908b9353edc045b0b2ebfd6bd57e064cb5 100644 (file)
@@ -27,10 +27,10 @@ $node->safe_psql( 'postgres', "CREATE FOREIGN TABLE t1 (a int) SERVER s1");
 my ($cmd, $stdout, $stderr, $result);
 
 command_fails_like(
-   [ "pg_dump",  '-p', $port, 'postgres', '--include-foreign-data=s0' ],
+   [ "pg_dump",  '-p', $port, '--include-foreign-data=s0', 'postgres' ],
    qr/foreign-data wrapper \"dummy\" has no handler\r?\npg_dump: error: query was:.*t0/,
    "correctly fails to dump a foreign table from a dummy FDW");
 
 command_ok(
-   [ "pg_dump", '-p', $port, 'postgres', '-a', '--include-foreign-data=s2' ] ,
+   [ "pg_dump", '-p', $port, '-a', '--include-foreign-data=s2', 'postgres' ] ,
    "dump foreign server with no tables");