From b59ded24b35f79d1f183bf4e0f1bce8908778a7e Mon Sep 17 00:00:00 2001 From: Brylie Christopher Oxley Date: Sat, 12 Sep 2026 21:35:36 +0300 Subject: [PATCH] test: assert single editable control on Board groupBy/swimlaneBy primary field collision (closes #272) --- src/lib/components/BoardCollectionView.svelte.test.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/lib/components/BoardCollectionView.svelte.test.ts b/src/lib/components/BoardCollectionView.svelte.test.ts index ff1fe70..d8f5e58 100644 --- a/src/lib/components/BoardCollectionView.svelte.test.ts +++ b/src/lib/components/BoardCollectionView.svelte.test.ts @@ -338,7 +338,7 @@ describe('BoardCollectionView', () => { expect(await screen.findByDisplayValue('Still primary')).toBeInTheDocument(); }); - it('falls back to a plain, non-editable title label when the primary field is also the grouping property (issue #104)', async () => { + it('falls back to a plain, non-editable title label when the primary field is also the grouping property (issue #104/#272)', async () => { createCollection(ydoc, { id: 'col-1', title: 'Board', @@ -366,11 +366,12 @@ describe('BoardCollectionView', () => { // plain text rather than its own editable control... expect(within(card).getByText('To do', { selector: 'span' })).toBeInTheDocument(); // ...so the "Move to column" select is the only editable control for - // this value anywhere on the card. + // this value anywhere on the card (titleEditableViaCell, issue #104/#272). expect(within(card).getAllByRole('combobox')).toHaveLength(1); + expect(within(card).getAllByDisplayValue('To do')).toHaveLength(1); }); - it('falls back to a plain, non-editable title label when the primary field is also the swimlane property (issue #104)', async () => { + it('falls back to a plain, non-editable title label when the primary field is also the swimlane property (issue #104/#272)', async () => { createCollection(ydoc, { id: 'col-1', title: 'Board', @@ -411,8 +412,9 @@ describe('BoardCollectionView', () => { expect(within(card).getByText('High', { selector: 'span' })).toBeInTheDocument(); // ...so the "Move to column"/"Move to swimlane" selects are the only // editable controls for their respective values — no third, redundant - // editable control for the primary field itself. + // editable control for the primary field itself (titleEditableViaCell, issue #104/#272). expect(within(card).getAllByRole('combobox')).toHaveLength(2); + expect(within(card).getAllByDisplayValue('High')).toHaveLength(1); }); it('adds a new option to a non-grouping select field from a card without touching the grouping property', async () => {