From ba457d693cbfdc5f845f2419bd7a8d382ac2049c Mon Sep 17 00:00:00 2001
From: Karim Jerbi <k4rim.jerbi@gmail.com>
Date: Sat, 17 Oct 2026 23:48:17 +0100
Subject: [PATCH] feat: verify Polar Stretch target editing

Exercise frame replacement and vertex rebinding across native DWG and DXF round trips.
---
 src/scene/block_actions/tests.rs | 72 ++++++++++++++++++++++++++++++++
 1 file changed, 72 insertions(+)

diff --git a/src/scene/block_actions/tests.rs b/src/scene/block_actions/tests.rs
index 145c2c1ab..09edd01a1 100644
--- a/src/scene/block_actions/tests.rs
+++ b/src/scene/block_actions/tests.rs
@@ -741,6 +741,78 @@ fn polar_stretch_evaluates_test_block_and_saved_reference() {
     assert_eq!(line.end, Vector3::new(2., 0., 0.));
 }
 
+#[test]
+fn polar_stretch_retarget_rebinds_vertices_and_roundtrips() {
+    let mut scene = Scene::new();
+    scene.define_block_raw(
+        "POLAR_RETARGET",
+        Vector3::ZERO,
+        vec![
+            EntityType::Line(codec::entities::Line::from_points(
+                Vector3::UNIT_X,
+                Vector3::new(2., 0., 0.),
+            )),
+            EntityType::Line(codec::entities::Line::from_points(
+                Vector3::new(4., 0., 0.),
+                Vector3::new(5., 0., 0.),
+            )),
+        ],
+    );
+    let record = scene.document.block_records.get("POLAR_RETARGET").unwrap();
+    let (definition, targets) = (record.handle, record.entity_handles.clone());
+    let mut doc = scene.document;
+    let parameter = distance::create(
+        &mut doc,
+        definition,
+        distance::Kind::Polar,
+        Vector3::ZERO,
+        Vector3::new(10., 0., 0.),
+        Vector3::new(5., 2., 0.),
+    )
+    .unwrap();
+    let action = create_with_frame(
+        &mut doc,
+        definition,
+        Kind::PolarStretch,
+        parameter,
+        &targets[..1],
+        DVec3::ZERO,
+        Some([DVec3::new(0.5, -0.5, 0.), DVec3::new(1.5, 0.5, 0.)]),
+    )
+    .unwrap();
+
+    assert!(retarget(
+        &mut doc,
+        definition,
+        action,
+        &targets[1..],
+        Some([DVec3::new(3.5, -0.5, 0.), DVec3::new(5.5, 0.5, 0.)]),
+    )
+    .unwrap());
+    let ObjectType::DynamicBlock(object) = &doc.objects[&action] else {
+        panic!()
+    };
+    let DynamicBlockData::PolarStretchAction(action_data) = &object.data else {
+        panic!()
+    };
+    assert_eq!(action_data.action.dependencies, vec![targets[1]]);
+    assert_eq!(action_data.bindings.len(), 1);
+    assert_eq!(action_data.bindings[0].handle, targets[1]);
+    assert_eq!(action_data.points.len(), 2);
+
+    for saved in [
+        DwgReader::from_stream(Cursor::new(DwgWriter::write_to_vec(&doc).unwrap()))
+            .read()
+            .unwrap(),
+        DxfReader::from_reader(Cursor::new(DxfWriter::new(&doc).write_to_vec().unwrap()))
+            .unwrap()
+            .read()
+            .unwrap(),
+    ] {
+        assert_eq!(saved.objects[&action], doc.objects[&action]);
+    }
+}
+
 #[test]
 fn block_actions_stretch_frozen_vertices_polyline_and_whole_objects_roundtrip() {
     let (doc, def, handles, p, action) = stretch_fixture();