fix test / migration / bug

packages/tldraw/src/lib/shapes/image/ImageShapeUtil.tsx

@@ -227,7 +227,7 @@ export class ImageShapeUtil extends BaseBoxShapeUtil<TLImageShape> {
 	}
 
 	override onDoubleClick = (shape: TLImageShape) => {
-		const zoomLevel = useValue('zoom level', () => this.editor.getZoomLevel(), [this.editor])
+		const zoomLevel = this.editor.getZoomLevel()
 		const asset = shape.props.assetId
 			? this.editor.getAsset<TLImageAsset>(shape.props.assetId)
 			: undefined

packages/tlschema/src/assets/TLImageAsset.ts

@@ -88,13 +88,13 @@ export const imageAssetMigrations = createRecordMigrationSequence({
 				if (asset.props.src === null) {
 					asset.props.sources = []
 				} else {
-					asset.props.sources = [{ scale: 1, src: asset.src }]
+					asset.props.sources = [{ scale: 1, src: asset.props.src }]
 				}
 				delete asset.props.src
 			},
 			down: (asset: any) => {
 				// get the largest source
-				const src = asset.props.sources.sort((a: any, b: any) => b.scale - a.scale)[0]?.src ?? null
+				const src = asset.props.sources[0]?.src ?? null
 				asset.props.src = src
 				delete asset.props.sources
 			},

packages/tlschema/src/migrations.test.ts

@@ -1561,6 +1561,22 @@ describe('Add text align to text shapes', () => {
 	})
 })
 
+describe('Add image sources to image shape', () => {
+	const { up, down } = getTestMigration(imageAssetVersions.AddSources)
+
+	test('up works as expected', () => {
+		expect(up({ props: { src: 'abc' } })).toEqual({
+			props: { sources: [{ scale: 1, src: 'abc' }] },
+		})
+	})
+
+	test('down works as expected', () => {
+		expect(down({ props: { sources: [{ scale: 1, src: 'abc' }] } })).toEqual({
+			props: { src: 'abc' },
+		})
+	})
+})
+
 /* ---  PUT YOUR MIGRATIONS TESTS ABOVE HERE --- */
 
 // check that all migrator fns were called at least once