eperm ci

Author: lukekarrys

.github/workflows/eperm.yml

@@ -1,10 +1,10 @@
-name: CI
+name: EPERM
 
 on:
   workflow_dispatch:
     inputs:
       iterations:
-        default: '20000'
+        description: How many iterations to run
 
 jobs:
   test:
@@ -26,6 +26,6 @@ jobs:
         run: npm install
 
       - name: Run Tests
-        run: npm test -- test/integration/eperm.ts
+        run: npm test -- test/integration/eperm.ts --disable-coverage
         env:
           RIMRAF_TEST_EPERM_ITERATIONS: ${{ inputs.iterations }}

test/integration/eperm.ts

@@ -2,17 +2,26 @@ import t, { Test } from 'tap'
 import { mkdirSync, readdirSync, writeFileSync } from 'fs'
 import { sep, join } from 'path'
 import { globSync } from 'glob'
-import { windows, windowsSync } from '../../src/index.js'
 import { randomBytes } from 'crypto'
 import assert from 'assert'
 
+const isWinCI = process.env.CI && process.platform === 'win32'
+
+const mockWindows = async (t: Test) =>
+  (await t.mockImport(
+    '../../src/index.js',
+  )) as typeof import('../../src/index.js')
+
 const setup = (t: Test) => {
-  const [iterations, depth, fileCount, fileKb] =
-    process.env.CI && process.platform === 'win32' ?
-      [Number(process.env?.RIMRAF_TEST_EPERM_ITERATIONS ?? 1000), 15, 7, 100]
-    : [200, 8, 3, 10]
+  const depth = 10
+  const fileCount = 7
+  const fileKb = 100
+  const iterations =
+    process.env?.RIMRAF_TEST_EPERM_ITERATIONS ?
+      +process.env.RIMRAF_TEST_EPERM_ITERATIONS
+    : isWinCI ? 1000
+    : 100
 
-  t.plan(10)
   const dir = t.testdir()
   const readdir = () => readdirSync(dir)
 
@@ -46,6 +55,14 @@ const setup = (t: Test) => {
         }
       }
 
+      const tick = () => {
+        if (iteration % (iterations / 10) === 0) {
+          const now = Date.now()
+          t.ok(true, `${iteration} (${now - previous}ms / ${now - start}ms)`)
+          previous = now
+        }
+      }
+
       const assertContents = (expected: boolean = false) => {
         const found = readdir()
         assert(
@@ -82,6 +99,7 @@ const setup = (t: Test) => {
       iteration += 1
       yield {
         matches,
+        tick,
         error: (error: unknown, path: string) =>
           new RunError('rimraf error', { path, error }),
         assertResult: (result: [string, boolean][]) => {
@@ -99,11 +117,7 @@ const setup = (t: Test) => {
             }),
           )
           assertContents()
-          if (iteration % (iterations / 10) === 0) {
-            const now = Date.now()
-            t.ok(true, `${iteration} (${now - previous}ms / ${now - start}ms)`)
-            previous = now
-          }
+          tick()
         },
       }
     }
@@ -117,7 +131,9 @@ const setup = (t: Test) => {
 // errors consistently in Windows CI environments.
 // https://github.com/sindresorhus/del/blob/chore/update-deps/test.js#L116
 t.test('windows does not throw EPERM', t => {
-  t.test('sync', t => {
+  t.test('sync', async t => {
+    t.plan(10)
+    const { windowsSync } = await mockWindows(t)
     for (const { matches, error, assertResult } of setup(t)()) {
       assertResult(
         matches.map(path => {
@@ -132,6 +148,8 @@ t.test('windows does not throw EPERM', t => {
   })
 
   t.test('async', async t => {
+    t.plan(10)
+    const { windows } = await mockWindows(t)
     for (const { matches, error, assertResult } of setup(t)()) {
       assertResult(
         await Promise.all(
@@ -147,5 +165,30 @@ t.test('windows does not throw EPERM', t => {
     }
   })
 
+  if (isWinCI) {
+    t.test('async error', async t => {
+      t.intercept(process, 'platform', { value: 'posix' })
+      const { windows } = await mockWindows(t)
+      let error = null
+      try {
+        for (const { matches, error, tick } of setup(t)()) {
+          await Promise.all(
+            matches.map(async path => {
+              try {
+                return [path, await windows(path)]
+              } catch (er) {
+                throw error(er, path)
+              }
+            }),
+          )
+          tick()
+        }
+      } catch (e) {
+        error = e
+      }
+      t.comment(error)
+    })
+  }
+
   t.end()
 })