Skip to content

Commit 29addf0

Browse files
committed
fix more tests
1 parent 571e0e3 commit 29addf0

File tree

8 files changed

+33
-37
lines changed

8 files changed

+33
-37
lines changed

packages/core/test/lib/prepareEvent.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import type { ScopeContext } from '../../src';
12
import { GLOBAL_OBJ, createStackParser, getGlobalScope, getIsolationScope } from '../../src';
23
import type {
34
Attachment,
@@ -7,7 +8,6 @@ import type {
78
Event,
89
EventHint,
910
EventProcessor,
10-
ScopeContext,
1111
} from '../../src/types-hoist';
1212

1313
import { Scope } from '../../src/scope';

packages/core/test/lib/utils/applyScopeDataToEvent.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1+
import type { ScopeData } from '../../../src';
12
import { startInactiveSpan } from '../../../src';
2-
import type { Attachment, Breadcrumb, Event, EventProcessor, EventType, ScopeData } from '../../../src/types-hoist';
3+
import type { Attachment, Breadcrumb, Event, EventProcessor, EventType } from '../../../src/types-hoist';
34
import {
45
applyScopeDataToEvent,
56
mergeAndOverwriteScopeData,

packages/core/test/utils-hoist/requestdata.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/* eslint-disable deprecation/deprecation */
22
import type * as net from 'net';
3-
import { addRequestDataToEvent, extractPathForTransaction, extractRequestData } from '@sentry/core';
3+
import { addRequestDataToEvent, extractPathForTransaction, extractRequestData } from '../../src';
44
import type { Event, PolymorphicRequest, TransactionSource, User } from '../../src/types-hoist';
55
import { getClientIPAddress } from '../../src/utils-hoist/vendor/getIpAddress';
66

packages/deno/test/__snapshots__/mod.test.ts.snap

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ snapshot[`captureException 1`] = `
5656
"});",
5757
"",
5858
"Deno.test('captureMessage', async t => {",
59-
" let ev: sentryTypes.Event | undefined;",
59+
" let ev: sentryCore.Event | undefined;",
6060
],
6161
pre_context: [
6262
" ev = event;",
@@ -86,7 +86,7 @@ snapshot[`captureException 1`] = `
8686
],
8787
pre_context: [
8888
"Deno.test('captureException', async t => {",
89-
" let ev: sentryTypes.Event | undefined;",
89+
" let ev: sentryCore.Event | undefined;",
9090
" const client = getTestClient(event => {",
9191
" ev = event;",
9292
" });",

packages/deno/test/build.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,2 @@
11
// We use this as the entry point to bundle Sentry dependencies that are used by the tests.
2-
export * as sentryTypes from '@sentry/core';
3-
export * as sentryUtils from '@sentry/core';
42
export * as sentryCore from '@sentry/core';

packages/deno/test/mod.test.ts

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,20 @@
11
import { assertEquals } from 'https://deno.land/std@0.202.0/assert/assert_equals.ts';
22
import { assertSnapshot } from 'https://deno.land/std@0.202.0/testing/snapshot.ts';
33

4-
import type { sentryTypes } from '../build-test/index.js';
5-
import { sentryUtils } from '../build-test/index.js';
4+
import { sentryCore } from '../build-test/index.js';
65
import { DenoClient, getCurrentScope, getDefaultIntegrations } from '../build/index.mjs';
76
import { getNormalizedEvent } from './normalize.ts';
87
import { makeTestTransport } from './transport.ts';
98

109
function getTestClient(
11-
callback: (event?: sentryTypes.Event) => void,
12-
integrations: sentryTypes.Integration[] = [],
10+
callback: (event?: sentryCore.Event) => void,
11+
integrations: sentryCore.Integration[] = [],
1312
): DenoClient {
1413
const client = new DenoClient({
1514
dsn: 'https://233a45e5efe34c47a3536797ce15dafa@nothing.here/5650507',
1615
debug: true,
1716
integrations: [...getDefaultIntegrations({}), ...integrations],
18-
stackParser: sentryUtils.createStackParser(sentryUtils.nodeStackLineParser()),
17+
stackParser: sentryCore.createStackParser(sentryCore.nodeStackLineParser()),
1918
transport: makeTestTransport(envelope => {
2019
callback(getNormalizedEvent(envelope));
2120
}),
@@ -34,7 +33,7 @@ function delay(time: number): Promise<void> {
3433
}
3534

3635
Deno.test('captureException', async t => {
37-
let ev: sentryTypes.Event | undefined;
36+
let ev: sentryCore.Event | undefined;
3837
const client = getTestClient(event => {
3938
ev = event;
4039
});
@@ -50,7 +49,7 @@ Deno.test('captureException', async t => {
5049
});
5150

5251
Deno.test('captureMessage', async t => {
53-
let ev: sentryTypes.Event | undefined;
52+
let ev: sentryCore.Event | undefined;
5453
const client = getTestClient(event => {
5554
ev = event;
5655
});
@@ -62,7 +61,7 @@ Deno.test('captureMessage', async t => {
6261
});
6362

6463
Deno.test('captureMessage twice', async t => {
65-
let ev: sentryTypes.Event | undefined;
64+
let ev: sentryCore.Event | undefined;
6665
const client = getTestClient(event => {
6766
ev = event;
6867
});

packages/deno/test/normalize.ts

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,20 @@
11
/* eslint-disable complexity */
2-
import type { sentryTypes } from '../build-test/index.js';
3-
import { sentryUtils } from '../build-test/index.js';
2+
import { sentryCore } from '../build-test/index.js';
43

5-
type EventOrSession = sentryTypes.Event | sentryTypes.Session;
4+
type EventOrSession = sentryCore.Event | sentryCore.Session;
65

7-
export function getNormalizedEvent(envelope: sentryTypes.Envelope): sentryTypes.Event | undefined {
8-
let event: sentryTypes.Event | undefined;
6+
export function getNormalizedEvent(envelope: sentryCore.Envelope): sentryCore.Event | undefined {
7+
let event: sentryCore.Event | undefined;
98

10-
sentryUtils.forEachEnvelopeItem(envelope, item => {
9+
sentryCore.forEachEnvelopeItem(envelope, item => {
1110
const [headers, body] = item;
1211

1312
if (headers.type === 'event') {
14-
event = body as sentryTypes.Event;
13+
event = body as sentryCore.Event;
1514
}
1615
});
1716

18-
return normalize(event) as sentryTypes.Event | undefined;
17+
return normalize(event) as sentryCore.Event | undefined;
1918
}
2019

2120
export function normalize(event: EventOrSession | undefined): EventOrSession | undefined {
@@ -24,14 +23,14 @@ export function normalize(event: EventOrSession | undefined): EventOrSession | u
2423
}
2524

2625
if (eventIsSession(event)) {
27-
return normalizeSession(event as sentryTypes.Session);
26+
return normalizeSession(event as sentryCore.Session);
2827
} else {
29-
return normalizeEvent(event as sentryTypes.Event);
28+
return normalizeEvent(event as sentryCore.Event);
3029
}
3130
}
3231

3332
export function eventIsSession(data: EventOrSession): boolean {
34-
return !!(data as sentryTypes.Session)?.sid;
33+
return !!(data as sentryCore.Session)?.sid;
3534
}
3635

3736
/**
@@ -40,7 +39,7 @@ export function eventIsSession(data: EventOrSession): boolean {
4039
* All properties that are timestamps, versions, ids or variables that may vary
4140
* by platform are replaced with placeholder strings
4241
*/
43-
function normalizeSession(session: sentryTypes.Session): sentryTypes.Session {
42+
function normalizeSession(session: sentryCore.Session): sentryCore.Session {
4443
if (session.sid) {
4544
session.sid = '{{id}}';
4645
}
@@ -66,7 +65,7 @@ function normalizeSession(session: sentryTypes.Session): sentryTypes.Session {
6665
* All properties that are timestamps, versions, ids or variables that may vary
6766
* by platform are replaced with placeholder strings
6867
*/
69-
function normalizeEvent(event: sentryTypes.Event): sentryTypes.Event {
68+
function normalizeEvent(event: sentryCore.Event): sentryCore.Event {
7069
if (event.sdk?.version) {
7170
event.sdk.version = '{{version}}';
7271
}

packages/deno/test/transport.ts

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,18 @@
1-
import type { sentryTypes } from '../build-test/index.js';
2-
import { sentryCore, sentryUtils } from '../build-test/index.js';
1+
import { sentryCore } from '../build-test/index.js';
32

4-
export interface TestTransportOptions extends sentryTypes.BaseTransportOptions {
5-
callback: (envelope: sentryTypes.Envelope) => void;
3+
export interface TestTransportOptions extends sentryCore.BaseTransportOptions {
4+
callback: (envelope: sentryCore.Envelope) => void;
65
}
76

87
/**
98
* Creates a Transport that uses the Fetch API to send events to Sentry.
109
*/
11-
export function makeTestTransport(callback: (envelope: sentryTypes.Envelope) => void) {
12-
return (options: sentryTypes.BaseTransportOptions): sentryTypes.Transport => {
10+
export function makeTestTransport(callback: (envelope: sentryCore.Envelope) => void) {
11+
return (options: sentryCore.BaseTransportOptions): sentryCore.Transport => {
1312
async function doCallback(
14-
request: sentryTypes.TransportRequest,
15-
): Promise<sentryTypes.TransportMakeRequestResponse> {
16-
await callback(sentryUtils.parseEnvelope(request.body));
13+
request: sentryCore.TransportRequest,
14+
): Promise<sentryCore.TransportMakeRequestResponse> {
15+
await callback(sentryCore.parseEnvelope(request.body));
1716

1817
return Promise.resolve({
1918
statusCode: 200,

0 commit comments

Comments
 (0)