fix: integration-browser error condition (#308)

seebees · web-flow · commit dab43a0d3678 · 2020-04-21T15:35:45.000-07:00
`spawnSync` does not throw
when getting a error exit code.
`status` needs to be investigated.

Also, `@trust/keyto` was removed from dependencies.
This error was not noticed because of the above problem.
diff --git a/modules/integration-browser/package.json b/modules/integration-browser/package.json
@@ -19,6 +19,7 @@
     "@aws-sdk/karma-credential-loader": "1.0.0-alpha.6",
     "@aws-sdk/util-base64-browser": "1.0.0-beta.2",
     "@aws-sdk/util-utf8-browser": "1.0.0-beta.2",
+    "@trust/keyto": "^1.0.1",
     "@types/got": "^9.6.9",
     "@types/stream-to-promise": "^2.2.0",
     "@types/yargs": "^15.0.3",
diff --git a/modules/integration-browser/src/cli.ts b/modules/integration-browser/src/cli.ts
@@ -99,10 +99,12 @@ if (!existsSync(fixtures)) {
   }
 
   if (karma) {
-    spawnSync('npm', ['run', 'karma'], {
+    const { status } = spawnSync('npm', ['run', 'karma'], {
       cwd: __dirname,
       stdio: 'inherit'
     })
+    /* Forward the status to the parent. */
+    process.exit(status || 0)
   }
 })(cli.argv)
   .catch(err => {
diff --git a/package-lock.json b/package-lock.json

Original file line number	Diff line number	Diff line change
`@@ -99,10 +99,12 @@ if (!existsSync(fixtures)) {`
`99`	`99`	`}`
`100`	`100`
`101`	`101`	`if (karma) {`
`102`		`- spawnSync('npm', ['run', 'karma'], {`
	`102`	`+ const { status } = spawnSync('npm', ['run', 'karma'], {`
`103`	`103`	`cwd: __dirname,`
`104`	`104`	`stdio: 'inherit'`
`105`	`105`	`})`
	`106`	`+ /* Forward the status to the parent. */`
	`107`	`+ process.exit(status \|\| 0)`
`106`	`108`	`}`
`107`	`109`	`})(cli.argv)`
`108`	`110`	`.catch(err => {`