Skip to content

Commit 49fb6cf

Browse files
committed
test(lint): the byte-for-byte hint control covers the rows describe() answers
A live row that opts in with authorWarn makes describe() throw before and after the ruling, so it has no hint to keep; the COVERAGE pin holds that it stays loud. main now ships one such row (mapping.connectorSource), which the control would otherwise trip over. Claude-Session: https://claude.ai/code/session_01JAhu8u8QfBvRjVZDox7CP9 Co-authored-by: Claude <noreply@anthropic.com>
1 parent b0705c1 commit 49fb6cf

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

‎packages/lint/src/lint-liveness-properties.test.ts‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1775,7 +1775,12 @@ describe('the hint a verdict-triggered row shows an author (#16094)', () => {
17751775
});
17761776

17771777
it('CONTROL, REAL LEDGER: every row that warned before the ruling keeps its hint byte for byte', () => {
1778-
const before = shippedRows().filter((r) => r.entry.authorWarn === true || r.entry.status === 'experimental');
1778+
// A `live` row that opts in is outside this population: `describe()` throws
1779+
// on it before and after the ruling (the COVERAGE pin above holds that it
1780+
// stays loud), so it has no hint to keep.
1781+
const before = shippedRows().filter(
1782+
(r) => (r.entry.authorWarn === true || r.entry.status === 'experimental') && r.entry.status !== 'live',
1783+
);
17791784
// Anti-vacuity: an opted-in row and an experimental row both fall back to
17801785
// their note today, so the control would see a change in either.
17811786
expect(before.some((r) => r.entry.authorWarn === true && r.entry.authorHint === undefined && typeof r.entry.note === 'string')).toBe(true);

0 commit comments

Comments
 (0)