Part 2 of updates

Author: sveinse

canopen/node/local.py

@@ -51,7 +51,6 @@ def associate_network(self, network: canopen.network.Network):
         network.subscribe(0, self.nmt.on_command)
 
     def remove_network(self) -> None:
-        # Make it safe to call this method multiple times
         if not self.has_network():
             return
         self.network.unsubscribe(self.sdo.rx_cobid, self.sdo.on_request)

canopen/node/remote.py

@@ -66,7 +66,6 @@ def associate_network(self, network: canopen.network.Network):
         network.subscribe(0, self.nmt.on_command)
 
     def remove_network(self) -> None:
-        # Make it safe to call this method multiple times
         if not self.has_network():
             return
         for sdo in self.sdo_channels:

test/test_node.py

@@ -3,8 +3,7 @@
 
 import canopen
 import canopen.network
-
-from .util import SAMPLE_EDS
+import canopen.objectdictionary
 
 
 def count_subscribers(network: canopen.Network) -> int:
@@ -23,17 +22,16 @@ def setUpClass(cls):
         cls.network.NOTIFIER_SHUTDOWN_TIMEOUT = 0.0
         cls.network.connect(interface="virtual")
 
-        cls.node = canopen.LocalNode(2, SAMPLE_EDS)
+        cls.node = canopen.LocalNode(2, canopen.objectdictionary.ObjectDictionary())
 
     @classmethod
     def tearDownClass(cls):
         cls.network.disconnect()
 
     def test_associate_network(self):
 
-        # Need to store the number of subscribers before associating because
-        # the current network implementation automatically adds subscribers
-        # to the list
+        # Need to store the number of subscribers before associating because the
+        # network implementation automatically adds subscribers to the list
         n_subscribers = count_subscribers(self.network)
 
         # Associating the network with the local node
@@ -75,17 +73,16 @@ def setUpClass(cls):
         cls.network.NOTIFIER_SHUTDOWN_TIMEOUT = 0.0
         cls.network.connect(interface="virtual")
 
-        cls.node = canopen.RemoteNode(2, SAMPLE_EDS)
+        cls.node = canopen.RemoteNode(2, canopen.objectdictionary.ObjectDictionary())
 
     @classmethod
     def tearDownClass(cls):
         cls.network.disconnect()
 
     def test_associate_network(self):
 
-        # Need to store the number of subscribers before associating because
-        # the current network implementation automatically adds subscribers
-        # to the list
+        # Need to store the number of subscribers before associating because the
+        # network implementation automatically adds subscribers to the list
         n_subscribers = count_subscribers(self.network)
 
         # Associating the network with the local node