Fix plan inconsistencies found in pre-flight review
This commit is contained in:
@@ -238,7 +238,7 @@ test("cfgFromSettings defaults baseUrl and omits empty apiKey", () => {
|
|||||||
|
|
||||||
test("shorten keeps short names and truncates long ones", () => {
|
test("shorten keeps short names and truncates long ones", () => {
|
||||||
assert.equal(shorten("Qwen3.8-27B-NVFP4"), "Qwen3.8-27B-NVFP4");
|
assert.equal(shorten("Qwen3.8-27B-NVFP4"), "Qwen3.8-27B-NVFP4");
|
||||||
assert.equal(shorten("DeepSeek-V4-Flash-0731"), "DeepSe…0731");
|
assert.equal(shorten("DeepSeek-V4-Flash-0731"), "DeepSeek-V…0731");
|
||||||
});
|
});
|
||||||
|
|
||||||
test("escapeXml escapes XML special characters", () => {
|
test("escapeXml escapes XML special characters", () => {
|
||||||
@@ -612,8 +612,7 @@ test("poller stores per-GPU history and computes aggregates", async () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
test("poller ring buffer caps at 60 samples", async () => {
|
test("poller ring buffer caps at 60 samples", async () => {
|
||||||
const low = FIXTURE.replace(/} 100\n/g, "} 100\n").replace(/} 51\n/g, "} 51\n");
|
const poller = new MetricsPoller(CFG, 5000, stubFetch([FIXTURE]));
|
||||||
const poller = new MetricsPoller(CFG, 5000, stubFetch([low]));
|
|
||||||
for (let i = 0; i < 70; i++) await poller.tick();
|
for (let i = 0; i < 70; i++) await poller.tick();
|
||||||
assert.equal(poller.getHistory("0", "fan").length, 60);
|
assert.equal(poller.getHistory("0", "fan").length, 60);
|
||||||
});
|
});
|
||||||
@@ -758,12 +757,8 @@ export class MetricsPoller {
|
|||||||
return this.lastError !== undefined;
|
return this.lastError !== undefined;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
export { gpuInfos };
|
|
||||||
```
|
```
|
||||||
|
|
||||||
Note: `gpuInfos` is re-exported for `datasources.ts` convenience; if unused there, remove the re-export in Task 9.
|
|
||||||
|
|
||||||
- [ ] **Step 4: Run test to verify it passes**
|
- [ ] **Step 4: Run test to verify it passes**
|
||||||
|
|
||||||
Run: `npm test`
|
Run: `npm test`
|
||||||
|
|||||||
Reference in New Issue
Block a user