test: introduce test_case and rework primitive tests

fujiapple852 · Jul 28, 2024 · b30dc83 · b30dc83
1 parent e4f5ba0
commit b30dc83
Show file tree

Hide file tree

Showing 3 changed files with 41 additions and 138 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -18,6 +18,7 @@ bounded-static-derive = { version = "0.8.0", path = "bounded-static-derive" }
 syn = { version = "2.0.38", features = [ "full" ] }
 quote = "1.0.33"
 proc-macro2 = "1.0.69"
+test-case = "3.3.1"
 smol_str = { version = "0.2.2", default-features = false }
 smallvec = { version = "1.13.2", default-features = false }
 smartstring = { version = "1.0.1", default-features = false }

diff --git a/bounded-static/Cargo.toml b/bounded-static/Cargo.toml
@@ -37,5 +37,8 @@ smartstring = { workspace = true, optional = true, default-features = false }
 ahash = { workspace = true, optional = true, default-features = false }
 chrono = { workspace = true, optional = true, default-features = false }
 
+[dev-dependencies]
+test-case.workspace = true
+
 [package.metadata.docs.rs]
 all-features = true
diff --git a/bounded-static/src/lib.rs b/bounded-static/src/lib.rs
@@ -1024,149 +1024,48 @@ make_copy_impl!(chrono::Local);
 #[cfg(test)]
 mod core_tests {
     use super::*;
+    use test_case::test_case;
 
     fn ensure_static<T: 'static>(t: T) {
         drop(t);
     }
 
-    #[test]
-    fn test_bool() {
-        ensure_static(false.to_static());
-    }
-
-    #[test]
-    fn test_char() {
-        ensure_static('a'.to_static());
-    }
-
-    #[test]
-    fn test_f32() {
-        ensure_static(0.0f32.to_static());
-    }
-
-    #[test]
-    fn test_f64() {
-        ensure_static(0.0f64.to_static());
-    }
-
-    #[test]
-    fn test_usize() {
-        ensure_static(0usize.to_static());
-    }
-
-    #[test]
-    fn test_u8() {
-        ensure_static(0u8.to_static());
-    }
-
-    #[test]
-    fn test_u16() {
-        ensure_static(0u16.to_static());
-    }
-
-    #[test]
-    fn test_u32() {
-        ensure_static(0u32.to_static());
-    }
-
-    #[test]
-    fn test_u64() {
-        ensure_static(0u64.to_static());
-    }
-
-    #[test]
-    fn test_u128() {
-        ensure_static(0u128.to_static());
-    }
-
-    #[test]
-    fn test_isize() {
-        ensure_static(0isize.to_static());
-    }
-
-    #[test]
-    fn test_i8() {
-        ensure_static(0i8.to_static());
-    }
-
-    #[test]
-    fn test_i16() {
-        ensure_static(0i16.to_static());
-    }
-
-    #[test]
-    fn test_i32() {
-        ensure_static(0i32.to_static());
-    }
-
-    #[test]
-    fn test_i64() {
-        ensure_static(0i64.to_static());
-    }
-
-    #[test]
-    fn test_i128() {
-        ensure_static(0i128.to_static());
-    }
-
-    #[test]
-    fn test_non_zero_usize() {
-        ensure_static(NonZeroUsize::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_u8() {
-        ensure_static(NonZeroU8::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_u16() {
-        ensure_static(NonZeroU16::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_u32() {
-        ensure_static(NonZeroU32::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_u64() {
-        ensure_static(NonZeroU64::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_u128() {
-        ensure_static(NonZeroU128::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_isize() {
-        ensure_static(NonZeroIsize::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_i8() {
-        ensure_static(NonZeroI8::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_i16() {
-        ensure_static(NonZeroI16::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_i32() {
-        ensure_static(NonZeroI32::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_i64() {
-        ensure_static(NonZeroI64::new(1).unwrap().to_static());
-    }
-
-    #[test]
-    fn test_non_zero_i128() {
-        ensure_static(NonZeroI128::new(1).unwrap().to_static());
+    #[test_case(false; "bool")]
+    #[test_case('a'; "char")]
+    #[test_case(0.0f32; "f32")]
+    #[test_case(0.0f64; "f64")]
+    #[test_case(0usize; "usize")]
+    #[test_case(0u8; "u8")]
+    #[test_case(0u16; "u16")]
+    #[test_case(0u32; "u32")]
+    #[test_case(0u64; "u64")]
+    #[test_case(0u128; "u128")]
+    #[test_case(0isize; "isize")]
+    #[test_case(0i8; "i8")]
+    #[test_case(0i16; "i16")]
+    #[test_case(0i32; "i32")]
+    #[test_case(0i64; "i64")]
+    #[test_case(0i128; "i128")]
+    #[allow(clippy::needless_pass_by_value)]
+    fn test_primitive<T: ToBoundedStatic>(t: T) {
+        ensure_static(t.to_static());
+    }
+
+    #[test_case(NonZeroUsize::new(1); "usize")]
+    #[test_case(NonZeroU8::new(1); "u8")]
+    #[test_case(NonZeroU16::new(1); "u16")]
+    #[test_case(NonZeroU32::new(1); "u32")]
+    #[test_case(NonZeroU64::new(1); "u64")]
+    #[test_case(NonZeroU128::new(1); "u128")]
+    #[test_case(NonZeroIsize::new(1); "isize")]
+    #[test_case(NonZeroI8::new(1); "i8")]
+    #[test_case(NonZeroI16::new(1); "i16")]
+    #[test_case(NonZeroI32::new(1); "i32")]
+    #[test_case(NonZeroI64::new(1); "i64")]
+    #[test_case(NonZeroI128::new(1); "i128")]
+    #[allow(clippy::needless_pass_by_value)]
+    fn test_non_zero<T: ToBoundedStatic>(t: T) {
+        ensure_static(t.to_static());
     }
 
     #[test]