Minor logging and scope changes

network-parser/src/helpers.rs

@@ -335,7 +335,6 @@ where
     D: Deserializer<'de>,
 {
     let str_sequence = String::deserialize(deserializer)?;
-    println!("str_sequence {}", str_sequence);
     if str_sequence.is_empty() {
         Ok(None)
     } else {

simulator/src/core_types/graph.rs

@@ -133,11 +133,11 @@ impl Graph {
         }
     }
 
-    /// Discard the given node from the graph
-    pub(crate) fn remove_node(&mut self, node: &ID) {
+    /// Discard the given node and its edges from the graph
+    pub fn remove_node(&mut self, node: &ID) {
         self.nodes.retain(|n| *n.id != *node);
-        for n in self.get_node_ids() {
-            self.remove_edge(node, &n);
+        for n in self.get_outedges(node) {
+            self.remove_edge(node, &n.destination);
         }
     }
 

simulator/src/stats/adversaries.rs

@@ -5,14 +5,14 @@ use crate::{
 };
 
 #[cfg(not(test))]
-use log::{error, info, warn};
+use log::{info, warn};
 use rayon::prelude::*;
 use std::{
     collections::HashMap,
     sync::{Arc, Mutex},
 };
 #[cfg(test)]
-use std::{println as info, println as warn, println as error};
+use std::{println as info, println as warn};
 
 impl Simulation {
     pub(crate) fn eval_adversaries(&mut self, run_all: bool) {
@@ -25,7 +25,7 @@ impl Simulation {
                 vec![1, 2, 3, 4, 5, 10, 12, 15, 20]
             };
         if !number_of_adversaries.is_empty() && self.adversary_selection.is_empty() {
-            error!("Aborting adversary evaluation as no strategy was passed.");
+            warn!("Aborting adversary evaluation as no strategy was passed.");
             return;
         }
         let selected_adversaries =