Skip to content

Commit d45f150

Browse files
committed
ci(changeset): document expanded test-suite for the refactor PR
1 parent 83536eb commit d45f150

2 files changed

Lines changed: 9 additions & 3 deletions

File tree

.changeset/test-suite.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
---
2+
'@platforma-open/milaboratories.sequence-properties.model': patch
3+
'@platforma-open/milaboratories.sequence-properties.test': patch
4+
---
5+
6+
Lock label-resolution behaviour with model-package vitest. `model/src/label.ts` centralises `resolveSubtitle` and `resolveTraceLabel`; `model/src/label.test.ts` and `model/src/dataModel.test.ts` cover the resolution chain and the `Ver_2026_05_18` migration backfill respectively. Stages SC IG fastq fixtures so the two-instance dedup integration test runs in CI instead of self-skipping.

model/src/dataModel.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,9 @@ export const blockDataModel = new DataModelBuilder()
3636
// Ver_2026_05_05 (DataModelBuilder skips matching-version migrations).
3737
.migrate<BlockDataV2>("Ver_2026_05_05", migrateV1toV2)
3838
// Backfills label fields onto V2-tagged projects. `?? ""` preserves any
39-
// interim-deployed value; missing fields default to "". The workflow's
40-
// `traceLabel := args.customBlockLabel || args.defaultBlockLabel`
41-
// requires both args to be strings, never undefined.
39+
// interim-deployed value; missing fields default to "". The args
40+
// projection (resolveTraceLabel in label.ts) requires both fields to be
41+
// strings, never undefined.
4242
.migrate<BlockData>("Ver_2026_05_18", migrateV2toV2_1)
4343
.init(() => ({
4444
tableState: createPlDataTableStateV2(),

0 commit comments

Comments
 (0)