Fix Windows smoke npm invocation
This commit is contained in:
@@ -1,5 +1,9 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## 0.1.24 - 2026-03-10
|
||||||
|
### Fixed
|
||||||
|
- Fixed the Windows desktop managed-daemon smoke test to rebuild packaged artifacts without relying on a bare `npm` executable lookup in GitHub Actions.
|
||||||
|
|
||||||
## 0.1.23 - 2026-03-10
|
## 0.1.23 - 2026-03-10
|
||||||
### Fixed
|
### Fixed
|
||||||
- Fixed the Windows desktop managed-daemon smoke test to resolve repo paths correctly on GitHub Actions runners.
|
- Fixed the Windows desktop managed-daemon smoke test to resolve repo paths correctly on GitHub Actions runners.
|
||||||
|
|||||||
@@ -362,8 +362,12 @@ async function ensurePackagedArtifact(binaryPath) {
|
|||||||
if (process.env.PASEO_MANAGED_SMOKE_SKIP_BUILD === "1") {
|
if (process.env.PASEO_MANAGED_SMOKE_SKIP_BUILD === "1") {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
const npmExecPath = process.env.npm_execpath;
|
||||||
|
if (!npmExecPath) {
|
||||||
|
throw new Error("npm_execpath is required to build the packaged desktop artifact during smoke tests.");
|
||||||
|
}
|
||||||
try {
|
try {
|
||||||
await execFileAsync("npm", ["run", "build"], {
|
await execFileAsync(process.execPath, [npmExecPath, "run", "build"], {
|
||||||
cwd: desktopRoot,
|
cwd: desktopRoot,
|
||||||
env: process.env,
|
env: process.env,
|
||||||
maxBuffer: 20 * 1024 * 1024,
|
maxBuffer: 20 * 1024 * 1024,
|
||||||
|
|||||||
Reference in New Issue
Block a user