-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
86 additions
and
83 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,101 +4,104 @@ import { sentryTest } from '../../../../utils/fixtures'; | |
| import { envelopeRequestParser, getEnvelopeType } from '../../../../utils/helpers'; | ||
| import { getCustomRecordingEvents, getReplayEvent, waitForReplayRequest } from '../../../../utils/replayHelpers'; | ||
|
|
||
| sentryTest('should capture feedback (@sentry-internal/feedback import)', async ({ forceFlushReplay, getLocalTestPath, page }) => { | ||
| if (process.env.PW_BUNDLE) { | ||
| sentryTest.skip(); | ||
| } | ||
|
|
||
| const reqPromise0 = waitForReplayRequest(page, 0); | ||
| const reqPromise1 = waitForReplayRequest(page, 1); | ||
| const reqPromise2 = waitForReplayRequest(page, 2); | ||
| const feedbackRequestPromise = page.waitForResponse(res => { | ||
| const req = res.request(); | ||
|
|
||
| const postData = req.postData(); | ||
| if (!postData) { | ||
| return false; | ||
| sentryTest( | ||
| 'should capture feedback (@sentry-internal/feedback import)', | ||
| async ({ forceFlushReplay, getLocalTestPath, page }) => { | ||
| if (process.env.PW_BUNDLE) { | ||
| sentryTest.skip(); | ||
| } | ||
|
|
||
| try { | ||
| return getEnvelopeType(req) === 'feedback'; | ||
| } catch (err) { | ||
| return false; | ||
| } | ||
| }); | ||
| const reqPromise0 = waitForReplayRequest(page, 0); | ||
| const reqPromise1 = waitForReplayRequest(page, 1); | ||
| const reqPromise2 = waitForReplayRequest(page, 2); | ||
| const feedbackRequestPromise = page.waitForResponse(res => { | ||
| const req = res.request(); | ||
|
|
||
| await page.route('https://dsn.ingest.sentry.io/**/*', route => { | ||
| return route.fulfill({ | ||
| status: 200, | ||
| contentType: 'application/json', | ||
| body: JSON.stringify({ id: 'test-id' }), | ||
| const postData = req.postData(); | ||
| if (!postData) { | ||
| return false; | ||
| } | ||
|
|
||
| try { | ||
| return getEnvelopeType(req) === 'feedback'; | ||
| } catch (err) { | ||
| return false; | ||
| } | ||
| }); | ||
| }); | ||
|
|
||
| const url = await getLocalTestPath({ testDir: __dirname }); | ||
| await page.route('https://dsn.ingest.sentry.io/**/*', route => { | ||
| return route.fulfill({ | ||
| status: 200, | ||
| contentType: 'application/json', | ||
| body: JSON.stringify({ id: 'test-id' }), | ||
| }); | ||
| }); | ||
|
|
||
| const [,,replayReq0] = await Promise.all([page.goto(url), page.getByText('Report a Bug').click(), reqPromise0]); | ||
| const url = await getLocalTestPath({ testDir: __dirname }); | ||
|
|
||
| // Inputs are slow, these need to be serial | ||
| await page.locator('[name="name"]').fill('Jane Doe'); | ||
| await page.locator('[name="email"]').fill('[email protected]'); | ||
| await page.locator('[name="message"]').fill('my example feedback'); | ||
| const [, , replayReq0] = await Promise.all([page.goto(url), page.getByText('Report a Bug').click(), reqPromise0]); | ||
|
|
||
| // Force flush here, as inputs are slow and can cause click event to be in unpredictable segments | ||
| await Promise.all([ | ||
| forceFlushReplay(), | ||
| reqPromise1, | ||
| ]); | ||
| // Inputs are slow, these need to be serial | ||
| await page.locator('[name="name"]').fill('Jane Doe'); | ||
| await page.locator('[name="email"]').fill('[email protected]'); | ||
| await page.locator('[name="message"]').fill('my example feedback'); | ||
|
|
||
| const [,feedbackResp, replayReq2] = await Promise.all([page.getByLabel('Send Bug Report').click(), feedbackRequestPromise, reqPromise2]); | ||
| // Force flush here, as inputs are slow and can cause click event to be in unpredictable segments | ||
| await Promise.all([forceFlushReplay(), reqPromise1]); | ||
|
|
||
| const feedbackEvent = envelopeRequestParser(feedbackResp.request()); | ||
| const replayEvent = getReplayEvent(replayReq0); | ||
| // Feedback breadcrumb is on second segment because we flush when "Report a Bug" is clicked | ||
| // And then the breadcrumb is sent when feedback form is submitted | ||
| const { breadcrumbs } = getCustomRecordingEvents(replayReq2); | ||
| const [, feedbackResp, replayReq2] = await Promise.all([ | ||
| page.getByLabel('Send Bug Report').click(), | ||
| feedbackRequestPromise, | ||
| reqPromise2, | ||
| ]); | ||
|
|
||
| const feedbackEvent = envelopeRequestParser(feedbackResp.request()); | ||
| const replayEvent = getReplayEvent(replayReq0); | ||
| // Feedback breadcrumb is on second segment because we flush when "Report a Bug" is clicked | ||
| // And then the breadcrumb is sent when feedback form is submitted | ||
| const { breadcrumbs } = getCustomRecordingEvents(replayReq2); | ||
|
|
||
| expect(breadcrumbs).toEqual( | ||
| expect.arrayContaining([ | ||
| expect.objectContaining({ | ||
| category: 'sentry.feedback', | ||
| data: { feedbackId: expect.any(String) }, | ||
| timestamp: expect.any(Number), | ||
| type: 'default', | ||
| }), | ||
| ]), | ||
| ); | ||
| expect(breadcrumbs).toEqual( | ||
| expect.arrayContaining([ | ||
| expect.objectContaining({ | ||
| category: 'sentry.feedback', | ||
| data: { feedbackId: expect.any(String) }, | ||
| timestamp: expect.any(Number), | ||
| type: 'default', | ||
| }), | ||
| ]), | ||
| ); | ||
|
|
||
| expect(feedbackEvent).toEqual({ | ||
| type: 'feedback', | ||
| breadcrumbs: expect.any(Array), | ||
| contexts: { | ||
| feedback: { | ||
| contact_email: '[email protected]', | ||
| message: 'my example feedback', | ||
| name: 'Jane Doe', | ||
| replay_id: replayEvent.event_id, | ||
| source: 'widget', | ||
| url: expect.stringContaining('/dist/index.html'), | ||
| expect(feedbackEvent).toEqual({ | ||
| type: 'feedback', | ||
| breadcrumbs: expect.any(Array), | ||
| contexts: { | ||
| feedback: { | ||
| contact_email: '[email protected]', | ||
| message: 'my example feedback', | ||
| name: 'Jane Doe', | ||
| replay_id: replayEvent.event_id, | ||
| source: 'widget', | ||
| url: expect.stringContaining('/dist/index.html'), | ||
| }, | ||
| }, | ||
| level: 'info', | ||
| timestamp: expect.any(Number), | ||
| event_id: expect.stringMatching(/\w{32}/), | ||
| environment: 'production', | ||
| sdk: { | ||
| integrations: expect.arrayContaining(['Feedback']), | ||
| version: expect.any(String), | ||
| name: 'sentry.javascript.browser', | ||
| packages: expect.anything(), | ||
| }, | ||
| }, | ||
| level: 'info', | ||
| timestamp: expect.any(Number), | ||
| event_id: expect.stringMatching(/\w{32}/), | ||
| environment: 'production', | ||
| sdk: { | ||
| integrations: expect.arrayContaining(['Feedback']), | ||
| version: expect.any(String), | ||
| name: 'sentry.javascript.browser', | ||
| packages: expect.anything(), | ||
| }, | ||
| request: { | ||
| url: expect.stringContaining('/dist/index.html'), | ||
| headers: { | ||
| 'User-Agent': expect.stringContaining(''), | ||
| request: { | ||
| url: expect.stringContaining('/dist/index.html'), | ||
| headers: { | ||
| 'User-Agent': expect.stringContaining(''), | ||
| }, | ||
| }, | ||
| }, | ||
| platform: 'javascript', | ||
| }); | ||
| }); | ||
| platform: 'javascript', | ||
| }); | ||
| }, | ||
| ); | ||