Skip to content

Commit ce6fb78

Browse files
author
MIYASHITA, Akihiro
committed
fix bug in executing multiple testcases with postgresql
1 parent e07c7c6 commit ce6fb78

File tree

3 files changed

+14
-15
lines changed

3 files changed

+14
-15
lines changed

lib/connection.js

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ class PostgreSQLCasette {
138138
});
139139
}
140140

141-
async function connect() {
141+
async function startServer() {
142142
return new Promise((_res, _rej) => {
143143
let done = false;
144144
const res = (...args) => {
@@ -178,19 +178,18 @@ class PostgreSQLCasette {
178178
});
179179
}
180180

181-
for (let retries = 5; retries > 0; retries--) {
182-
if (fs.existsSync('/var/lib/postgresql/data/postmaster.pid')) {
183-
fs.unlinkSync('/var/lib/postgresql/data/postmaster.pid');
184-
}
185-
try {
186-
await connect();
187-
break;
188-
} catch (e) {
189-
if (options.verbose) {
190-
console.error(e);
181+
if (!fs.existsSync('/var/run/postgresql/.s.PGSQL.5432')) {
182+
for (let retries = 5; retries > 0; retries--) {
183+
try {
184+
await startServer();
185+
break;
186+
} catch (e) {
187+
if (options.verbose) {
188+
console.error(e);
189+
}
191190
}
191+
await sleep(1000);
192192
}
193-
await sleep(1000);
194193
}
195194

196195
for (let retries = 20; retries > 0; retries--) {

package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "track-db-test-library",
3-
"version": "2.5.0-rc3",
3+
"version": "2.5.0-rc4",
44
"description": "Test utility for Track database challenges",
55
"main": "index.js",
66
"scripts": {

0 commit comments

Comments
 (0)