Skip to content

Commit

Permalink
chore: update resolveCliArgsFromVSCodeExecutablePath too
Browse files Browse the repository at this point in the history
  • Loading branch information
connor4312 committed Feb 3, 2022
1 parent 50a3fde commit ce0285a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/util.ts
Expand Up @@ -137,7 +137,7 @@ export async function getLatestInsidersMetadata(platform: string) {
* Usually you will want {@link resolveCliArgsFromVSCodeExecutablePath} instead.
*/
export function resolveCliPathFromVSCodeExecutablePath(vscodeExecutablePath: string, platform: DownloadPlatform) {
if (platform === 'win32') {
if (platform === 'win32-archive' || platform === 'win32-x64-archive') {
if (vscodeExecutablePath.endsWith('Code - Insiders.exe')) {
return path.resolve(vscodeExecutablePath, '../bin/code-insiders.cmd');
} else {
Expand Down Expand Up @@ -172,7 +172,7 @@ export function resolveCliPathFromVSCodeExecutablePath(vscodeExecutablePath: str
* @param vscodeExecutablePath The `vscodeExecutablePath` from `downloadAndUnzipVSCode`.
*/
export function resolveCliArgsFromVSCodeExecutablePath(vscodeExecutablePath: string, options?: Pick<TestOptions, 'reuseMachineInstall' | 'platform'>) {
const args = [resolveCliPathFromVSCodeExecutablePath(vscodeExecutablePath, options?.platform ?? process.platform)];
const args = [resolveCliPathFromVSCodeExecutablePath(vscodeExecutablePath, options?.platform ?? systemDefaultPlatform)];
if (!options?.reuseMachineInstall) {
args.push(...getProfileArguments(args));
}
Expand Down

0 comments on commit ce0285a

Please sign in to comment.