Skip to content

Commit

Permalink
Merge pull request #6 from sawyercroteau/scro/update-success-state
Browse files Browse the repository at this point in the history
Not running tests shouldn't be a success state
  • Loading branch information
paddlefish authored Sep 17, 2024
2 parents ae2653d + 3ddc8cb commit 367c177
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/cli/cli.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ const runHandlerNoTry = async (args: CliRunOptions): Promise<number> => {
const done = (): void => {
console.log(JSON.stringify(testResults))
srv.close()
if (testResults.fail === 0) {
if (testResults.fail === 0 && testResults.test > 0) {
resolve(0)
}
resolve(1)
Expand Down

0 comments on commit 367c177

Please sign in to comment.