From b00e5cdbd0a8086bae44d495a8de3f82b64eba5e Mon Sep 17 00:00:00 2001 From: Johannes Rieken Date: Thu, 16 Dec 2021 16:25:48 +0100 Subject: [PATCH] use randomBytes for rnd file name generation, https://github.com/microsoft/vscode/issues/139217 --- .../src/singlefolder-tests/workspace.test.ts | 6 +++--- extensions/vscode-api-tests/src/utils.ts | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/extensions/vscode-api-tests/src/singlefolder-tests/workspace.test.ts b/extensions/vscode-api-tests/src/singlefolder-tests/workspace.test.ts index d535935d5fa..a0c072998aa 100644 --- a/extensions/vscode-api-tests/src/singlefolder-tests/workspace.test.ts +++ b/extensions/vscode-api-tests/src/singlefolder-tests/workspace.test.ts @@ -981,7 +981,7 @@ suite('vscode API - workspace', () => { assert.strictEqual(document.getText(), expected); }); - test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867', async function () { + test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867, 1/3', async function () { const file1 = await createRandomFile(); const file2 = await createRandomFile(); @@ -1008,7 +1008,7 @@ suite('vscode API - workspace', () => { assert.strictEqual(e.files[1].oldUri.toString(), file2.toString()); }); - test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867', async function () { + test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867, 2/3', async function () { const event = new Promise(resolve => { let sub = vscode.workspace.onWillCreateFiles(e => { @@ -1032,7 +1032,7 @@ suite('vscode API - workspace', () => { assert.strictEqual(e.files[1].toString(), file2.toString()); }); - test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867', async function () { + test('Should send a single FileWillRenameEvent instead of separate events when moving multiple files at once#111867, 3/3', async function () { const file1 = await createRandomFile(); const file2 = await createRandomFile(); diff --git a/extensions/vscode-api-tests/src/utils.ts b/extensions/vscode-api-tests/src/utils.ts index 7bf10716227..2c9e312fb87 100644 --- a/extensions/vscode-api-tests/src/utils.ts +++ b/extensions/vscode-api-tests/src/utils.ts @@ -4,11 +4,12 @@ *--------------------------------------------------------------------------------------------*/ import * as assert from 'assert'; +import * as crypto from 'crypto'; import * as vscode from 'vscode'; import { TestFS } from './memfs'; export function rndName() { - return Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 10); + return crypto.randomBytes(8).toString('hex'); } export const testFs = new TestFS('fake-fs', true);