From adce12035a442834a7545f122bb3425168baebd9 Mon Sep 17 00:00:00 2001 From: Jeroen Schweitzer Date: Wed, 3 Jun 2026 16:31:19 +0200 Subject: [PATCH] fix(simulation): route atlas_response fixtures in the serialize test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit all_fixtures_deserialize panicked with "unknown fixture naming convention: atlas_response_ready" — the test reads every .msgpack in client/tests/fixtures/msgpack but had no branch for the atlas_response prefix (the AtlasLayerResponse fixtures gen_fixtures emits, #969). A pre-existing gap (present on main; the suite isn't gated by the pre-push hook, which runs cargo-deny/ruff/JSON/stamp, not cargo test). Add an atlas_response branch deserializing AtlasLayerResponse, and regenerate atlas_response_ready.msgpack so its bytes match the now-integer GeographicAttractor types (#955). Co-Authored-By: Claude Opus 4.8 (1M context) --- .../msgpack/atlas_response_ready.msgpack | Bin 314 -> 307 bytes server/tests/serialization.rs | 5 +++++ 2 files changed, 5 insertions(+) diff --git a/client/tests/fixtures/msgpack/atlas_response_ready.msgpack b/client/tests/fixtures/msgpack/atlas_response_ready.msgpack index 39e40b709e23b753544cad1e03f4f3516b4064f8..acc81f0072312c65f79b4cde500de05b674e7607 100644 GIT binary patch delta 34 mcmdnRw3%tbOU9^)?~KLHtXh^{l$jD=ewK*=!pboOI1^|jC5!nC$ diff --git a/server/tests/serialization.rs b/server/tests/serialization.rs index 62b9002b1..01f5e8902 100644 --- a/server/tests/serialization.rs +++ b/server/tests/serialization.rs @@ -1,5 +1,6 @@ //! IPC serialization round-trip tests (D-030 Layer 1: fixture-based). +use settled_reach_server::atlas::layer_proxy::AtlasLayerResponse; use settled_reach_server::bridge::types::*; use settled_reach_server::simulation::time::{DayPhase, TickRate}; use std::fs; @@ -173,6 +174,10 @@ fn all_fixtures_deserialize() { rmp_serde::from_slice::(&bytes).unwrap_or_else(|e| { panic!("deserialize boundary snapshot fixture {}: {}", name, e) }); + } else if name.starts_with("atlas_response") { + // Atlas layer-stream responses (#969, D-225) — Ready/Pending/NotFound. + rmp_serde::from_slice::(&bytes) + .unwrap_or_else(|e| panic!("deserialize atlas_response fixture {}: {}", name, e)); } else if name.starts_with("snapshot") { rmp_serde::from_slice::(&bytes) .unwrap_or_else(|e| panic!("deserialize snapshot fixture {}: {}", name, e));