Clippy and format to fix CI

framework/runner/src/cli.rs

@@ -43,9 +43,15 @@ pub struct WindTunnelScenarioCli {
 
 fn parse_agent_behaviour(s: &str) -> anyhow::Result<(String, usize)> {
     let mut parts = s.split(':');
-    let name = parts.next().map(|s| s.to_string()).ok_or(anyhow::anyhow!("No name specified for behaviour"))?;
-
-    let count = parts.next().and_then(|s| s.parse::<usize>().ok()).unwrap_or(1);
+    let name = parts
+        .next()
+        .map(|s| s.to_string())
+        .ok_or(anyhow::anyhow!("No name specified for behaviour"))?;
+
+    let count = parts
+        .next()
+        .and_then(|s| s.parse::<usize>().ok())
+        .unwrap_or(1);
 
     Ok((name, count))
 }

framework/runner/src/definition.rs

@@ -192,7 +192,7 @@ impl<RV: UserValuesConstraint, V: UserValuesConstraint> ScenarioDefinitionBuilde
         let unknown_behaviours = requested_behaviours
             .difference(&registered_behaviours)
             .collect::<Vec<&String>>();
-        if unknown_behaviours.len() > 0 {
+        if !unknown_behaviours.is_empty() {
             return Err(anyhow::anyhow!(
                 "Unknown behaviours requested: {:?}",
                 unknown_behaviours

framework/runner/src/run.rs

@@ -36,7 +36,7 @@ pub fn run<RV: UserValuesConstraint, V: UserValuesConstraint>(
     }
 
     // After the setup has run, and if this is a time bounded scenario, then we need to take additional actions
-    if let Some(duration) = definition.duration_s.clone() {
+    if let Some(duration) = definition.duration_s {
         if !definition.no_progress {
             // If the scenario is time bounded then start the progress monitor to show the user how long is left
             start_progress(
@@ -68,7 +68,10 @@ pub fn run<RV: UserValuesConstraint, V: UserValuesConstraint>(
         let runner_context = runner_context.clone();
 
         let setup_agent_fn = definition.setup_agent_fn;
-        let agent_behaviour_fn = definition.agent_behaviour.get(&assigned_behaviours[agent_index]).cloned();
+        let agent_behaviour_fn = definition
+            .agent_behaviour
+            .get(&assigned_behaviours[agent_index])
+            .cloned();
         let teardown_agent_fn = definition.teardown_agent_fn;
 
         // For us to check if the agent should shut down between behaviour cycles

framework/runner/src/shutdown.rs

@@ -1,10 +1,10 @@
 use std::{borrow::BorrowMut, sync::Arc};
 
+use tokio::sync::Mutex;
 use tokio::{
     signal,
     sync::broadcast::{Receiver, Sender},
 };
-use tokio::sync::Mutex;
 
 #[derive(Debug, Clone)]
 pub(crate) struct ShutdownHandle {