From cf47eb41ff7ad5328b4fe2dfa8079ab36d7f7d9d Mon Sep 17 00:00:00 2001
From: Doug Parker <dgp1130@users.noreply.github.com>
Date: Fri, 10 Apr 2026 15:55:55 -0700
Subject: [PATCH] test: use strict mode for `ng_elements` integration test

Apparently the Rollup bundle for these tests defaults to `es` format, meaning it expects to be loaded at runtime as native ESM. This was not happening because it was loaded as a regular `<script src="...">` tag (note the lack of `type="module"`).

This is problematic because Rollup assumed it would be running in a scoped environment, meaning [this function](https://github.com/angular/angular/blob/adb8d1078d5f127085952ca81951c18e0178a038/packages/core/primitives/event-dispatch/src/event.ts#L45), which happens to be named `addEventListener` but does *not* implement the `EventTarget.prototype.addEventListener` contract, was being bundled as a simple:

```javascript
function addEventListener(element, ...) {
  // ...
}
```

Since this was loaded with no `type="module"` or `'use strict';`, the script executed in "sloppy mode", meaning all `var` statements and function definitions are implicitly global. Since `window` *is* the `globalThis` object, this random `addEventListener` function clobbers the actual `window.addEventListener` and breaks any calls to it because they're not implementing the same contract.

Fix is to just use `<script src="..." type="module">`. Alternatively we could bundle in an IIFE, which Rollup does support, but in theory we could depend on external ES modules which aren't bundled, so the `type="module"` seems a little safer and more future-proof.
---
 integration/ng_elements/src/hello-world.html | 2 +-
 integration/ng_elements/src/slots.html       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/integration/ng_elements/src/hello-world.html b/integration/ng_elements/src/hello-world.html
index df65bc2b8fa..a0fa70d0c51 100644
--- a/integration/ng_elements/src/hello-world.html
+++ b/integration/ng_elements/src/hello-world.html
@@ -11,6 +11,6 @@
     <hello-world-el></hello-world-el>
     <hello-world-onpush-el></hello-world-onpush-el>
     <hello-world-shadow-el></hello-world-shadow-el>
-    <script src="dist/bundle.js"></script>
+    <script src="dist/bundle.js" type="module"></script>
   </body>
 </html>
diff --git a/integration/ng_elements/src/slots.html b/integration/ng_elements/src/slots.html
index 04c1d3ebf86..e042eb64d38 100644
--- a/integration/ng_elements/src/slots.html
+++ b/integration/ng_elements/src/slots.html
@@ -12,6 +12,6 @@
       <p>TestCardContent</p>
       <span slot="card-footer">TestCardFooter</span>
     </test-card>
-    <script src="dist/bundle.js"></script>
+    <script src="dist/bundle.js" type="module"></script>
   </body>
 </html>