@jsenv/core 25.4.9 → 26.0.0

package/src/{executeTestPlan.js → execute_test_plan.js}

@@ -11,13 +11,11 @@ import { createLogger, createDetailedMessage } from "@jsenv/logger"
 import {
   assertProjectDirectoryUrl,
   assertProjectDirectoryExists,
-} from "./internal/argUtils.js"
-import { executePlan } from "./internal/executing/executePlan.js"
-import { executionIsPassed } from "./internal/executing/executionIsPassed.js"
-import { generateCoverageJsonFile } from "./internal/executing/coverage_reporter/coverage_reporter_json_file.js"
-import { generateCoverageHtmlDirectory } from "./internal/executing/coverage_reporter/coverage_reporter_html_directory.js"
-import { generateCoverageTextLog } from "./internal/executing/coverage_reporter/coverage_reporter_text_log.js"
-import { jsenvCoverageConfig } from "./jsenvCoverageConfig.js"
+} from "@jsenv/core/src/internal/jsenv_params_assertions.js"
+import { executePlan } from "./internal/executing/execute_plan.js"
+import { generateCoverageJsonFile } from "./internal/coverage/coverage_reporter_json_file.js"
+import { generateCoverageHtmlDirectory } from "./internal/coverage/coverage_reporter_html_directory.js"
+import { generateCoverageTextLog } from "./internal/coverage/coverage_reporter_text_log.js"
 
 /**
  * Execute a list of files and log how it goes
@@ -28,6 +26,7 @@ import { jsenvCoverageConfig } from "./jsenvCoverageConfig.js"
  * @param {boolean} [testPlanParameters.completedExecutionLogMerging=false] Merge completed execution logs to shorten terminal output
  * @param {number} [testPlanParameters.maxExecutionsInParallel=1] Maximum amount of execution in parallel
  * @param {number} [testPlanParameters.defaultMsAllocatedPerExecution=30000] Milliseconds after which execution is aborted and considered as failed by timeout
+ * @param {boolean} [testPlanParameters.failFast=false] Fails immediatly when a test execution fails
  * @param {number} [testPlanParameters.cooldownBetweenExecutions=0] Millisecond to wait between each execution
  * @param {boolean} [testPlanParameters.logMemoryHeapUsage=false] Add memory heap usage during logs
  * @param {boolean} [testPlanParameters.coverage=false] Controls if coverage is collected during files executions
@@ -48,15 +47,17 @@ export const executeTestPlan = async ({
 
   testPlan,
 
+  logSummary = true,
   logMemoryHeapUsage = false,
+  logFileRelativeUrl = ".jsenv/test_plan_debug.txt",
   completedExecutionLogAbbreviation = false,
   completedExecutionLogMerging = false,
-  logSummary = true,
   updateProcessExitCode = true,
   windowsProcessExitFix = true,
 
   maxExecutionsInParallel = 1,
   defaultMsAllocatedPerExecution = 30000,
+  failFast = false,
   // stopAfterExecute: true to ensure runtime is stopped once executed
   // because we have what we wants: execution is completed and
   // we have associated coverage and capturedConsole
@@ -69,7 +70,13 @@ export const executeTestPlan = async ({
   coverage = process.argv.includes("--cover") ||
     process.argv.includes("--coverage"),
   coverageTempDirectoryRelativeUrl = "./coverage/tmp/",
-  coverageConfig = jsenvCoverageConfig,
+  coverageConfig = {
+    "./index.js": true,
+    "./main.js": true,
+    "./src/**/*.js": true,
+    "./**/*.test.*": false, // contains .test. -> nope
+    "./**/test/": false, // inside a test folder -> nope,
+  },
   coverageIncludeMissing = true,
   coverageAndExecutionAllowed = false,
   coverageForceIstanbul = false,
@@ -103,14 +110,11 @@ export const executeTestPlan = async ({
   jsenvDirectoryClean,
 }) => {
   const logger = createLogger({ logLevel })
-
   projectDirectoryUrl = assertProjectDirectoryUrl({ projectDirectoryUrl })
   await assertProjectDirectoryExists({ projectDirectoryUrl })
-
   if (typeof testPlan !== "object") {
     throw new Error(`testPlan must be an object, got ${testPlan}`)
   }
-
   if (coverage) {
     if (typeof coverageConfig !== "object") {
       throw new TypeError(
@@ -155,7 +159,6 @@ export const executeTestPlan = async ({
       }
     }
   }
-
   const result = await executePlan(testPlan, {
     signal,
     handleSIGINT,
@@ -171,13 +174,15 @@ export const executeTestPlan = async ({
     importResolutionMethod,
     importDefaultExtension,
 
+    logSummary,
     logMemoryHeapUsage,
+    logFileRelativeUrl,
     completedExecutionLogMerging,
     completedExecutionLogAbbreviation,
-    logSummary,
 
-    defaultMsAllocatedPerExecution,
     maxExecutionsInParallel,
+    defaultMsAllocatedPerExecution,
+    failFast,
     stopAfterExecute,
     cooldownBetweenExecutions,
     gcBetweenExecutions,
@@ -205,11 +210,12 @@ export const executeTestPlan = async ({
     importMapInWebWorkers,
     customCompilers,
   })
-
-  if (updateProcessExitCode && !executionIsPassed(result)) {
+  if (
+    updateProcessExitCode &&
+    result.planSummary.executionCount !== result.planSummary.completedCount
+  ) {
     process.exitCode = 1
   }
-
   const planCoverage = result.planCoverage
   // planCoverage can be null when execution is aborted
   if (planCoverage) {
@@ -260,7 +266,6 @@ export const executeTestPlan = async ({
     }
     await Promise.all(promises)
   }
-
   // Sometimes on windows test plan scripts never ends
   // I suspect it's some node process keeping the process alive
   // because not properly killed for some reason.
@@ -271,7 +276,6 @@ export const executeTestPlan = async ({
       process.exit()
     }, 2000).unref()
   }
-
   return {
     testPlanAborted: result.aborted,
     testPlanSummary: result.planSummary,

package/src/{importUsingChildProcess.js → import_using_child_process.js}

@@ -1,5 +1,5 @@
-import { launchAndExecute } from "./internal/executing/launchAndExecute.js"
-import { createControllableNodeProcess } from "./internal/node_launcher/createControllableNodeProcess.js"
+import { launchAndExecute } from "./internal/executing/launch_and_execute.js"
+import { createControllableNodeProcess } from "./internal/node_launcher/node_controllable_process.js"
 
 export const importUsingChildProcess = async (
   fileUrl,

package/src/internal/{browser_runtime/browser_runtime.js → browser_client/browser_client.js}

@@ -1,14 +1,12 @@
-/* eslint-env browser */
+import { fetchUrl } from "@jsenv/core/src/internal/browser_utils/fetch_browser.js"
+import { fetchAndEval } from "@jsenv/core/src/internal/browser_utils/fetch_and_eval.js"
+import { unevalException } from "@jsenv/core/src/internal/runtime_client/uneval_exception.js"
+import { memoize } from "@jsenv/core/src/internal/memoize.js"
 
-import { installBrowserErrorStackRemapping } from "../error-stack-remapping/installBrowserErrorStackRemapping.js"
-import { fetchUrl } from "../browser_utils/fetch_browser.js"
-import { fetchAndEvalUsingFetch } from "../browser_utils/fetchAndEvalUsingFetch.js"
-import { unevalException } from "../unevalException.js"
-import { memoize } from "../memoize.js"
-
-import { displayErrorInDocument } from "./displayErrorInDocument.js"
-import { displayErrorNotification } from "./displayErrorNotification.js"
-import { createBrowserRuntime } from "./createBrowserRuntime.js"
+import { createBrowserClient } from "./browser_client_factory.js"
+import { installBrowserErrorStackRemapping } from "./browser_error_stack_remap.js"
+import { displayErrorInDocument } from "./error_in_document.js"
+import { displayErrorNotification } from "./error_in_notification.js"
 
 const getNavigationStartTime = () => {
   try {
@@ -177,7 +175,7 @@ const getBrowserRuntime = memoize(async () => {
   const parts = afterJsenvDirectory.split("/")
   const compileId = parts[0]
 
-  const browserRuntime = await createBrowserRuntime({
+  const browserClient = await createBrowserClient({
     compileServerOrigin,
     jsenvDirectoryRelativeUrl,
     compileId,
@@ -186,9 +184,7 @@ const getBrowserRuntime = memoize(async () => {
   if (errorStackRemapping && Error.captureStackTrace) {
     const { sourcemapMainFileRelativeUrl, sourcemapMappingFileRelativeUrl } =
       compileServerMeta
-    await fetchAndEvalUsingFetch(
-      `${compileServerOrigin}/${sourcemapMainFileRelativeUrl}`,
-    )
+    await fetchAndEval(`${compileServerOrigin}/${sourcemapMainFileRelativeUrl}`)
     const { SourceMapConsumer } = window.sourceMap
     SourceMapConsumer.initialize({
       "lib/mappings.wasm": `${compileServerOrigin}/${sourcemapMappingFileRelativeUrl}`,
@@ -204,12 +200,12 @@ const getBrowserRuntime = memoize(async () => {
       error.stack = originalStack
       return error
     }
-    const executeFile = browserRuntime.executeFile
-    browserRuntime.executeFile = (file, options = {}) => {
+    const executeFile = browserClient.executeFile
+    browserClient.executeFile = (file, options = {}) => {
       return executeFile(file, { errorTransform, ...options })
     }
   }
-  return browserRuntime
+  return browserClient
 })
 
 const livereloadingCallbacks = {}

package/src/internal/{browser_runtime/createBrowserRuntime.js → browser_client/browser_client_factory.js}

@@ -1,19 +1,17 @@
-/* eslint-env browser */
-
 import { normalizeImportMap } from "@jsenv/importmap/src/normalizeImportMap.js"
 
 // do not use memoize from @jsenv/filesystem to avoid pulling @jsenv/filesystem code into the browser build
-import { fetchUrl } from "../browser_utils/fetch_browser.js"
-import { createImportResolverForImportmap } from "../import-resolution/import-resolver-importmap.js"
-import { memoize } from "../memoize.js"
-import { measureAsyncFnPerf } from "../perf_browser.js"
+import { fetchUrl } from "@jsenv/core/src/internal/browser_utils/fetch_browser.js"
+import { createImportResolverForImportmap } from "@jsenv/core/src/internal/import_resolution/import_resolver_importmap.js"
+import { memoize } from "@jsenv/core/src/internal/memoize.js"
 
 import { createBrowserSystem } from "./browser_system.js"
-import { makeNamespaceTransferable } from "./makeNamespaceTransferable.js"
+import { measureAsyncFnPerf } from "./perf_browser.js"
+import { makeModuleNamespaceTransferable } from "./module_namespace_transfer.js"
 
 const memoizedCreateBrowserSystem = memoize(createBrowserSystem)
 
-export const createBrowserRuntime = async ({
+export const createBrowserClient = async ({
   compileServerOrigin,
   jsenvDirectoryRelativeUrl,
   compileId,
@@ -103,7 +101,7 @@ export const createBrowserRuntime = async ({
         let namespace = await browserSystem.import(specifier)
 
         if (transferableNamespace) {
-          namespace = makeNamespaceTransferable(namespace)
+          namespace = makeModuleNamespaceTransferable(namespace)
         }
 
         return {

package/src/internal/{error-stack-remapping/installBrowserErrorStackRemapping.js → browser_client/browser_error_stack_remap.js}

@@ -1,6 +1,4 @@
-/* eslint-env browser */
-
-import { installErrorStackRemapping } from "./installErrorStackRemapping.js"
+import { installErrorStackRemapping } from "@jsenv/core/src/internal/error_stack_remap/install_error_stack_remapping.js"
 
 export const installBrowserErrorStackRemapping = (options = {}) =>
   installErrorStackRemapping({

package/src/internal/{browser_runtime → browser_client}/browser_system.js

@@ -1,7 +1,5 @@
-/* eslint-env browser */
-
-import { getJavaScriptModuleResponseError } from "../runtime/module-registration.js"
-import "../runtime/s.js"
+import { getJavaScriptModuleResponseError } from "@jsenv/core/src/internal/runtime_client/module_registration.js"
+import "@jsenv/core/src/internal/runtime_client/s.js"
 
 export const createBrowserSystem = ({
   compileServerOrigin,

package/src/internal/{browser_runtime/displayErrorInDocument.js → browser_client/error_in_document.js}

@@ -1,5 +1,3 @@
-/* eslint-env browser */
-
 export const displayErrorInDocument = (error) => {
   const title = "An error occured"
   let theme

package/src/internal/{browser_runtime/displayErrorNotification.js → browser_client/error_in_notification.js}

@@ -1,4 +1,3 @@
-/* eslint-env browser */
 const { Notification } = window
 
 const displayErrorNotificationNotAvailable = () => {}

package/src/internal/{browser_runtime/makeNamespaceTransferable.js → browser_client/module_namespace_transfer.js}

@@ -1,4 +1,4 @@
-export const makeNamespaceTransferable = (namespace) => {
+export const makeModuleNamespaceTransferable = (namespace) => {
   const transferableNamespace = {}
   Object.keys(namespace).forEach((key) => {
     const value = namespace[key]

package/src/internal/{perf_browser.js → browser_client/perf_browser.js}

@@ -1,5 +1,3 @@
-/* eslint-env browser */
-
 const { performance } = window
 
 export const measureAsyncFnPerf = performance

package/src/internal/browser_detection/browser_detection.js

@@ -2,15 +2,15 @@
 // https://github.com/lancedikson/bowser/blob/master/src/parser-browsers.js#L1
 
 import { detectFromUserAgentData } from "./user_agent_data.js"
-import { detectAndroid } from "./detectAndroid.js"
-import { detectInternetExplorer } from "./detectInternetExplorer.js"
-import { detectOpera } from "./detectOpera.js"
-import { detectEdge } from "./detectEdge.js"
-import { detectFirefox } from "./detectFirefox.js"
-import { detectChrome } from "./detectChrome.js"
-import { detectSafari } from "./detectSafari.js"
-import { detectElectron } from "./detectElectron.js"
-import { detectIOS } from "./detectIOS.js"
+import { detectAndroid } from "./detect_android.js"
+import { detectInternetExplorer } from "./detect_internet_explorer.js"
+import { detectOpera } from "./detect_opera.js"
+import { detectEdge } from "./detect_edge.js"
+import { detectFirefox } from "./detect_firefox.js"
+import { detectChrome } from "./detect_chrome.js"
+import { detectSafari } from "./detect_safari.js"
+import { detectElectron } from "./detect_electron.js"
+import { detectIOS } from "./detect_ios.js"
 
 const detectorCompose = (detectors) => () => {
   let i = 0
@@ -40,7 +40,6 @@ const detector = detectorCompose([
 
 export const detectBrowser = () => {
   const { name = "other", version = "unknown" } = detector() || {}
-
   return {
     name: normalizeName(name),
     version: normalizeVersion(version),
@@ -57,12 +56,10 @@ const normalizeVersion = (version) => {
     // remove extraneous .
     return parts.slice(0, 3).join(".")
   }
-
   if (version.indexOf("_") > -1) {
     const parts = version.split("_")
     // remove extraneous _
     return parts.slice(0, 3).join("_")
   }
-
   return version
 }

package/src/internal/browser_detection/user_agent_data.js

@@ -9,7 +9,6 @@ export const detectFromUserAgentData = () => {
   if (!userAgentData) {
     return null
   }
-
   const { brands } = userAgentData
   let i = 0
   while (i < brands.length) {

package/src/internal/browser_launcher/{executeHtmlFile.js → execute_html_file.js}

@@ -6,10 +6,11 @@ import {
 } from "@jsenv/filesystem"
 
 import { COMPILE_PROXY_BUILD_URL } from "@jsenv/core/dist/build_manifest.js"
-import { filterV8Coverage } from "@jsenv/core/src/internal/executing/coverage_utils/v8_coverage_from_directory.js"
-import { composeTwoFileByFileIstanbulCoverages } from "@jsenv/core/src/internal/executing/coverage_utils/istanbul_coverage_composition.js"
-import { evalSource } from "../node_runtime/evalSource.js"
-import { escapeRegexpSpecialCharacters } from "../escapeRegexpSpecialCharacters.js"
+import { filterV8Coverage } from "@jsenv/core/src/internal/coverage/v8_coverage_from_directory.js"
+import { composeTwoFileByFileIstanbulCoverages } from "@jsenv/core/src/internal/coverage/istanbul_coverage_composition.js"
+import { evalSource } from "@jsenv/core/src/internal/node_launcher/eval_source.js"
+import { escapeRegexpSpecialCharacters } from "@jsenv/core/src/internal/regexp_escape.js"
+
 import { getBrowserRuntimeProfile } from "./browser_runtime_profile.js"
 
 export const executeHtmlFile = async (

package/src/internal/browser_launcher/from_playwright.js

@@ -9,8 +9,8 @@ import {
 } from "@jsenv/abort"
 import { memoize } from "@jsenv/filesystem"
 
-import { trackPageToNotify } from "./trackPageToNotify.js"
-import { executeHtmlFile } from "./executeHtmlFile.js"
+import { trackPageErrorsAndLogs } from "./page_errors_and_logs.js"
+import { executeHtmlFile } from "./execute_html_file.js"
 
 export const createRuntimeFromPlaywright = ({
   browserName,
@@ -113,7 +113,7 @@ export const createRuntimeFromPlaywright = ({
         throw e
       }
     })
-    const stopTrackingToNotify = trackPageToNotify(page, {
+    const stopTrackingToNotify = trackPageErrorsAndLogs(page, {
       onError: (error) => {
         error = transformErrorHook(error)
         if (!ignoreErrorHook(error)) {

package/src/internal/browser_launcher/{trackPageToNotify.js → page_errors_and_logs.js}

@@ -1,4 +1,4 @@
-export const trackPageToNotify = (page, { onError, onConsole }) => {
+export const trackPageErrorsAndLogs = (page, { onError, onConsole }) => {
   // https://github.com/GoogleChrome/puppeteer/blob/v1.4.0/docs/api.md#event-error
   const removeErrorListener = registerEvent({
     object: page,

package/src/internal/browser_utils/{fetchAndEvalUsingFetch.js → fetch_and_eval.js}

@@ -1,10 +1,8 @@
-/* eslint-env browser */
-
 import { createDetailedMessage } from "@jsenv/logger"
 
 import { fetchUrl } from "./fetch_browser.js"
 
-export const fetchAndEvalUsingFetch = async (url) => {
+export const fetchAndEval = async (url) => {
   const response = await fetchUrl(url)
 
   if (response.status >= 200 && response.status <= 299) {

package/src/internal/browser_utils/{fetchAndEvalUsingScript.js → fetch_and_eval_using_script.js}

@@ -1,5 +1,3 @@
-/* eslint-env browser */
-
 export const fetchAndEvalUsingScript = async (src) => {
   return new Promise((resolve, reject) => {
     const script = document.createElement("script")

package/src/internal/browser_utils/fetch_browser.js

@@ -1,4 +1,4 @@
-import { fetchUsingXHR } from "./fetchUsingXHR.js"
+import { fetchUsingXHR } from "./fetch_using_xhr.js"
 
 const fetchNative = async (url, { mode = "cors", ...options } = {}) => {
   const response = await window.fetch(url, {

package/src/internal/browser_utils/{fetchUsingXHR.js → fetch_using_xhr.js}

@@ -1,5 +1,3 @@
-/* eslint-env browser */
-
 export const fetchUsingXHR = async (
   url,
   {

package/src/internal/building/build_logs.js

@@ -1,8 +1,9 @@
 import { ANSI, UNICODE } from "@jsenv/log"
 
-import { getHtmlNodeLocation } from "@jsenv/core/src/internal/compiling/compileHtml.js"
-import { byteAsFileSize } from "../logs/byteAsFileSize.js"
-import { msAsDuration } from "../logs/msAsDuration.js"
+import { getHtmlNodeLocation } from "@jsenv/core/src/internal/compile_server/html/html_ast.js"
+
+import { byteAsFileSize } from "../logs/byte_as_file_size.js"
+import { msAsDuration } from "../logs/ms_as_duration.js"
 import { stringifyUrlSite } from "./url_trace.js"
 
 export const formatBuildStartLog = ({ entryPoints }) => {
@@ -61,37 +62,17 @@ const formatBuildDoneDetails = ({ buildStats, buildDirectoryRelativeUrl }) => {
     )})`
   })
   const buildFileCount = buildFiles.length
-
   const { buildSourcemapFileSizes } = buildStats
-  const sourcemapFiles = Object.keys(buildSourcemapFileSizes).map((key) => {
-    const buildSourcemapFileSize = buildSourcemapFileSizes[key]
-    return `${buildDirectoryRelativeUrl}${key} (${byteAsFileSize(
-      buildSourcemapFileSize,
-    )})`
-  })
-  const sourcemapFileCount = sourcemapFiles.length
-
-  const buildFilesDescription =
-    buildFileCount === 1
-      ? "file in the build: 1"
-      : `files in the build: ${buildFileCount}`
-
-  const buildSourcemapFilesDescription =
-    sourcemapFileCount === 0
-      ? ""
-      : sourcemapFileCount === 1
-      ? "sourcemap file in the build: 1"
-      : `sourcemap files in the build: ${sourcemapFileCount}`
-
+  const sourcemapFileCount = Object.keys(buildSourcemapFileSizes).length
+  let buildFilesDescription =
+    buildFileCount === 1 ? "build file" : `build files`
+  if (sourcemapFileCount === 1) {
+    buildFilesDescription += ` (excluding sourcemap file)`
+  } else if (sourcemapFileCount > 1) {
+    buildFilesDescription += ` (excluding sourcemap files)`
+  }
   let message = `--- ${buildFilesDescription} ---
 ${buildFiles.join("\n")}`
-
-  if (buildSourcemapFilesDescription) {
-    message += `
---- ${buildSourcemapFilesDescription} ---
-${sourcemapFiles.join("\n")}`
-  }
-
   return message
 }
 
@@ -101,24 +82,40 @@ const formatBuildSummary = ({ buildStats }) => {
     projectFileSizes,
     projectTotalFileSize,
     buildFileSizes,
-    buildTotalFileSize,
+    buildFileTotalSize,
+    buildSourcemapFileSizes,
+    buildSourcemapFileTotalSize,
   } = buildStats
 
   const projectFileCount = Object.keys(projectFileSizes).length
   const buildFileCount = Object.keys(buildFileSizes).length
+  const buildSourcemapFileCount = Object.keys(buildSourcemapFileSizes).length
 
   return `------- build summary -------
-${ANSI.color(
-  `project files:`,
-  ANSI.GREY,
-)} ${projectFileCount} (${byteAsFileSize(projectTotalFileSize)})
-${ANSI.color(`build files:`, ANSI.GREY)} ${buildFileCount} (${byteAsFileSize(
-    buildTotalFileSize,
-  )})
-${ANSI.color(`build duration:`, ANSI.GREY)} ${msAsDuration(buildDuration)}
+${formatSummaryContent({
+  "project files": `${projectFileCount} (${byteAsFileSize(
+    projectTotalFileSize,
+  )})`,
+  "build files": `${buildFileCount} (${byteAsFileSize(buildFileTotalSize)})`,
+  ...(buildSourcemapFileCount === 0
+    ? {}
+    : {
+        "build sourcemap files": `${buildSourcemapFileCount} (${byteAsFileSize(
+          buildSourcemapFileTotalSize,
+        )})`,
+      }),
+  "build duration": msAsDuration(buildDuration),
+})}
 ------------------------------`
 }
 
+const formatSummaryContent = (summaryData) => {
+  return Object.keys(summaryData).map((key) => {
+    return `${ANSI.color(`${key}:`, ANSI.GREY)} ${summaryData[key]}`
+  }).join(`
+`)
+}
+
 const showHtmlSourceLocation = ({
   htmlNode,
   htmlUrl,

package/src/internal/building/build_stats.js

@@ -19,8 +19,9 @@ export const createBuildStats = ({
     projectFileSizes: projectFileSizeInfo.fileSizes,
     projectTotalFileSize: projectFileSizeInfo.totalSize,
     buildFileSizes: buildFileSizeInfo.fileSizes,
-    buildTotalFileSize: buildFileSizeInfo.totalSize,
+    buildFileTotalSize: buildFileSizeInfo.totalSize,
     buildSourcemapFileSizes: sourcemapFileSizeInfo.fileSizes,
+    buildSourcemapFileTotalSize: sourcemapFileSizeInfo.totalSize,
     buildDuration,
   }
 }

package/src/internal/building/build_url_generator.js

@@ -1,6 +1,6 @@
 import { urlToBasename, urlToExtension } from "@jsenv/filesystem"
 
-import { generateContentHash } from "./url_versioning.js"
+import { generateContentHash } from "./url_versioning/url_versioning.js"
 
 export const createBuildUrlGenerator = ({
   entryPointUrls,