fix(results): exit with status code when lint contains errors

lib/file-client/results-writer.ts

@@ -90,5 +90,5 @@ export function writeResults(results: LintMessage[], repository: string): void {
 }
 
 export function getResults(): string[] {
-    return RESULTS.length > 0 ? RESULTS : ['No errors'];
+    return RESULTS;
 }

lib/ui/components/Results.tsx

@@ -5,6 +5,7 @@ import { getResults } from '@file-client';
 import { useExitAfterRender } from '../hooks';
 
 const START_MESSAGE = '\nResults:';
+const NO_ERRORS = ['No errors'];
 
 /**
  * Results of the scan
@@ -13,7 +14,11 @@ const START_MESSAGE = '\nResults:';
 export default function Results(): JSX.Element {
     useExitAfterRender();
 
-    const items: string[] = [START_MESSAGE, ...getResults()];
+    const results = getResults();
+    const items: string[] = [
+        START_MESSAGE,
+        ...(results.length > 0 ? results : NO_ERRORS),
+    ];
 
     return (
         <Static items={items}>

lib/ui/hooks/useExitAfterRender.ts

@@ -1,6 +1,8 @@
 import { useLayoutEffect, useState } from 'react';
 import { useApp } from 'ink';
 
+import { getResults } from '@file-client';
+
 /**
  * Hook for exiting application and process after one final re-render
  */
@@ -14,8 +16,10 @@ export function useExitAfterRender(): void {
 
     useLayoutEffect(() => {
         if (hasRendered) {
+            const exitCode = getResults().length > 0 ? 1 : undefined;
+
             exit();
-            process.exit();
+            process.exit(exitCode);
         }
     }, [hasRendered, exit]);
 }

test/integration/integration.ci.test.ts

@@ -151,4 +151,8 @@ describe('CI mode', () => {
         expect(secondRunWrites.some(call => /PULLING/.test(call))).toBe(true);
         expect(fs.existsSync(cachedRepository)).toBe(true);
     });
+
+    test('exits process with error code', async () => {
+        expect(process.exit).toHaveBeenCalledWith(1);
+    });
 });