Removing unnecessary test

Author: Graydon Hope

packages/cli/src/commands/scan/output-scan-report.mts

@@ -96,6 +96,12 @@ export async function outputScanReport(
     process.exitCode = 1;
   }
 
+  // I don't think we emit the default error message with banner for an unhealthy report, do we?
+  // if (!scanReport.data.healthy) {
+  //   logger.fail(failMsgWithBadge(scanReport.message, scanReport.cause))
+  //   return
+  // }
+
   if (
     outputKind === OUTPUT_JSON ||
     (outputKind === OUTPUT_TEXT && filepath && filepath.endsWith('.json'))

packages/cli/test/unit/commands/ci/handle-ci.test.mts

@@ -266,25 +266,4 @@ describe('handleCi', () => {
     expect(mockDebug).toHaveBeenCalledWith('Failed to get default org slug')
     expect(mockDebugDir).toHaveBeenCalledWith({ orgSlugCResult: error })
   })
-
-  it('sets exit code to 1 when scan report is unhealthy',
-    async () => {
-      mockGetDefaultOrgSlug.mockResolvedValue({
-        ok: true,
-        data: 'test-org',
-      })
-      
-      mockGitBranch.mockResolvedValue('main')
-      mockGetRepoName.mockResolvedValue('test-repo')
-
-      mockHandleCreateNewScan.mockImplementation(async () => {
-        process.exitCode = 1;
-      })
-
-      await handleCi(false)
-
-      expect(process.exitCode).toBe(1)
-      expect(mockHandleCreateNewScan).toHaveBeenCalled()
-    }
-  )
 })