Skip to content

Commit

Permalink
Revert "debug playwright test"
Browse files Browse the repository at this point in the history
This reverts commit 7cf7b73.
  • Loading branch information
billyvg committed Jan 11, 2024
1 parent 7cf7b73 commit d001529
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,6 @@ sentryTest('sets up canvas when adding ReplayCanvas integration first', async ({

await page.goto(url);

page.on('console', (m) => {
console.log(m);
});
const replay = await getReplaySnapshot(page);
const canvasOptions = replay._canvas;
expect(canvasOptions?.sampling.canvas).toBe(2);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,6 @@ sentryTest('sets up canvas when adding ReplayCanvas integration after Replay', a

await page.goto(url);

page.on('console', (m) => {
console.log(m);
});
const replay = await getReplaySnapshot(page);
const canvasOptions = replay._canvas;
expect(canvasOptions?.sampling.canvas).toBe(2);
Expand Down
1 change: 0 additions & 1 deletion packages/replay-canvas/src/canvas.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ const replayCanvasIntegration = ((options: Partial<ReplayCanvasOptions> = {}) =>
getOptions(): ReplayCanvasIntegrationOptions {
const { quality } = _canvasOptions;

console.log('getoptions');
return {
recordCanvas: true,
getCanvasManager: (options: ConstructorParameters<typeof CanvasManager>[0]) => new CanvasManager(options),
Expand Down
3 changes: 0 additions & 3 deletions packages/replay/src/integration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -357,20 +357,17 @@ Sentry.init({ replaysOnErrorSampleRate: ${errorSampleRate} })`,
private _maybeLoadFromReplayCanvasIntegration(): void {
// To save bundle size, we skip checking for stuff here
// and instead just try-catch everything - as generally this should all be defined
console.log("maybeload")
/* eslint-disable @typescript-eslint/no-non-null-assertion */
try {
const client = getClient()!;
const canvasIntegration = client.getIntegrationById!('ReplayCanvas') as Integration & {
getOptions(): ReplayCanvasIntegrationOptions;
};
if (!canvasIntegration) {
console.log('no canvas integration');
return;
}

this._replay!['_canvas'] = canvasIntegration.getOptions();
console.log(this._replay!['_canvas'])
} catch {
// ignore errors here
}
Expand Down

0 comments on commit d001529

Please sign in to comment.