Skip to content

Commit acce7d2

Browse files
committed
🐛 feedback: auto-execute worker implementation
1 parent 29e94d9 commit acce7d2

File tree

1 file changed

+7
-32
lines changed

1 file changed

+7
-32
lines changed

test/e2e/scenario/logs.scenario.ts

Lines changed: 7 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -16,21 +16,13 @@ test.describe('logs', () => {
1616
.withLogs()
1717
.withWorker(
1818
function (self) {
19-
self.addEventListener('message', (event) => {
20-
const message = (event as MessageEvent<string>).data
21-
22-
self.DD_LOGS!.logger.log(message)
23-
})
19+
self.DD_LOGS!.logger.log('Some message')
2420
},
2521
{ type: 'module' }
2622
)
27-
.run(async ({ flushEvents, intakeRegistry, browserName, interactWithWorker }) => {
23+
.run(async ({ flushEvents, intakeRegistry, browserName }) => {
2824
test.skip(browserName !== 'chromium', 'Non-Chromium browsers do not support ES modules in Service Workers')
2925

30-
await interactWithWorker((worker) => {
31-
worker.postMessage('Some message')
32-
})
33-
3426
await flushEvents()
3527

3628
expect(intakeRegistry.logsRequests).toHaveLength(1)
@@ -40,22 +32,13 @@ test.describe('logs', () => {
4032
createTest('service worker with worker logs - importScripts')
4133
.withLogs()
4234
.withWorker(function (self) {
43-
self.addEventListener('message', (event) => {
44-
const message = (event as MessageEvent<string>).data
45-
46-
self.DD_LOGS!.logger.log(message)
47-
})
35+
self.DD_LOGS!.logger.log('Other message')
4836
})
49-
.run(async ({ flushEvents, intakeRegistry, browserName, interactWithWorker }) => {
37+
.run(async ({ flushEvents, intakeRegistry, browserName }) => {
5038
test.skip(
5139
browserName === 'webkit',
5240
'BrowserStack overrides the localhost URL with bs-local.com and cannot be used to install a Service Worker'
5341
)
54-
55-
await interactWithWorker((worker) => {
56-
worker.postMessage('Other message')
57-
})
58-
5942
await flushEvents()
6043

6144
expect(intakeRegistry.logsRequests).toHaveLength(1)
@@ -64,23 +47,15 @@ test.describe('logs', () => {
6447

6548
createTest('service worker console forwarding')
6649
.withLogs({ forwardConsoleLogs: 'all', forwardErrorsToLogs: true })
67-
.withWorker(function (self) {
68-
self.addEventListener('message', (event) => {
69-
const message = (event as MessageEvent<string>).data
70-
71-
console.log(message)
72-
})
50+
.withWorker(function () {
51+
console.log('SW console log test')
7352
})
74-
.run(async ({ flushEvents, intakeRegistry, interactWithWorker, browserName }) => {
53+
.run(async ({ flushEvents, intakeRegistry, browserName }) => {
7554
test.skip(
7655
browserName === 'webkit',
7756
'BrowserStack overrides the localhost URL with bs-local.com and cannot be used to install a Service Worker'
7857
)
7958

80-
await interactWithWorker((worker) => {
81-
worker.postMessage('SW console log test')
82-
})
83-
8459
await flushEvents()
8560

8661
// Expect logs for console, error, and report events from service worker

0 commit comments

Comments
 (0)