Skip to content

Commit 65985c1

Browse files
author
Benjamin Pasero
committed
💄 editor tests
1 parent 8aeca9f commit 65985c1

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

src/vs/workbench/contrib/files/test/browser/fileEditorTracker.test.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
*--------------------------------------------------------------------------------------------*/
55

66
import * as assert from 'assert';
7+
import { Event } from 'vs/base/common/event';
78
import { FileEditorTracker } from 'vs/workbench/contrib/files/browser/editors/fileEditorTracker';
89
import { toResource } from 'vs/base/test/common/utils';
910
import { IEditorService } from 'vs/workbench/services/editor/common/editorService';
@@ -106,7 +107,7 @@ suite('Files - FileEditorTracker', () => {
106107

107108
model.textEditorModel.setValue('Super Good');
108109

109-
await timeout(300 /* 250ms debounce delay in text file editor model manager */);
110+
await awaitEditorOpening(editorService);
110111
assert.ok(editorService.isOpen({ resource }));
111112

112113
part.dispose();
@@ -138,11 +139,17 @@ suite('Files - FileEditorTracker', () => {
138139

139140
model.textEditorModel.setValue('Super Good');
140141

141-
await timeout(300 /* 250ms debounce delay in tracker */);
142+
await awaitEditorOpening(editorService);
142143
assert.ok(editorService.isOpen(untitledEditor));
143144

144145
part.dispose();
145146
tracker.dispose();
146147
model.dispose();
147148
});
149+
150+
function awaitEditorOpening(editorService: IEditorService): Promise<void> {
151+
return new Promise(c => {
152+
Event.once(editorService.onDidActiveEditorChange)(c);
153+
});
154+
}
148155
});

src/vs/workbench/services/configuration/test/electron-browser/configurationService.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1094,7 +1094,7 @@ suite('WorkspaceConfigurationService - Folder', () => {
10941094
test('no change event when there are no global tasks', async () => {
10951095
const target = sinon.spy();
10961096
testObject.onDidChangeConfiguration(target);
1097-
await timeout(500);
1097+
await timeout(5);
10981098
assert.ok(target.notCalled);
10991099
});
11001100

0 commit comments

Comments
 (0)