tldraw 4.2.2 → 4.2.3

package/src/lib/shapes/draw/DrawShapeUtil.test.ts

@@ -1,146 +0,0 @@
-import { createShapeId, TLDrawShape } from '@tldraw/editor'
-import { TestEditor } from '../../../test/TestEditor'
-import { createDrawSegments, pointsToBase64 } from '../../utils/test-helpers'
-
-let editor: TestEditor
-
-beforeEach(() => {
-	editor = new TestEditor()
-})
-
-afterEach(() => {
-	editor?.dispose()
-})
-
-describe('DrawShapeUtil dot detection', () => {
-	const shapeId = createShapeId('test-draw')
-
-	function createDrawShape(segments: TLDrawShape['props']['segments']): TLDrawShape {
-		editor.createShapes([
-			{
-				id: shapeId,
-				type: 'draw',
-				props: { segments },
-			},
-		])
-		return editor.getShape(shapeId) as TLDrawShape
-	}
-
-	describe('getIsDot behavior via hideResizeHandles', () => {
-		it('treats a shape with one segment and zero points as a dot', () => {
-			const shape = createDrawShape([{ type: 'free', points: '' }])
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-
-		it('treats a shape with one segment and one point as a dot', () => {
-			const shape = createDrawShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-
-		it('treats a shape with one segment and two points as NOT a dot', () => {
-			const shape = createDrawShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('treats a shape with one segment and many points as NOT a dot', () => {
-			const shape = createDrawShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-						{ x: 20, y: 5, z: 0.5 },
-						{ x: 30, y: 15, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('treats a shape with multiple segments as NOT a dot', () => {
-			const shape = createDrawShape(
-				createDrawSegments([[{ x: 0, y: 0, z: 0.5 }], [{ x: 10, y: 10, z: 0.5 }]])
-			)
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-	})
-
-	describe('hideRotateHandle mirrors hideResizeHandles for dots', () => {
-		it('hides rotate handle for dots', () => {
-			const shape = createDrawShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideRotateHandle(shape)).toBe(true)
-		})
-
-		it('shows rotate handle for non-dots', () => {
-			const shape = createDrawShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideRotateHandle(shape)).toBe(false)
-		})
-	})
-
-	describe('hideSelectionBoundsFg mirrors hideResizeHandles for dots', () => {
-		it('hides selection bounds for dots', () => {
-			const shape = createDrawShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideSelectionBoundsFg(shape)).toBe(true)
-		})
-
-		it('shows selection bounds for non-dots', () => {
-			const shape = createDrawShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideSelectionBoundsFg(shape)).toBe(false)
-		})
-	})
-
-	describe('base64 encoding boundary conditions', () => {
-		it('correctly handles the boundary at exactly 16 base64 characters (2 points)', () => {
-			// Each point is 8 base64 characters (3 Float16s = 6 bytes = 8 base64 chars)
-			// 2 points = 16 characters, which should NOT be a dot
-			const twoPointsBase64 = pointsToBase64([
-				{ x: 0, y: 0, z: 0.5 },
-				{ x: 1, y: 1, z: 0.5 },
-			])
-			expect(twoPointsBase64.length).toBe(16)
-
-			const shape = createDrawShape([{ type: 'free', points: twoPointsBase64 }])
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('correctly handles the boundary at exactly 8 base64 characters (1 point)', () => {
-			// 1 point = 8 characters, which should be a dot
-			const onePointBase64 = pointsToBase64([{ x: 0, y: 0, z: 0.5 }])
-			expect(onePointBase64.length).toBe(8)
-
-			const shape = createDrawShape([{ type: 'free', points: onePointBase64 }])
-			const util = editor.getShapeUtil('draw')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-	})
-})

package/src/lib/shapes/highlight/HighlightShapeUtil.test.ts

@@ -1,146 +0,0 @@
-import { createShapeId, TLHighlightShape } from '@tldraw/editor'
-import { TestEditor } from '../../../test/TestEditor'
-import { createDrawSegments, pointsToBase64 } from '../../utils/test-helpers'
-
-let editor: TestEditor
-
-beforeEach(() => {
-	editor = new TestEditor()
-})
-
-afterEach(() => {
-	editor?.dispose()
-})
-
-describe('HighlightShapeUtil dot detection', () => {
-	const shapeId = createShapeId('test-highlight')
-
-	function createHighlightShape(segments: TLHighlightShape['props']['segments']): TLHighlightShape {
-		editor.createShapes([
-			{
-				id: shapeId,
-				type: 'highlight',
-				props: { segments },
-			},
-		])
-		return editor.getShape(shapeId) as TLHighlightShape
-	}
-
-	describe('getIsDot behavior via hideResizeHandles', () => {
-		it('treats a shape with one segment and zero points as a dot', () => {
-			const shape = createHighlightShape([{ type: 'free', points: '' }])
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-
-		it('treats a shape with one segment and one point as a dot', () => {
-			const shape = createHighlightShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-
-		it('treats a shape with one segment and two points as NOT a dot', () => {
-			const shape = createHighlightShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('treats a shape with one segment and many points as NOT a dot', () => {
-			const shape = createHighlightShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-						{ x: 20, y: 5, z: 0.5 },
-						{ x: 30, y: 15, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('treats a shape with multiple segments as NOT a dot', () => {
-			const shape = createHighlightShape(
-				createDrawSegments([[{ x: 0, y: 0, z: 0.5 }], [{ x: 10, y: 10, z: 0.5 }]])
-			)
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-	})
-
-	describe('hideRotateHandle mirrors hideResizeHandles for dots', () => {
-		it('hides rotate handle for dots', () => {
-			const shape = createHighlightShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideRotateHandle(shape)).toBe(true)
-		})
-
-		it('shows rotate handle for non-dots', () => {
-			const shape = createHighlightShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideRotateHandle(shape)).toBe(false)
-		})
-	})
-
-	describe('hideSelectionBoundsFg mirrors hideResizeHandles for dots', () => {
-		it('hides selection bounds for dots', () => {
-			const shape = createHighlightShape(createDrawSegments([[{ x: 0, y: 0, z: 0.5 }]]))
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideSelectionBoundsFg(shape)).toBe(true)
-		})
-
-		it('shows selection bounds for non-dots', () => {
-			const shape = createHighlightShape(
-				createDrawSegments([
-					[
-						{ x: 0, y: 0, z: 0.5 },
-						{ x: 10, y: 10, z: 0.5 },
-					],
-				])
-			)
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideSelectionBoundsFg(shape)).toBe(false)
-		})
-	})
-
-	describe('base64 encoding boundary conditions', () => {
-		it('correctly handles the boundary at exactly 16 base64 characters (2 points)', () => {
-			// Each point is 8 base64 characters (3 Float16s = 6 bytes = 8 base64 chars)
-			// 2 points = 16 characters, which should NOT be a dot
-			const twoPointsBase64 = pointsToBase64([
-				{ x: 0, y: 0, z: 0.5 },
-				{ x: 1, y: 1, z: 0.5 },
-			])
-			expect(twoPointsBase64.length).toBe(16)
-
-			const shape = createHighlightShape([{ type: 'free', points: twoPointsBase64 }])
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(false)
-		})
-
-		it('correctly handles the boundary at exactly 8 base64 characters (1 point)', () => {
-			// 1 point = 8 characters, which should be a dot
-			const onePointBase64 = pointsToBase64([{ x: 0, y: 0, z: 0.5 }])
-			expect(onePointBase64.length).toBe(8)
-
-			const shape = createHighlightShape([{ type: 'free', points: onePointBase64 }])
-			const util = editor.getShapeUtil('highlight')
-			expect(util.hideResizeHandles(shape)).toBe(true)
-		})
-	})
-})

package/src/lib/shapes/shared/useEfficientZoomThreshold.ts

@@ -1,10 +0,0 @@
-import { useEditor, useValue } from '@tldraw/editor'
-
-/** Returns true when zoomed out far enough that shapes should render in a simplified "solid" style. */
-export function useEfficientZoomThreshold(threshold = 0.25) {
-	const editor = useEditor()
-	return useValue('efficient zoom threshold', () => editor.getEfficientZoomLevel() < threshold, [
-		editor,
-		threshold,
-	])
-}

package/src/lib/utils/test-helpers.ts

@@ -1,60 +0,0 @@
-import type { TLDrawShapeSegment, VecModel } from '@tldraw/editor'
-import { b64Vecs, compressLegacySegments } from '@tldraw/editor'
-
-/**
- * Helper function to convert draw shape points from VecModel[] to base64 string.
- * This is useful for tests that create draw shapes with the legacy array format.
- *
- * @example
- * ```ts
- * const segments = [{ type: 'free', points: pointsToBase64([{x: 0, y: 0, z: 0.5}]) }]
- * ```
- *
- * @public
- */
-export function pointsToBase64(points: VecModel[]): string {
-	return b64Vecs.encodePoints(points)
-}
-
-/**
- * Helper function to convert base64 string back to VecModel[] points.
- * This is useful for tests that need to inspect draw shape points.
- *
- * @example
- * ```ts
- * const points = base64ToPoints(shape.props.segments[0].points)
- * expect(points[0].x).toBe(0)
- * ```
- *
- * @public
- */
-export function base64ToPoints(base64: string): VecModel[] {
-	return b64Vecs.decodePoints(base64)
-}
-
-/**
- * Helper function to create draw shape segments from legacy array format.
- * This allows tests to use the old format while the shape uses the new base64 format.
- *
- * @example
- * ```ts
- * editor.createShapes([{
- *   type: 'draw',
- *   props: {
- *     segments: createDrawSegments([[{x: 0, y: 0}, {x: 10, y: 10}]])
- *   }
- * }])
- * ```
- * @public
- */
-export function createDrawSegments(
-	pointArrays: VecModel[][],
-	type: 'free' | 'straight' = 'free'
-): TLDrawShapeSegment[] {
-	return compressLegacySegments(
-		pointArrays.map((points) => ({
-			type,
-			points,
-		}))
-	)
-}

package/src/test/commands/cameraState.test.ts

@@ -1,299 +0,0 @@
-import { Box } from '@tldraw/editor'
-import { TestEditor } from '../TestEditor'
-
-let editor: TestEditor
-
-beforeEach(() => {
-	editor = new TestEditor()
-	editor.updateViewportScreenBounds(new Box(0, 0, 1600, 900))
-})
-
-describe('getCameraState', () => {
-	it('starts as idle', () => {
-		expect(editor.getCameraState()).toBe('idle')
-	})
-
-	it('becomes moving when the camera changes via setCamera', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.setCamera({ x: 100, y: 100, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-
-	it('becomes moving when the camera changes via pan', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.pan({ x: 100, y: 100 })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-
-	it('becomes moving when the camera changes via zoomIn', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-
-	it('returns to idle after the timeout elapses', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.setCamera({ x: 100, y: 100, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// The default timeout is 64ms (options.cameraMovingTimeoutMs)
-		// Each tick is 16ms, so we need ~4 ticks to elapse
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-	})
-
-	it('stays moving while camera continues to change', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.setCamera({ x: 100, y: 100, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Move again before timeout elapses
-		editor.forceTick(2)
-		editor.setCamera({ x: 200, y: 200, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Move again
-		editor.forceTick(2)
-		editor.setCamera({ x: 300, y: 300, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Now let it settle
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-	})
-
-	it('stays idle when camera position does not actually change', () => {
-		expect(editor.getCameraState()).toBe('idle')
-
-		// Setting the same camera position should not trigger moving state
-		const currentCamera = editor.getCamera()
-		editor.setCamera({ x: currentCamera.x, y: currentCamera.y, z: currentCamera.z })
-		expect(editor.getCameraState()).toBe('idle')
-	})
-
-	it('does not add multiple tick listeners when camera changes rapidly', () => {
-		// This test verifies the fix: we should not have redundant listeners
-		expect(editor.getCameraState()).toBe('idle')
-
-		// Change camera multiple times rapidly
-		editor.setCamera({ x: 100, y: 100, z: 1 })
-		editor.setCamera({ x: 200, y: 200, z: 1 })
-		editor.setCamera({ x: 300, y: 300, z: 1 })
-
-		expect(editor.getCameraState()).toBe('moving')
-
-		// After timeout, should return to idle exactly once
-		// If there were multiple listeners, the state might behave unexpectedly
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-	})
-
-	it('resets timeout when camera changes while already moving', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.setCamera({ x: 100, y: 100, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Wait almost until timeout
-		editor.forceTick(3)
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Change camera again - should reset timeout
-		editor.setCamera({ x: 200, y: 200, z: 1 })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Wait 3 more ticks - would have been idle if timeout wasn't reset
-		editor.forceTick(3)
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Now let it fully settle
-		editor.forceTick(3)
-		expect(editor.getCameraState()).toBe('idle')
-	})
-})
-
-describe('camera state with zoom', () => {
-	it('becomes moving on zoomOut', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.zoomOut(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-
-	it('becomes moving on centerOnPoint', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		editor.centerOnPoint({ x: 500, y: 500 })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-
-	it('becomes moving on zoomToFit', () => {
-		// Create a shape so zoomToFit has something to fit
-		editor.createShape({ type: 'geo', x: 100, y: 100, props: { w: 200, h: 200 } })
-		expect(editor.getCameraState()).toBe('idle')
-		editor.zoomToFit({ immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-	})
-})
-
-describe('getDebouncedZoomLevel', () => {
-	it('returns the current zoom level when camera is idle', () => {
-		expect(editor.getCameraState()).toBe('idle')
-		expect(editor.getDebouncedZoomLevel()).toBe(editor.getZoomLevel())
-
-		// Change zoom and let it settle
-		editor.zoomIn(undefined, { immediate: true })
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-		expect(editor.getDebouncedZoomLevel()).toBe(editor.getZoomLevel())
-	})
-
-	it('captures zoom when camera starts moving', () => {
-		expect(editor.getCameraState()).toBe('idle')
-
-		// Start zooming - the debounced zoom is captured when movement starts
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// The debounced zoom is captured at the moment movement starts (after first change)
-		const capturedZoom = editor.getDebouncedZoomLevel()
-		expect(capturedZoom).toBe(editor.getZoomLevel())
-	})
-
-	it('keeps captured zoom during continued camera movement', () => {
-		// Start zooming
-		editor.zoomIn(undefined, { immediate: true })
-		const capturedZoom = editor.getDebouncedZoomLevel()
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Zoom again while still moving - debounced value should stay the same
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-		expect(editor.getDebouncedZoomLevel()).toBe(capturedZoom)
-
-		// But current zoom should have changed
-		expect(editor.getZoomLevel()).not.toBe(capturedZoom)
-	})
-
-	it('updates debounced zoom when camera becomes idle again', () => {
-		// Start zooming
-		editor.zoomIn(undefined, { immediate: true })
-		const capturedZoom = editor.getDebouncedZoomLevel()
-
-		// Zoom again while moving to change the current zoom
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getDebouncedZoomLevel()).toBe(capturedZoom)
-
-		// Let camera settle
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-
-		// Debounced zoom should now match current zoom
-		expect(editor.getDebouncedZoomLevel()).toBe(editor.getZoomLevel())
-	})
-
-	it('captures new zoom at the start of each new movement', () => {
-		// First zoom and settle
-		editor.zoomIn(undefined, { immediate: true })
-		const firstCapturedZoom = editor.getDebouncedZoomLevel()
-		editor.forceTick(5)
-		expect(editor.getCameraState()).toBe('idle')
-
-		// Second zoom - should capture new zoom level
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-		// The captured zoom should be different from the first capture
-		expect(editor.getDebouncedZoomLevel()).not.toBe(firstCapturedZoom)
-		// And it should match the current zoom (since we just started moving)
-		expect(editor.getDebouncedZoomLevel()).toBe(editor.getZoomLevel())
-	})
-
-	describe('with debouncedZoom option disabled', () => {
-		let editorWithoutDebouncedZoom: TestEditor
-
-		beforeEach(() => {
-			editorWithoutDebouncedZoom = new TestEditor({
-				options: { debouncedZoom: false },
-			})
-			editorWithoutDebouncedZoom.updateViewportScreenBounds(new Box(0, 0, 1600, 900))
-		})
-
-		it('always returns the current zoom level even when camera is moving', () => {
-			const initialZoom = editorWithoutDebouncedZoom.getZoomLevel()
-
-			editorWithoutDebouncedZoom.zoomIn(undefined, { immediate: true })
-			expect(editorWithoutDebouncedZoom.getCameraState()).toBe('moving')
-
-			// Should return the current zoom, not the captured one
-			expect(editorWithoutDebouncedZoom.getDebouncedZoomLevel()).toBe(
-				editorWithoutDebouncedZoom.getZoomLevel()
-			)
-			expect(editorWithoutDebouncedZoom.getDebouncedZoomLevel()).not.toBe(initialZoom)
-		})
-	})
-})
-
-describe('getEfficientZoomLevel', () => {
-	it('returns current zoom level when below shape threshold', () => {
-		// Default threshold is 500 shapes, we have 0
-		expect(editor.getZoomLevel()).toBe(editor.getEfficientZoomLevel())
-
-		// Add a few shapes - still below threshold
-		for (let i = 0; i < 10; i++) {
-			editor.createShape({ type: 'geo', x: i * 100, y: 0, props: { w: 50, h: 50 } })
-		}
-		expect(editor.getCurrentPageShapeIds().size).toBe(10)
-
-		// Start zooming
-		editor.zoomIn(undefined, { immediate: true })
-		expect(editor.getCameraState()).toBe('moving')
-
-		// Should still return current zoom because we're below threshold
-		expect(editor.getEfficientZoomLevel()).toBe(editor.getZoomLevel())
-	})
-
-	describe('with many shapes above threshold', () => {
-		let editorWithManyShapes: TestEditor
-
-		beforeEach(() => {
-			// Use a lower threshold for testing
-			editorWithManyShapes = new TestEditor({
-				options: { debouncedZoomThreshold: 5 },
-			})
-			editorWithManyShapes.updateViewportScreenBounds(new Box(0, 0, 1600, 900))
-
-			// Add shapes above the threshold
-			for (let i = 0; i < 10; i++) {
-				editorWithManyShapes.createShape({
-					type: 'geo',
-					x: i * 100,
-					y: 0,
-					props: { w: 50, h: 50 },
-				})
-			}
-		})
-
-		it('returns debounced zoom level when above shape threshold and camera is moving', () => {
-			// First zoom to capture a debounced value
-			editorWithManyShapes.zoomIn(undefined, { immediate: true })
-			const capturedZoom = editorWithManyShapes.getEfficientZoomLevel()
-			expect(editorWithManyShapes.getCameraState()).toBe('moving')
-
-			// Zoom again while still moving
-			editorWithManyShapes.zoomIn(undefined, { immediate: true })
-			expect(editorWithManyShapes.getCameraState()).toBe('moving')
-
-			// Should return the captured zoom, not the current zoom
-			expect(editorWithManyShapes.getEfficientZoomLevel()).toBe(capturedZoom)
-			expect(editorWithManyShapes.getEfficientZoomLevel()).not.toBe(
-				editorWithManyShapes.getZoomLevel()
-			)
-		})
-
-		it('returns current zoom level when above threshold but camera is idle', () => {
-			editorWithManyShapes.zoomIn(undefined, { immediate: true })
-			editorWithManyShapes.forceTick(5)
-			expect(editorWithManyShapes.getCameraState()).toBe('idle')
-
-			// Should return current zoom because camera is idle
-			expect(editorWithManyShapes.getEfficientZoomLevel()).toBe(editorWithManyShapes.getZoomLevel())
-		})
-	})
-})