chore(3155): address CodeRabbit changeset ref and test diagnostics

This commit is contained in:
Tom Boucher
2026-05-05 18:53:35 -04:00
parent 53879d8c93
commit b694d31239
2 changed files with 16 additions and 2 deletions

View File

@@ -1,5 +1,5 @@
---
type: Fixed
pr: 3150
pr: 3155
---
**`stats.json` decimal phase ordering now has explicit regression coverage** — added a fixture ensuring `06.7/06.8/06.9` remain present when `06.10` exists, preventing dropped-phase regressions in mixed decimal phase ranges.

View File

@@ -26,7 +26,21 @@ describe('bug #3150: stats.json includes contiguous decimal phases when .10 exis
const result = runGsdTools('stats json', tmpDir);
assert.ok(result.success, `Command failed: ${result.error}`);
const output = JSON.parse(result.output);
let output;
assert.doesNotThrow(
() => {
output = JSON.parse(result.output);
},
`Command output must be valid JSON. Raw output prefix: ${result.output.slice(0, 200)}`
);
assert.ok(
output && typeof output === 'object' && !Array.isArray(output),
`Expected object output, got: ${typeof output}`
);
assert.ok(
Array.isArray(output.phases),
`Expected output.phases array. Raw output prefix: ${result.output.slice(0, 200)}`
);
const phaseNumbers = output.phases.map((p) => p.number);
assert.deepStrictEqual(