Change tests to use exposeFunction

Author: Luca Forstner

packages/integration-tests/suites/public-api/instrumentation/eventListener-instrumentation-behaviour/subject.js

@@ -1,7 +1,6 @@
 // Simple function event listener
 const eventListener1 = () => {
-  // Trigger something that puppeteer can listen to - like console.log
-  console.log('eventListener1');
+  testCallback1();
 };
 
 // Attach event listener twice
@@ -11,8 +10,7 @@ window.addEventListener('click', eventListener1);
 // Event listener that has handleEvent() method: https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener#listener
 class EventHandlerClass {
   handleEvent() {
-    // Trigger something that puppeteer can listen to - like console.log
-    console.log('eventListener2');
+    testCallback2();
   }
 }
 

packages/integration-tests/suites/public-api/instrumentation/eventListener-instrumentation-behaviour/test.ts

@@ -6,29 +6,20 @@ sentryTest('should attach the same event listener only once', async ({ getLocalT
   const url = await getLocalTestPath({ testDir: __dirname });
   await page.goto(url);
 
-  const testCompletionPromise = new Promise<void>(resolve => {
-    let eventListener1Calls = 0;
-    let eventListener2Calls = 0;
-
-    page.on('console', async msg => {
-      const msgText = msg.text();
+  let testCallback1Calls = 0;
+  await page.exposeFunction('testCallback1', () => {
+    testCallback1Calls = testCallback1Calls + 1;
+  });
 
-      if (msgText === 'eventListener1') {
-        eventListener1Calls = eventListener1Calls + 1;
-      } else if (msgText === 'eventListener2') {
-        eventListener2Calls = eventListener2Calls + 1;
-      } else if (msgText === 'done') {
-        expect(eventListener1Calls).toBe(2);
-        expect(eventListener2Calls).toBe(2);
-        resolve();
-      }
-    });
+  let testCallback2Calls = 0;
+  await page.exposeFunction('testCallback2', () => {
+    testCallback2Calls = testCallback2Calls + 1;
   });
 
-  // Trigger event listeners twice and signal completion afterwards
+  // Trigger event listeners twice
   await page.evaluate('document.body.click()');
   await page.evaluate('document.body.click()');
-  await page.evaluate('console.log("done")');
 
-  return testCompletionPromise;
+  expect(testCallback1Calls).toBe(2);
+  expect(testCallback2Calls).toBe(2);
 });