Skip to content

Commit ed2bc64

Browse files
committed
test(spec): the first system/ file group's test titles state each cited decision in words instead of a tracker number (stage 26)
Stage 26 of the spec lane's share of the runtime-string burn-down: the test strings shipped under packages/spec/src, the first name-ordered system/ group (auth-config.test.ts through metadata-form-declared-rows.pin.test.ts, with constants/system-names.test.ts in its path position). 91 literals in 18 files (83 test titles and 8 declared expect messages) lose 97 tracker ids: 23 literals now state the cited decision in words, 11 get their subject back in words, and 57 drop a number the title already explains. No needle sits in this group. Text only: one line per literal, no assertion, identifier, test count, code comment or file name changes. Claude-Session: https://claude.ai/code/session_01T9u38rswFp5Rw8DswRUReJ Co-authored-by: Claude <noreply@anthropic.com>
1 parent dcf3eb4 commit ed2bc64

18 files changed

Lines changed: 91 additions & 91 deletions

‎packages/spec/src/system/auth-config.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -420,7 +420,7 @@ describe('AuthConfigSchema – new passthrough fields', () => {
420420
});
421421
});
422422

423-
describe('AudienceConfigSchema (#11739)', () => {
423+
describe('AudienceConfigSchema — one closed audience posture, invite_only when undeclared', () => {
424424
it('defaults an undeclared posture to invite_only (the ruled safe default)', () => {
425425
const parsed = AudienceConfigSchema.parse({});
426426
expect(parsed.posture).toBe('invite_only');
@@ -517,7 +517,7 @@ describe('AudienceConfigSchema (#11739)', () => {
517517
// `updateAge: this.config.session?.updateAge || 60 * 60 * 24` straight into
518518
// better-auth's `session.updateAge`, whose unit is seconds. Only `expiresIn`
519519
// carried the marker; this closes the pair.
520-
describe('AuthConfig.session.updateAge declares seconds by mirror (#18124)', () => {
520+
describe('AuthConfig.session.updateAge declares seconds by mirror', () => {
521521
it('keeps the bare name, the default and the value it always accepted', () => {
522522
expect(AuthConfigSchema.parse({}).session).toBeUndefined();
523523
const parsed = AuthConfigSchema.parse({ session: { updateAge: 3 * 86_400 } });

‎packages/spec/src/system/book.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ describe('resolveDocAudiences — union over claiming books (§6.7)', () => {
363363
});
364364

365365
// ── Inline translation maps retired in 17.0.0 (#4667, ADR-0049) ─────────────
366-
describe('retired book translation maps (#4667)', () => {
366+
describe('retired book translation maps', () => {
367367
it('rejects a book-level `translations` map and names the live neighbour', () => {
368368
// The prescription has to mention `doc.translations`: that key is live on
369369
// every doc render path and is what the author actually wanted. Without it
@@ -410,7 +410,7 @@ describe('retired book translation maps (#4667)', () => {
410410
// book-tree response contract (#12038)
411411
// ==========================================
412412

413-
describe('ResolvedBookSchema is the book-tree response contract (#12038)', () => {
413+
describe('ResolvedBookSchema is the book-tree response contract', () => {
414414
// The conformance suite for the `GET /meta/book/:name/tree` ledger rows
415415
// (#3877's no-row-without-conformance rule). Stronger than the handwritten
416416
// captures its meta.* siblings use: `resolveBookTree()` is pure and lives in

‎packages/spec/src/system/cache.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ describe('DistributedCacheConfigSchema', () => {
333333
// Asserted on the issue CODE and the prescription text, never on "it threw":
334334
// a bare `toThrow()` would stay green against a schema that rejected for any
335335
// other reason, which is the failure this pin exists to catch.
336-
describe('cache duration keys carry their unit (#15679)', () => {
336+
describe('cache duration keys carry their unit in the key name', () => {
337337
it('REFUSES the retired `CacheTier.ttl` with the rename in the message', () => {
338338
const result = CacheTierSchema.safeParse({ name: 'l1', type: 'memory', ttl: 600 });
339339
expect(result.success).toBe(false);

‎packages/spec/src/system/collaboration.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1002,7 +1002,7 @@ describe('CollaborationSessionSchema', () => {
10021002
// collision that got this population ruled: milliseconds here, seconds on the
10031003
// tenant surface. Both old spellings are `retiredKey()` tombstones; asserted on
10041004
// the issue CODE and the prescription, never on a bare `toThrow()`.
1005-
describe('collaboration session durations carry their unit (#15679)', () => {
1005+
describe('collaboration session durations carry their unit in the key name', () => {
10061006
it('REFUSES the retired `idleTimeout` with the rename in the message', () => {
10071007
const result = CollaborationSessionConfigSchema.safeParse({ mode: 'ot', idleTimeout: 600000 });
10081008
expect(result.success).toBe(false);

‎packages/spec/src/system/compliance-families-retirement.test.ts‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -148,15 +148,15 @@ const MUST_SURVIVE_SYSTEM = ['DataClassificationSchema', 'ComplianceFrameworkSch
148148
const SPEC_ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '../..');
149149
const SRC_ROOT = path.join(SPEC_ROOT, 'src');
150150

151-
describe('[#15513] system/ compliance families retirement — the public surface', () => {
151+
describe('system/ compliance families retirement — the public surface', () => {
152152
it('every retired name has ZERO holders on any public entry; the survivors still stand', () => {
153153
for (const needed of ['.', './system', './kernel']) {
154154
expect(EXPORT_ENTRY_POINTS, `exports map must include ${needed}`).toContain(needed);
155155
}
156156
expect(exportNamesOf('./system').length, './system must export a non-trivial surface').toBeGreaterThan(100);
157157

158158
for (const name of RETIRED_NAMES) {
159-
expect(holdersOf(name), `${name} must have zero holders after #15513`).toEqual([]);
159+
expect(holdersOf(name), `${name} must have zero holders after the families' retirement`).toEqual([]);
160160
}
161161
const systemNames = exportNamesOf('./system');
162162
for (const name of MUST_SURVIVE_SYSTEM) {
@@ -207,7 +207,7 @@ describe('[#15513] system/ compliance families retirement — the public surface
207207
}
208208
};
209209
walk(SRC_ROOT);
210-
expect(importers, 'a resurrected import means the retirement is being undone — re-read #15513').toEqual([]);
210+
expect(importers, 'a resurrected import means the retirement is being undone — the three families were retired whole, none roadmapped').toEqual([]);
211211
});
212212

213213
it('the generated shards no longer list any of the nineteen defs or forty-five names', () => {
@@ -238,7 +238,7 @@ describe('[#15513] system/ compliance families retirement — the public surface
238238
});
239239
});
240240

241-
describe('[#15513] ADR-0087 registration', () => {
241+
describe('compliance families retirement — ADR-0087 registration', () => {
242242
it('declares all nineteen defs under major 18, with the three D3 semantic entries wired and no D2 conversion', () => {
243243
for (const def of RETIRED_DEFS) {
244244
expect(RETIRED_DEFS_BY_MAJOR[18], `${def} must be declared`).toContain(def);
@@ -265,7 +265,7 @@ describe('[#15513] ADR-0087 registration', () => {
265265
expect(step!.rationale).toMatch(/retires those three compliance-shaped families WHOLE/);
266266
});
267267

268-
it("keeps PR #15514's fourteen deadline-key registrations and three D3 entries as history", () => {
268+
it("keeps the earlier deadline-key retirement's fourteen registrations and three D3 entries as history", () => {
269269
for (const key of HISTORY_KEYS) {
270270
expect(RETIRED_KEYS_BY_MAJOR[18], `${key} is history — keep it`).toContain(key);
271271
}
@@ -312,7 +312,7 @@ describe('[#15513] ADR-0087 registration', () => {
312312
// the very declaration the self-test refuses.)
313313
// The residue this leg covers is everything `tsc` does not compile: JSON,
314314
// YAML, MD, MDX, and (under `packages/`) untyped `.js` / `.mjs` / `.cjs`.
315-
describe('[#15513] tree-scoped absence: nothing inside the declared radius references a retired name', () => {
315+
describe('tree-scoped absence: nothing inside the declared radius references a retired name', () => {
316316
const REPO_ROOT = path.resolve(SPEC_ROOT, '../..');
317317
const THIS_FILE = path.relative(REPO_ROOT, fileURLToPath(import.meta.url)).split(path.sep).join('/');
318318

@@ -491,6 +491,6 @@ describe('[#15513] tree-scoped absence: nothing inside the declared radius refer
491491
for (const root of WALK_ROOTS) walk(path.join(REPO_ROOT, root));
492492
// Anti-vacuity: the walk really covered the tree.
493493
expect(visited).toBeGreaterThan(1000);
494-
expect(offenders, 'a reference to a retired name means the retirement is being undone — re-read #15513').toEqual([]);
494+
expect(offenders, 'a reference to a retired name means the retirement is being undone — the three families were retired whole, none roadmapped').toEqual([]);
495495
});
496496
});

‎packages/spec/src/system/constants/system-names.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ describe('SystemFieldName', () => {
103103
// authorable, pinned in `../../data/object.test.ts`. The column reaching a row
104104
// and the tier being declarable are now BOTH true — a change from the split
105105
// state this comment used to record, not a restatement of it.
106-
it('registers the ADR-0117 business-unit ownership stamp, distinct from the user attribute (#4611)', () => {
106+
it('registers the ADR-0117 business-unit ownership stamp, distinct from the user attribute', () => {
107107
expect(SystemFieldName.OWNING_BUSINESS_UNIT_ID).toBe('owning_business_unit_id');
108108
// Guard the naming discipline ADR-0117 D10 spells out: the record stamp must
109109
// NOT be confused with `sys_user.primary_business_unit_id`, which is a USER

‎packages/spec/src/system/core-service-provider.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('CORE_SERVICE_PROVIDER', () => {
3131
}
3232
});
3333

34-
it('carries no entry for retired or never-real slots (#4451)', () => {
34+
it('carries no entry for retired or never-real slots', () => {
3535
for (const slot of ['workflow', 'graphql']) {
3636
expect(
3737
Object.prototype.hasOwnProperty.call(CORE_SERVICE_PROVIDER, slot),

‎packages/spec/src/system/data-migration-flag-column-move.pin.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ function baseRow(over: Partial<DataMigrationFlag> = {}): DataMigrationFlag {
6969
};
7070
}
7171

72-
describe('columns_moved_at — absence is the contract (#16185, ruling on #15989 Q1)', () => {
72+
describe('columns_moved_at — absence is the contract (ruled: one nullable datetime on the flag row)', () => {
7373
it('parses green when the member is absent — every row alive today', () => {
7474
const result = DataMigrationFlagSchema.safeParse(baseRow());
7575
expect(result.success, JSON.stringify(result.error?.issues)).toBe(true);
@@ -123,7 +123,7 @@ describe('columns_moved_at — absence is the contract (#16185, ruling on #15989
123123
});
124124
});
125125

126-
describe('the ONE arbiter is unchanged by this card (#16185 constraint 2)', () => {
126+
describe('the ONE arbiter is unchanged by the new member — read beside it, never inside it', () => {
127127
/** `(verified_at, blocking)` and the verdict recorded BEFORE the new member existed. */
128128
const CASES: Array<{ label: string; over: Partial<DataMigrationFlag>; verified: boolean }> = [
129129
{ label: 'verified, no blocking', over: { verified_at: MOVED_AT, blocking: 0 }, verified: true },

‎packages/spec/src/system/disaster-recovery.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ import { z } from 'zod';
232232
// neighbouring `dns.ttl` is a DECLARED `externalVocabulary` mirror and must
233233
// survive this rename untouched, and no gate can catch its loss (the marker
234234
// exempts the key either way), so the pin is the only guard.
235-
describe('FailoverConfig.healthCheckInterval carries its unit (#15679)', () => {
235+
describe('FailoverConfig.healthCheckInterval carries its unit in the key name', () => {
236236
const regions = [
237237
{ name: 'us-east-1', role: 'primary' as const },
238238
{ name: 'eu-west-1', role: 'secondary' as const },

‎packages/spec/src/system/email-config.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ describe('EmailProviderSchema', () => {
5353
expect(accepted).toEqual([...DELIVERABLE_PROVIDERS]);
5454
});
5555

56-
it('accepts smtp — shipped by plugin-email since #5087 (ADR-0012)', () => {
56+
it('accepts smtp — shipped by the SMTP transport in plugin-email (ADR-0012)', () => {
5757
// The single assertion #5104 is about: green after the enum gained
5858
// 'smtp', red on any revert of it.
5959
expect(EmailProviderSchema.safeParse('smtp').success).toBe(true);
@@ -123,7 +123,7 @@ describe('EmailServiceConfigSchema', () => {
123123
// assertion below reads `parsed.data`, so each one is green after the
124124
// declaration and red on a revert of it.
125125
// ─────────────────────────────────────────────────────────────────────────────
126-
describe('EmailServiceConfigSchema — keys the CLI reads (#5307)', () => {
126+
describe('EmailServiceConfigSchema — keys the CLI reads', () => {
127127
it('carries queueDelivery through the parse, not into the bin', () => {
128128
for (const queueDelivery of [true, false]) {
129129
const parsed = EmailServiceConfigSchema.safeParse({ queueDelivery });

0 commit comments

Comments
 (0)