From 5168d3d975a95fd7f04b92a1d3e7496f7bc1379d Mon Sep 17 00:00:00 2001 From: Anthony Dresser Date: Fri, 19 Jun 2020 01:02:19 -0700 Subject: [PATCH] Better Smoke test setup (#11005) * add handling for 404 * make the error message make more sense --- test/smoke/src/sql/main.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/test/smoke/src/sql/main.ts b/test/smoke/src/sql/main.ts index 9100e43541..1d7c2c8f15 100644 --- a/test/smoke/src/sql/main.ts +++ b/test/smoke/src/sql/main.ts @@ -82,11 +82,13 @@ function getVersion(remote: boolean) { } function fetch(url: string): Promise { - return new Promise((resolve, reject) => { + return new Promise((resolve, reject) => { const buffers: Buffer[] = []; - const req = request(url, async res => { + const req = request(url, res => { if (res.headers.location) { - resolve(await fetch(res.headers.location)); + resolve(fetch(res.headers.location)); + } else if (res.statusCode === 404) { + reject(`${url}: ${res.statusMessage}`); } else { res.on('data', chunk => buffers.push(chunk)); res.on('end', () => {