test: test case for item split (toeverything#450)

* test: add test case

* chore: restore debug impl

* chore: cleanup code

Author: darkskygit

libs/jwst-codec/src/doc/codec/content.rs

@@ -365,6 +365,52 @@ mod tests {
         }
     }
 
+    #[test]
+    fn test_content_split() {
+        let contents = [
+            Content::String("hello".to_string()),
+            Content::JSON(vec![
+                None,
+                Some("test_1".to_string()),
+                Some("test_2".to_string()),
+            ]),
+            Content::Any(vec![Any::BigInt64(42), Any::String("Test Any".to_string())]),
+            Content::Binary(vec![]),
+        ];
+
+        {
+            let (left, right) = contents[0].split(1).unwrap();
+            assert!(contents[0].splittable());
+            assert_eq!(left, Content::String("h".to_string()));
+            assert_eq!(right, Content::String("ello".to_string()));
+        }
+
+        {
+            let (left, right) = contents[1].split(1).unwrap();
+            assert!(contents[1].splittable());
+            assert_eq!(left, Content::JSON(vec![None, Some("test_1".to_string())]));
+            assert_eq!(right, Content::JSON(vec![Some("test_2".to_string())]));
+        }
+
+        {
+            let (left, right) = contents[2].split(1).unwrap();
+            assert!(contents[2].splittable());
+            assert_eq!(
+                left,
+                Content::Any(vec![Any::BigInt64(42), Any::String("Test Any".to_string())])
+            );
+            assert_eq!(right, Content::Any(vec![]));
+        }
+
+        {
+            assert!(!contents[3].splittable());
+            assert_eq!(
+                contents[3].split(2),
+                Err(JwstCodecError::ContentSplitNotSupport(2))
+            );
+        }
+    }
+
     proptest! {
         #[test]
         fn test_random_content(contents in vec(any::<Content>(), 0..10)) {

libs/jwst-codec/src/doc/codec/utils/items.rs

@@ -50,3 +50,29 @@ impl ItemBuilder {
         self.item
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_item_builder() {
+        let item = ItemBuilder::new()
+            .id(Id::new(0, 1))
+            .left_id(Some(Id::new(2, 3)))
+            .right_id(Some(Id::new(4, 5)))
+            .parent(Some(Parent::String("test".into())))
+            .content(Content::Any(vec![Any::String("Hello".into())]))
+            .build();
+
+        assert_eq!(item.id, Id::new(0, 1));
+        assert_eq!(item.left_id, Some(Id::new(2, 3)));
+        assert_eq!(item.right_id, Some(Id::new(4, 5)));
+        assert!(matches!(item.parent, Some(Parent::String(text)) if text == "test"));
+        assert_eq!(item.parent_sub, None);
+        assert_eq!(
+            item.content,
+            std::sync::Arc::new(Content::Any(vec![Any::String("Hello".into())]))
+        );
+    }
+}

libs/jwst-codec/src/doc/store.rs

@@ -545,7 +545,7 @@ impl DocStore {
                         item.delete();
                     } else {
                         // adjust parent length
-                        if item.parent_sub.is_none() && item.flags.countable() {
+                        if item.parent_sub.is_none() && item.countable() {
                             parent.content_len += item.len();
                             parent.item_len += 1;
                         }