development/vscode
development/vscode
1
0
Fork 0
mirror of https://github.com/Microsoft/vscode synced 2024-10-13 06:48:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

move tests to /node/ or make them not use node globals

Browse source
This commit is contained in:
Johannes Rieken 2020-02-05 09:44:49 +01:00
parent a2d76d6133
commit 5e5f9fb3e6
4 changed files with 10 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/vs/base/test/common/utils.ts
View file

@ -26,25 +26,22 @@ export class DeferredPromise<T> {
public complete(value: T) {
return new Promise(resolve => {
process.nextTick(() => {
this.completeCallback(value);
resolve();
});
this.completeCallback(value);
resolve();
});
}
public error(err: any) {
return new Promise(resolve => {
process.nextTick(() => {
this.errorCallback(err);
resolve();
});
this.errorCallback(err);
resolve();
});
}
public cancel() {
process.nextTick(() => {
new Promise(resolve => {
this.errorCallback(canceled());
resolve();
});
}
}

0
src/vs/base/test/common/path.test.ts → src/vs/base/test/node/path.test.ts
View file

5
src/vs/workbench/contrib/debug/test/browser/baseDebugView.test.ts
View file

@ -15,6 +15,7 @@ import { workbenchInstantiationService } from 'vs/workbench/test/browser/workben
import { createMockSession } from 'vs/workbench/contrib/debug/test/browser/callStack.test';
import { isStatusbarInDebugMode } from 'vs/workbench/contrib/debug/browser/statusbarColorProvider';
import { State } from 'vs/workbench/contrib/debug/common/debug';
import { isWindows } from 'vs/base/common/platform';
const $ = dom.$;
suite('Debug - Base Debug View', () => {
@ -76,7 +77,7 @@ suite('Debug - Base Debug View', () => {
renderExpressionValue(expression, container, { colorize: true, maxValueLength: 4, linkDetector });
assert.equal(container.textContent, 'this...');
expression.value = process.platform === 'win32' ? 'C:\\foo.js:5' : '/foo.js:5';
expression.value = isWindows ? 'C:\\foo.js:5' : '/foo.js:5';
container = $('.container');
renderExpressionValue(expression, container, { colorize: true, linkDetector });
assert.ok(container.querySelector('a'));
@ -109,7 +110,7 @@ suite('Debug - Base Debug View', () => {
assert.equal(label.element.textContent, 'foo:');
assert.equal(label.element.title, 'string');
variable.value = process.platform === 'win32' ? 'C:\\foo.js:5' : '/foo.js:5';
variable.value = isWindows ? 'C:\\foo.js:5' : '/foo.js:5';
expression = $('.');
name = $('.');
value = $('.');

2
src/vs/workbench/services/extensions/test/common/rpcProtocol.test.ts
View file

@ -24,7 +24,7 @@ suite('RPCProtocol', () => {
}
public send(buffer: VSBuffer): void {
process.nextTick(() => {
Promise.resolve().then(() => {
this._pair!._onMessage.fire(buffer);
});
}