assert no RPC leakage in notebook tests

microsoft · Feb 19, 2021 · 61df162 · 61df162
1 parent 3723081
commit 61df162
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/extensions/vscode-api-tests/src/singlefolder-tests/notebook.document.test.ts b/extensions/vscode-api-tests/src/singlefolder-tests/notebook.document.test.ts
@@ -33,7 +33,7 @@ suite('Notebook Document', function () {
 	const disposables: vscode.Disposable[] = [];
 
 	suiteTeardown(async function () {
-		// utils.assertNoRpc();
+		utils.assertNoRpc();
 		await utils.revertAllDirty();
 		await utils.closeAllEditors();
 		utils.disposeAll(disposables);

diff --git a/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts b/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts
@@ -6,7 +6,7 @@
 import 'mocha';
 import * as assert from 'assert';
 import * as vscode from 'vscode';
-import { createRandomFile, asPromise, disposeAll, closeAllEditors, revertAllDirty, saveAllEditors } from '../utils';
+import { createRandomFile, asPromise, disposeAll, closeAllEditors, revertAllDirty, saveAllEditors, assertNoRpc } from '../utils';
 
 // Since `workbench.action.splitEditor` command does await properly
 // Notebook editor/document events are not guaranteed to be sent to the ext host when promise resolves
@@ -70,6 +70,9 @@ suite('Notebook API tests', function () {
 	const disposables: vscode.Disposable[] = [];
 
 	suiteTeardown(async function () {
+
+		assertNoRpc();
+
 		await revertAllDirty();
 		await closeAllEditors();