E589 Always kill web servers used in tests · percona/postgres@77d4807 · GitHub
[go: up one dir, main page]
Skip to content

Commit 77d4807

Browse files
Anders ÅstrandAndersAstrand
Anders Åstrand
authored and
AndersAstrand
committed
Always kill web servers used in tests
Previously these processes would stay around if the test script crashed or was otherwise aborted before reaching the kill command.
1 parent 3e0639c commit 77d4807

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

‎contrib/pg_tde/t/003_remote_config.pl

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@
4949
}
5050

5151
my $pid = MyWebServer->new(8888)->background();
52+
END { kill('TERM', $pid); }
5253

5354
PGTDE::setup_files_dir(basename($0));
5455

@@ -85,8 +86,6 @@
8586

8687
$node->stop;
8788

88-
kill('TERM', $pid);
89-
9089
# Compare the expected and out file
9190
my $compare = PGTDE->compare_results();
9291

‎contrib/pg_tde/t/006_remote_vault_config.pl

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@
5858
}
5959

6060
my $pid = MyWebServer->new(8889)->background();
61+
END { kill('TERM', $pid); }
6162

6263
PGTDE::setup_files_dir(basename($0));
6364

@@ -94,8 +95,6 @@
9495

9596
$node->stop;
9697

97-
kill('TERM', $pid);
98-
9998
# Compare the expected and out file
10099
my $compare = PGTDE->compare_results();
101100

0 commit comments

Comments
 (0)
0