@gjsify/unit 0.1.15 → 0.3.0

package/lib/esm/index.js

@@ -8,6 +8,8 @@ let countTestsFailed = 0;
 let countTestsIgnored = 0;
 let runtime = "";
 let runStartTime = 0;
+let currentSuite = "";
+let testErrors = [];
 const DEFAULT_TIMEOUT_CONFIG = {
   testTimeout: 5e3,
   suiteTimeout: 3e4,
@@ -62,7 +64,8 @@ const formatDuration = (ms) => {
   if (ms >= 100) return `${Math.round(ms)}ms`;
   return `${ms.toFixed(1)}ms`;
 };
-const print = globalThis.print || console.log;
+const _isGjsProcess = typeof globalThis.process?.versions?.gjs === "string";
+const print = !_isGjsProcess && typeof globalThis.document !== "undefined" ? console.log : globalThis.print || console.log;
 class MatcherFactory {
   constructor(actualValue, positive, negated) {
     this.actualValue = actualValue;
@@ -330,6 +333,8 @@ class MatcherFactory {
 const describe = async function(moduleName, callback, options) {
   const suiteTimeoutMs = typeof options === "number" ? options : options?.timeout ?? timeoutConfig.suiteTimeout;
   print("\n" + moduleName);
+  const prevSuite = currentSuite;
+  currentSuite = moduleName;
   const t0 = now();
   try {
     await withTimeout(callback, suiteTimeoutMs, `describe: ${moduleName}`);
@@ -341,6 +346,7 @@ const describe = async function(moduleName, callback, options) {
       throw e;
     }
   }
+  currentSuite = prevSuite;
   const duration = now() - t0;
   print(`  ${GRAY}\u21B3 ${formatDuration(duration)}${RESET}`);
   beforeEachCb = null;
@@ -432,6 +438,7 @@ const it = async function(expectation, callback, options) {
     if (!e.__testFailureCounted) {
       ++countTestsFailed;
     }
+    testErrors.push({ suite: currentSuite, test: expectation, message: e.message ?? String(e) });
     const icon = e instanceof TimeoutError ? "\u23F1" : "\u274C";
     print(`  ${RED}${icon}${RESET} ${GRAY}${expectation}  (${formatDuration(duration)})${RESET}`);
     print(`${RED}${e.message}${RESET}`);
@@ -502,6 +509,17 @@ const runTests = async function(namespaces) {
     }
   }
 };
+const browserSignalDone = () => {
+  const doc = globalThis.document;
+  if (!doc) return;
+  globalThis.__gjsify_test_results = {
+    passed: countTestsOverall - countTestsFailed,
+    failed: countTestsFailed,
+    total: countTestsOverall,
+    errors: testErrors
+  };
+  doc.documentElement.dataset.testsDone = "true";
+};
 const printResult = () => {
   const totalMs = runStartTime > 0 ? now() - runStartTime : 0;
   const durationStr = totalMs > 0 ? `  ${GRAY}(${formatDuration(totalMs)})` : "";
@@ -523,23 +541,27 @@ const getRuntime = async () => {
   }
   if (globalThis.Deno?.version?.deno) {
     return "Deno " + globalThis.Deno?.version?.deno;
-  } else {
+  }
+  {
     let process = globalThis.process;
     if (!process) {
       try {
         process = await import("process");
-      } catch (error) {
-        console.error(error);
-        console.warn(error.message);
-        runtime = "Unknown";
+      } catch (_e) {
       }
     }
     if (process?.versions?.gjs) {
       runtime = "Gjs " + process.versions.gjs;
+      return runtime;
     } else if (process?.versions?.node) {
       runtime = "Node.js " + process.versions.node;
+      return runtime;
     }
   }
+  if (typeof globalThis.document !== "undefined") {
+    runtime = "Browser";
+    return runtime;
+  }
   return runtime || "Unknown";
 };
 const printRuntime = async () => {
@@ -573,6 +595,7 @@ ${RED}\u23F1 ${e.message}${RESET}`);
     }
   }).then(async () => {
     printResult();
+    browserSignalDone();
     print();
     quitMainLoop();
     mainloop?.quit();

package/lib/types/index.d.ts

@@ -14,7 +14,7 @@ export interface Namespaces {
 }
 export type Callback = () => Promise<void>;
 export type Runtime = 'Gjs' | 'Deno' | 'Node.js' | 'Unknown' | 'Browser' | 'Display';
-export declare const print: typeof globalThis.print;
+export declare const print: (...data: any[]) => void;
 declare class MatcherFactory {
     protected readonly actualValue: any;
     protected readonly positive: boolean;
@@ -101,6 +101,6 @@ declare const _default: {
         skip(moduleName: string, _callback?: Callback): Promise<void>;
     };
     configure: (overrides: Partial<TimeoutConfig>) => void;
-    print: typeof globalThis.print;
+    print: (...data: any[]) => void;
 };
 export default _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gjsify/unit",
-  "version": "0.1.15",
+  "version": "0.3.0",
   "description": "A BDD-style testing framework for Gjs",
   "module": "lib/esm/index.js",
   "types": "lib/types/index.d.ts",
@@ -21,7 +21,7 @@
     "build:test:gjs": "gjsify build src/test.mts --app gjs --outfile test.gjs.mjs",
     "build:test:node": "gjsify build src/test.mts --app node --outfile test.node.mjs",
     "test": "yarn build:gjsify && yarn build:test && yarn test:node && yarn test:gjs",
-    "test:gjs": "gjs -m test.gjs.mjs",
+    "test:gjs": "gjsify run test.gjs.mjs",
     "test:node": "node test.node.mjs"
   },
   "repository": {
@@ -41,15 +41,15 @@
   },
   "homepage": "https://github.com/gjsify/unit#readme",
   "devDependencies": {
-    "@girs/gjs": "^4.0.0-rc.3",
-    "@girs/glib-2.0": "^2.88.0-4.0.0-rc.3",
-    "@gjsify/cli": "^0.1.15",
+    "@girs/gjs": "^4.0.0-rc.9",
+    "@girs/glib-2.0": "^2.88.0-4.0.0-rc.9",
+    "@gjsify/cli": "^0.3.0",
     "@types/node": "^25.6.0",
-    "typescript": "^6.0.2"
+    "typescript": "^6.0.3"
   },
   "dependencies": {
-    "@gjsify/assert": "^0.1.15",
-    "@gjsify/process": "^0.1.15",
-    "@gjsify/utils": "^0.1.15"
+    "@gjsify/assert": "^0.3.0",
+    "@gjsify/process": "^0.3.0",
+    "@gjsify/utils": "^0.3.0"
   }
 }

package/src/index.ts

@@ -14,6 +14,8 @@ let countTestsFailed = 0;
 let countTestsIgnored = 0;
 let runtime = '';
 let runStartTime = 0;
+let currentSuite = '';
+let testErrors: Array<{ suite: string; test: string; message: string }> = [];
 
 export interface TimeoutConfig {
 	/** Per-it() timeout in ms. Default: 5000. 0 = disabled. */
@@ -102,7 +104,14 @@ export type Callback = () => Promise<void>;
 export type Runtime = 'Gjs' | 'Deno' | 'Node.js' | 'Unknown' | 'Browser' | 'Display';
 
 // Makes this work on Gjs and Node.js
-export const print = globalThis.print || console.log;
+// In browsers, globalThis.print is window.print() (the print dialog), not text output.
+// Use console.log in browser contexts to avoid triggering print dialogs.
+// GJS check takes priority: @gjsify/dom-elements can set globalThis.document on GJS,
+// which would otherwise cause a false-positive browser detection.
+const _isGjsProcess = typeof (globalThis as any).process?.versions?.gjs === 'string';
+export const print = (!_isGjsProcess && typeof (globalThis as any).document !== 'undefined')
+    ? console.log
+    : (globalThis.print || console.log);
 
 class MatcherFactory {
 
@@ -373,6 +382,8 @@ export const describe = async function(moduleName: string, callback: Callback, o
 
 	print('\n' + moduleName);
 
+	const prevSuite = currentSuite;
+	currentSuite = moduleName;
 	const t0 = now();
 	try {
 		await withTimeout(callback, suiteTimeoutMs, `describe: ${moduleName}`);
@@ -384,6 +395,7 @@ export const describe = async function(moduleName: string, callback: Callback, o
 			throw e;
 		}
 	}
+	currentSuite = prevSuite;
 	const duration = now() - t0;
 	print(`  ${GRAY}↳ ${formatDuration(duration)}${RESET}`);
 
@@ -508,6 +520,7 @@ export const it = async function(expectation: string, callback: () => void | Pro
 		if (!e.__testFailureCounted) {
 			++countTestsFailed;
 		}
+		testErrors.push({ suite: currentSuite, test: expectation, message: e.message ?? String(e) });
 		const icon = e instanceof TimeoutError ? '⏱' : '❌';
 		print(`  ${RED}${icon}${RESET} ${GRAY}${expectation}  (${formatDuration(duration)})${RESET}`);
 		print(`${RED}${e.message}${RESET}`);
@@ -596,6 +609,18 @@ const runTests = async function(namespaces: Namespaces) {
 	}
 }
 
+const browserSignalDone = () => {
+	const doc = (globalThis as any).document;
+	if (!doc) return;
+	(globalThis as any).__gjsify_test_results = {
+		passed: countTestsOverall - countTestsFailed,
+		failed: countTestsFailed,
+		total: countTestsOverall,
+		errors: testErrors,
+	};
+	doc.documentElement.dataset.testsDone = 'true';
+};
+
 const printResult = () => {
 	const totalMs = runStartTime > 0 ? now() - runStartTime : 0;
 	const durationStr = totalMs > 0 ? `  ${GRAY}(${formatDuration(totalMs)})` : '';
@@ -622,25 +647,38 @@ const getRuntime = async () => {
 
 	if(globalThis.Deno?.version?.deno) {
 		return 'Deno ' + globalThis.Deno?.version?.deno;
-	} else {
+	}
+
+	// Check process (GJS / Node) BEFORE document: @gjsify/dom-elements can set
+	// globalThis.document on GJS, which would otherwise cause a false browser-positive.
+	// dynamic import('process') throws in the browser so this stays safe there.
+	{
 		let process = globalThis.process;
 
 		if(!process) {
 			try {
 				process = await import('process');
-			} catch (error) {
-				console.error(error)
-				console.warn(error.message);
-				runtime = 'Unknown'
+			} catch (_e) {
+				// browser or runtime without process — fall through to document check
 			}
 		}
 
 		if(process?.versions?.gjs) {
 			runtime = 'Gjs ' + process.versions.gjs;
+			return runtime;
 		} else if (process?.versions?.node) {
 			runtime = 'Node.js ' + process.versions.node;
+			return runtime;
 		}
 	}
+
+	// Only treat as Browser after confirming no Node/GJS process is present.
+	// dynamic imports throw in browsers, so we are safely past that path here.
+	if (typeof (globalThis as any).document !== 'undefined') {
+		runtime = 'Browser';
+		return runtime;
+	}
+
 	return runtime || 'Unknown';
 }
 
@@ -679,6 +717,7 @@ export const run = async (namespaces: Namespaces, options?: { timeout?: number;
 	})
 	.then(async () => {
 		printResult();
+		browserSignalDone();
 		print();
 
 		quitMainLoop(); // Pre-quit ensureMainLoop's loop so it exits immediately when the hook fires