Skip to content

Commit 22feffe

Browse files
committed
test(react): the gate's carried-defect row is not a CONTROL; its guard is the core builder's
Ablating the builder's binding condition reddens it, so by this suite's naming convention it is not a control. Moved into its own block. Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BA3nKVUwKQJf8DBxrSVtNC
1 parent f156819 commit 22feffe

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

‎packages/react/src/element-data-source/__tests__/ElementDataSourceGate.test.tsx‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -946,13 +946,18 @@ describe('ElementDataSourceGate — a refused binding `limit` is not authored (o
946946
});
947947
});
948948

949-
describe('CONTROLS — rows the ruling does not move', () => {
950-
it('CONTROL — binding 3 + refused view cap ⇒ 3, and NO view warning (the carried defect, silent here already)', async () => {
949+
describe('the carried defect’s condition, read from the builder on this caller too', () => {
950+
// Not a CONTROL: this gate was already silent here, but the condition that
951+
// keeps it silent now lives in the core builder, shared with
952+
// `ViewDataProvider`, so ablating the builder's condition reddens this row.
953+
it('binding 3 + refused view cap ⇒ 3, and NO view warning', async () => {
951954
const { result, spy } = await row({ type: 'list-view', dataSource: bindingOf(3) }, VIEW_REFUSED);
952955
expect(result.current.schema.pagination).toEqual({ pageSize: 3 });
953956
expect(spy).not.toHaveBeenCalled();
954957
});
958+
});
955959

960+
describe('CONTROLS — rows the ruling does not move', () => {
956961
it('CONTROL — binding 3 + usable view cap ⇒ 3, and nothing said', async () => {
957962
const { result, spy } = await row({ type: 'list-view', dataSource: bindingOf(3) }, VIEW_USABLE);
958963
expect(result.current.schema.pagination).toEqual({ pageSize: 3 });

0 commit comments

Comments
 (0)