fix the certification signing

joyeecheung · Jan 10, 2024 · cea9a5d · cea9a5d
1 parent e12780f
commit cea9a5d
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/test/common/sea.js b/test/common/sea.js
@@ -78,17 +78,18 @@ function injectAndCodeSign(targetExecutable, resource) {
     }
     if (signtoolFound) {
       let certificatesFound = false;
-      let stderr;
+      let stderr, child;
       try {
-        ({ stderr } = spawnSyncAndExitWithoutError('signtool', [ 'sign', '/fd', 'SHA256', targetExecutable ], {}));
+        ({ child, stderr } = spawnSyncAndExitWithoutError('signtool', [ 'sign', '/fd', 'SHA256', targetExecutable ], { status: undefined }));
+        if (child.status !== 0) throw new Error(`- process terminated with status ${child.status}, expected 0`);
         certificatesFound = true;
       } catch (err) {
         if (!/SignTool Error: No certificates were found that met all the given criteria/.test(stderr)) {
           throw err;
         }
       }
       if (certificatesFound) {
-        spawnSyncAndExitWithoutError('signtool', 'verify', '/pa', 'SHA256', targetExecutable, {});
+        spawnSyncAndExitWithoutError('signtool', [ 'verify', '/pa', 'SHA256', targetExecutable ], {});
       }
     }
   }