fix: text-slot bug in testing framework

Author: TtTRz

float-pigment-forest/src/node.rs

@@ -315,9 +315,9 @@ impl Node {
     pub(crate) unsafe fn baseline_func(&self) -> Option<&BaselineFn<Len>> {
         (*self.baseline_func.get()).as_deref()
     }
-    pub unsafe fn set_baseline_func(&self, baseline_func: Option<Box<BaselineFn<Len>>>) {
+    pub fn set_baseline_func(&self, baseline_func: Option<Box<BaselineFn<Len>>>) {
         drop(std::mem::replace(
-            &mut *self.baseline_func.get(),
+            unsafe { &mut *self.baseline_func.get() },
             baseline_func,
         ));
     }

float-pigment-forest/tests/custom/css_margin.rs

@@ -365,6 +365,20 @@ fn margin_collapse_between_sibling_and_empty_block_4() {
     )
 }
 
+#[test]
+fn margin_collapse_between_sibling_and_empty_block_5() {
+    assert_xml!(
+        r#"
+            <div style="height: 300px;">
+                <div style="height: 30px; margin-bottom: 50px" expect_height="30" expect_top="0"></div>
+                <text-slot len="0"></text-slot>
+                <div style="height: 30px; margin-top: 100px" expect_height="30" expect_top="130"></div>
+            </div>
+        "#,
+        true
+    )
+}
+
 #[test]
 fn margin_collapse_between_parent_and_empty_block_1() {
     assert_xml!(

float-pigment-forest/tests/mod.rs

@@ -125,6 +125,12 @@ impl TextInfo {
         max_content_height: Len,
     ) -> Size<Len> {
         let text_len = self.text_len;
+        if text_len == 0 {
+            println!(
+                "text_info: {self:?}, width: {min_width:?} ~ {max_width:?}, height: {min_height:?} ~ {max_height:?}, max_content_width: {max_content_width:?}, max_content_height: {max_content_height:?}, measured_width: 0, measured_height: 0",
+            );
+            return Size::new(Len::zero(), Len::zero());
+        }
         let text_width = self.font_size * text_len as f32;
         let max_w = max_width.min(max_content_width);
         let max_h = max_height.min(max_content_height);
@@ -188,6 +194,11 @@ fn convert_font_size_to_px(font_size: float_pigment_css::typing::Length) -> f32
 #[inline(always)]
 fn prepare_measure_node(node: *mut Node, text_info: TextInfo) {
     let node = unsafe { &mut *node };
+    unsafe {
+        node.set_display(Display::Inline);
+        node.set_node_type(float_pigment_forest::NodeType::Text);
+    }
+    node.set_baseline_func(Some(Box::new(|_, _, _| Len::from_f32(16.))));
     node.set_measure_func(Some(Box::new(
         move |_,
               max_width,
@@ -208,11 +219,6 @@ fn prepare_measure_node(node: *mut Node, text_info: TextInfo) {
             )
         },
     )));
-    unsafe {
-        node.set_display(Display::Inline);
-        node.set_baseline_func(Some(Box::new(|_, _, _| Len::from_f32(16.))));
-        node.set_node_type(float_pigment_forest::NodeType::Text);
-    }
 }
 
 impl TestCtx {
@@ -385,7 +391,6 @@ impl TestCtx {
                 self.set_expect_layout_pos(node, e.attributes());
 
                 if is_measure_text_slot(e.tag()) {
-                    let node = Node::new_ptr();
                     let text_len = e
                         .attributes()
                         .get("len")