fix(solo): propagate exit status

Agoric · May 2, 2021 · 1f6537e · 1f6537e
1 parent 5e236e6
commit 1f6537e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/packages/solo/src/entrypoint.cjs b/packages/solo/src/entrypoint.cjs
@@ -19,7 +19,7 @@ const solo = esmRequire('./main.js').default;
 
 const baseprog = path.basename(process.argv[1]);
 solo(baseprog, process.argv.slice(2)).then(
-  _res => 0,
+  res => process.exit(res || 0),
   reason => {
     console.log(`error running ag-solo:`, reason);
     console.error(`\

diff --git a/packages/solo/src/main.js b/packages/solo/src/main.js
@@ -62,7 +62,7 @@ start
       const { netconfig } = parseArgs(argv.slice(1));
       if (!AG_SOLO_BASEDIR) {
         console.error(`setup: you must set $AG_SOLO_BASEDIR`);
-        return;
+        return 1;
       }
       if (!fs.existsSync(AG_SOLO_BASEDIR)) {
         await solo(progname, ['init', AG_SOLO_BASEDIR, ...argv.slice(1)]);
@@ -145,11 +145,13 @@ start
       const cp = spawnSync(`${__dirname}/../../${argv[0]}.js`, argv.slice(1), {
         stdio: 'inherit',
       });
-      process.exit(cp.status);
+      return cp.status;
     }
     default: {
       log.error(`unrecognized command ${argv[0]}`);
       log.error(`try one of: init, set-gci-ingress, start`);
+      return 1;
     }
   }
+  return 0;
 }