Skip to content

Commit afede06

Browse files
author
Fergus Bisset
committed
chore: comment out verbose test console output for cleaner CI logs
1 parent 1781ed6 commit afede06

File tree

3 files changed

+28
-28
lines changed

3 files changed

+28
-28
lines changed

src/components/DataVisualisation/charts/SPC/SPCChart/logic_v2/__tests__/centralised.datasets.parity.test.ts

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -197,25 +197,25 @@ describe("logic_v2: centralised datasets parity (healthcare + grouped)", () => {
197197

198198
// Emit a concise report for visibility in CI logs
199199
// Using console.info to avoid failing on formatting issues
200-
console.info(
201-
"SPC v2 grouped metrics report (count=",
202-
perMetric.length,
203-
")"
204-
);
205-
console.table?.(
206-
perMetric.map((m) => ({
207-
Metric: m.metric,
208-
Dir: m.dir,
209-
Len: m.length,
210-
Eligible: m.eligible,
211-
LastIcon: m.lastIcon,
212-
Single: m.anySinglePoint,
213-
TwoSig: m.anyTwoSigma,
214-
Shift: m.anyShift,
215-
Trend: m.anyTrend,
216-
}))
217-
);
218-
console.info("Icon totals:", totals);
200+
// console.info(
201+
// "SPC v2 grouped metrics report (count=",
202+
// perMetric.length,
203+
// ")"
204+
// );
205+
// console.table?.(
206+
// perMetric.map((m) => ({
207+
// Metric: m.metric,
208+
// Dir: m.dir,
209+
// Len: m.length,
210+
// Eligible: m.eligible,
211+
// LastIcon: m.lastIcon,
212+
// Single: m.anySinglePoint,
213+
// TwoSig: m.anyTwoSigma,
214+
// Shift: m.anyShift,
215+
// Trend: m.anyTrend,
216+
// }))
217+
// );
218+
// console.info("Icon totals:", totals);
219219

220220
// Basic sanity: at least one metric per category over the full suite (dataset dependent but usually true)
221221
// We don't hard-require each bucket to be >0 to avoid brittleness; assert totals sum

src/components/DataVisualisation/charts/SPC/SPCChart/logic_v2/__tests__/colours.snapshot.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ describe("Golden colours snapshot", () => {
8282
const metric = getMetricOptions()[0];
8383
const { names } = buildIconNamesForGrouped(metric);
8484
const last12 = names.slice(-12);
85-
logArray("EXPECTED_GROUPED_LAST12", last12);
85+
// logArray("EXPECTED_GROUPED_LAST12", last12);
8686
if (EXPECTED_GROUPED_LAST12.length === 0) {
8787
// Scaffold mode: just ensure we produced something deterministic
8888
expect(last12.length).toBeGreaterThan(0);
@@ -99,7 +99,7 @@ describe("Golden colours snapshot", () => {
9999
toV2HealthcareDir(def.direction)
100100
);
101101
const last12 = names.slice(-12);
102-
logArray("EXPECTED_HEALTHCARE_LAST12", last12);
102+
// logArray("EXPECTED_HEALTHCARE_LAST12", last12);
103103
if (EXPECTED_HEALTHCARE_LAST12.length === 0) {
104104
expect(last12.length).toBeGreaterThan(0);
105105
} else {

src/components/DataVisualisation/charts/SPC/SPCChart/logic_v2/__tests__/specialCauseConflict.strategySweep.test.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -65,14 +65,14 @@ describe("logic_v2: strategy sweep for Special cause conflict - High is good", (
6565

6666
const results = strategies.map(computeMatchesForStrategy);
6767

68-
console.table(results.map(r => ({ strategy: strategyName(r.strategy), matched: `${r.totalMatched}/${r.totalCompared}`, fullMatch: r.totalMatched === r.totalCompared })));
68+
//console.table(results.map(r => ({ strategy: strategyName(r.strategy), matched: `${r.totalMatched}/${r.totalCompared}`, fullMatch: r.totalMatched === r.totalCompared })));
6969

70-
const anyFull = results.some(r => r.totalCompared > 0 && r.totalMatched === r.totalCompared);
71-
if (!anyFull) {
72-
console.warn("No full matches; first two mismatch sets:",
73-
results.slice(0,2).map(r => ({ strategy: strategyName(r.strategy), mismatches: r.mismatches.slice(0,10) }))
74-
);
75-
}
70+
// const anyFull = results.some(r => r.totalCompared > 0 && r.totalMatched === r.totalCompared);
71+
// if (!anyFull) {
72+
// console.warn("No full matches; first two mismatch sets:",
73+
// results.slice(0,2).map(r => ({ strategy: strategyName(r.strategy), mismatches: r.mismatches.slice(0,10) }))
74+
// );
75+
// }
7676
// Report-only: ensure the sweep ran comparisons but don't fail the build if no full match exists.
7777
const totalCompared = results.reduce((a,b)=>a+b.totalCompared,0);
7878
expect(totalCompared).toBeGreaterThan(0);

0 commit comments

Comments
 (0)