aboutsummaryrefslogtreecommitdiff
path: root/src/test/modules
diff options
context:
space:
mode:
authorBruce Momjian <bruce@momjian.us>2017-05-17 19:01:23 -0400
committerBruce Momjian <bruce@momjian.us>2017-05-17 19:01:23 -0400
commitce554810329b9b8e862eade08b598148931eb456 (patch)
treeddc702112bdabfcdf78d1c4648614b92fd11dadd /src/test/modules
parenta6fd7b7a5f7bf3a8aa3f3d076cf09d922c1c6dd2 (diff)
downloadpostgresql-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.pl3
-rw-r--r--src/test/modules/commit_ts/t/004_restart.pl4
-rw-r--r--src/test/modules/test_pg_dump/t/001_base.pl70
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/^