Fix coverage issue

Author: owjs3901

.github/workflows/publish.yml

@@ -40,6 +40,9 @@ jobs:
       - run: |
           pnpm build
           pnpm lint
+          # rust coverage issue
+          echo "max_width = 1000\ntab_spaces = 4\nnewline_style = \"Unix\"\nfn_call_width = 1000\nfn_params_layout = \"Compressed\"\nchain_width = 1000\nmerge_derives = true\nuse_small_heuristics = \"Default\"\n" > .rustfmt.toml
+          cargo fmt
           pnpm test
       - name: Benchmark
         run: pnpm benchmark

libs/css/src/lib.rs

@@ -454,7 +454,7 @@ mod tests {
                 "cls",
                 Some(&StyleSelector::Media {
                     query: "print".to_string(),
-                    selector: None,
+                    selector: None
                 })
             ),
             ".cls"
@@ -465,7 +465,7 @@ mod tests {
                 "cls",
                 Some(&StyleSelector::Media {
                     query: "print".to_string(),
-                    selector: Some("&:hover".to_string()),
+                    selector: Some("&:hover".to_string())
                 })
             ),
             ".cls:hover"

libs/css/src/optimize_value.rs

@@ -114,7 +114,7 @@ pub fn optimize_value(value: &str) -> String {
 }
 
 fn optimize_color(value: &str) -> String {
-    let mut ret = value.to_string().to_uppercase();
+    let mut ret = value.to_uppercase();
 
     if ret.len() == 6 {
         let ch = ret.chars().collect::<Vec<char>>();

libs/extractor/src/css_utils.rs

@@ -157,7 +157,7 @@ pub fn optimize_css_block(css: &str) -> String {
         .split(";")
         .map(|s| {
             if !s.contains(":") {
-                s.to_string().trim().to_string()
+                s.trim().to_string()
             } else {
                 let mut iter = s.split(":");
                 let property = iter.next().unwrap().trim();

libs/extractor/src/gen_class_name.rs

@@ -124,7 +124,7 @@ pub fn merge_expression_for_class_name<'a>(
     let mut unknown_expr = vec![];
     for expr in expressions {
         if let Expression::StringLiteral(str) = &expr {
-            class_names.push(str.value.to_string().trim().to_string())
+            class_names.push(str.value.trim().to_string())
         } else {
             unknown_expr.push(expr);
         }

libs/extractor/src/lib.rs

@@ -200,10 +200,7 @@ mod tests {
                 r#"import {Box} from '@devup-ui/core'
         <Box padding={1} ref={ref} data-test={1} role={2} children={[]} onClick={()=>{}} aria-valuenow={24} key={2} tabIndex={1} id="id" />
         "#,
-                ExtractOption {
-                    package: "@devup-ui/core".to_string(),
-                    css_file: None
-                }
+                ExtractOption { package: "@devup-ui/core".to_string(), css_file: None }
             )
             .unwrap()
         ));
@@ -2595,37 +2592,13 @@ e(o, { className: "a", bg: variable, style: { color: "blue" }, ...props })
     #[serial]
     fn support_transpile_cjs() {
         reset_class_map();
-        assert_debug_snapshot!(ToBTreeSet::from(extract(
-            "test.cjs",
-            r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e.jsxs("div",{children:[e.jsx(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e.jsx(r.Text,{typography:"header",children:"typo"}),e.jsx(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#,
-            ExtractOption {
-                package: "@devup-ui/react".to_string(),
-                css_file: None
-            }
-        )
-        .unwrap()));
+        assert_debug_snapshot!(ToBTreeSet::from(extract("test.cjs", r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e.jsxs("div",{children:[e.jsx(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e.jsx(r.Text,{typography:"header",children:"typo"}),e.jsx(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#, ExtractOption { package: "@devup-ui/react".to_string(), css_file: None }).unwrap()));
 
         reset_class_map();
-        assert_debug_snapshot!(ToBTreeSet::from(extract(
-            "test.cjs",
-            r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const {jsx:e1, jsxs:e2}=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e2("div",{children:[e1(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e1(r.Text,{typography:"header",children:"typo"}),e1(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#,
-            ExtractOption {
-                package: "@devup-ui/react".to_string(),
-                css_file: None
-            }
-        )
-        .unwrap()));
+        assert_debug_snapshot!(ToBTreeSet::from(extract("test.cjs", r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const {jsx:e1, jsxs:e2}=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e2("div",{children:[e1(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e1(r.Text,{typography:"header",children:"typo"}),e1(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#, ExtractOption { package: "@devup-ui/react".to_string(), css_file: None }).unwrap()));
 
         reset_class_map();
-        assert_debug_snapshot!(ToBTreeSet::from(extract(
-            "test.js",
-            r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e.jsxs("div",{children:[e.jsx(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e.jsx(r.Text,{typography:"header",children:"typo"}),e.jsx(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#,
-            ExtractOption {
-                package: "@devup-ui/react".to_string(),
-                css_file: None
-            }
-        )
-        .unwrap()));
+        assert_debug_snapshot!(ToBTreeSet::from(extract("test.js", r#""use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("react/jsx-runtime"),r=require("@devup-ui/react");function t(){return e.jsxs("div",{children:[e.jsx(r.Box,{_hover:{bg:"blue"},bg:"$text",color:"red",children:"hello"}),e.jsx(r.Text,{typography:"header",children:"typo"}),e.jsx(r.Flex,{as:"section",mt:2,children:"section"})]})}exports.Lib=t;"#, ExtractOption { package: "@devup-ui/react".to_string(), css_file: None }).unwrap()));
     }
 
     #[test]
@@ -2638,10 +2611,7 @@ e(o, { className: "a", bg: variable, style: { color: "blue" }, ...props })
                 r#"import {Flex} from '@devup-ui/core'
         <Flex opacity={1} zIndex={2} fontWeight={900} scale={2} flex={1} lineHeight={1} tabSize={4} MozTabSize={4} />
         "#,
-                ExtractOption {
-                    package: "@devup-ui/core".to_string(),
-                    css_file: None
-                }
+                ExtractOption { package: "@devup-ui/core".to_string(), css_file: None }
             )
             .unwrap()
         ));