chore(simulation): update tests and fixtures for Sprint 23

- Add debug_response: None to all ObserverSnapshot constructors in
  integration tests
- Bump PROTOCOL_VERSION assertion 17 → 18 in serialization tests
- Regenerate golden proof_room_tick_10.json (BoundaryWall tiles)
- Regenerate client msgpack fixtures for new snapshot fields
- Fix debug.rs resource optionality (Option<ResMut> for
  ContaminationActive/EventQueue)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
2026-03-03 19:12:19 +01:00
co-authored by Claude Opus 4.6
parent c30197db0e
commit c797a72fc2
19 changed files with 1972 additions and 45 deletions
+1 -1
View File
@@ -1092,7 +1092,7 @@ dependencies = [
[[package]]
name = "settled-reach-server"
version = "0.1.20"
version = "0.1.22"
dependencies = [
"bevy_app",
"bevy_ecs",
+1
View File
@@ -74,6 +74,7 @@ fn snapshot_roundtrip_over_unix_socket() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
+1
View File
@@ -60,6 +60,7 @@ fn snapshot_roundtrip_over_tcp() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
+1
View File
@@ -295,6 +295,7 @@ fn snapshot_with_sim_errors_roundtrips() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: Some(0xDEADBEEF),
debug_response: None,
sim_errors: vec![
SimError {
kind: SimErrorKind::ProtocolError,
+3
View File
@@ -50,6 +50,7 @@ fn fixture_snapshot(tick: u64, entities: Vec<VisibleEntity>) -> ObserverSnapshot
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
}
}
@@ -244,6 +245,7 @@ fn generate_msgpack_fixtures() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
write_fixture(
@@ -408,6 +410,7 @@ fn generate_msgpack_fixtures() {
}),
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
write_fixture(
File diff suppressed because it is too large Load Diff
+4 -1
View File
@@ -38,6 +38,7 @@ fn test_snapshot(tick: u64, entities: Vec<VisibleEntity>) -> ObserverSnapshot {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
}
}
@@ -299,6 +300,7 @@ fn snapshot_v2_fields_roundtrip() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
@@ -354,7 +356,7 @@ fn protocol_version_constant_matches_snapshot() {
let snapshot = test_snapshot(0, vec![]);
assert_eq!(snapshot.version, PROTOCOL_VERSION);
assert_eq!(
PROTOCOL_VERSION, 17,
PROTOCOL_VERSION, 18,
"bump this assertion when protocol version changes"
);
}
@@ -406,6 +408,7 @@ fn all_facing_direction_variants_roundtrip() {
save_result: None,
triangle_crisis_events: vec![],
state_hash: None,
debug_response: None,
sim_errors: vec![],
};
let bytes = rmp_serde::to_vec_named(&snapshot).expect("serialize");