Replace forEach(...) for for (_ of ...)

It's faster...it reads better.

Author: andyleejordan

src/controls/checkboxQuickPick.ts

@@ -38,11 +38,12 @@ function getQuickPickItems(items: ICheckboxQuickPickItem[]): vscode.QuickPickIte
     const quickPickItems: vscode.QuickPickItem[] = [];
     quickPickItems.push({ label: confirmItemLabel, description: "" });
 
-    items.forEach((item) =>
+    for (const item of items) {
         quickPickItems.push({
-                label: convertToCheckBox(item),
-                description: item.description,
-        }));
+            label: convertToCheckBox(item),
+            description: item.description,
+        });
+    }
 
     return quickPickItems;
 }

src/features/Console.ts

@@ -129,9 +129,9 @@ function showChoicePrompt(
             });
 
         // Select the defaults
-        promptDetails.defaultChoices.forEach((choiceIndex) => {
-            checkboxQuickPickItems[choiceIndex].isSelected = true;
-        });
+        for (const choice of promptDetails.defaultChoices) {
+            checkboxQuickPickItems[choice].isSelected = true;
+        };
 
         resultThenable =
             showCheckboxQuickPick(

src/features/CustomViews.ts

@@ -22,7 +22,9 @@ export class CustomViewsFeature extends LanguageClientConsumer {
     }
 
     public dispose() {
-        this.commands.forEach((d) => d.dispose());
+        for (const command of this.commands) {
+            command.dispose();
+        }
     }
 
     public setLanguageClient(languageClient: LanguageClient) {
@@ -177,23 +179,17 @@ class HtmlContentView extends CustomView {
         let styleTags = "";
         if (this.htmlContent.styleSheetPaths &&
             this.htmlContent.styleSheetPaths.length > 0) {
-            this.htmlContent.styleSheetPaths.forEach(
-                (styleSheetPath) => {
-                    styleTags += `<link rel="stylesheet" href="${
-                        styleSheetPath.toString().replace("file://", "vscode-resource://")
-                    }">\n`;
-                });
+            for (const styleSheetPath of this.htmlContent.styleSheetPaths) {
+                styleTags += `<link rel="stylesheet" href="${styleSheetPath.toString().replace("file://", "vscode-resource://")}">\n`;
+            }
         }
 
         let scriptTags = "";
         if (this.htmlContent.javaScriptPaths &&
             this.htmlContent.javaScriptPaths.length > 0) {
-            this.htmlContent.javaScriptPaths.forEach(
-                (javaScriptPath) => {
-                    scriptTags += `<script src="${
-                        javaScriptPath.toString().replace("file://", "vscode-resource://")
-                    }"></script>\n`;
-                });
+            for (const javaScriptPath of this.htmlContent.javaScriptPaths) {
+                scriptTags += `<script src="${javaScriptPath.toString().replace("file://", "vscode-resource://")}"></script>\n`;
+            }
         }
 
         // Return an HTML page with the specified content

src/features/ExtensionCommands.ts

@@ -447,7 +447,7 @@ export class ExtensionCommandsFeature extends LanguageClientConsumer {
             case "file":
                 // If the file to save can't be found, just complete the request
                 if (!this.findTextDocument(this.normalizeFilePath(currentFileUri.fsPath))) {
-                    this.log.writeAndShowError(`File to save not found: ${currentFileUri.fsPath}.`);
+                    await this.log.writeAndShowError(`File to save not found: ${currentFileUri.fsPath}.`);
                     return EditorOperationResponse.Completed;
                 }
 

src/logging.ts

@@ -1,9 +1,10 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
+"use strict";
+
 import fs = require("fs");
 import os = require("os");
-import path = require("path");
 import vscode = require("vscode");
 
 export enum LogLevel {
@@ -19,16 +20,15 @@ export enum LogLevel {
  *  This will allow for easy mocking of the logger during unit tests.
  */
 export interface ILogger {
-    write(message: string, ...additionalMessages: string[]);
-    writeDiagnostic(message: string, ...additionalMessages: string[]);
-    writeVerbose(message: string, ...additionalMessages: string[]);
-    writeWarning(message: string, ...additionalMessages: string[]);
-    writeAndShowWarning(message: string, ...additionalMessages: string[]);
-    writeError(message: string, ...additionalMessages: string[]);
+    write(message: string, ...additionalMessages: string[]): void;
+    writeDiagnostic(message: string, ...additionalMessages: string[]): void;
+    writeVerbose(message: string, ...additionalMessages: string[]): void;
+    writeWarning(message: string, ...additionalMessages: string[]): void;
+    writeAndShowWarning(message: string, ...additionalMessages: string[]): void;
+    writeError(message: string, ...additionalMessages: string[]): void;
 }
 
 export class Logger implements ILogger {
-
     public logBasePath: vscode.Uri;
     public logSessionPath: vscode.Uri;
     public MinimumLogLevel: LogLevel = LogLevel.Normal;
@@ -40,54 +40,55 @@ export class Logger implements ILogger {
     constructor(logBasePath: vscode.Uri) {
         this.logChannel = vscode.window.createOutputChannel("PowerShell Extension Logs");
         this.logBasePath = vscode.Uri.joinPath(logBasePath, "logs");
-
         this.commands = [
             vscode.commands.registerCommand(
                 "PowerShell.ShowLogs",
                 () => { this.showLogPanel(); }),
 
             vscode.commands.registerCommand(
                 "PowerShell.OpenLogFolder",
-                () => { this.openLogFolder(); }),
+                async () => { await this.openLogFolder(); }),
         ];
     }
 
     public dispose() {
-        this.commands.forEach((command) => { command.dispose(); });
         this.logChannel.dispose();
+        for (const command of this.commands) {
+            command.dispose();
+        }
     }
 
     public getLogFilePath(baseName: string): vscode.Uri {
         return vscode.Uri.joinPath(this.logSessionPath, `${baseName}.log`);
     }
 
-    public writeAtLevel(logLevel: LogLevel, message: string, ...additionalMessages: string[]) {
+    public writeAtLevel(logLevel: LogLevel, message: string, ...additionalMessages: string[]): void {
         if (logLevel >= this.MinimumLogLevel) {
             this.writeLine(message, logLevel);
 
-            additionalMessages.forEach((line) => {
-                this.writeLine(line, logLevel);
-            });
+            for (const additionalMessage of additionalMessages) {
+                this.writeLine(additionalMessage, logLevel);
+            };
         }
     }
 
-    public write(message: string, ...additionalMessages: string[]) {
+    public write(message: string, ...additionalMessages: string[]): void {
         this.writeAtLevel(LogLevel.Normal, message, ...additionalMessages);
     }
 
-    public writeDiagnostic(message: string, ...additionalMessages: string[]) {
+    public writeDiagnostic(message: string, ...additionalMessages: string[]): void {
         this.writeAtLevel(LogLevel.Diagnostic, message, ...additionalMessages);
     }
 
-    public writeVerbose(message: string, ...additionalMessages: string[]) {
+    public writeVerbose(message: string, ...additionalMessages: string[]): void {
         this.writeAtLevel(LogLevel.Verbose, message, ...additionalMessages);
     }
 
-    public writeWarning(message: string, ...additionalMessages: string[]) {
+    public writeWarning(message: string, ...additionalMessages: string[]): void {
         this.writeAtLevel(LogLevel.Warning, message, ...additionalMessages);
     }
 
-    public writeAndShowWarning(message: string, ...additionalMessages: string[]) {
+    public writeAndShowWarning(message: string, ...additionalMessages: string[]): void {
         this.writeWarning(message, ...additionalMessages);
 
         vscode.window.showWarningMessage(message, "Show Logs").then((selection) => {
@@ -97,23 +98,22 @@ export class Logger implements ILogger {
         });
     }
 
-    public writeError(message: string, ...additionalMessages: string[]) {
+    public writeError(message: string, ...additionalMessages: string[]): void {
         this.writeAtLevel(LogLevel.Error, message, ...additionalMessages);
     }
 
-    public writeAndShowError(message: string, ...additionalMessages: string[]) {
+    public async writeAndShowError(message: string, ...additionalMessages: string[]): Promise<void> {
         this.writeError(message, ...additionalMessages);
 
-        vscode.window.showErrorMessage(message, "Show Logs").then((selection) => {
-            if (selection !== undefined) {
-                this.showLogPanel();
-            }
-        });
+        const choice = await vscode.window.showErrorMessage(message, "Show Logs");
+        if (choice !== undefined) {
+            this.showLogPanel();
+        }
     }
 
     public async writeAndShowErrorWithActions(
         message: string,
-        actions: { prompt: string; action: () => Promise<void> }[]) {
+        actions: { prompt: string; action: () => Promise<void> }[]): Promise<void> {
         this.writeError(message);
 
         const fullActions = [
@@ -134,7 +134,7 @@ export class Logger implements ILogger {
         }
     }
 
-    public async startNewLog(minimumLogLevel: string = "Normal") {
+    public async startNewLog(minimumLogLevel: string = "Normal"): Promise<void> {
         this.MinimumLogLevel = this.logLevelNameToValue(minimumLogLevel.trim());
 
         this.logSessionPath =
@@ -158,19 +158,19 @@ export class Logger implements ILogger {
         }
     }
 
-    private showLogPanel() {
+    private showLogPanel(): void {
         this.logChannel.show();
     }
 
-    private openLogFolder() {
+    private async openLogFolder(): Promise<void> {
         if (this.logSessionPath) {
             // Open the folder in VS Code since there isn't an easy way to
             // open the folder in the platform's file browser
-            vscode.commands.executeCommand("vscode.openFolder", this.logSessionPath, true);
+            await vscode.commands.executeCommand("vscode.openFolder", this.logSessionPath, true);
         }
     }
 
-    private writeLine(message: string, level: LogLevel = LogLevel.Normal) {
+    private writeLine(message: string, level: LogLevel = LogLevel.Normal): void {
         const now = new Date();
         const timestampedMessage =
             `${now.toLocaleDateString()} ${now.toLocaleTimeString()} [${LogLevel[level].toUpperCase()}] - ${message}`;

src/main.ts

@@ -183,13 +183,13 @@ export async function activate(context: vscode.ExtensionContext): Promise<IPower
 
 export function deactivate(): void {
     // Clean up all extension features
-    languageClientConsumers.forEach((languageClientConsumer) => {
+    for (const languageClientConsumer of languageClientConsumers) {
         languageClientConsumer.dispose();
-    });
+    };
 
-    commandRegistrations.forEach((commandRegistration) => {
+    for (const commandRegistration of commandRegistrations) {
         commandRegistration.dispose();
-    });
+    };
 
     // Dispose of the current session
     sessionManager.dispose();

src/session.ts

@@ -98,7 +98,6 @@ export class SessionManager implements Middleware {
     private debugSessionProcess: PowerShellProcess;
     private debugEventHandler: vscode.Disposable;
     private versionDetails: IPowerShellVersionDetails;
-    // TODO: Perhaps consumers should also register their handler disposables.
     private registeredHandlers: vscode.Disposable[] = [];
     private registeredCommands: vscode.Disposable[] = [];
     private languageClient: LanguageClient = undefined;
@@ -536,27 +535,27 @@ Type 'help' to get help.
             this.sessionDetails = await this.languageServerProcess.start("EditorServices");
         } catch (error) {
             this.log.write("PowerShell process failed to start.");
-            this.setSessionFailure("PowerShell process failed to start: ", error);
+            await this.setSessionFailure("PowerShell process failed to start: ", error);
         }
 
         if (this.sessionDetails.status === "started") {
             this.log.write("Language server started.");
             try {
                 await this.startLanguageClient(this.sessionDetails);
             } catch (error) {
-                this.setSessionFailure("Language client failed to start: ", error);
+                await this.setSessionFailure("Language client failed to start: ", error);
             }
         } else if (this.sessionDetails.status === "failed") {
             if (this.sessionDetails.reason === "unsupported") {
-                this.setSessionFailure(
+                await this.setSessionFailure(
                     "PowerShell language features are only supported on PowerShell version 5.1 and 7+. " +
                     `The current version is ${this.sessionDetails.powerShellVersion}.`);
             } else if (this.sessionDetails.reason === "languageMode") {
-                this.setSessionFailure(
+                await this.setSessionFailure(
                     "PowerShell language features are disabled due to an unsupported LanguageMode: " +
                     `${this.sessionDetails.detail}`);
             } else {
-                this.setSessionFailure(
+                await this.setSessionFailure(
                     `PowerShell could not be started for an unknown reason '${this.sessionDetails.reason}'`);
             }
         } else {
@@ -658,7 +657,7 @@ Type 'help' to get help.
         try {
             await this.languageClient.start();
         } catch (error) {
-            this.setSessionFailure("Could not start language service: ", error);
+            await this.setSessionFailure("Could not start language service: ", error);
             return;
         }
 
@@ -799,9 +798,9 @@ Type 'help' to get help.
         this.setSessionStatus(version, SessionStatus.Running);
     }
 
-    private setSessionFailure(message: string, ...additionalMessages: string[]) {
-        this.log.writeAndShowError(message, ...additionalMessages);
+    private async setSessionFailure(message: string, ...additionalMessages: string[]) {
         this.setSessionStatus("Initialization Error", SessionStatus.Failed);
+        await this.log.writeAndShowError(message, ...additionalMessages);
     }
 
     private async changePowerShellDefaultVersion(exePath: IPowerShellExeDetails) {

test/features/CustomViews.test.ts

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 import * as assert from "assert";
-import fs = require("fs");
 import path = require("path");
 import rewire = require("rewire");
 import vscode = require("vscode");
@@ -30,7 +29,7 @@ function convertToVSCodeResourceScheme(filePath: string): string {
     return vscode.Uri.file(filePath).toString().replace("file://", "vscode-resource://");
 }
 
-describe("CustomViews feature", function () {
+describe("CustomViews feature", async function () {
     const testCases: IHtmlContentViewTestCase[] = [
         {
             name: "with no JavaScript or CSS",
@@ -106,20 +105,20 @@ hello
     ];
 
     for (const testCase of testCases) {
-        it(`Correctly creates an HtmlContentView ${testCase.name}`, function () {
+        it(`Correctly creates an HtmlContentView ${testCase.name}`, async function () {
             const htmlContentView = new HtmlContentView();
 
-            const jsPaths = testCase.javaScriptFiles.map((jsFile) => {
-                const jsPath: string = path.join(__dirname, jsFile.fileName);
-                fs.writeFileSync(jsPath, jsFile.content);
-                return vscode.Uri.file(jsPath).toString();
-            });
+            const jsPaths = await Promise.all(testCase.javaScriptFiles.map(async (jsFile) => {
+                const jsPath: vscode.Uri = vscode.Uri.file(path.join(__dirname, jsFile.fileName));
+                await vscode.workspace.fs.writeFile(jsPath, Buffer.from(jsFile.content));
+                return jsPath.toString();
+            }));
 
-            const cssPaths = testCase.cssFiles.map((cssFile) => {
-                const cssPath: string = path.join(__dirname, cssFile.fileName);
-                fs.writeFileSync(cssPath, cssFile.content);
-                return vscode.Uri.file(cssPath).toString();
-            });
+            const cssPaths = await Promise.all(testCase.cssFiles.map(async (cssFile) => {
+                const cssPath: vscode.Uri = vscode.Uri.file(path.join(__dirname, cssFile.fileName));
+                await vscode.workspace.fs.writeFile(cssPath, Buffer.from(cssFile.content));
+                return cssPath.toString();
+            }));
 
             htmlContentView.htmlContent = {
                 bodyContent: testCase.htmlContent,
@@ -129,8 +128,12 @@ hello
             try {
                 assert.strictEqual(htmlContentView.getContent(), testCase.expectedHtmlString);
             } finally {
-                jsPaths.forEach((jsPath) => fs.unlinkSync(vscode.Uri.parse(jsPath).fsPath));
-                cssPaths.forEach((cssPath) => fs.unlinkSync(vscode.Uri.parse(cssPath).fsPath));
+                for (const jsPath of jsPaths) {
+                    await vscode.workspace.fs.delete(vscode.Uri.parse(jsPath));
+                }
+                for (const cssPath of cssPaths) {
+                    await vscode.workspace.fs.delete(vscode.Uri.parse(cssPath));
+                }
             }
         });
     }