Skip to content

Commit 7750ed1

Browse files
committed
fixes to tests
1 parent c4b5112 commit 7750ed1

3 files changed

Lines changed: 5 additions & 2 deletions

src/test/extension.jupyter.comms.jupyterClient.test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,10 @@ suite('JupyterClient', () => {
7272
output = new MockOutputChannel('Jupyter');
7373
jupyter = new JupyterClientAdapter(output, __dirname);
7474
});
75-
suiteTeardown(done => {
75+
suiteTeardown(() => {
7676
disposable.dispose();
7777
});
78-
teardown(done => {
78+
teardown(() => {
7979
process.env['PYTHON_DONJAYAMANNE_TEST'] = '1';
8080
process.env['DEBUG_DJAYAMANNE_IPYTHON'] = '0';
8181
output.dispose();
@@ -551,6 +551,7 @@ suite('JupyterClient', () => {
551551
});
552552
}).catch(reason => {
553553
assert.fail(reason, undefined, 'Failed to retrieve kernelspecs', '');
554+
done();
554555
});
555556
});
556557
});

src/test/extension.jupyter.comms.jupyterKernel.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -385,6 +385,7 @@ suite('Jupyter Kernel', () => {
385385
});
386386
}).catch(reason => {
387387
assert.fail(reason, undefined, 'Failed to interrupt Kernel', '');
388+
done();
388389
});
389390
});
390391
});

src/test/extension.jupyter.comms.jupyterKernelManager.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@ suite('Kernel Manager', () => {
133133
done();
134134
}).catch(reason => {
135135
assert.fail(reason, null, 'Some error', '');
136+
done();
136137
});
137138
});
138139
});

0 commit comments

Comments
 (0)