fix flaky tests

sandy081 · sandy081 · commit 4a651127fffc · 2020-03-27T16:14:15.000+01:00
diff --git a/src/vs/platform/configuration/common/configurationService.ts b/src/vs/platform/configuration/common/configurationService.ts
@@ -29,6 +29,7 @@ export class ConfigurationService extends Disposable implements IConfigurationSe
 		fileService: IFileService
 	) {
 		super();
+		this._register(fileService.watch(settingsResource));
 		this.userConfiguration = this._register(new UserSettings(this.settingsResource, undefined, fileService));
 		this.configuration = new Configuration(new DefaultConfigurationModel(), new ConfigurationModel());
 
diff --git a/src/vs/platform/configuration/test/node/configurationService.test.ts b/src/vs/platform/configuration/test/node/configurationService.test.ts
@@ -22,6 +22,7 @@ import { IDisposable } from 'vs/base/common/lifecycle';
 import { DiskFileSystemProvider } from 'vs/platform/files/node/diskFileSystemProvider';
 import { Schemas } from 'vs/base/common/network';
 import { IFileService } from 'vs/platform/files/common/files';
+import { VSBuffer } from 'vs/base/common/buffer';
 
 suite('ConfigurationService - Node', () => {
 
@@ -110,18 +111,18 @@ suite('ConfigurationService - Node', () => {
 
 	test('trigger configuration change event when file does not exist', async () => {
 		const res = await testFile('config', 'config.json');
-
-		const service = new ConfigurationService(URI.file(res.testFile), fileService);
+		const settingsFile = URI.file(res.testFile);
+		const service = new ConfigurationService(settingsFile, fileService);
 		await service.initialize();
-		return new Promise((c, e) => {
+		return new Promise(async (c, e) => {
 			const disposable = Event.filter(service.onDidChangeConfiguration, e => e.source === ConfigurationTarget.USER)(async (e) => {
 				disposable.dispose();
 				assert.equal(service.getValue('foo'), 'bar');
 				service.dispose();
 				await res.cleanUp();
 				c();
 			});
-			fs.writeFileSync(res.testFile, '{ "foo": "bar" }');
+			await fileService.writeFile(settingsFile, VSBuffer.fromString('{ "foo": "bar" }'));
 		});
 
 	});
