Failure should result in non-zero exit code (#174)

- Set error code to 1 on failure
danvk · Mar 28, 2020 · f352bfa · f352bfa
1 parent d9c5749
commit f352bfa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/cli.ts b/src/cli.ts
@@ -287,5 +287,7 @@ if (require.main === module) {
       } else {
         logError('Failed to explore', error);
       }
+
+      process.exitCode = 1;
     });
 }
diff --git a/tests/e2e/cli.test.ts b/tests/e2e/cli.test.ts
@@ -38,9 +38,7 @@ describe('CLI', () => {
   it('should output result as tsv', async () => {
     const result = await execute(SCRIPT_PATH, ['data/inline-map.js', '--tsv']);
 
-    expect(result)
-      .to.have.string('Source\tSize')
-      .and.have.string('src/bar.js\t104');
+    expect(result).to.have.string('Source\tSize').and.have.string('src/bar.js\t104');
   });
 
   it('should output result as tsv excluding source map bytes', async () => {
@@ -59,9 +57,7 @@ describe('CLI', () => {
   it('should output result as tsv excluding unmapped bytes', async () => {
     const result = await execute(SCRIPT_PATH, ['data/with-unmapped.js', '--tsv', '--only-mapped']);
 
-    expect(result)
-      .to.have.string('Source\tSize')
-      .and.not.have.string('[unmapped]\t');
+    expect(result).to.have.string('Source\tSize').and.not.have.string('[unmapped]\t');
   });
 
   it('should output multiple results as tsv', async () => {