Disabled optical unit tests moved to Dummy.

Fixed tests for group. All tests pass again.

src/nodes/dummy.rs

@@ -26,6 +26,11 @@ pub struct Dummy {
     name: String,
 }
 
+impl Default for Dummy {
+    fn default() -> Self {
+        Self { is_inverted: Default::default(), name: String::from("dummy") }
+    }
+}
 impl Dummy {
     /// Creates a new [`Dummy`] with a given name.
     pub fn new(name: &str) -> Self {
@@ -39,6 +44,9 @@ impl Optical for Dummy {
     fn set_name(&mut self, name: &str) {
         self.name = name.to_owned()
     }
+    fn name(&self) -> &str {
+        &self.name
+    }
     /// Returns "dummy" as node type.
     fn node_type(&self) -> &str {
         "dummy"
@@ -70,6 +78,60 @@ impl Optical for Dummy {
     fn set_inverted(&mut self, inverted: bool) {
         self.is_inverted = inverted;
     }
+    fn inverted(&self) -> bool {
+        self.is_inverted
+    }
 }
 
 impl Dottable for Dummy {}
+
+#[cfg(test)]
+mod test {
+    use super::*;
+    #[test]
+    fn new() {
+         let node = Dummy::new("Test");
+         assert_eq!(node.name, "Test");
+         assert_eq!(node.inverted(), false);
+    }
+    #[test]
+    fn default() {
+         let node = Dummy::default();
+         assert_eq!(node.name, "dummy");
+         assert_eq!(node.inverted(), false);
+    }
+    #[test]
+    fn set_name() {
+        let mut node = Dummy::default();
+        node.set_name("Test1");
+        assert_eq!(node.name, "Test1")
+    }
+    #[test]
+    fn name() {
+        let mut node = Dummy::default();
+        node.set_name("Test1");
+        assert_eq!(node.name(), "Test1")
+    }
+    #[test]
+    fn set_inverted() {
+        let mut node = Dummy::default();
+        node.set_inverted(true);
+        assert_eq!(node.is_inverted, true)
+    }
+    #[test]
+    fn inverted() {
+        let mut node = Dummy::default();
+        node.set_inverted(true);
+        assert_eq!(node.inverted(), true)
+    }
+    #[test]
+    fn is_detector() {
+        let node = Dummy::default();
+        assert_eq!(node.is_detector(), false);
+    }
+    #[test]
+    fn node_type() {
+        let node = Dummy::default();
+        assert_eq!(node.node_type(), "dummy");
+    }
+}

src/nodes/group.rs

@@ -733,7 +733,7 @@ mod test {
     fn map_input_port_half_connected_nodes() {
         let mut og = NodeGroup::new();
         let sn1_i = og.add_node(Dummy::new("n1"));
-        let sn2_i = og.add_node(Dummy::new("n2"));
+        let sn2_i = og.add_node(BeamSplitter::default());
         og.connect_nodes(sn1_i, "rear", sn2_i, "input1").unwrap();
 
         // node port already internally connected

src/nodes/source.rs

@@ -42,7 +42,7 @@ impl Source {
     /// nodes::Source,
     /// spectrum::create_he_ne_spectrum};
     ///
-    /// let source=Source::new(LightData::Energy(DataEnergy {spectrum: create_he_ne_spectrum(1.0)}));
+    /// let source=Source::new("My Source", LightData::Energy(DataEnergy {spectrum: create_he_ne_spectrum(1.0)}));
     /// ```
     pub fn new(name: &str, light: LightData) -> Self {
         Source {

src/optical.rs

@@ -86,43 +86,6 @@ impl Debug for dyn Optical {
 
 #[cfg(test)]
 mod test {
-    // use crate::nodes::{Detector, Dummy};
-    // #[test]
-    // fn new() {
-    //     let node = OpticNode::new("Test", Dummy::default());
-    //     assert_eq!(node.name, "Test");
-    //     assert_eq!(node.inverted(), false);
-    // }
-    // #[test]
-    // fn set_name() {
-    //     let mut node = OpticNode::new("Test", Dummy::default());
-    //     node.set_name("Test2".into());
-    //     assert_eq!(node.name, "Test2")
-    // }
-    // #[test]
-    // fn name() {
-    //     let node = OpticNode::new("Test", Dummy::default());
-    //     assert_eq!(node.name(), "Test")
-    // }
-    // #[test]
-    // fn set_inverted() {
-    //     let mut node = OpticNode::new("Test", Dummy::default());
-    //     node.set_inverted(true);
-    //     assert_eq!(node.inverted(), true)
-    // }
-    // #[test]
-    // fn inverted() {
-    //     let mut node = OpticNode::new("Test", Dummy::default());
-    //     node.set_inverted(true);
-    //     assert_eq!(node.inverted(), true)
-    // }
-    // #[test]
-    // fn is_detector() {
-    //     let node = OpticNode::new("Test", Dummy::default());
-    //     assert_eq!(node.is_detector(), false);
-    //     let node = OpticNode::new("Test", Detector::default());
-    //     assert_eq!(node.is_detector(), true)
-    // }
     // #[test]
     // #[ignore]
     // fn to_dot() {