Rename sleep to timeout for better wording consistency

Author: anthonykim1

src/common/utils/asyncUtils.ts

@@ -1,9 +1,5 @@
-export async function sleep(milliseconds: number) {
+export async function timeout(milliseconds: number): Promise<void> {
     return new Promise<void>((resolve) => setTimeout(resolve, milliseconds));
 }
 
-export function timeout(ms: number): Promise<void> {
-    return new Promise((resolve) => setTimeout(resolve, ms));
-}
-
 // TODO: Advanced timeout from core async: https://github.com/microsoft/vscode-python-environments/issues/953

src/features/terminal/shells/common/shellUtils.ts

@@ -1,7 +1,7 @@
 import { PythonCommandRunConfiguration, PythonEnvironment } from '../../../../api';
 import { traceInfo } from '../../../../common/logging';
 import { getGlobalPersistentState } from '../../../../common/persistentState';
-import { sleep } from '../../../../common/utils/asyncUtils';
+import { timeout } from '../../../../common/utils/asyncUtils';
 import { isWindows } from '../../../../common/utils/platformUtils';
 import { activeTerminalShellIntegration } from '../../../../common/window.apis';
 import { getConfiguration } from '../../../../common/workspace.apis';
@@ -106,11 +106,11 @@ export function extractProfilePath(content: string): string | undefined {
 
 export async function shellIntegrationForActiveTerminal(name: string, profile?: string): Promise<boolean> {
     let hasShellIntegration = activeTerminalShellIntegration();
-    let timeout = 0;
+    let timeOutstamp = 0;
 
-    while (!hasShellIntegration && timeout < SHELL_INTEGRATION_TIMEOUT) {
-        await sleep(SHELL_INTEGRATION_POLL_INTERVAL);
-        timeout += SHELL_INTEGRATION_POLL_INTERVAL;
+    while (!hasShellIntegration && timeOutstamp < SHELL_INTEGRATION_TIMEOUT) {
+        await timeout(SHELL_INTEGRATION_POLL_INTERVAL);
+        timeOutstamp += SHELL_INTEGRATION_POLL_INTERVAL;
         hasShellIntegration = activeTerminalShellIntegration();
     }
 

src/test/features/creators/autoFindProjects.unit.test.ts

@@ -1,15 +1,15 @@
+import assert from 'assert';
 import * as path from 'path';
 import * as sinon from 'sinon';
 import * as typmoq from 'typemoq';
-import * as wapi from '../../../common/workspace.apis';
-import * as winapi from '../../../common/window.apis';
-import { PythonProjectManager } from '../../../internal.api';
-import { createDeferred } from '../../../common/utils/deferred';
-import { AutoFindProjects } from '../../../features/creators/autoFindProjects';
-import assert from 'assert';
 import { Uri } from 'vscode';
 import { PythonProject } from '../../../api';
-import { sleep } from '../../../common/utils/asyncUtils';
+import { timeout } from '../../../common/utils/asyncUtils';
+import { createDeferred } from '../../../common/utils/deferred';
+import * as winapi from '../../../common/window.apis';
+import * as wapi from '../../../common/workspace.apis';
+import { AutoFindProjects } from '../../../features/creators/autoFindProjects';
+import { PythonProjectManager } from '../../../internal.api';
 
 suite('Auto Find Project tests', () => {
     let findFilesStub: sinon.SinonStub;
@@ -45,7 +45,7 @@ suite('Auto Find Project tests', () => {
         const result = await autoFindProjects.create();
         assert.equal(result, undefined, 'Result should be undefined');
 
-        await Promise.race([deferred.promise, sleep(100)]);
+        await Promise.race([deferred.promise, timeout(100)]);
         assert.ok(errorShown, 'Error message should have been shown');
     });
 
@@ -64,7 +64,7 @@ suite('Auto Find Project tests', () => {
         const result = await autoFindProjects.create();
         assert.equal(result, undefined, 'Result should be undefined');
 
-        await Promise.race([deferred.promise, sleep(100)]);
+        await Promise.race([deferred.promise, timeout(100)]);
         assert.ok(errorShown, 'Error message should have been shown');
     });