copy (#340)

Co-authored-by: yacovm <yacovm@users.noreply.github.com>

Author: samliok

testutil/comm.go

@@ -62,10 +62,9 @@ func (c *TestComm) Send(msg *simplex.Message, destination simplex.NodeID) {
 
 	// cannot send if either [from] or [destination] is not connected
 	if c.net.IsDisconnected(destination) || c.net.IsDisconnected(c.from) {
-
 		for _, instance := range c.net.GetInstances() {
 			if bytes.Equal(instance.E.ID, destination) {
-				instance.l.Info("Node is disconnect not sending message")
+				instance.l.Info("Node is disconnected not sending message")
 			}
 		}
 
@@ -76,7 +75,7 @@ func (c *TestComm) Send(msg *simplex.Message, destination simplex.NodeID) {
 
 	for _, instance := range c.net.GetInstances() {
 		if bytes.Equal(instance.E.ID, c.from) {
-			instance.l.Info("Enqueing message")
+			instance.l.Info("Enqueuing message")
 			continue
 		}
 		if bytes.Equal(instance.E.ID, destination) {

testutil/network.go

@@ -119,7 +119,9 @@ func (b *BasicInMemoryNetwork) GetInstances() []*BasicNode {
 	b.lock.RLock()
 	defer b.lock.RUnlock()
 
-	return b.instances
+	nodes := make([]*BasicNode, len(b.instances))
+	copy(nodes, b.instances)
+	return nodes
 }
 
 func (b *BasicInMemoryNetwork) ReplaceNode(node *BasicNode) {