Update browser integration tests for captureFeedback

Author: andreiborza

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/simple_feedback/subject.js

@@ -1,6 +1,6 @@
 Sentry.captureFeedback({
   eventId: 'test_event_id',
   email: 'test_email',
-  comments: 'test_comments',
+  message: 'test_comments',
   name: 'test_name',
 });

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/simple_feedback/test.ts

@@ -1,18 +1,19 @@
 import { expect } from '@playwright/test';
-import type { UserFeedback } from '@sentry/core';
+import type { FeedbackEvent } from '@sentry/core';
 
 import { sentryTest } from '../../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest } from '../../../../utils/helpers';
 
 sentryTest('should capture simple user feedback', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
 
-  const eventData = await getFirstSentryEnvelopeRequest<UserFeedback>(page, url);
+  const eventData = await getFirstSentryEnvelopeRequest<FeedbackEvent>(page, url);
 
-  expect(eventData).toMatchObject({
-    eventId: 'test_event_id',
-    email: 'test_email',
-    comments: 'test_comments',
-    name: 'test_name',
-  });
+  expect(eventData.contexts).toMatchObject(expect.objectContaining({
+    feedback: {
+      contact_email: 'test_email',
+      message: 'test_comments',
+      name: 'test_name',
+    }
+  }))
 });

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureException/init.js

@@ -6,10 +6,10 @@ Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   beforeSend(event) {
     Sentry.captureFeedback({
-      event_id: event.event_id,
+      associatedEventId: event.event_id,
       name: 'John Doe',
       email: 'john@doe.com',
-      comments: 'This feedback should be attached associated with the captured error',
+      message: 'This feedback should be attached associated with the captured error',
     });
     return event;
   },

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureException/test.ts

@@ -1,23 +1,25 @@
 import { expect } from '@playwright/test';
-import type { Event, UserFeedback } from '@sentry/core';
+import type { Event, FeedbackEvent } from '@sentry/core';
 
 import { sentryTest } from '../../../../utils/fixtures';
 import { getMultipleSentryEnvelopeRequests } from '../../../../utils/helpers';
 
 sentryTest('capture user feedback when captureException is called', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
 
-  const data = (await getMultipleSentryEnvelopeRequests(page, 2, { url })) as (Event | UserFeedback)[];
+  const data = (await getMultipleSentryEnvelopeRequests(page, 2, { url })) as (Event | FeedbackEvent)[];
 
   expect(data).toHaveLength(2);
 
   const errorEvent = ('exception' in data[0] ? data[0] : data[1]) as Event;
-  const feedback = ('exception' in data[0] ? data[1] : data[0]) as UserFeedback;
+  const feedback = ('exception' in data[0] ? data[1] : data[0]) as FeedbackEvent;
 
-  expect(feedback).toEqual({
-    comments: 'This feedback should be attached associated with the captured error',
-    email: 'john@doe.com',
-    event_id: errorEvent.event_id,
-    name: 'John Doe',
-  });
+  expect(feedback.contexts).toEqual(expect.objectContaining({
+    feedback: {
+      associated_event_id: errorEvent.event_id,
+      message: 'This feedback should be attached associated with the captured error',
+      contact_email: 'john@doe.com',
+      name: 'John Doe',
+    }
+  }));
 });

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureMessage/init.js

@@ -6,10 +6,10 @@ Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   beforeSend(event) {
     Sentry.captureFeedback({
-      event_id: event.event_id,
+      associatedEventId: event.event_id,
       name: 'John Doe',
       email: 'john@doe.com',
-      comments: 'This feedback should be attached associated with the captured message',
+      message: 'This feedback should be attached associated with the captured message',
     });
     return event;
   },

dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureMessage/test.ts

@@ -1,23 +1,23 @@
 import { expect } from '@playwright/test';
-import type { Event, UserFeedback } from '@sentry/core';
+import type { Event, FeedbackEvent } from '@sentry/core';
 
 import { sentryTest } from '../../../../utils/fixtures';
 import { getMultipleSentryEnvelopeRequests } from '../../../../utils/helpers';
 
 sentryTest('capture user feedback when captureMessage is called', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
 
-  const data = (await getMultipleSentryEnvelopeRequests(page, 2, { url })) as (Event | UserFeedback)[];
+  const data = (await getMultipleSentryEnvelopeRequests(page, 2, { url })) as (Event | FeedbackEvent)[];
 
   expect(data).toHaveLength(2);
 
   const errorEvent = ('exception' in data[0] ? data[0] : data[1]) as Event;
-  const feedback = ('exception' in data[0] ? data[1] : data[0]) as UserFeedback;
+  const feedback = ('exception' in data[0] ? data[1] : data[0]) as FeedbackEvent;
 
-  expect(feedback).toEqual({
-    comments: 'This feedback should be attached associated with the captured message',
-    email: 'john@doe.com',
-    event_id: errorEvent.event_id,
+  expect(feedback.contexts).toEqual(expect.objectContaining({
+    message: 'This feedback should be attached associated with the captured message',
+    contact_email: 'john@doe.com',
+    associated_event_id: errorEvent.event_id,
     name: 'John Doe',
-  });
+  }));
 });