Skip to main content

max / makenotwork

Fix stale AppState test harnesses (cost_allocation, pricing_comparison) The integration and load test harnesses constructed AppState with a removed field (cost_allocation / tier_prices::CostAllocation, deleted in bdd88fb) and were missing pricing_comparison (added in 446b939). Neither surfaced under `cargo test --lib`, only when the tests/ targets compile — Sando's cargo_test gate caught it. Drop the dead field, add pricing_comparison via the canonical assumptions.toml, matching main.rs. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author: Max Johnson <me@maxj.phd> · 2026-07-08 18:39 UTC
Commit: 7921ac8b1d43afb5ebb1644cb444bf451aaeb649
Parent: fbabd18
2 files changed, +6 insertions, -2 deletions
@@ -371,8 +371,10 @@ impl TestHarness {
371 371 db: pool.clone(),
372 372 config,
373 373 tier_prices: makenotwork::tier_prices::TierPrices::default(),
374 - cost_allocation: makenotwork::tier_prices::CostAllocation::default(),
375 374 runway_config: makenotwork::tier_prices::RunwayConfig::default(),
375 + pricing_comparison: makenotwork::pricing_comparison::PricingComparison::load(
376 + "docs/business/assumptions.toml",
377 + ),
376 378 s3,
377 379 synckit_s3,
378 380 public_s3,
@@ -107,8 +107,10 @@ pub async fn run(config: LoadConfig) {
107 107 db: pool.clone(),
108 108 config: app_config,
109 109 tier_prices: makenotwork::tier_prices::TierPrices::default(),
110 - cost_allocation: makenotwork::tier_prices::CostAllocation::default(),
111 110 runway_config: makenotwork::tier_prices::RunwayConfig::default(),
111 + pricing_comparison: makenotwork::pricing_comparison::PricingComparison::load(
112 + "docs/business/assumptions.toml",
113 + ),
112 114 s3: None,
113 115 synckit_s3: None,
114 116 public_s3: None,