vitest.dev / vitest
[READ-ONLY] Mirror of https://github.com/vitest-dev/vitest. Next generation testing framework powered by Vite. vitest.dev
test testing-tools vite
12
Fork 0

Configure Feed

Select the types of activity you want to include in your feed.

fix(web-worker): ensure `removeEventListener` is bound to worker (#7631)

authored by

Joel Gallant and committed by
GitHub (Mar 11, 2025, 8:40 AM +0200) ff42bcb3 94b27af5

+11 -4
+3 -1
packages/web-worker/src/shared-worker.ts
··· 97 97 addEventListener: (...args: any[]) => { 98 98 return this._vw_workerTarget.addEventListener(...args as [any, any]) 99 99 }, 100 - removeEventListener: this._vw_workerTarget.removeEventListener, 100 + removeEventListener: (...args: any[]) => { 101 + return this._vw_workerTarget.removeEventListener(...args as [any, any]) 102 + }, 101 103 get self() { 102 104 return selfProxy 103 105 },
+3 -1
packages/web-worker/src/worker.ts
··· 75 75 } 76 76 return this._vw_workerTarget.addEventListener(...args as [any, any]) 77 77 }, 78 - removeEventListener: this._vw_workerTarget.removeEventListener, 78 + removeEventListener: (...args: any[]) => { 79 + return this._vw_workerTarget.removeEventListener(...args as [any, any]) 80 + }, 79 81 postMessage: (...args: any[]) => { 80 82 if (!args.length) { 81 83 throw new SyntaxError(
+5 -2
test/core/src/web-worker/eventListenerWorker.ts
··· 1 - self.addEventListener('message', (e) => { 1 + function callback(e: MessageEvent) { 2 2 self.postMessage(`${e.data} world`) 3 - }) 3 + self.removeEventListener('message', callback) 4 + } 5 + 6 + self.addEventListener('message', callback)