Fix coverage issue

Author: owjs3901

libs/extractor/src/css_utils.rs

@@ -470,11 +470,20 @@ mod tests {
             ("50%", vec![("color", "red"), ("background", "blue")]),
         ],
     )]
+    // error case
+    #[case(
+        "50% { color: red        ; background: blue ",
+        vec![
+        ],
+    )]
     fn test_keyframes_to_keyframes_style(
         #[case] input: &str,
         #[case] expected: Vec<(&str, Vec<(&str, &str)>)>,
     ) {
         let styles = keyframes_to_keyframes_style(input);
+        if styles.len() != expected.len() {
+            panic!("styles.len() != expected.len()");
+        }
         for (expected_key, expected_styles) in styles.iter() {
             let styles = expected_styles;
             let mut result: Vec<(&str, &str)> = styles

libs/extractor/src/gen_style.rs

@@ -32,29 +32,25 @@ fn gen_style<'a>(
     let mut properties = vec![];
     match style {
         ExtractStyleProp::Static(st) => {
-            if let Some(ex) = st.extract() {
-                match ex {
-                    StyleProperty::ClassName(_) => {}
-                    StyleProperty::Variable {
-                        variable_name,
-                        identifier,
-                        ..
-                    } => {
-                        properties.push(ast_builder.object_property_kind_object_property(
-                            SPAN,
-                            PropertyKind::Init,
-                            PropertyKey::StringLiteral(ast_builder.alloc_string_literal(
-                                SPAN,
-                                ast_builder.atom(&variable_name),
-                                None,
-                            )),
-                            ast_builder.expression_identifier(SPAN, ast_builder.atom(&identifier)),
-                            false,
-                            false,
-                            false,
-                        ));
-                    }
-                }
+            if let Some(StyleProperty::Variable {
+                variable_name,
+                identifier,
+                ..
+            }) = st.extract()
+            {
+                properties.push(ast_builder.object_property_kind_object_property(
+                    SPAN,
+                    PropertyKind::Init,
+                    PropertyKey::StringLiteral(ast_builder.alloc_string_literal(
+                        SPAN,
+                        ast_builder.atom(&variable_name),
+                        None,
+                    )),
+                    ast_builder.expression_identifier(SPAN, ast_builder.atom(&identifier)),
+                    false,
+                    false,
+                    false,
+                ));
             }
         }
         ExtractStyleProp::StaticArray(res) => {

libs/extractor/src/lib.rs

@@ -4981,6 +4981,28 @@ keyframes({
   [`50%`]: { opacity: 0.5 },
   [`100%`]: { opacity: 1 }
 })
+"#,
+                ExtractOption {
+                    package: "@devup-ui/core".to_string(),
+                    css_file: None
+                }
+            )
+            .unwrap()
+        ));
+    }
+    #[test]
+    #[serial]
+    fn extract_wrong_keyframs() {
+        reset_class_map();
+        assert_debug_snapshot!(ToBTreeSet::from(
+            extract(
+                "test.tsx",
+                r#"import { keyframes } from "@devup-ui/core";
+keyframes({
+  from: { opacity: 0 },
+  [true]: { opacity: 0.5 },
+  to: { opacity: 1, color: dy }
+})
 "#,
                 ExtractOption {
                     package: "@devup-ui/core".to_string(),

libs/extractor/src/snapshots/extractor__tests__extract_wrong_keyframs.snap

@@ -0,0 +1,33 @@
+---
+source: libs/extractor/src/lib.rs
+expression: "ToBTreeSet::from(extract(\"test.tsx\",\nr#\"import { keyframes } from \"@devup-ui/core\";\nkeyframes({\n  from: { opacity: 0 },\n  [true]: { opacity: 0.5 },\n  to: { opacity: 1 }\n})\n\"#,\nExtractOption\n{ package: \"@devup-ui/core\".to_string(), css_file: None }).unwrap())"
+---
+ToBTreeSet {
+    styles: {
+        Keyframes(
+            ExtractKeyframes {
+                keyframes: {
+                    "from": [
+                        ExtractStaticStyle {
+                            property: "opacity",
+                            value: "0",
+                            level: 0,
+                            selector: None,
+                            style_order: None,
+                        },
+                    ],
+                    "to": [
+                        ExtractStaticStyle {
+                            property: "opacity",
+                            value: "1",
+                            level: 0,
+                            selector: None,
+                            style_order: None,
+                        },
+                    ],
+                },
+            },
+        ),
+    },
+    code: "import \"@devup-ui/core/devup-ui.css\";\n\"k0\";\n",
+}