Skip to content

Commit 5ea8e14

Browse files
committed
test(catcher): Catcher release feature tests moved to separate file
1 parent fbd1089 commit 5ea8e14

File tree

2 files changed

+23
-20
lines changed

2 files changed

+23
-20
lines changed

packages/javascript/tests/catcher.context.test.ts

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -16,28 +16,9 @@ describe('Catcher', () => {
1616

1717
// ── Context enrichment ────────────────────────────────────────────────────
1818
//
19-
// The Catcher attaches contextual information to every event: an optional
20-
// release version and arbitrary developer-supplied context data.
19+
// The Catcher attaches arbitrary developer-supplied context data to every event.
2120
describe('context enrichment', () => {
2221

23-
it('should include release version when configured', async () => {
24-
const { sendSpy, transport } = createTransport();
25-
26-
createCatcher(transport, { release: '1.2.3' }).send(new Error('e'));
27-
await wait();
28-
29-
expect(getLastPayload(sendSpy).release).toBe('1.2.3');
30-
});
31-
32-
it('should omit release when not configured', async () => {
33-
const { sendSpy, transport } = createTransport();
34-
35-
createCatcher(transport).send(new Error('e'));
36-
await wait();
37-
38-
expect(getLastPayload(sendSpy).release).toBeFalsy();
39-
});
40-
4122
it('should include global context set via setContext()', async () => {
4223
const { sendSpy, transport } = createTransport();
4324
const hawk = createCatcher(transport);
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
import { describe, expect, it } from "vitest";
2+
import { createCatcher, createTransport, getLastPayload, wait } from "./catcher.helpers";
3+
4+
describe('Catcher', () => {
5+
it('should include release version when configured', async () => {
6+
const { sendSpy, transport } = createTransport();
7+
8+
createCatcher(transport, { release: '1.2.3' }).send(new Error('e'));
9+
await wait();
10+
11+
expect(getLastPayload(sendSpy).release).toBe('1.2.3');
12+
});
13+
14+
it('should omit release when not configured', async () => {
15+
const { sendSpy, transport } = createTransport();
16+
17+
createCatcher(transport).send(new Error('e'));
18+
await wait();
19+
20+
expect(getLastPayload(sendSpy).release).toBeFalsy();
21+
});
22+
});

0 commit comments

Comments
 (0)