Skip to content

Commit d888924

Browse files
committed
refactor: strip audit prefixes from test names for long-term readability
Remove numbered (1., 2., ...), B-prefixed, TC-NNN:, and INDENT-XN: prefixes that were added during audit sessions. These identifiers lose context over time and clutter test output.
1 parent 5aa4ef8 commit d888924

6 files changed

Lines changed: 268 additions & 268 deletions

tests/unit/import-manager.blank-lines.test.ts

Lines changed: 37 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ suite('Blank Lines - Mode "one" (default)', () => {
8383
config.override('disableImportRemovalOnOrganize', true); // Don't remove unused imports for blank line tests
8484
});
8585

86-
test('TC-001: 0 blank lines after → 1 blank line after', async () => {
86+
test('0 blank lines after → 1 blank line after', async () => {
8787
const input = `import { A } from './a';\nexport class Test {}`;
8888
const expected = `import { A } from './a';\n\nexport class Test {}`;
8989

@@ -99,7 +99,7 @@ suite('Blank Lines - Mode "one" (default)', () => {
9999
}
100100
});
101101

102-
test('TC-002: 1 blank line after → 1 blank line after (preserved)', async () => {
102+
test('1 blank line after → 1 blank line after (preserved)', async () => {
103103
const input = `import { A } from './a';\n\nexport class Test {}`;
104104
const expected = `import { A } from './a';\n\nexport class Test {}`;
105105

@@ -115,7 +115,7 @@ suite('Blank Lines - Mode "one" (default)', () => {
115115
}
116116
});
117117

118-
test('TC-003: 2 blank lines after → 1 blank line after (normalized)', async () => {
118+
test('2 blank lines after → 1 blank line after (normalized)', async () => {
119119
const input = `import { A } from './a';\n\n\nexport class Test {}`;
120120
const expected = `import { A } from './a';\n\nexport class Test {}`;
121121

@@ -131,7 +131,7 @@ suite('Blank Lines - Mode "one" (default)', () => {
131131
}
132132
});
133133

134-
test('TC-004: 3 blank lines after → 1 blank line after (normalized)', async () => {
134+
test('3 blank lines after → 1 blank line after (normalized)', async () => {
135135
const input = `import { A } from './a';\n\n\n\nexport class Test {}`;
136136
const expected = `import { A } from './a';\n\nexport class Test {}`;
137137

@@ -147,7 +147,7 @@ suite('Blank Lines - Mode "one" (default)', () => {
147147
}
148148
});
149149

150-
test('TC-005: 5 blank lines after → 1 blank line after (normalized)', async () => {
150+
test('5 blank lines after → 1 blank line after (normalized)', async () => {
151151
const input = `import { A } from './a';\n\n\n\n\n\nexport class Test {}`;
152152
const expected = `import { A } from './a';\n\nexport class Test {}`;
153153

@@ -173,7 +173,7 @@ suite('Blank Lines - Mode "two"', () => {
173173
config.override('disableImportRemovalOnOrganize', true);
174174
});
175175

176-
test('TC-010: 0 blank lines after → 2 blank lines after', async () => {
176+
test('0 blank lines after → 2 blank lines after', async () => {
177177
const input = `import { A } from './a';\nexport class Test {}`;
178178
const expected = `import { A } from './a';\n\n\nexport class Test {}`;
179179

@@ -189,7 +189,7 @@ suite('Blank Lines - Mode "two"', () => {
189189
}
190190
});
191191

192-
test('TC-011: 1 blank line after → 2 blank lines after', async () => {
192+
test('1 blank line after → 2 blank lines after', async () => {
193193
const input = `import { A } from './a';\n\nexport class Test {}`;
194194
const expected = `import { A } from './a';\n\n\nexport class Test {}`;
195195

@@ -205,7 +205,7 @@ suite('Blank Lines - Mode "two"', () => {
205205
}
206206
});
207207

208-
test('TC-012: 2 blank lines after → 2 blank lines after (preserved)', async () => {
208+
test('2 blank lines after → 2 blank lines after (preserved)', async () => {
209209
const input = `import { A } from './a';\n\n\nexport class Test {}`;
210210
const expected = `import { A } from './a';\n\n\nexport class Test {}`;
211211

@@ -221,7 +221,7 @@ suite('Blank Lines - Mode "two"', () => {
221221
}
222222
});
223223

224-
test('TC-013: 3 blank lines after → 2 blank lines after (normalized)', async () => {
224+
test('3 blank lines after → 2 blank lines after (normalized)', async () => {
225225
const input = `import { A } from './a';\n\n\n\nexport class Test {}`;
226226
const expected = `import { A } from './a';\n\n\nexport class Test {}`;
227227

@@ -247,7 +247,7 @@ suite('Blank Lines - Mode "preserve"', () => {
247247
config.override('disableImportRemovalOnOrganize', true);
248248
});
249249

250-
test('TC-020: 0 blank lines after → 0 blank lines after', async () => {
250+
test('0 blank lines after → 0 blank lines after', async () => {
251251
const input = `import { A } from './a';\nexport class Test {}`;
252252
const expected = `import { A } from './a';\nexport class Test {}`;
253253

@@ -263,7 +263,7 @@ suite('Blank Lines - Mode "preserve"', () => {
263263
}
264264
});
265265

266-
test('TC-021: 1 blank line after → 1 blank line after', async () => {
266+
test('1 blank line after → 1 blank line after', async () => {
267267
const input = `import { A } from './a';\n\nexport class Test {}`;
268268
const expected = `import { A } from './a';\n\nexport class Test {}`;
269269

@@ -279,7 +279,7 @@ suite('Blank Lines - Mode "preserve"', () => {
279279
}
280280
});
281281

282-
test('TC-022: 2 blank lines after → 2 blank lines after', async () => {
282+
test('2 blank lines after → 2 blank lines after', async () => {
283283
const input = `import { A } from './a';\n\n\nexport class Test {}`;
284284
const expected = `import { A } from './a';\n\n\nexport class Test {}`;
285285

@@ -295,7 +295,7 @@ suite('Blank Lines - Mode "preserve"', () => {
295295
}
296296
});
297297

298-
test('TC-023: 3 blank lines after → 3 blank lines after', async () => {
298+
test('3 blank lines after → 3 blank lines after', async () => {
299299
const input = `import { A } from './a';\n\n\n\nexport class Test {}`;
300300
const expected = `import { A } from './a';\n\n\n\nexport class Test {}`;
301301

@@ -311,7 +311,7 @@ suite('Blank Lines - Mode "preserve"', () => {
311311
}
312312
});
313313

314-
test('TC-024: 5 blank lines after → 5 blank lines after', async () => {
314+
test('5 blank lines after → 5 blank lines after', async () => {
315315
const input = `import { A } from './a';\n\n\n\n\n\nexport class Test {}`;
316316
const expected = `import { A } from './a';\n\n\n\n\n\nexport class Test {}`;
317317

@@ -338,7 +338,7 @@ suite('Blank Lines - Header Detection', () => {
338338
config.override('disableImportRemovalOnOrganize', true);
339339
});
340340

341-
test('TC-100: Leading blank lines removed (no header)', async () => {
341+
test('Leading blank lines removed (no header)', async () => {
342342
const input = `\n\nimport { A } from './a';\nexport class Test {}`;
343343
const expected = `import { A } from './a';\n\nexport class Test {}`;
344344

@@ -354,7 +354,7 @@ suite('Blank Lines - Header Detection', () => {
354354
}
355355
});
356356

357-
test('TC-101: Leading blank lines before comment removed', async () => {
357+
test('Leading blank lines before comment removed', async () => {
358358
const input = `\n\n// Comment\nimport { A } from './a';\nexport class Test {}`;
359359
const expected = `// Comment\nimport { A } from './a';\n\nexport class Test {}`;
360360

@@ -370,7 +370,7 @@ suite('Blank Lines - Header Detection', () => {
370370
}
371371
});
372372

373-
test('TC-110: Header with no blank lines preserved', async () => {
373+
test('Header with no blank lines preserved', async () => {
374374
const input = `// Comment\nimport { A } from './a';\nexport class Test {}`;
375375
const expected = `// Comment\nimport { A } from './a';\n\nexport class Test {}`;
376376

@@ -386,7 +386,7 @@ suite('Blank Lines - Header Detection', () => {
386386
}
387387
});
388388

389-
test('TC-111: Header with 1 blank line preserved', async () => {
389+
test('Header with 1 blank line preserved', async () => {
390390
const input = `// Comment\n\nimport { A } from './a';\nexport class Test {}`;
391391
const expected = `// Comment\n\nimport { A } from './a';\n\nexport class Test {}`;
392392

@@ -402,7 +402,7 @@ suite('Blank Lines - Header Detection', () => {
402402
}
403403
});
404404

405-
test('TC-112: Header with 2 blank lines preserved', async () => {
405+
test('Header with 2 blank lines preserved', async () => {
406406
const input = `// Comment\n\n\nimport { A } from './a';\nexport class Test {}`;
407407
const expected = `// Comment\n\n\nimport { A } from './a';\n\nexport class Test {}`;
408408

@@ -418,7 +418,7 @@ suite('Blank Lines - Header Detection', () => {
418418
}
419419
});
420420

421-
test('TC-113: Multiple header lines with blank preserved', async () => {
421+
test('Multiple header lines with blank preserved', async () => {
422422
const input = `// Copyright 2025\n// Info\n\nimport { A } from './a';\nexport class Test {}`;
423423
const expected = `// Copyright 2025\n// Info\n\nimport { A } from './a';\n\nexport class Test {}`;
424424

@@ -434,7 +434,7 @@ suite('Blank Lines - Header Detection', () => {
434434
}
435435
});
436436

437-
test('TC-120: Shebang with no blank preserved', async () => {
437+
test('Shebang with no blank preserved', async () => {
438438
const input = `#!/usr/bin/env node\nimport { A } from './a';\nexport class Test {}`;
439439
const expected = `#!/usr/bin/env node\nimport { A } from './a';\n\nexport class Test {}`;
440440

@@ -450,7 +450,7 @@ suite('Blank Lines - Header Detection', () => {
450450
}
451451
});
452452

453-
test('TC-121: Shebang with blank preserved', async () => {
453+
test('Shebang with blank preserved', async () => {
454454
const input = `#!/usr/bin/env node\n\nimport { A } from './a';\nexport class Test {}`;
455455
const expected = `#!/usr/bin/env node\n\nimport { A } from './a';\n\nexport class Test {}`;
456456

@@ -466,7 +466,7 @@ suite('Blank Lines - Header Detection', () => {
466466
}
467467
});
468468

469-
test('TC-130: use strict with no blank preserved', async () => {
469+
test('use strict with no blank preserved', async () => {
470470
const input = `'use strict';\nimport { A } from './a';\nexport class Test {}`;
471471
const expected = `'use strict';\nimport { A } from './a';\n\nexport class Test {}`;
472472

@@ -482,7 +482,7 @@ suite('Blank Lines - Header Detection', () => {
482482
}
483483
});
484484

485-
test('TC-131: use strict with blank preserved', async () => {
485+
test('use strict with blank preserved', async () => {
486486
const input = `'use strict';\n\nimport { A } from './a';\nexport class Test {}`;
487487
const expected = `'use strict';\n\nimport { A } from './a';\n\nexport class Test {}`;
488488

@@ -498,7 +498,7 @@ suite('Blank Lines - Header Detection', () => {
498498
}
499499
});
500500

501-
test('TC-132: double-quoted use strict preserved', async () => {
501+
test('double-quoted use strict preserved', async () => {
502502
const input = `"use strict";\nimport { A } from './a';\nexport class Test {}`;
503503
const expected = `"use strict";\nimport { A } from './a';\n\nexport class Test {}`;
504504

@@ -524,7 +524,7 @@ suite('Blank Lines - Import Group Separation', () => {
524524
config.override('disableImportRemovalOnOrganize', true);
525525
});
526526

527-
test('TC-200: Modules only - no group separation', async () => {
527+
test('Modules only - no group separation', async () => {
528528
const input = `import { A } from '@angular/core';\nimport { B } from 'rxjs';\nexport class Test {}`;
529529
const expected = `import { A } from '@angular/core';\nimport { B } from 'rxjs';\n\nexport class Test {}`;
530530

@@ -540,7 +540,7 @@ suite('Blank Lines - Import Group Separation', () => {
540540
}
541541
});
542542

543-
test('TC-201: Workspace only - no group separation', async () => {
543+
test('Workspace only - no group separation', async () => {
544544
const input = `import { A } from './a';\nimport { B } from './b';\nexport class Test {}`;
545545
const expected = `import { A } from './a';\nimport { B } from './b';\n\nexport class Test {}`;
546546

@@ -556,7 +556,7 @@ suite('Blank Lines - Import Group Separation', () => {
556556
}
557557
});
558558

559-
test('TC-202: Modules + Workspace - 1 blank between groups', async () => {
559+
test('Modules + Workspace - 1 blank between groups', async () => {
560560
const input = `import { A } from '@angular/core';\nimport { B } from './b';\nexport class Test {}`;
561561
const expected = `import { A } from '@angular/core';\n\nimport { B } from './b';\n\nexport class Test {}`;
562562

@@ -572,7 +572,7 @@ suite('Blank Lines - Import Group Separation', () => {
572572
}
573573
});
574574

575-
test('TC-203: Plains + Modules + Workspace - 2 blanks total (1 between each)', async () => {
575+
test('Plains + Modules + Workspace - 2 blanks total (1 between each)', async () => {
576576
const input = `import './polyfills';\nimport { A } from '@angular/core';\nimport { B } from './b';\nexport class Test {}`;
577577
const expected = `import './polyfills';\n\nimport { A } from '@angular/core';\n\nimport { B } from './b';\n\nexport class Test {}`;
578578

@@ -597,7 +597,7 @@ suite('Blank Lines - Combined Scenarios', () => {
597597
config.override('disableImportRemovalOnOrganize', true);
598598
});
599599

600-
test('TC-300: Mode "one" + Header with blanks', async () => {
600+
test('Mode "one" + Header with blanks', async () => {
601601
config.override('blankLinesAfterImports', 'one');
602602

603603
const input = `// Header\n\nimport { A } from './a';\n\n\nexport class Test {}`;
@@ -615,7 +615,7 @@ suite('Blank Lines - Combined Scenarios', () => {
615615
}
616616
});
617617

618-
test('TC-310: Mode "preserve" + Header', async () => {
618+
test('Mode "preserve" + Header', async () => {
619619
config.override('blankLinesAfterImports', 'preserve');
620620

621621
const input = `// Header\n\nimport { A } from './a';\n\n\nexport class Test {}`;
@@ -644,7 +644,7 @@ suite('Blank Lines - Edge Cases', () => {
644644
config.override('disableImportRemovalOnOrganize', true);
645645
});
646646

647-
test('TC-400: File with only imports (no code after)', async () => {
647+
test('File with only imports (no code after)', async () => {
648648
const input = `import { A } from './a';`;
649649

650650
const doc = await createTempDocument(input);
@@ -664,7 +664,7 @@ suite('Blank Lines - Edge Cases', () => {
664664
}
665665
});
666666

667-
test('TC-401: File with no imports', async () => {
667+
test('File with no imports', async () => {
668668
const input = `export class Test {}`;
669669

670670
const doc = await createTempDocument(input);
@@ -679,7 +679,7 @@ suite('Blank Lines - Edge Cases', () => {
679679
}
680680
});
681681

682-
test('TC-402: Empty file', async () => {
682+
test('Empty file', async () => {
683683
const input = ``;
684684

685685
const doc = await createTempDocument(input);
@@ -693,7 +693,7 @@ suite('Blank Lines - Edge Cases', () => {
693693
}
694694
});
695695

696-
test('TC-403: Whitespace-only file becomes empty', async () => {
696+
test('Whitespace-only file becomes empty', async () => {
697697
const input = `\n\n\n`;
698698

699699
const doc = await createTempDocument(input);
@@ -708,7 +708,7 @@ suite('Blank Lines - Edge Cases', () => {
708708
}
709709
});
710710

711-
test('TC-404: CRLF line endings preserved', async () => {
711+
test('CRLF line endings preserved', async () => {
712712
const input = `import { A } from './a';\r\nexport class Test {}`;
713713
const expected = `import { A } from './a';\r\n\r\nexport class Test {}`;
714714

@@ -724,7 +724,7 @@ suite('Blank Lines - Edge Cases', () => {
724724
}
725725
});
726726

727-
test('TC-405: Mixed import types handled correctly', async () => {
727+
test('Mixed import types handled correctly', async () => {
728728
const input = `import './polyfills';\nimport * as React from 'react';\nimport { A } from './a';\nexport class Test {}`;
729729
const expected = `import './polyfills';\n\nimport * as React from 'react';\n\nimport { A } from './a';\n\nexport class Test {}`;
730730

0 commit comments

Comments
 (0)