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/001_base.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/test/modules/commit_ts/t/001_base.pl') diff --git a/src/test/modules/commit_ts/t/001_base.pl b/src/test/modules/commit_ts/t/001_base.pl index dd41936658d..0c504421d44 100644 --- a/src/test/modules/commit_ts/t/001_base.pl +++ b/src/test/modules/commit_ts/t/001_base.pl @@ -10,7 +10,7 @@ use TestLib; use Test::More tests => 2; use PostgresNode; -my $node = get_new_node('foxtrot'); +my $node = PostgresNode->new('foxtrot'); $node->init; $node->append_conf('postgresql.conf', 'track_commit_timestamp = on'); $node->start; -- cgit v1.2.3