microsoft#47262 Fix tests

sandy081 · sandy081 · commit 7ed2a1d3d7cf · 2018-05-17T15:50:15.000+02:00
diff --git a/src/vs/workbench/api/node/extHostTreeViews.ts b/src/vs/workbench/api/node/extHostTreeViews.ts
@@ -133,7 +133,7 @@ class ExtHostTreeView<T> extends Disposable {
 					this.refreshPromise = this.refreshPromise.then(() => refreshingPromise);
 				}
 				return last ? [...last, current] : [current];
-			}, 200, true)(elements => {
+			}, 200)(elements => {
 				const _promiseCallback = promiseCallback;
 				refreshingPromise = null;
 				this.refresh(elements).then(() => _promiseCallback());
diff --git a/src/vs/workbench/test/electron-browser/api/extHostTreeViews.test.ts b/src/vs/workbench/test/electron-browser/api/extHostTreeViews.test.ts
@@ -252,13 +252,9 @@ suite('ExtHostTreeView', function () {
 	});
 
 	test('refresh calls are throttled on roots', function (done) {
-		let counter = 0;
 		target.onRefresh.event(actuals => {
-			counter++;
 			assert.equal(undefined, actuals);
-			if (counter > 1) {
-				done();
-			}
+			done();
 		});
 		onDidChangeTreeNode.fire();
 		onDidChangeTreeNode.fire();
@@ -267,18 +263,9 @@ suite('ExtHostTreeView', function () {
 	});
 
 	test('refresh calls are throttled on elements', function (done) {
-		let counter = 0;
 		target.onRefresh.event(actuals => {
-			counter++;
-			if (counter === 1) {
-				assert.deepEqual(['0/0:a'], Object.keys(actuals));
-			}
-			if (counter === 2) {
-				assert.deepEqual(['0/0:a', '0/0:b'], Object.keys(actuals));
-			}
-			if (counter > 1) {
-				done();
-			}
+			assert.deepEqual(['0/0:a', '0/0:b'], Object.keys(actuals));
+			done();
 		});
 
 		onDidChangeTreeNode.fire(getNode('a'));

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ class ExtHostTreeView<T> extends Disposable {`
`133`	`133`	`this.refreshPromise = this.refreshPromise.then(() => refreshingPromise);`
`134`	`134`	`}`
`135`	`135`	`return last ? [...last, current] : [current];`
`136`		`- }, 200, true)(elements => {`
	`136`	`+ }, 200)(elements => {`
`137`	`137`	`const _promiseCallback = promiseCallback;`
`138`	`138`	`refreshingPromise = null;`
`139`	`139`	`this.refresh(elements).then(() => _promiseCallback());`