Skip to content

Commit f6ab9a2

Browse files
committed
lint
1 parent 84295c4 commit f6ab9a2

File tree

3 files changed

+11
-17
lines changed

3 files changed

+11
-17
lines changed

packages/replay/src/types.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ export interface WorkerResponse {
184184
export type AddEventResult = void;
185185

186186
export interface BeforeAddRecoringEvent {
187-
(event: RecordingEvent): RecordingEvent | null | undefined
187+
(event: RecordingEvent): RecordingEvent | null | undefined;
188188
}
189189

190190
export interface ReplayNetworkOptions {

packages/replay/src/util/addEvent.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,17 +40,17 @@ export async function addEvent(
4040

4141
const replayOptions = replay.getOptions();
4242

43-
console.log('add event: ', typeof replayOptions.beforeAddRecordingEvent, event.type)
44-
const eventAfterPossibleCallback = typeof replayOptions.beforeAddRecordingEvent === 'function' ?
45-
replayOptions.beforeAddRecordingEvent(event) : event;
43+
const eventAfterPossibleCallback =
44+
typeof replayOptions.beforeAddRecordingEvent === 'function'
45+
? replayOptions.beforeAddRecordingEvent(event)
46+
: event;
4647

4748
if (!eventAfterPossibleCallback) {
4849
return;
4950
}
5051

5152
return await replay.eventBuffer.addEvent(eventAfterPossibleCallback);
5253
} catch (error) {
53-
console.error(error)
5454
__DEBUG_BUILD__ && logger.error(error);
5555
await replay.stop('addEvent');
5656

packages/replay/test/integration/beforeAddRecordingEvent.test.ts

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,8 @@ import type { Transport } from '@sentry/types';
33
import * as SentryUtils from '@sentry/utils';
44

55
import type { Replay } from '../../src';
6-
import { DEFAULT_FLUSH_MIN_DELAY, WINDOW } from '../../src/constants';
76
import type { ReplayContainer } from '../../src/replay';
87
import { clearSession } from '../../src/session/clearSession';
9-
import { addEvent } from '../../src/util/addEvent';
108
import * as SendReplayRequest from '../../src/util/sendReplayRequest';
119
import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
1210
import { useFakeTimers } from '../utils/use-fake-timers';
@@ -38,20 +36,19 @@ describe('Integration | beforeAddRecordingEvent', () => {
3836

3937
({ replay, integration } = await mockSdk({
4038
replayOptions: {
41-
beforeAddRecordingEvent: (event) => {
39+
beforeAddRecordingEvent: event => {
4240
const eventData = event.data as Record<string, any>;
4341

4442
if (eventData.tag === 'breadcrumb' && eventData.payload.category === 'ui.click') {
45-
4643
return {
4744
...event,
4845
data: {
4946
...eventData,
5047
payload: {
5148
...eventData.payload,
5249
message: 'beforeAddRecordingEvent',
53-
}
54-
}
50+
},
51+
},
5552
};
5653
}
5754

@@ -106,7 +103,7 @@ describe('Integration | beforeAddRecordingEvent', () => {
106103
await advanceTimers(5000);
107104

108105
expect(replay).toHaveLastSentReplay({
109-
recordingPayloadHeader: {segment_id: 0},
106+
recordingPayloadHeader: { segment_id: 0 },
110107
recordingData: JSON.stringify([
111108
{
112109
type: 5,
@@ -122,7 +119,6 @@ describe('Integration | beforeAddRecordingEvent', () => {
122119
},
123120
},
124121
},
125-
126122
]),
127123
});
128124
});
@@ -136,10 +132,8 @@ describe('Integration | beforeAddRecordingEvent', () => {
136132
jest.runAllTimers();
137133
await new Promise(process.nextTick);
138134
expect(replay).toHaveLastSentReplay({
139-
recordingPayloadHeader: {segment_id: 0},
140-
recordingData: JSON.stringify([
141-
{"data":{"isCheckout":true},"timestamp":BASE_TIMESTAMP,"type":2}
142-
]),
135+
recordingPayloadHeader: { segment_id: 0 },
136+
recordingData: JSON.stringify([{ data: { isCheckout: true }, timestamp: BASE_TIMESTAMP, type: 2 }]),
143137
});
144138
});
145139
});

0 commit comments

Comments
 (0)