Fix test errors

Author: KushalMeghani1644

benches/cow_bench.rs

@@ -0,0 +1,51 @@
+use criterion::{criterion_group, criterion_main, Criterion};
+use std::hint::black_box;
+
+use heapless::{
+    cow::CowStr,
+    string::{String, StringView},
+};
+
+fn bench_cowstr(c: &mut Criterion) {
+    // Owned string
+    let mut owned_buf: String<32> = String::new();
+    owned_buf.push_str("owned hello").unwrap();
+    let owned_s: String<32> = owned_buf;
+
+    // Borrowed string (temporary buffer)
+    let mut temp_buf: String<32> = String::new();
+    temp_buf.push_str("hello").unwrap();
+
+    // Static string
+    let mut static_buf: String<32> = String::new();
+    static_buf.push_str("static hello").unwrap();
+    let static_sv: &'static StringView<usize> = Box::leak(Box::new(static_buf)).as_view();
+
+    // Borrowed benchmark
+    c.bench_function("CowStr::Borrowed to_owned", |b| {
+        b.iter(|| {
+            let sv = temp_buf.as_view();
+            let cow: CowStr<32> = CowStr::Borrowed(sv);
+            black_box(cow.to_owned());
+        });
+    });
+
+    // Static benchmark
+    c.bench_function("CowStr::Static to_owned", |b| {
+        b.iter(|| {
+            let cow: CowStr<32> = CowStr::Static(static_sv);
+            black_box(cow.to_owned());
+        });
+    });
+
+    // Owned benchmark
+    c.bench_function("CowStr::Owned clone", |b| {
+        b.iter(|| {
+            let cow: CowStr<32> = CowStr::Owned(owned_s.clone());
+            black_box(cow.to_owned());
+        });
+    });
+}
+
+criterion_group!(benches, bench_cowstr);
+criterion_main!(benches);

src/cow.rs

@@ -117,16 +117,14 @@ where
 #[cfg(test)]
 mod tests {
     use super::*;
-
-    // Helper function to create a StringView from a static str
-    fn sv_from_str(s: &'static str) -> StringView<usize> {
-        StringView::new(s)
-    }
+    use crate::string::StringView;
+    use crate::String;
 
     #[test]
     fn test_borrowed_variant() {
-        let view = sv_from_str("hello");
-        let cow: CowStr<16> = CowStr::Borrowed(&view);
+        let s: String<16> = String::try_from("hello").unwrap();
+        let view = s.as_view();
+        let cow: CowStr<16> = CowStr::Borrowed(view);
 
         assert!(cow.is_borrowed());
         assert!(!cow.is_static());
@@ -139,8 +137,9 @@ mod tests {
 
     #[test]
     fn test_static_variant() {
-        static VIEW: StringView<usize> = StringView::new("world");
-        let cow: CowStr<16> = CowStr::from_static(&VIEW);
+        let s: String<16> = String::try_from("world").unwrap();
+        let view: &'static StringView<usize> = Box::leak(Box::new(s));
+        let cow: CowStr<16> = CowStr::Static(view.as_view());
 
         assert!(!cow.is_borrowed());
         assert!(cow.is_static());
@@ -167,8 +166,9 @@ mod tests {
 
     #[test]
     fn test_from_stringview() {
-        let view = sv_from_str("from_borrowed");
-        let cow: CowStr<16> = CowStr::from(&view);
+        let s: String<16> = String::try_from("from_borrowed").unwrap();
+        let view = s.as_view();
+        let cow: CowStr<16> = CowStr::from(view);
 
         assert!(cow.is_borrowed());
         assert_eq!(cow.as_str(), "from_borrowed");