Define clippy restrictions for the crate

This way, they don't have to be specified from the CLI each time.

Signed-off-by: Sahas Subramanian <sahas.subramanian@proton.me>

Author: shsms

src/lib.rs

@@ -3,6 +3,18 @@
 
 //! High-level interface for the Microgrid API.
 
+#![cfg_attr(
+    not(test),
+    deny(
+        clippy::unwrap_used,
+        clippy::expect_used,
+        clippy::panic,
+        clippy::unimplemented,
+        clippy::todo,
+        clippy::unreachable,
+    )
+)]
+
 mod bounds;
 pub use bounds::Bounds;
 

src/logical_meter/formula/async_formula.rs

@@ -157,7 +157,11 @@ async fn synchronize_receivers<Q: Quantity>(
     for item in formula_items.iter_mut() {
         match item.recv().await {
             Ok(vv) => latest.push(vv),
-            Err(_) => todo!(),
+            Err(e) => {
+                return Err(crate::Error::internal(format!(
+                    "Failed to receive from formula operand: {e}"
+                )));
+            }
         };
     }
 

src/microgrid/telemetry_tracker/battery_pool_telemetry_tracker.rs

@@ -113,12 +113,10 @@ impl BatteryPoolTelemetryTracker {
             if !group_batteries.is_subset(&self.component_ids) {
                 let e = format!(
                     concat!(
-                        "Inverter {} is connected to batteries {:?} which are not all in the ",
-                        "requested component IDs {:?}"
+                        "Inverters {:?} are connected to batteries {:?} which are not all in ",
+                        "the requested component IDs {:?}"
                     ),
-                    group_inverters.iter().next().unwrap(),
-                    group_batteries,
-                    self.component_ids
+                    group_inverters, group_batteries, self.component_ids
                 );
 
                 tracing::error!("{}", e);