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 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J53sR6gEaVt56soFDpTck8
This commit is contained in:
1 parent
08cf526fe2
commit
cfdd8a533b
1 file changed
-2
@@ -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);
|
||||
|
||||
Reference in new issue
Block a user