Skip to content

Commit f0b02ed

Browse files
committed
cleanup tests and verify messageId
1 parent 5749ce6 commit f0b02ed

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

packages/browser-integration-tests/src/segment-retries.test.ts

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,16 +42,16 @@ test.describe('Standalone tests', () => {
4242
times: 1,
4343
}
4444
)
45-
const requestFailure = new Promise((resolve) => {
46-
page.once('requestfailed', resolve)
45+
const requestFailure = new Promise<Record<string, any>>((resolve) => {
46+
page.once('requestfailed', (request) => resolve(request.postDataJSON()))
4747
})
4848

4949
// trigger an event
5050
await page.evaluate(() => {
5151
void window.analytics.track('test event')
5252
})
5353

54-
await requestFailure
54+
const { messageId } = await requestFailure
5555
await page.reload()
5656

5757
// load analytics.js again and wait for a new request.
@@ -61,16 +61,14 @@ test.describe('Standalone tests', () => {
6161
])
6262

6363
expect(request.method()).toBe('POST')
64+
expect(request.postDataJSON().messageId).toBe(messageId)
6465
})
6566

6667
test('supports retrying in-flight requests on page navigation', async ({
6768
page,
6869
}) => {
6970
// Load analytics.js
7071
await page.goto('/standalone.html')
71-
await page.evaluate(() =>
72-
window.addEventListener('pagehide', () => console.log('hidden'))
73-
)
7472
await page.evaluate(() => window.analytics.load('fake-key'))
7573

7674
// blackhole the request so that it stays in-flight when we reload the page
@@ -85,11 +83,11 @@ test.describe('Standalone tests', () => {
8583
)
8684

8785
// Detect when we've seen a track request initiated by the browser
88-
const requestSent = new Promise<void>((resolve) => {
86+
const requestSent = new Promise<Record<string, any>>((resolve) => {
8987
const onRequest: (req: Request) => void = (req) => {
9088
if (req.url() === 'https://api.segment.io/v1/t') {
9189
page.off('request', onRequest)
92-
resolve()
90+
resolve(req.postDataJSON())
9391
}
9492
}
9593

@@ -101,7 +99,7 @@ test.describe('Standalone tests', () => {
10199
void window.analytics.track('test event')
102100
})
103101

104-
await requestSent
102+
const { messageId } = await requestSent
105103
await page.reload()
106104

107105
// load analytics.js again and wait for a new request.
@@ -111,6 +109,7 @@ test.describe('Standalone tests', () => {
111109
])
112110

113111
expect(request.method()).toBe('POST')
112+
expect(request.postDataJSON().messageId).toBe(messageId)
114113
})
115114
})
116115
})

0 commit comments

Comments
 (0)