fix: ci flakiness (#1423)

raineorshine · Jun 12, 2024 · 42075d6 · 42075d6
1 parent 0f17df2
commit 42075d6
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 5 deletions.
diff --git a/test/doctor.test.ts b/test/doctor.test.ts
@@ -109,8 +109,16 @@ describe('doctor', function () {
       await fs.rm(lockfilePath, { recursive: true, force: true })
       await fs.rm(nodeModulesPath, { recursive: true, force: true })
 
-      // stderr should be empty
-      stderr.should.equal('')
+      stderr = stripAnsi(stderr).trim()
+      if (stderr !== '') {
+        stderr.should.equal(`> test
+> node test.js
+
+
+
+> test
+> node test.js`)
+      }
 
       // stdout should include normal output
       stripAnsi(stdout).should.containIgnoreCase('Tests pass')
@@ -154,8 +162,16 @@ describe('doctor', function () {
       await fs.rm(lockfilePath, { recursive: true, force: true })
       await fs.rm(nodeModulesPath, { recursive: true, force: true })
 
-      // stderr should be empty
-      stderr.should.equal('')
+      stderr = stripAnsi(stderr).trim()
+      if (stderr !== '') {
+        stripAnsi(stderr).should.equal(`> test
+> echo 'Test Success'
+
+
+
+> test
+> echo 'Test Success'`)
+      }
 
       // stdout should include normal output
       stripAnsi(stdout).should.containIgnoreCase('Tests pass')

diff --git a/test/helpers/doctorHelpers.ts b/test/helpers/doctorHelpers.ts
@@ -77,7 +77,16 @@ export const testPass = ({ packageManager }: { packageManager: PackageManagerNam
     if (packageManager === 'bun') {
       stripAnsi(stderr).should.equal('$ echo Success\n\n$ echo Success\n\n')
     } else {
-      stderr.should.equal('')
+      stderr = stripAnsi(stderr).trim()
+      if (stderr !== '') {
+        stderr.should.equal(`> test
+> echo Success
+
+
+
+> test
+> echo Success`)
+      }
     }
 
     // stdout should include normal output