fix tests

Author: billyvg

packages/replay-canvas/src/canvas.ts

@@ -70,7 +70,7 @@ export const _replayCanvasIntegration = ((options: Partial<ReplayCanvasOptions>
       maxCanvasHeight ?
         Math.min(maxCanvasHeight, DEFAULT_MAX_CANVAS_SIZE)
         : DEFAULT_MAX_CANVAS_SIZE
-    ],
+    ] as [number, number],
   };
 
   let canvasManagerResolve: (value: CanvasManager) => void;

packages/replay-canvas/test/canvas.test.ts

@@ -1,9 +1,18 @@
 import { _replayCanvasIntegration } from '../src/canvas';
+import { CanvasManager } from '@sentry-internal/rrweb';
+
+jest.mock('@sentry-internal/rrweb');
+
+
+beforeEach(() => {
+  jest.clearAllMocks();
+})
 
 it('initializes with default options', () => {
   const rc = _replayCanvasIntegration();
+  const options = rc.getOptions();
 
-  expect(rc.getOptions()).toEqual({
+  expect(options).toEqual({
     recordCanvas: true,
     getCanvasManager: expect.any(Function),
     sampling: {
@@ -14,12 +23,20 @@ it('initializes with default options', () => {
       quality: 0.4,
     },
   });
+
+  // @ts-expect-error don't care about the normal options we need to call this with, just want to test maxCanvasSize
+  options.getCanvasManager({});
+
+  expect(CanvasManager).toHaveBeenCalledWith(expect.objectContaining({
+    maxCanvasSize: [1280, 1280],
+  }))
 });
 
 it('initializes with quality option and manual snapshot', () => {
   const rc = _replayCanvasIntegration({ enableManualSnapshot: true, quality: 'low' });
+  const options = rc.getOptions();
 
-  expect(rc.getOptions()).toEqual({
+  expect(options).toEqual({
     enableManualSnapshot: true,
     recordCanvas: true,
     getCanvasManager: expect.any(Function),
@@ -30,13 +47,40 @@ it('initializes with quality option and manual snapshot', () => {
       type: 'image/webp',
       quality: 0.25,
     },
-    maxCanvasSize: [1280, 1280],
   });
+
+
+  // @ts-expect-error don't care about the normal options we need to call this with, just want to test maxCanvasSize
+  options.getCanvasManager({});
+
+  expect(CanvasManager).toHaveBeenCalledWith(expect.objectContaining({
+    maxCanvasSize: [1280, 1280],
+  }))
 });
 
 it('enforces a max canvas size', () => {
   const rc = _replayCanvasIntegration({ enableManualSnapshot: true, quality: 'low', maxCanvasSize: [2000, 2000]});
+  const options = rc.getOptions();
+
+  expect(options).toEqual({
+    enableManualSnapshot: true,
+    recordCanvas: true,
+    getCanvasManager: expect.any(Function),
+    sampling: {
+      canvas: 1,
+    },
+    dataURLOptions: {
+      type: 'image/webp',
+      quality: 0.25,
+    },
+  });
 
-  expect(rc.getOptions().maxCanvasSize).toEqual([1280, 1280]);
+
+  // @ts-expect-error don't care about the normal options we need to call this with, just want to test maxCanvasSize
+  options.getCanvasManager({});
+
+  expect(CanvasManager).toHaveBeenCalledWith(expect.objectContaining({
+    maxCanvasSize: [1280, 1280],
+  }))
 });