unit test fix

scripts/clippy_fix_fmt.sh

@@ -20,7 +20,7 @@ echo
 
 echo 'Fixing clippy (might need a "git commit" and a rerun, if "cargo fix" changed the source)'
 
-cargo +nightly clippy --fix --tests --examples --benches --all-features --allow-dirty --allow-staged
+cargo +nightly clippy --fix --examples --benches --all-features --allow-dirty --allow-staged
 
 echo "[+] Done fixing clippy"
 echo

src/cli/ziggy.rs

@@ -171,10 +171,7 @@ impl ZiggyConfig {
         f.render_widget(hello_world, chunks[0]);
 
         let ziggy_output = Paragraph::new(Line::from(
-            app.ziggy_output
-                .iter()
-                .map(|line| Span::raw(line))
-                .collect::<Vec<_>>(),
+            app.ziggy_output.iter().map(Span::raw).collect::<Vec<_>>(),
         ))
         .scroll((app.ziggy_output.len() as u16, 0))
         .block(Block::default().borders(Borders::ALL).title("Ziggy Output"));

src/contract/custom/mod.rs

@@ -1,4 +1,4 @@
-mod custom;
+pub mod custom;
 
 use sp_core::storage::Storage;
 pub struct Preferences {}

src/cover/coverage.rs

@@ -126,11 +126,12 @@ impl InputCoverage {
             );
         }
 
+        #[allow(clippy::unnecessary_cast)]
         /// We assume that the instrumentation will never insert more than
         /// `2_000` artificial branches This value should be big enough
         /// to handle most of smart-contract, even the biggest
         seq_macro::seq!(x in 0..= 2_000 {
-            if flattened_cov.contains(&(x as u64)) {
+            if flattened_cov.contains(&(x)) {
                 let _ = black_box(x + 1);
             }
         });

src/fuzzer/fuzz.rs

@@ -321,29 +321,55 @@ fn check_invariants(
 
 #[cfg(test)]
 mod tests {
-    use std::path::Path;
-
     use super::*;
 
+    use std::{
+        path::Path,
+        process::{
+            Command,
+            Stdio,
+        },
+        sync::Once,
+    };
+
+    static BUILD: Once = Once::new();
+
+    pub fn build() {
+        println!("executing build.sh");
+        BUILD.call_once(|| {
+            let _status = Command::new("bash")
+                .current_dir("sample") // Change to the 'sample' directory
+                .arg("build.sh")
+                .stdout(Stdio::null())
+                .stderr(Stdio::null())
+                .status()
+                .expect("failed to execute process");
+        });
+    }
     #[test]
     fn test_parse_input() {
+        build();
         let metadata_path = Path::new("sample/dns/target/ink/dns.json");
         let transcoder = Mutex::new(
             ContractMessageTranscoder::load(metadata_path)
-                .expect("Failed to load ContractMessageTranscoder"),
+                .expect("Failed to load `ContractMessageTranscoder`"),
         );
 
         let encoded_bytes =
             hex::decode("229b553f9400000000000000000027272727272727272700002727272727272727272727")
                 .expect("Failed to decode hex string");
 
-        let hex = transcoder
-            .lock()
-            .unwrap()
-            .decode_contract_message(&mut &encoded_bytes[..])
-            .expect("Failed to decode contract message");
+        assert!(
+            transcoder
+                .lock()
+                .unwrap()
+                .decode_contract_message(&mut &encoded_bytes[..])
+                .is_ok(),
+            "Failed to decode contract message"
+        );
 
         let binding = transcoder.lock().unwrap();
         let messages = binding.metadata().spec().messages();
+        assert!(!messages.is_empty(), "Ther should be some messages here");
     }
 }

src/instrumenter/instrumentation.rs

@@ -27,6 +27,7 @@ use syn::{
 use walkdir::WalkDir;
 
 /// The objective of this `struct` is to assist Phink in instrumenting ink!
+///
 /// smart contracts. In a fuzzing context, instrumenting a smart contract
 /// involves modifying the target (i.e., the WASM blob), for example, by adding
 /// additional code to branches to obtain a coverage map during the execution of

tests/cli_fuzz_integration_test.rs

@@ -68,7 +68,6 @@ mod tests {
 
                     // The content of selectors.dict should be
                     //
-                    //
                     // # Dictionary file for selectors
                     // # Lines starting with '#' and empty lines are ignored.
                     // delimiter="********"

tests/shared/mod.rs

@@ -33,6 +33,7 @@ use std::{
         Instant,
     },
 };
+#[allow(clippy::too_long_first_doc_paragraph)]
 
 pub const DEFAULT_TEST_PHINK_TOML: &str = "phink_temp_test.toml";