@tldraw/tlschema 4.2.1 → 4.2.2

package/src/assets/TLBookmarkAsset.test.ts

@@ -1,96 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { getTestMigration } from '../__tests__/migrationTestUtils'
-import { bookmarkAssetVersions } from './TLBookmarkAsset'
-
-describe('TLBookmarkAsset', () => {
-	describe('MakeUrlsValid migration', () => {
-		const { up, down } = getTestMigration(bookmarkAssetVersions.MakeUrlsValid)
-
-		it('should clean invalid src URLs and preserve valid ones', () => {
-			const assetWithInvalidSrc = {
-				id: 'asset:bookmark1',
-				type: 'bookmark',
-				props: {
-					title: 'Test Bookmark',
-					description: 'Test Description',
-					image: 'https://example.com/image.jpg',
-					src: 'invalid-url-format',
-				},
-			}
-
-			const result = up(assetWithInvalidSrc)
-			expect(result.props.src).toBe('')
-
-			// Test valid URL is preserved
-			const assetWithValidSrc = {
-				...assetWithInvalidSrc,
-				props: { ...assetWithInvalidSrc.props, src: 'https://example.com' },
-			}
-			const validResult = up(assetWithValidSrc)
-			expect(validResult.props.src).toBe('https://example.com')
-		})
-
-		it('should be a no-op down migration', () => {
-			const asset = {
-				id: 'asset:bookmark1',
-				type: 'bookmark',
-				props: {
-					title: 'Test Bookmark',
-					description: 'Test Description',
-					image: 'https://example.com/image.jpg',
-					src: 'https://example.com',
-				},
-			}
-
-			const result = down(asset)
-			expect(result).toEqual(asset)
-		})
-	})
-
-	describe('AddFavicon migration', () => {
-		const { up, down } = getTestMigration(bookmarkAssetVersions.AddFavicon)
-
-		it('should add favicon property and clean invalid URLs', () => {
-			// Test adding favicon property to asset without one
-			const assetWithoutFavicon = {
-				id: 'asset:bookmark1',
-				type: 'bookmark',
-				props: {
-					title: 'Test Bookmark',
-					description: 'Test Description',
-					image: 'https://example.com/image.jpg',
-					src: 'https://example.com',
-				},
-			}
-
-			const result = up(assetWithoutFavicon)
-			expect(result.props.favicon).toBe('')
-
-			// Test cleaning invalid favicon URL
-			const assetWithInvalidFavicon = {
-				...assetWithoutFavicon,
-				props: { ...assetWithoutFavicon.props, favicon: 'invalid-url' },
-			}
-			const cleanResult = up(assetWithInvalidFavicon)
-			expect(cleanResult.props.favicon).toBe('')
-		})
-
-		it('should remove favicon property in down migration', () => {
-			const assetWithFavicon = {
-				id: 'asset:bookmark1',
-				type: 'bookmark',
-				props: {
-					title: 'Test Bookmark',
-					description: 'Test Description',
-					image: 'https://example.com/image.jpg',
-					src: 'https://example.com',
-					favicon: 'https://example.com/favicon.ico',
-				},
-			}
-
-			const result = down(assetWithFavicon)
-			expect(result.props).not.toHaveProperty('favicon')
-			expect(result.props.title).toBe('Test Bookmark')
-		})
-	})
-})

package/src/assets/TLImageAsset.test.ts

@@ -1,213 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { getTestMigration } from '../__tests__/migrationTestUtils'
-import { imageAssetVersions } from './TLImageAsset'
-
-describe('TLImageAsset', () => {
-	describe('AddIsAnimated migration', () => {
-		const { up, down } = getTestMigration(imageAssetVersions.AddIsAnimated)
-
-		it('should add isAnimated property in up migration', () => {
-			const assetWithoutIsAnimated = {
-				id: 'asset:image1',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-				},
-			}
-
-			const result = up(assetWithoutIsAnimated)
-			expect(result.props.isAnimated).toBe(false)
-		})
-
-		it('should remove isAnimated property in down migration', () => {
-			const assetWithIsAnimated = {
-				id: 'asset:image3',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-					isAnimated: false,
-				},
-			}
-
-			const result = down(assetWithIsAnimated)
-			expect(result.props).not.toHaveProperty('isAnimated')
-		})
-	})
-
-	describe('RenameWidthHeight migration', () => {
-		const { up, down } = getTestMigration(imageAssetVersions.RenameWidthHeight)
-
-		it('should rename width and height to w and h in up migration', () => {
-			const assetWithWidthHeight = {
-				id: 'asset:image1',
-				type: 'image',
-				props: {
-					width: 800,
-					height: 600,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-				},
-			}
-
-			const result = up(assetWithWidthHeight)
-			expect(result.props.w).toBe(800)
-			expect(result.props.h).toBe(600)
-			expect(result.props).not.toHaveProperty('width')
-			expect(result.props).not.toHaveProperty('height')
-		})
-
-		it('should rename w and h to width and height in down migration', () => {
-			const assetWithWH = {
-				id: 'asset:image3',
-				type: 'image',
-				props: {
-					w: 1024,
-					h: 768,
-					name: 'test.png',
-					isAnimated: false,
-					mimeType: 'image/png',
-					src: 'https://example.com/test.png',
-				},
-			}
-
-			const result = down(assetWithWH)
-			expect(result.props.width).toBe(1024)
-			expect(result.props.height).toBe(768)
-			expect(result.props).not.toHaveProperty('w')
-			expect(result.props).not.toHaveProperty('h')
-		})
-	})
-
-	describe('MakeUrlsValid migration', () => {
-		const { up, down: _down } = getTestMigration(imageAssetVersions.MakeUrlsValid)
-
-		it('should clean invalid src URLs in up migration', () => {
-			const assetWithInvalidSrc = {
-				id: 'asset:image1',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'invalid-url-format',
-				},
-			}
-
-			const result = up(assetWithInvalidSrc)
-			expect(result.props.src).toBe('')
-		})
-
-		it('should preserve valid src URLs in up migration', () => {
-			const assetWithValidSrc = {
-				id: 'asset:image2',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-				},
-			}
-
-			const result = up(assetWithValidSrc)
-			expect(result.props.src).toBe('https://example.com/test.jpg')
-		})
-	})
-
-	describe('AddFileSize migration', () => {
-		const { up, down } = getTestMigration(imageAssetVersions.AddFileSize)
-
-		it('should add fileSize property with -1 default in up migration', () => {
-			const assetWithoutFileSize = {
-				id: 'asset:image1',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-				},
-			}
-
-			const result = up(assetWithoutFileSize)
-			expect(result.props.fileSize).toBe(-1)
-		})
-
-		it('should remove fileSize property in down migration', () => {
-			const assetWithFileSize = {
-				id: 'asset:image3',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-					fileSize: 50000,
-				},
-			}
-
-			const result = down(assetWithFileSize)
-			expect(result.props).not.toHaveProperty('fileSize')
-		})
-	})
-
-	describe('MakeFileSizeOptional migration', () => {
-		const { up, down } = getTestMigration(imageAssetVersions.MakeFileSizeOptional)
-
-		it('should convert fileSize -1 to undefined in up migration', () => {
-			const assetWithNegativeFileSize = {
-				id: 'asset:image1',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-					fileSize: -1,
-				},
-			}
-
-			const result = up(assetWithNegativeFileSize)
-			expect(result.props.fileSize).toBeUndefined()
-		})
-
-		it('should convert undefined fileSize to -1 in down migration', () => {
-			const assetWithUndefinedFileSize = {
-				id: 'asset:image5',
-				type: 'image',
-				props: {
-					w: 100,
-					h: 100,
-					name: 'test.jpg',
-					isAnimated: false,
-					mimeType: 'image/jpeg',
-					src: 'https://example.com/test.jpg',
-					fileSize: undefined,
-				},
-			}
-
-			const result = down(assetWithUndefinedFileSize)
-			expect(result.props.fileSize).toBe(-1)
-		})
-	})
-})

package/src/assets/TLVideoAsset.test.ts

@@ -1,105 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { getTestMigration } from '../__tests__/migrationTestUtils'
-import { videoAssetVersions } from './TLVideoAsset'
-
-describe('TLVideoAsset', () => {
-	describe('migrations', () => {
-		it('should handle AddIsAnimated migration', () => {
-			const { up, down } = getTestMigration(videoAssetVersions.AddIsAnimated)
-
-			const assetWithoutIsAnimated = {
-				id: 'asset:video1',
-				type: 'video',
-				props: {
-					w: 640,
-					h: 480,
-					name: 'test.mp4',
-					mimeType: 'video/mp4',
-					src: 'https://example.com/test.mp4',
-				},
-			}
-
-			const upResult = up(assetWithoutIsAnimated)
-			expect(upResult.props.isAnimated).toBe(false)
-
-			const downResult = down(upResult)
-			expect(downResult.props).not.toHaveProperty('isAnimated')
-		})
-
-		it('should handle RenameWidthHeight migration', () => {
-			const { up, down } = getTestMigration(videoAssetVersions.RenameWidthHeight)
-
-			const assetWithWidthHeight = {
-				id: 'asset:video1',
-				type: 'video',
-				props: {
-					width: 1920,
-					height: 1080,
-					name: 'test.mp4',
-					isAnimated: true,
-					mimeType: 'video/mp4',
-					src: 'https://example.com/test.mp4',
-				},
-			}
-
-			const upResult = up(assetWithWidthHeight)
-			expect(upResult.props.w).toBe(1920)
-			expect(upResult.props.h).toBe(1080)
-			expect(upResult.props).not.toHaveProperty('width')
-			expect(upResult.props).not.toHaveProperty('height')
-
-			const downResult = down(upResult)
-			expect(downResult.props.width).toBe(1920)
-			expect(downResult.props.height).toBe(1080)
-			expect(downResult.props).not.toHaveProperty('w')
-			expect(downResult.props).not.toHaveProperty('h')
-		})
-
-		it('should handle MakeUrlsValid migration', () => {
-			const { up } = getTestMigration(videoAssetVersions.MakeUrlsValid)
-
-			const assetWithInvalidSrc = {
-				id: 'asset:video1',
-				type: 'video',
-				props: {
-					w: 640,
-					h: 480,
-					name: 'test.mp4',
-					isAnimated: true,
-					mimeType: 'video/mp4',
-					src: 'invalid-url-format',
-				},
-			}
-
-			const result = up(assetWithInvalidSrc)
-			expect(result.props.src).toBe('')
-		})
-
-		it('should handle MakeFileSizeOptional migration', () => {
-			const { up, down } = getTestMigration(videoAssetVersions.MakeFileSizeOptional)
-
-			const assetWithNegativeFileSize = {
-				id: 'asset:video1',
-				type: 'video',
-				props: {
-					w: 640,
-					h: 480,
-					name: 'test.mp4',
-					isAnimated: true,
-					mimeType: 'video/mp4',
-					src: 'https://example.com/test.mp4',
-					fileSize: -1,
-				},
-			}
-
-			const upResult = up(assetWithNegativeFileSize)
-			expect(upResult.props.fileSize).toBeUndefined()
-
-			const downResult = down({
-				...assetWithNegativeFileSize,
-				props: { ...assetWithNegativeFileSize.props, fileSize: undefined },
-			})
-			expect(downResult.props.fileSize).toBe(-1)
-		})
-	})
-})

package/src/bindings/TLArrowBinding.test.ts

@@ -1,55 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { getTestMigration } from '../__tests__/migrationTestUtils'
-import { arrowBindingVersions } from './TLArrowBinding'
-
-describe('TLArrowBinding', () => {
-	describe('AddSnap migration', () => {
-		const { up, down } = getTestMigration(arrowBindingVersions.AddSnap)
-
-		it('should add snap property with default value "none"', () => {
-			const oldRecord = {
-				props: {
-					terminal: 'end',
-					normalizedAnchor: { x: 0.5, y: 0.5 },
-					isExact: true,
-					isPrecise: false,
-				},
-			}
-
-			const result = up(oldRecord)
-			expect(result.props.snap).toBe('none')
-			expect(result.props.terminal).toBe('end')
-		})
-
-		it('should remove snap property on down migration', () => {
-			const newRecord = {
-				props: {
-					terminal: 'end',
-					normalizedAnchor: { x: 0.5, y: 0.5 },
-					isExact: true,
-					isPrecise: false,
-					snap: 'center',
-				},
-			}
-
-			const result = down(newRecord)
-			expect(result.props.snap).toBeUndefined()
-			expect(result.props.terminal).toBe('end')
-		})
-
-		it('should be reversible', () => {
-			const originalRecord = {
-				props: {
-					terminal: 'start',
-					normalizedAnchor: { x: 0.5, y: 0.5 },
-					isExact: true,
-					isPrecise: false,
-				},
-			}
-
-			const upResult = up(originalRecord)
-			const downResult = down(upResult)
-			expect(downResult.props).toEqual(originalRecord.props)
-		})
-	})
-})

package/src/misc/id-validator.test.ts

@@ -1,50 +0,0 @@
-import type { RecordId, UnknownRecord } from '@tldraw/store'
-import { describe, expect, it } from 'vitest'
-import { idValidator } from './id-validator'
-
-// Mock record types for testing
-interface MockShapeRecord extends UnknownRecord {
-	typeName: 'shape'
-}
-
-interface MockPageRecord extends UnknownRecord {
-	typeName: 'page'
-}
-
-type MockShapeId = RecordId<MockShapeRecord>
-type MockPageId = RecordId<MockPageRecord>
-
-describe('idValidator', () => {
-	it('should validate correct IDs with proper prefix', () => {
-		const shapeValidator = idValidator<MockShapeId>('shape')
-		const pageValidator = idValidator<MockPageId>('page')
-
-		expect(shapeValidator.validate('shape:abc123')).toBe('shape:abc123')
-		expect(shapeValidator.validate('shape:')).toBe('shape:')
-		expect(pageValidator.validate('page:main')).toBe('page:main')
-	})
-
-	it('should reject IDs with wrong prefix', () => {
-		const shapeValidator = idValidator<MockShapeId>('shape')
-
-		expect(() => shapeValidator.validate('page:abc123')).toThrow(
-			'shape ID must start with "shape:"'
-		)
-		expect(() => shapeValidator.validate('asset:xyz789')).toThrow(
-			'shape ID must start with "shape:"'
-		)
-		expect(() => shapeValidator.validate('abc123')).toThrow('shape ID must start with "shape:"')
-		expect(() => shapeValidator.validate('')).toThrow('shape ID must start with "shape:"')
-	})
-
-	it('should work with different prefixes independently', () => {
-		const shapeValidator = idValidator<MockShapeId>('shape')
-		const pageValidator = idValidator<MockPageId>('page')
-
-		expect(shapeValidator.isValid('shape:abc123')).toBe(true)
-		expect(shapeValidator.isValid('page:abc123')).toBe(false)
-
-		expect(pageValidator.isValid('shape:abc123')).toBe(false)
-		expect(pageValidator.isValid('page:abc123')).toBe(true)
-	})
-})