chore: exit successfully when test report directory doesn't exist (#2…

…7986)
cypress-io · Oct 6, 2023 · 92cc6da · 92cc6da · cypress-bot · Oct 6, 2023
1 parent 9659e18
commit 92cc6da
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/scripts/unit/verify-mocha-results-spec.js b/scripts/unit/verify-mocha-results-spec.js
@@ -7,6 +7,8 @@ if (process.platform !== 'win32') {
   describe('verify-mocha-results', () => {
     let cachedEnv = { ...process.env }
 
+    let fsAccessStub
+
     afterEach(() => {
       sinon.restore()
       Object.assign(process.env, cachedEnv)
@@ -19,11 +21,19 @@ if (process.platform !== 'win32') {
         Dispatched: { TaskInfo: { Environment: { somekey: 'someval' } } },
       }))
 
+      fsAccessStub = sinon.stub(fs, 'access').withArgs('/tmp/cypress/junit').resolves()
+
       sinon.stub(fs, 'readdir').withArgs('/tmp/cypress/junit').resolves([
         'report.xml',
       ])
     })
 
+    it('exits normally when report directory does not exist', async () => {
+      fsAccessStub.rejects()
+
+      await verifyMochaResults()
+    })
+
     it('does not fail with normal report', async () => {
       fs.readFile
       .withArgs('/tmp/cypress/junit/report.xml')

diff --git a/scripts/verify-mocha-results.js b/scripts/verify-mocha-results.js
@@ -94,6 +94,14 @@ async function checkReportFiles (filenames) {
 
 async function verifyMochaResults () {
   try {
+    try {
+      await fs.access(REPORTS_PATH)
+    } catch {
+      console.log('Reports directory does not exist - assuming no tests ran')
+
+      return
+    }
+
     const filenames = await fs.readdir(REPORTS_PATH)
 
     const resultCount = filenames.length