npm - @exodus/test - Versions diffs - 1.0.0-rc.76 → 1.0.0-rc.78 - Mend

@exodus/test 1.0.0-rc.76 → 1.0.0-rc.78

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 (7) hide show

package/README.md CHANGED Viewed

@@ -18,8 +18,8 @@ It can run your existing tests on [all runtimes and also browsers](#engines), wi
   - Actual `expect` module, also `jest-extended` and `jest-when` just work on top
   - Snapshots, including snapshot matchers
   - Function and timer mocks
-  - [test.concurrent]()
-  - Module mocks (on top of Node.js runtime only), including for ESM modules
+  - [test.concurrent](https://jestjs.io/docs/api#testconcurrentname-fn-timeout)
+  - Module mocks, including for ESM modules (already loaded ESM modules can be mocked only on `node:test`)
   - Loads Jest configuration
   - It works on Hermes too!
 - Built-in network record/replay for offline tests, mocking `fetch` and `WebSocket` sessions

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/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/test",
-  "version": "1.0.0-rc.76",
+  "version": "1.0.0-rc.78",
   "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 = {

package/src/jest.timers.js CHANGED Viewed

@@ -66,8 +66,14 @@ export function runOnlyPendingTimers() {
 }
 export function advanceTimersByTime(time) {
-  assert(Number.isSafeInteger(time) && time > 0)
+  assert(Number.isSafeInteger(time) && time >= 0)
   assertEnabledTimers()
+  if (time === 0) {
+    mock.timers.tick(0)
+    return this
+  }
   // We split this into multiple steps to run timers scheduled during the time we are running
   const minSteps = Math.min(1000, time) // usually just split e.g. 5 seconds into 1000 * 5ms
   const step = Number(Math.floor(time / minSteps).toPrecision(1))