fix(instrumentation-web-exception): properly add bounded error listener

packages/instrumentation-web-exception/src/instrumentation.ts

@@ -49,7 +49,6 @@ export interface GlobalErrorsInstrumentationConfig
 export class ExceptionInstrumentation extends InstrumentationBase<GlobalErrorsInstrumentationConfig> {
   constructor(config: GlobalErrorsInstrumentationConfig = {}) {
     super(PACKAGE_NAME, PACKAGE_VERSION, config);
-    this.onError = this.onError.bind(this);
   }
 
   init() {}
@@ -101,6 +100,7 @@ export class ExceptionInstrumentation extends InstrumentationBase<GlobalErrorsIn
   }
 
   override enable(): void {
+    this.onError = this.onError.bind(this);
     window.addEventListener('error', this.onError);
     window.addEventListener('unhandledrejection', this.onError);
   }

packages/instrumentation-web-exception/test/instrumentation.test.ts

@@ -98,17 +98,15 @@ describe('ExceptionInstrumentation', () => {
   });
 
   describe('throwing an error', () => {
-    const instr = new ExceptionInstrumentation();
+    let disableInstrumentations: () => void;
     beforeEach(() => {
-      registerInstrumentations({
-        instrumentations: [instr],
+      disableInstrumentations = registerInstrumentations({
+        instrumentations: [new ExceptionInstrumentation()],
       });
-
-      instr.enable();
     });
 
     afterEach(() => {
-      instr.disable();
+      disableInstrumentations();
       exporter.reset();
     });
 
@@ -179,21 +177,24 @@ describe('ExceptionInstrumentation', () => {
         'app.custom.exception': error.message.toLocaleUpperCase(),
       };
     };
-    const instr = new ExceptionInstrumentation({
-      applyCustomAttributes: applyCustomAttrs,
-    });
+
+    let disableInstrumentations: () => void;
+
     beforeEach(() => {
-      registerInstrumentations({
-        instrumentations: [instr],
+      disableInstrumentations = registerInstrumentations({
+        instrumentations: [
+          new ExceptionInstrumentation({
+            applyCustomAttributes: applyCustomAttrs,
+          }),
+        ],
       });
-
-      instr.enable();
     });
 
     afterEach(() => {
-      instr.disable();
+      disableInstrumentations();
       exporter.reset();
     });
+
     it('should add custom attributes to the event', async () => {
       setTimeout(() => {
         throwErr('Something happened!');