ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-16 10:58:30 -05:00
Code Issues Packages Projects Releases Wiki Activity

Notebook Port Test Fix (#5433)

Browse Source 
* Test fixes to not rely on 7100

* PR comments
This commit is contained in:
Chris LaFreniere
2019-05-08 13:16:09 -07:00
committed by GitHub
parent 7df793f208
commit 1cc3cb5408
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
extensions/notebook/src/test/common/port.test.ts
View File

@@ -26,8 +26,8 @@ describe('Ports', () => {
// once listening, find another free port and assert that the port is different from the opened one
ports.findFreePort(7000, 50, 300000).then(freePort => {
assert.ok(freePort >= 7000 && freePort !== initialPort);
server.close();
assert.ok(freePort >= 7000 && freePort !== initialPort);
done();
}, err => done(err));
@@ -51,11 +51,10 @@ describe('Ports', () => {
// once listening, find another free port and assert that the port is different from the opened one
options.minPort = initialPort;
options.maxRetriesPerStartPort = 1;
options.totalRetryLoops = 50;
options.totalRetryLoops = 10;
ports.strictFindFreePort(options).then(freePort => {
assert.ok(freePort >= 7100 && freePort !== initialPort);
server.close();
assert(freePort !== initialPort, `Expected freePort !== initialPort, Actual: ${freePort}`);
done();
}, err => done(err));
});