From c41c343492a0d1101532926a0d1a9e9803405ef4 Mon Sep 17 00:00:00 2001 From: Roman Krasiuk Date: Thu, 16 Feb 2023 10:27:25 +0200 Subject: [PATCH] fix: convert fuzz test names to snake case (#1390) --- Cargo.lock | 1 + crates/storage/codecs/derive/Cargo.toml | 1 + crates/storage/codecs/derive/src/compact/generator.rs | 7 +++++-- crates/storage/codecs/derive/src/compact/mod.rs | 6 +++--- 4 files changed, 10 insertions(+), 5 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 4bcd594fb..7fcee2731 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -724,6 +724,7 @@ checksum = "67ba02a97a2bd10f4b59b25c7973101c79642302776489e030cd13cdab09ed15" name = "codecs-derive" version = "0.1.0" dependencies = [ + "convert_case 0.6.0", "parity-scale-codec", "proc-macro2 1.0.50", "quote 1.0.23", diff --git a/crates/storage/codecs/derive/Cargo.toml b/crates/storage/codecs/derive/Cargo.toml index 98f73635b..66c2a56ca 100644 --- a/crates/storage/codecs/derive/Cargo.toml +++ b/crates/storage/codecs/derive/Cargo.toml @@ -21,6 +21,7 @@ proc-macro = true proc-macro2 = "1.0.47" quote = "1.0" syn = { version = "1.0", features = ["full"] } +convert_case = "0.6.0" # codecs serde = { version = "1.0.*", default-features = false } diff --git a/crates/storage/codecs/derive/src/compact/generator.rs b/crates/storage/codecs/derive/src/compact/generator.rs index dcc71924d..caec5b929 100644 --- a/crates/storage/codecs/derive/src/compact/generator.rs +++ b/crates/storage/codecs/derive/src/compact/generator.rs @@ -1,6 +1,7 @@ //! Code generator for the `Compact` trait. use super::*; +use convert_case::{Case, Casing}; /// Generates code to implement the `Compact` trait for a data type. pub fn generate_from_to(ident: &Ident, fields: &FieldList) -> TokenStream2 { @@ -9,8 +10,10 @@ pub fn generate_from_to(ident: &Ident, fields: &FieldList) -> TokenStream2 { let to_compact = generate_to_compact(fields, ident); let from_compact = generate_from_compact(fields, ident); - let fuzz = format_ident!("fuzz_test_{ident}"); - let test = format_ident!("fuzz_{ident}"); + let snake_case_ident = ident.to_string().to_case(Case::Snake); + + let fuzz = format_ident!("fuzz_test_{snake_case_ident}"); + let test = format_ident!("fuzz_{snake_case_ident}"); // Build function quote! { diff --git a/crates/storage/codecs/derive/src/compact/mod.rs b/crates/storage/codecs/derive/src/compact/mod.rs index 4cc254a95..fb003ccc4 100644 --- a/crates/storage/codecs/derive/src/compact/mod.rs +++ b/crates/storage/codecs/derive/src/compact/mod.rs @@ -238,15 +238,15 @@ mod tests { #[cfg(test)] #[allow(dead_code)] #[test_fuzz::test_fuzz] - fn fuzz_test_TestStruct(obj: TestStruct) { + fn fuzz_test_test_struct(obj: TestStruct) { let mut buf = vec![]; let len = obj.clone().to_compact(&mut buf); let (same_obj, buf) = TestStruct::from_compact(buf.as_ref(), len); assert_eq!(obj, same_obj); } #[test] - pub fn fuzz_TestStruct() { - fuzz_test_TestStruct(TestStruct::default()) + pub fn fuzz_test_struct() { + fuzz_test_test_struct(TestStruct::default()) } impl Compact for TestStruct { fn to_compact(self, buf: &mut impl bytes::BufMut) -> usize {