Fix reassignment tracking (#4486)

src/ast/variables/LocalVariable.ts

@@ -155,7 +155,7 @@ export default class LocalVariable extends Variable {
 		if (path.length === 0) return false;
 		if (this.isReassigned) return true;
 		return (this.init &&
-			!context.accessed.trackEntityAtPathAndGetIfTracked(path, this) &&
+			!context.assigned.trackEntityAtPathAndGetIfTracked(path, this) &&
 			this.init.hasEffectsWhenAssignedAtPath(path, context))!;
 	}
 

test/function/samples/track-reassignments/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+	description: 'properly track reassignments (#4468)'
+};

test/function/samples/track-reassignments/main.js

@@ -0,0 +1,15 @@
+import { patchEventTarget } from './patchEventTarget.js';
+
+class EventTarget {
+	addEventListener(callback) {
+		callback();
+	}
+}
+global.window = { EventTarget };
+
+let patchCalled = false;
+patchEventTarget(() => (patchCalled = true));
+const target = new EventTarget();
+target.addEventListener()
+
+assert.ok(patchCalled, 'patch');

test/function/samples/track-reassignments/patchEventTarget.js

@@ -0,0 +1,10 @@
+export function patchEventTarget(callback) {
+	var proto = window.EventTarget.prototype;
+	var nativeAddEventListener = proto.addEventListener;
+
+	proto.addEventListener = function () {
+		return nativeAddEventListener(callback);
+	};
+
+	return proto;
+}