Fix comments.

Author: KristianDD

lib/commands/update.ts

@@ -15,16 +15,23 @@ export class UpdateCommand implements ICommand {
 		this.$projectData.initializeProjectData();
 	}
 
-	static readonly folders: string[] = ["lib", "hooks", "platforms", "node_modules"];
+	static readonly folders: string[] = [
+		constants.LIB_DIR_NAME,
+		constants.HOOKS_DIR_NAME,
+		constants.PLATFORMS_DIR_NAME,
+		constants.NODE_MODULES_FOLDER_NAME
+	];
 	static readonly tempFolder: string = ".tmp_backup";
+	static readonly updateFailMessage: string = "Could not update the project!";
+	static readonly backupFailMessage: string = "Could not backup project folders!";
 
 	public async execute(args: string[]): Promise<void> {
 		const tmpDir = path.join(this.$projectData.projectDir, UpdateCommand.tempFolder);
 
 		try {
 			this.backup(tmpDir);
 		} catch (error) {
-			this.$logger.error("Could not backup project folders!");
+			this.$logger.error(UpdateCommand.backupFailMessage);
 			this.$fs.deleteDirectory(tmpDir);
 			return;
 		}
@@ -33,7 +40,7 @@ export class UpdateCommand implements ICommand {
 			await this.executeCore(args);
 		} catch (error) {
 			this.restoreBackup(tmpDir);
-			this.$logger.error("Could not update the project!");
+			this.$logger.error(UpdateCommand.updateFailMessage);
 		} finally {
 			this.$fs.deleteDirectory(tmpDir);
 		}

lib/constants.ts

@@ -16,6 +16,8 @@ export const TEST_RUNNER_NAME = "nativescript-unit-test-runner";
 export const LIVESYNC_EXCLUDED_FILE_PATTERNS = ["**/*.js.map", "**/*.ts"];
 export const XML_FILE_EXTENSION = ".xml";
 export const PLATFORMS_DIR_NAME = "platforms";
+export const HOOKS_DIR_NAME = "hooks";
+export const LIB_DIR_NAME = "lib";
 export const CODE_SIGN_ENTITLEMENTS = "CODE_SIGN_ENTITLEMENTS";
 export const AWAIT_NOTIFICATION_TIMEOUT_SECONDS = 9;
 export const SRC_DIR = "src";

test/update.ts

@@ -178,7 +178,7 @@ describe("update command method tests", () => {
 			});
 		});
 
-		it("calls remove for all falders", async () => {
+		it("calls remove for all folders", async () => {
 			const testInjector = createTestInjector();
 			const fs = testInjector.resolve("fs");
 			const deleteDirectory: sinon.SinonStub = sandbox.stub(fs, "deleteDirectory");