From df506c2d9c47521eb8551b5bcc78d30fbc00edc2 Mon Sep 17 00:00:00 2001 From: Nikolai Laevskii Date: Fri, 12 May 2023 17:34:18 +0200 Subject: [PATCH] Refactor convertInstallPathToAbsolute --- dist/index.js | 15 +++++---------- src/installer.ts | 17 ++++++----------- 2 files changed, 11 insertions(+), 21 deletions(-) diff --git a/dist/index.js b/dist/index.js index e0bc888..77d557d 100644 --- a/dist/index.js +++ b/dist/index.js @@ -350,7 +350,6 @@ class DotnetInstallScript { setupScriptBash() { return __awaiter(this, void 0, void 0, function* () { (0, fs_1.chmodSync)(this.escapedScript, '777'); - this.scriptArguments = []; this.scriptPath = yield io.which(this.escapedScript, true); }); } @@ -385,15 +384,11 @@ class DotnetInstallScript { exports.DotnetInstallScript = DotnetInstallScript; class DotnetInstallDir { static convertInstallPathToAbsolute(installDir) { - let transformedPath; - if (path_1.default.isAbsolute(installDir)) { - transformedPath = installDir; - } - else { - transformedPath = installDir.startsWith('~') - ? path_1.default.join(os_1.default.homedir(), installDir.slice(1)) - : (transformedPath = path_1.default.join(process.cwd(), installDir)); - } + if (path_1.default.isAbsolute(installDir)) + return path_1.default.normalize(installDir); + const transformedPath = installDir.startsWith('~') + ? path_1.default.join(os_1.default.homedir(), installDir.slice(1)) + : path_1.default.join(process.cwd(), installDir); return path_1.default.normalize(transformedPath); } static addToPath() { diff --git a/src/installer.ts b/src/installer.ts index e0df8ee..5147df9 100644 --- a/src/installer.ts +++ b/src/installer.ts @@ -154,9 +154,6 @@ export class DotnetInstallScript { private async setupScriptBash() { chmodSync(this.escapedScript, '777'); - - this.scriptArguments = []; - this.scriptPath = await io.which(this.escapedScript, true); } @@ -214,14 +211,12 @@ export abstract class DotnetInstallDir { : DotnetInstallDir.default[getPlatform()]; private static convertInstallPathToAbsolute(installDir: string): string { - let transformedPath; - if (path.isAbsolute(installDir)) { - transformedPath = installDir; - } else { - transformedPath = installDir.startsWith('~') - ? path.join(os.homedir(), installDir.slice(1)) - : (transformedPath = path.join(process.cwd(), installDir)); - } + if (path.isAbsolute(installDir)) return path.normalize(installDir); + + const transformedPath = installDir.startsWith('~') + ? path.join(os.homedir(), installDir.slice(1)) + : path.join(process.cwd(), installDir); + return path.normalize(transformedPath); }