refactored code

src/PseudoWindow.js

@@ -15,44 +15,40 @@ export default {
 	},
 
 	mounted() {
-		this.bindEventListeners();
+		// Bind events
+		Object.entries(this.$listeners).forEach(($l) => {
+			const e = this.normalizeEvent(
+				this.document ? window.document : window,
+				$l[0], // event name
+				$l[1], // event handler
+			);
+			e.target.addEventListener(e.name, e.handler, e.opts);
+			this.handlers.push(e);
+		});
 	},
+
 	destroyed() {
-		this.unbindEventListeners();
+		// Unbind events
+		while (this.handlers.length) {
+			const e = this.handlers.shift();
+			e.target.removeEventListener(e.name, e.handler, e.opts);
+		}
 	},
 
 	methods: {
-		bindEventListeners() {
-			const { $listeners } = this;
-			for (const event in $listeners) {
-				if (!$listeners.hasOwnProperty(event)) {
-					continue;
-				}
-				const e = this.normalizeEvent(
-					this.document ? window.document : window,
-					event,
-					$listeners[event],
-				);
-				e.target.addEventListener(e.name, e.handler, e.opts);
-				this.handlers.push(e);
-			}
-		},
-
-		unbindEventListeners() {
-			while (this.handlers.length) {
-				const e = this.handlers.shift();
-				e.target.removeEventListener(e.name, e.handler, e.opts);
-			}
-		},
+		// https://github.com/vuejs/vue/blob/6fe07ebf5ab3fea1860c59fe7cdd2ec1b760f9b0/src/core/vdom/helpers/update-listeners.js#L14
+		normalizeEvent(target, rawName, handler) {
+			let name = rawName;
 
-		normalizeEvent(target, name, handler) {
-			const passive = name.charAt(0) === '&';
+			const passive = name[0] === '&';
 			name = passive ? name.slice(1) : name;
-			const once = name.charAt(0) === '~'; // Prefixed last, checked first
 
+			const once = name[0] === '~'; // Prefixed last, checked first
 			name = once ? name.slice(1) : name;
-			const capture = name.charAt(0) === '!';
+
+			const capture = name[0] === '!';
 			name = capture ? name.slice(1) : name;
+
 			return {
 				target,
 				name,

test/PseudoWindow.spec.js

@@ -2,6 +2,17 @@ import { mount } from '@vue/test-utils';
 import PseudoWindow from '@/PseudoWindow';
 
 describe('Window', () => {
+	it('should not catch "click" event', () => {
+		const myMockFn = jest.fn();
+		const wrapper = mount(PseudoWindow, {
+			attachToDocument: true,
+		});
+
+		global.window.dispatchEvent(new Event('click'));
+		expect(myMockFn).not.toBeCalled();
+		expect(wrapper.element).toMatchSnapshot();
+	});
+
 	it('should catch "resize" event', () => {
 		const myMockFn = jest.fn();
 		const wrapper = mount(PseudoWindow, {
@@ -16,7 +27,7 @@ describe('Window', () => {
 		expect(wrapper.element).toMatchSnapshot();
 	});
 
-	it('should not catch "resize" event', () => {
+	it('should cleanup event handler and not catch "resize" event', () => {
 		const myMockFn = jest.fn();
 		const wrapper = mount(PseudoWindow, {
 			attachToDocument: true,
@@ -46,6 +57,22 @@ describe('Window', () => {
 		expect(myMockFn.mock.calls.length).toBe(1);
 		expect(wrapper.element).toMatchSnapshot();
 	});
+
+	it('should not catch "resize" event', () => {
+		const myMockFn = jest.fn();
+		const wrapper = mount(PseudoWindow, {
+			attachToDocument: true,
+			listeners: {
+				'~resize': myMockFn,
+			},
+		});
+
+		wrapper.destroy();
+
+		global.window.dispatchEvent(new Event('resize'));
+		expect(myMockFn).not.toBeCalled();
+		expect(wrapper.element).toMatchSnapshot();
+	});
 });
 
 describe('Document', () => {

test/__snapshots__/PseudoWindow.spec.js.snap

@@ -8,4 +8,8 @@ exports[`Window should catch "resize" event 1`] = `<!---->`;
 
 exports[`Window should catch "resize" event once 1`] = `<!---->`;
 
+exports[`Window should cleanup event handler and not catch "resize" event 1`] = `<!---->`;
+
+exports[`Window should not catch "click" event 1`] = `<!---->`;
+
 exports[`Window should not catch "resize" event 1`] = `<!---->`;