Add testcase

Author: owjs3901

libs/css/src/lib.rs

@@ -794,6 +794,24 @@ mod tests {
         );
     }
 
+    #[test]
+    #[serial]
+    fn test_debug_sheet_to_classname_with_filename() {
+        reset_class_map();
+        set_debug(true);
+        // Debug mode + filename triggers the file_suffix branch (lines 234-235)
+        let class_name =
+            sheet_to_classname("background", 0, Some("red"), None, None, Some("test.tsx"));
+        assert!(class_name.contains("background-0-red--255-"));
+        // Should have a file number suffix
+        let parts: Vec<&str> = class_name.split('-').collect();
+        assert!(
+            parts.len() >= 6,
+            "Expected file suffix in debug classname: {class_name}"
+        );
+        set_debug(false);
+    }
+
     #[test]
     fn test_merge_selector() {
         assert_eq!(merge_selector("cls", Some(&"hover".into())), ".cls:hover");

libs/css/src/optimize_value.rs

@@ -42,10 +42,6 @@ pub fn optimize_value(value: &str) -> String {
         if let std::borrow::Cow::Owned(s) = replaced {
             ret = s;
         }
-        let trimmed = ret.trim();
-        if trimmed.len() != ret.len() {
-            ret = trimmed.to_string();
-        }
     }
     let replaced = F_RGBA_RE.replace_all(&ret, |c: &regex::Captures| {
         let r = c[1].parse::<i32>().unwrap();

libs/sheet/src/lib.rs

@@ -2207,4 +2207,42 @@ mod tests {
         assert!(css.contains("@layer components"));
         assert_debug_snapshot!(css.split("*/").nth(1).unwrap());
     }
+
+    #[test]
+    fn test_stylesheet_css_extract() {
+        let css_entry = StyleSheetCss {
+            css: "div{display:flex}".to_string(),
+        };
+        assert_eq!(css_entry.extract(), "div{display:flex}");
+
+        let empty = StyleSheetCss { css: String::new() };
+        assert_eq!(empty.extract(), "");
+    }
+
+    #[test]
+    fn test_keyframes_multi_property() {
+        let mut sheet = StyleSheet::default();
+        let mut keyframes: BTreeMap<String, Vec<(String, String)>> = BTreeMap::new();
+        // Multiple properties in a single keyframe step to cover the semicolon separator (line 548)
+        keyframes.insert(
+            String::from("from"),
+            vec![
+                (String::from("opacity"), String::from("0")),
+                (String::from("transform"), String::from("scale(0.5)")),
+            ],
+        );
+        keyframes.insert(
+            String::from("to"),
+            vec![
+                (String::from("opacity"), String::from("1")),
+                (String::from("transform"), String::from("scale(1)")),
+            ],
+        );
+        sheet.add_keyframes("slideIn", keyframes, None);
+        let css = sheet.create_css(None, false);
+        // Verify semicolon separator between multiple properties in a keyframe step
+        assert!(css.contains("opacity:0;transform:scale(0.5)"));
+        assert!(css.contains("opacity:1;transform:scale(1)"));
+        assert_debug_snapshot!(css.split("*/").nth(1).unwrap());
+    }
 }

libs/sheet/src/snapshots/sheet__tests__keyframes_multi_property.snap

@@ -0,0 +1,5 @@
+---
+source: libs/sheet/src/lib.rs
+expression: "css.split(\"*/\").nth(1).unwrap()"
+---
+"@keyframes slideIn{from{opacity:0;transform:scale(0.5)}to{opacity:1;transform:scale(1)}}"