Fix jasmine runs for local testing and tape runs on CI

README.md

@@ -40,11 +40,11 @@ hyperfine --warmup 1 \
 ## Suites
 
 - `jasmine`: This is our baseline, using Jasmine and happy-dom.
-    - *NOTE*: Jasmine doesn't support shapshot testing so those tests do a basic comparison
+    - *NOTE*: Jasmine doesn't support shapshot testing so those tests do a basic object comparison
 - `bun`: Same test suite, but running using Bun.
 - `jest`: Same test suite, but running using Jest.
 - `tape`: Same test suite, but running using Tape and ts-node.
-    - *NOTE*: Tape doesn't support shapshot testing so those tests do a basic comparison
+    - *NOTE*: Tape doesn't support shapshot testing so those tests do a basic object  comparison
 - `vitest`: Same test suite, but running using Vitest. *NOTE*: That benchmarks use `--poolOptions.threads.isolate=false` as it has the best performance (see [this comment](https://github.com/vitest-dev/vitest/issues/579#issuecomment-1946462435))
 
 ## Results

benchmarks/jasmine/spec/helpers/happydom.ts

@@ -7,9 +7,6 @@ global.window = window;
 // Register global window extensions
 [
 	'document',
-	// NOTE: This is need for running tests on GitHub Actions CI, but causes
-	// failure on local builds due to this error:
-	// > Cannot set property navigator of #<Object> which has only a getter
 	'navigator',
 	'Element',
 	'getComputedStyle',
@@ -19,5 +16,11 @@ global.window = window;
 	try {
 		global[key] = global.window[key];
 	} catch (e) {
+		// NOTE: This is need for running tests on GitHub Actions CI, but causes
+		// failure on local builds due to this error:
+		// > Cannot set property navigator of #<Object> which has only a getter
+		if (!e.message.includes('Cannot set property navigator')) {
+			throw e;
+		}
 	}
 });

benchmarks/tape/helpers/happydom.ts

@@ -7,10 +7,20 @@ global.window = window;
 // Register global window extensions
 [
 	'document',
+	'navigator',
 	'Element',
 	'getComputedStyle',
 	'HTMLElement',
 	'SVGElement'
 ].forEach((key) => {
-	global[key] = global.window[key];
+	try {
+		global[key] = global.window[key];
+	} catch (e) {
+		// NOTE: This is need for running tests on GitHub Actions CI, but causes
+		// failure on local builds due to this error:
+		// > Cannot set property navigator of #<Object> which has only a getter
+		if (!e.message.includes('Cannot set property navigator')) {
+			throw e;
+		}
+	}
 });