npm - @exodus/test - Versions diffs - 1.0.0-rc.77 → 1.0.0-rc.79 - Mend

@exodus/test 1.0.0-rc.77 → 1.0.0-rc.79

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (6) hide show

package/bin/browsers.js +14 -1
package/bin/inband.js +3 -1
package/bin/index.js +6 -1
package/loaders/typescript.loader.js +1 -1
package/package.json +1 -1
package/src/jest.js +13 -5

package/bin/browsers.js CHANGED Viewed

@@ -43,7 +43,20 @@ async function newPage(runner, browser, { binary, dropNetwork }) {
     return newPage(runner, browser, { binary, dropNetwork })
   }
-  await page.goto('file:///dev/null') // Need to load a secure origin for e.g. crypto.subtle to be available
+  // Need to load a secure origin for e.g. crypto.subtle to be available
+  if (runner === 'playwright' && binary === 'webkit') {
+    // Can attempt to download /dev/null, so we apply a work-around
+    await page.route('https://www.secure-context-top-level-domain-for-tests/*', (route) =>
+      route.fulfill({
+        status: 200,
+        contentType: 'text/html; charset=utf-8',
+        body: '<!doctype html><html><body></body></html>',
+      })
+    )
+    await page.goto('https://www.secure-context-top-level-domain-for-tests/')
+  } else {
+    await page.goto('file:///dev/null')
+  }
   if (dropNetwork && context.setOffline) await context.setOffline(true)
   if (dropNetwork && page.setOfflineMode) await page.setOfflineMode(true)

package/bin/inband.js CHANGED Viewed

@@ -1,5 +1,5 @@
 import { resolve } from 'node:path'
-import { describe } from '../src/engine.js'
+import { describe, after } from '../src/engine.js'
 const files = JSON.parse(process.env.EXODUS_TEST_INBAND)
 if (!Array.isArray(files)) throw new Error('Unexpected')
@@ -9,3 +9,5 @@ for (const file of files.sort()) {
     await import(resolve(file))
   })
 }
+if (globalThis.EXODUS_TEST_AFTER_INBAND) after(globalThis.EXODUS_TEST_AFTER_INBAND)

package/bin/index.js CHANGED Viewed

@@ -572,7 +572,7 @@ if (options.pure) {
   const missUnhandled = ['jsc'].includes(options.platform) || options.browsers
   if (missUnhandled) warnHuman(`Warning: ${engineName} does not have unhandled rejections tracking`)
-  const runOne = async (inputFile) => {
+  const runOne = async (inputFile, attempt = 0) => {
     const bundled = buildFile ? await buildFile(inputFile) : undefined
     if (buildFile) assert(bundled.file)
     const file = buildFile ? bundled.file : inputFile
@@ -589,6 +589,11 @@ if (options.pure) {
       const ms = Number(process.hrtime.bigint() - start) / 1e6
       return { ok: code === 0, output: [stdout, stderr], ms }
     } catch (err) {
+      if (options.engine === 'xs:bundle' && err.signal === 'SIGSEGV' && attempt < 2) {
+        // xs sometimes randomly crashes with SIGSEGV on CI. Allow 3 attempts (allow 0 - 1 to fail)
+        return runOne(inputFile, attempt + 1)
+      }
       const ms = Number(process.hrtime.bigint() - start) / 1e6
       const { code, stdout = '', stderr = '', signal, killed } = err
       if (code === null) {

package/loaders/typescript.loader.js CHANGED Viewed

@@ -9,7 +9,7 @@ export async function load(url, context, nextLoad) {
     const source = sourceBuf.toString('utf8')
     const { code: transformed } = transformSync(source, { isModule: true })
     const transformedBuf = Buffer.from(transformed)
-    if (sourceBuf.length !== transformed.length) throw new Error('length mismatch')
+    if (sourceBuf.length !== transformedBuf.length) throw new Error('length mismatch')
     // eslint-disable-next-line unicorn/no-for-loop
     for (let i = 0; i < transformedBuf.length; i++) {
       // should match either the source buffer or spaces or semicolon: https://github.com/swc-project/swc/issues/9331

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/test",
-  "version": "1.0.0-rc.77",
+  "version": "1.0.0-rc.79",
   "author": "Exodus Movement, Inc.",
   "description": "A test suite runner",
   "homepage": "https://github.com/ExodusMovement/test",

package/src/jest.js CHANGED Viewed

@@ -12,13 +12,14 @@ import { format as prettyFormat } from 'pretty-format'
 const { getCallerLocation, installLocationInNextTest } = createCallerLocationHook()
-let addStatefulApis = true
+let inband = false
 if (process.env.EXODUS_TEST_ENVIRONMENT !== 'bundle') {
-  // We can't provide snapshots in inband tests yet, and mocks/timers are unsafe there
   const files = process.argv.slice(1)
-  if (files.length === 1 && files[0].endsWith('/inband.js')) addStatefulApis = false
+  if (files.length === 1 && files[0].endsWith('/inband.js')) inband = true
 }
+// We can't provide snapshots in inband tests yet, and mocks/timers are unsafe there
+const addStatefulApis = !inband
 if (addStatefulApis) setupSnapshots(expect)
 let defaultTimeout = Number(process.env.EXODUS_TEST_TIMEOUT) || jestConfig().testTimeout // overridable via jest.setTimeout()
@@ -199,7 +200,8 @@ node.afterEach(() => {
 if (process.env.EXODUS_TEST_PLATFORM !== 'deno' && globalThis.process) {
   // TODO: deno, other engines
-  node.after(() => {
+  // This doesn't work with async imported tests, so for inband, we delay
+  const after = () => {
     jestTimers.useRealTimers()
     const prefix = `Tests completed, but still have asynchronous activity after`
@@ -217,7 +219,13 @@ if (process.env.EXODUS_TEST_PLATFORM !== 'deno' && globalThis.process) {
         console.warn(`${prefix} ${warnTimeout}ms. Waiting for ${timeout}ms to pass to finish...`)
       }, warnTimeout).unref()
     }
-  })
+  }
+  if (inband) {
+    globalThis.EXODUS_TEST_AFTER_INBAND = after
+  } else {
+    node.after(after)
+  }
 }
 export const jest = {