From cfdd8a533b38b164ac539211857e9e10904e20f3 Mon Sep 17 00:00:00 2001 From: Iraklis Kourtis Date: Mon, 12 Oct 2026 09:48:42 -0400 Subject: [PATCH] test(doc): drop two timing assertions on when a source's mesh is built The world-placement tests checked that the source body's mesh was built before a placement adopted its names. Once names follow history they no longer need the mesh, so that is a side effect's timing, not behaviour. What the tests protect, a placement sharing its source's names, both still assert through the public API: Arc::ptr_eq(face_names(placed), face_names(source)). Co-Authored-By: Claude Opus 5.5 Claude-Session: https://claude.ai/code/session_01J53sR6gEaVt56soFDpTck8 --- crates/doc/src/eval.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/crates/doc/src/eval.rs b/crates/doc/src/eval.rs index a55f2c6a..aaee71c3 100644 --- a/crates/doc/src/eval.rs +++ b/crates/doc/src/eval.rs @@ -3703,7 +3703,6 @@ mod world_placement_cache_tests { assert!(source.names.origin.is_some()); assert!(source.mesh.get().is_none()); let world = world_state(&doc, &state); - assert!(source.mesh.get().is_some(), "source naming is prepared before adoption, as before"); let names = crate::naming::face_names(&world.bodies[0]); assert!(Arc::ptr_eq(&names, &crate::naming::face_names(source))); assert!(names.iter().all(|name| name.starts_with(&format!("F{feature}:")))); @@ -3731,7 +3730,6 @@ mod world_placement_cache_tests { let first = world.body("Block").unwrap(); let second_name = doc.world_body_name(&state, "Block", component, &[second]); let placed = world.body(&second_name).unwrap(); - assert!(source.mesh.get().is_some(), "persistent source names stay prepared before adoption"); assert!(Arc::ptr_eq(&source.mesh, &first.mesh), "unchanged geometry must retain the shared display mesh cell"); assert_eq!(first.key, 0); assert_eq!(placed.key, 0);