@tldraw/tlschema 5.5.0-next.4d6473b0bda1 → 5.5.0-next.56e885b43469

package/dist-cjs/index.js

@@ -222,7 +222,7 @@ var import_translations = require("./translations/translations");
 var import_b64Vecs = require("./misc/b64Vecs");
 (0, import_utils.registerTldrawLibraryVersion)(
   "@tldraw/tlschema",
-  "5.5.0-next.4d6473b0bda1",
+  "5.5.0-next.56e885b43469",
   "cjs"
 );
 //# sourceMappingURL=index.js.map

package/dist-esm/index.mjs

@@ -214,7 +214,7 @@ import {
 } from "./translations/translations.mjs";
 registerTldrawLibraryVersion(
   "@tldraw/tlschema",
-  "5.5.0-next.4d6473b0bda1",
+  "5.5.0-next.56e885b43469",
   "esm"
 );
 import { DIM_2D, DIM_3D, b64Vecs } from "./misc/b64Vecs.mjs";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tldraw/tlschema",
   "description": "tldraw infinite canvas SDK (schema).",
-  "version": "5.5.0-next.4d6473b0bda1",
+  "version": "5.5.0-next.56e885b43469",
   "author": {
     "name": "tldraw Inc.",
     "email": "hello@tldraw.com"
@@ -54,13 +54,13 @@
     "kleur": "^4.1.5",
     "lazyrepo": "0.0.0-alpha.27",
     "react": "^19.2.1",
-    "vitest": "^4.1.7"
+    "vitest": "^5.0.0"
   },
   "dependencies": {
-    "@tldraw/state": "5.5.0-next.4d6473b0bda1",
-    "@tldraw/store": "5.5.0-next.4d6473b0bda1",
-    "@tldraw/utils": "5.5.0-next.4d6473b0bda1",
-    "@tldraw/validate": "5.5.0-next.4d6473b0bda1"
+    "@tldraw/state": "5.5.0-next.56e885b43469",
+    "@tldraw/store": "5.5.0-next.56e885b43469",
+    "@tldraw/utils": "5.5.0-next.56e885b43469",
+    "@tldraw/validate": "5.5.0-next.56e885b43469"
   },
   "peerDependencies": {
     "react": "^18.2.0 || ^19.2.1",

package/src/__tests__/migrationTestUtils.ts

@@ -1,6 +1,5 @@
 import { devFreeze, Migration, MigrationId } from '@tldraw/store'
 import { mockUniqueId, structuredClone } from '@tldraw/utils'
-import { vi } from 'vitest'
 import { createTLSchema } from '../createTLSchema'
 
 let nextNanoId = 0
@@ -8,11 +7,24 @@ mockUniqueId(() => `nanoid_${++nextNanoId}`)
 
 export const testSchema = createTLSchema()
 
-// mock all migrator fns
+// Records which migrator fns ran, so migrations.test.ts can assert every one is covered by a
+// test. This deliberately isn't a vi.fn: vitest clears mock call history before each test, which
+// would wipe the record before the coverage check runs.
+const calledMigrators = new Set<string>()
+
+/** Whether any test in this run exercised the given migrator. */
+export function wasMigratorCalled(id: MigrationId, direction: 'up' | 'down') {
+	return calledMigrators.has(`${id}/${direction}`)
+}
+
 for (const migration of testSchema.sortedMigrations) {
-	;(migration as any).up = vi.fn(migration.up as any)
-	if (typeof migration.down === 'function') {
-		;(migration as any).down = vi.fn(migration.down as any)
+	for (const direction of ['up', 'down'] as const) {
+		const fn = migration[direction]
+		if (typeof fn !== 'function') continue
+		;(migration as any)[direction] = (...args: any[]) => {
+			calledMigrators.add(`${migration.id}/${direction}`)
+			return (fn as any)(...args)
+		}
 	}
 }
 

package/src/migrations.test.ts

@@ -1,5 +1,5 @@
 import { createRecordType } from '@tldraw/store'
-import { getTestMigration, testSchema } from './__tests__/migrationTestUtils'
+import { getTestMigration, testSchema, wasMigratorCalled } from './__tests__/migrationTestUtils'
 import { bookmarkAssetVersions } from './assets/TLBookmarkAsset'
 import { imageAssetVersions } from './assets/TLImageAsset'
 import { videoAssetVersions } from './assets/TLVideoAsset'
@@ -2800,9 +2800,9 @@ describe('DrawShapeSegment dim validation', () => {
 describe('all migrator fns were called at least once', () => {
 	for (const migration of testSchema.sortedMigrations) {
 		it(`migration ${migration.id}`, () => {
-			expect((migration as any).up).toHaveBeenCalled()
+			expect(wasMigratorCalled(migration.id, 'up')).toBe(true)
 			if (typeof migration.down === 'function') {
-				expect((migration as any).down).toHaveBeenCalled()
+				expect(wasMigratorCalled(migration.id, 'down')).toBe(true)
 			}
 		})
 	}