fix(jsdom): use jsdom ArrayBuffer instead of Node, if jsdom is enabled (

#1727) * chore: cleanup lockfile * fix: use jsdom ArrayBuffer instead of Node, if jsdom is enabled
vitest-dev · Jul 27, 2022 · 021aca2 · 021aca2
1 parent 68951e0
commit 021aca2
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 9 deletions.
diff --git a/packages/vitest/src/integrations/env/utils.ts b/packages/vitest/src/integrations/env/utils.ts
@@ -4,6 +4,7 @@ const allowRewrite = [
   'Event',
   'EventTarget',
   'MessageEvent',
+  'ArrayBuffer',
 ]
 
 const skipKeys = [

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/test/core/test/dom.test.ts b/test/core/test/dom.test.ts
@@ -139,3 +139,18 @@ it('can extend global class', () => {
 
   expect(SuperBlob).toBeDefined()
 })
+
+it('uses jsdom ArrayBuffer', async () => {
+  const blob = new Blob(['Hello'], { type: 'text/plain' })
+
+  const arraybuffer = await new Promise<ArrayBuffer>((resolve, reject) => {
+    const reader = new FileReader()
+    reader.onload = () => resolve(reader.result as ArrayBuffer)
+    reader.onerror = () => reject(reader.error)
+    reader.readAsArrayBuffer(blob)
+  })
+
+  expect(arraybuffer.constructor.name).toBe('ArrayBuffer')
+  expect(arraybuffer instanceof ArrayBuffer).toBeTruthy()
+  expect(arraybuffer.constructor === ArrayBuffer).toBeTruthy()
+})