most unit tests work now

Author: juslesan

packages/sdk/src/signature/SignatureValidationWorker.ts

@@ -11,14 +11,22 @@ const workerApi = {
 export type SignatureValidationWorkerApi = typeof workerApi
 
 // Detect environment and expose accordingly
-if (typeof self !== 'undefined') {
-    // Browser Web Worker
-    Comlink.expose(workerApi)
-} else {
-    // Node.js Worker Thread
+// Check for Node.js worker_threads first, since `self` is defined in both environments
+// but only browser Web Workers have WorkerGlobalScope with addEventListener
+let parentPort: import('worker_threads').MessagePort | null = null
+try {
     // eslint-disable-next-line @typescript-eslint/no-require-imports
-    const { parentPort } = require('worker_threads')
+    parentPort = require('worker_threads').parentPort
+} catch {
+    // Not in Node.js environment
+}
+
+if (parentPort) {
+    // Node.js Worker Thread
     // eslint-disable-next-line @typescript-eslint/no-require-imports
     const nodeEndpoint = require('comlink/dist/umd/node-adapter')
     Comlink.expose(workerApi, nodeEndpoint(parentPort))
+} else {
+    // Browser Web Worker
+    Comlink.expose(workerApi)
 }

packages/sdk/test/unit/SignatureValidator.test.ts

@@ -24,7 +24,7 @@ describe('SignatureValidator', () => {
 
     describe('SECP256K1', () => {
 
-        it.only('unencrypted message passes signature validation', async () => {
+        it('unencrypted message passes signature validation', async () => {
             const message = new StreamMessage({
                 messageId: new MessageID(
                     toStreamID('streamr.eth/foo/bar'),