From 201a76183e2056c2217129e12d68c25ec9c559c8 Mon Sep 17 00:00:00 2001 From: Andrew Dunstan Date: Thu, 29 Jul 2021 05:58:08 -0400 Subject: Unify PostgresNode's new() and get_new_node() methods There is only one constructor now for PostgresNode, with the idiomatic name 'new'. The method is not exported by the class, and must be called as "PostgresNode->new('name',[args])". All the TAP tests that use PostgresNode are modified accordingly. Third party scripts will need adjusting, which is a fairly mechanical process (I just used a sed script). --- src/test/modules/commit_ts/t/004_restart.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/test/modules/commit_ts/t/004_restart.pl') diff --git a/src/test/modules/commit_ts/t/004_restart.pl b/src/test/modules/commit_ts/t/004_restart.pl index bc9931944f3..5d92c5a2e6d 100644 --- a/src/test/modules/commit_ts/t/004_restart.pl +++ b/src/test/modules/commit_ts/t/004_restart.pl @@ -8,7 +8,7 @@ use PostgresNode; use TestLib; use Test::More tests => 16; -my $node_primary = get_new_node('primary'); +my $node_primary = PostgresNode->new('primary'); $node_primary->init(allows_streaming => 1); $node_primary->append_conf('postgresql.conf', 'track_commit_timestamp = on'); $node_primary->start; -- cgit v1.2.3