@jsenv/test 1.3.0 → 1.4.0

package/dist/js/execute_using_dynamic_import.js

@@ -799,7 +799,7 @@ function safeDefineProperty(object, propertyNameOrSymbol, descriptor) {
  * So the best solution remains NODE_V8_COVERAGE
  * This profiler strategy remains useful when:
  * - As fallback when NODE_V8_COVERAGE is not configured
- * - If explicitely enabled with coverageMethodForNodeJs: 'Profiler'
+ * - If explicitely enabled with coverageMethodForNodeJs: "Profiler"
  *   - Used by jsenv during automated tests about coverage
  *   - Anyone prefering this approach over NODE_V8_COVERAGE and assuming
  *     it will not fork subprocess or don't care if coverage is missed for this code

package/dist/jsenv_test.js

@@ -1919,7 +1919,7 @@ function _supportsColor(haveStream, {streamIsTTY, sniffFlags = true} = {}) {
 	}
 
 	if ('CI' in env) {
-		if ('GITHUB_ACTIONS' in env) {
+		if ('GITHUB_ACTIONS' in env || 'GITEA_ACTIONS' in env) {
 			return 3;
 		}
 
@@ -3361,7 +3361,7 @@ const relativeUrlToEmptyCoverage = async (
     const { metadata } = await applyBabelPlugins({
       babelPlugins: [babelPluginInstrument],
       input: content,
-      inputIsJsModule: false,
+      inputIsJsModule: true,
       inputUrl: fileUrl,
     });
     const { coverage } = metadata;
@@ -4297,7 +4297,7 @@ const executeSteps = async (
             rootDirectoryUrl,
             coverageConfig,
             coverageIncludeMissing,
-            coverageMethodForBrowsers,
+            coverageMethodForNodeJs,
             coverageV8ConflictWarning,
           });
           executePlanReturnValue.planCoverage = planCoverage;
@@ -4384,6 +4384,10 @@ const executeSteps = async (
             status: "executing",
           },
         };
+        if (typeof executionParams.allocatedMs === "function") {
+          executionParams.allocatedMs =
+            executionParams.allocatedMs(beforeExecutionInfo);
+        }
         let spinner;
         if (executionSpinner) {
           spinner = startSpinner({
@@ -4413,10 +4417,7 @@ const executeSteps = async (
           executionResult = await run({
             signal: multipleExecutionsOperation.signal,
             logger,
-            allocatedMs:
-              typeof executionParams.allocatedMs === "function"
-                ? executionParams.allocatedMs(beforeExecutionInfo)
-                : executionParams.allocatedMs,
+            allocatedMs: executionParams.allocatedMs,
             keepRunning,
             mirrorConsole: false, // file are executed in parallel, log would be a mess to read
             collectConsole: executionParams.collectConsole,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/test",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -34,11 +34,11 @@
   },
   "dependencies": {
     "@jsenv/abort": "4.2.4",
-    "@jsenv/ast": "4.1.0",
-    "@jsenv/filesystem": "4.2.3",
-    "@jsenv/log": "3.3.5",
-    "@jsenv/plugin-supervisor": "1.1.1",
-    "@jsenv/urls": "2.0.0",
+    "@jsenv/ast": "5.0.0",
+    "@jsenv/filesystem": "4.2.4",
+    "@jsenv/log": "3.4.0",
+    "@jsenv/plugin-supervisor": "1.2.0",
+    "@jsenv/urls": "2.1.0",
     "@jsenv/url-meta": "8.1.0",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/utils": "2.0.1",

package/src/coverage/empty_coverage_factory.js

@@ -21,7 +21,7 @@ export const relativeUrlToEmptyCoverage = async (
     const { metadata } = await applyBabelPlugins({
       babelPlugins: [babelPluginInstrument],
       input: content,
-      inputIsJsModule: false,
+      inputIsJsModule: true,
       inputUrl: fileUrl,
     });
     const { coverage } = metadata;

package/src/execution/execute_steps.js

@@ -104,7 +104,7 @@ export const executeSteps = async (
             rootDirectoryUrl,
             coverageConfig,
             coverageIncludeMissing,
-            coverageMethodForBrowsers,
+            coverageMethodForNodeJs,
             coverageV8ConflictWarning,
           });
           executePlanReturnValue.planCoverage = planCoverage;
@@ -191,6 +191,10 @@ export const executeSteps = async (
             status: "executing",
           },
         };
+        if (typeof executionParams.allocatedMs === "function") {
+          executionParams.allocatedMs =
+            executionParams.allocatedMs(beforeExecutionInfo);
+        }
         let spinner;
         if (executionSpinner) {
           spinner = startSpinner({
@@ -220,10 +224,7 @@ export const executeSteps = async (
           executionResult = await run({
             signal: multipleExecutionsOperation.signal,
             logger,
-            allocatedMs:
-              typeof executionParams.allocatedMs === "function"
-                ? executionParams.allocatedMs(beforeExecutionInfo)
-                : executionParams.allocatedMs,
+            allocatedMs: executionParams.allocatedMs,
             keepRunning,
             mirrorConsole: false, // file are executed in parallel, log would be a mess to read
             collectConsole: executionParams.collectConsole,

package/src/runtime_node/profiler_v8_coverage.js

@@ -4,7 +4,7 @@
  * So the best solution remains NODE_V8_COVERAGE
  * This profiler strategy remains useful when:
  * - As fallback when NODE_V8_COVERAGE is not configured
- * - If explicitely enabled with coverageMethodForNodeJs: 'Profiler'
+ * - If explicitely enabled with coverageMethodForNodeJs: "Profiler"
  *   - Used by jsenv during automated tests about coverage
  *   - Anyone prefering this approach over NODE_V8_COVERAGE and assuming
  *     it will not fork subprocess or don't care if coverage is missed for this code