diff options
author | Bruce Momjian <bruce@momjian.us> | 2017-05-17 19:01:23 -0400 |
---|---|---|
committer | Bruce Momjian <bruce@momjian.us> | 2017-05-17 19:01:23 -0400 |
commit | ce554810329b9b8e862eade08b598148931eb456 (patch) | |
tree | ddc702112bdabfcdf78d1c4648614b92fd11dadd /src/test/modules | |
parent | a6fd7b7a5f7bf3a8aa3f3d076cf09d922c1c6dd2 (diff) | |
download | postgresql-ce554810329b9b8e862eade08b598148931eb456.tar.gz postgresql-ce554810329b9b8e862eade08b598148931eb456.zip |
Post-PG 10 beta1 pgperltidy run
Diffstat (limited to 'src/test/modules')
-rw-r--r-- | src/test/modules/commit_ts/t/002_standby.pl | 3 | ||||
-rw-r--r-- | src/test/modules/commit_ts/t/004_restart.pl | 4 | ||||
-rw-r--r-- | src/test/modules/test_pg_dump/t/001_base.pl | 70 |
3 files changed, 26 insertions, 51 deletions
diff --git a/src/test/modules/commit_ts/t/002_standby.pl b/src/test/modules/commit_ts/t/002_standby.pl index e7221e982be..1437519aa19 100644 --- a/src/test/modules/commit_ts/t/002_standby.pl +++ b/src/test/modules/commit_ts/t/002_standby.pl @@ -44,8 +44,7 @@ is($master_ts, $standby_ts, "standby gives same value as master"); $master->append_conf('postgresql.conf', 'track_commit_timestamp = off'); $master->restart; $master->safe_psql('postgres', 'checkpoint'); -$master_lsn = - $master->safe_psql('postgres', 'select pg_current_wal_lsn()'); +$master_lsn = $master->safe_psql('postgres', 'select pg_current_wal_lsn()'); $standby->poll_query_until('postgres', qq{SELECT '$master_lsn'::pg_lsn <= pg_last_wal_replay_lsn()}) or die "slave never caught up"; diff --git a/src/test/modules/commit_ts/t/004_restart.pl b/src/test/modules/commit_ts/t/004_restart.pl index b686925a7e0..daf42d3a029 100644 --- a/src/test/modules/commit_ts/t/004_restart.pl +++ b/src/test/modules/commit_ts/t/004_restart.pl @@ -22,12 +22,12 @@ like( ($ret, $stdout, $stderr) = $node_master->psql('postgres', qq[SELECT pg_xact_commit_timestamp('1');]); -is($ret, 0, 'getting ts of BootstrapTransactionId succeeds'); +is($ret, 0, 'getting ts of BootstrapTransactionId succeeds'); is($stdout, '', 'timestamp of BootstrapTransactionId is null'); ($ret, $stdout, $stderr) = $node_master->psql('postgres', qq[SELECT pg_xact_commit_timestamp('2');]); -is($ret, 0, 'getting ts of FrozenTransactionId succeeds'); +is($ret, 0, 'getting ts of FrozenTransactionId succeeds'); is($stdout, '', 'timestamp of FrozenTransactionId is null'); # Since FirstNormalTransactionId will've occurred during initdb, long before we diff --git a/src/test/modules/test_pg_dump/t/001_base.pl b/src/test/modules/test_pg_dump/t/001_base.pl index 3e45ccb005f..de70f4716be 100644 --- a/src/test/modules/test_pg_dump/t/001_base.pl +++ b/src/test/modules/test_pg_dump/t/001_base.pl @@ -41,12 +41,9 @@ my $tempdir_short = TestLib::tempdir_short; my %pgdump_runs = ( binary_upgrade => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/binary_upgrade.sql", - '--schema-only', - '--binary-upgrade', - '--dbname=postgres', ], }, + 'pg_dump', '--no-sync', + "--file=$tempdir/binary_upgrade.sql", '--schema-only', + '--binary-upgrade', '--dbname=postgres', ], }, clean => { dump_cmd => [ 'pg_dump', "--file=$tempdir/clean.sql", @@ -63,19 +60,16 @@ my %pgdump_runs = ( 'postgres', ], }, column_inserts => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/column_inserts.sql", - '-a', - '--column-inserts', - 'postgres', ], }, + 'pg_dump', '--no-sync', + "--file=$tempdir/column_inserts.sql", '-a', + '--column-inserts', 'postgres', ], }, createdb => { dump_cmd => [ 'pg_dump', '--no-sync', "--file=$tempdir/createdb.sql", '-C', - '-R', # no-op, just for testing + '-R', # no-op, just for testing 'postgres', ], }, data_only => { dump_cmd => [ @@ -83,7 +77,7 @@ my %pgdump_runs = ( '--no-sync', "--file=$tempdir/data_only.sql", '-a', - '-v', # no-op, just make sure it works + '-v', # no-op, just make sure it works 'postgres', ], }, defaults => { dump_cmd => [ 'pg_dump', '-f', "$tempdir/defaults.sql", 'postgres', ], @@ -126,52 +120,35 @@ my %pgdump_runs = ( "$tempdir/defaults_tar_format.tar", ], }, pg_dumpall_globals => { dump_cmd => [ - 'pg_dumpall', - '--no-sync', - "--file=$tempdir/pg_dumpall_globals.sql", - '-g', ], - }, + 'pg_dumpall', '--no-sync', + "--file=$tempdir/pg_dumpall_globals.sql", '-g', ], }, no_privs => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/no_privs.sql", - '-x', + 'pg_dump', '--no-sync', + "--file=$tempdir/no_privs.sql", '-x', 'postgres', ], }, no_owner => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/no_owner.sql", - '-O', + 'pg_dump', '--no-sync', + "--file=$tempdir/no_owner.sql", '-O', 'postgres', ], }, schema_only => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/schema_only.sql", - '-s', - 'postgres', ], - }, + 'pg_dump', '--no-sync', "--file=$tempdir/schema_only.sql", + '-s', 'postgres', ], }, section_pre_data => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/section_pre_data.sql", - '--section=pre-data', + 'pg_dump', '--no-sync', + "--file=$tempdir/section_pre_data.sql", '--section=pre-data', 'postgres', ], }, section_data => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/section_data.sql", - '--section=data', + 'pg_dump', '--no-sync', + "--file=$tempdir/section_data.sql", '--section=data', 'postgres', ], }, section_post_data => { dump_cmd => [ - 'pg_dump', - '--no-sync', - "--file=$tempdir/section_post_data.sql", + 'pg_dump', '--no-sync', "--file=$tempdir/section_post_data.sql", '--section=post-data', 'postgres', ], },); ############################################################### @@ -492,9 +469,8 @@ my %tests = ( pg_dumpall_globals => 1, section_post_data => 1, }, }, - 'GRANT SELECT(col2) ON regress_pg_dump_table TO regress_dump_test_role' - => { - create_order => 4, + 'GRANT SELECT(col2) ON regress_pg_dump_table TO regress_dump_test_role' => + { create_order => 4, create_sql => 'GRANT SELECT(col2) ON regress_pg_dump_table TO regress_dump_test_role;', regexp => qr/^ |