feat: fixed transaction conflicts in discovery

Author: aryanjassal

src/discovery/Discovery.ts

@@ -309,6 +309,7 @@ class Discovery {
       }
     }
     await Promise.all(taskPromises);
+    this.logger.info(`Stopped all tasks for ${this.constructor.name}`);
     this.taskManager.deregisterHandler(this.discoverVertexHandlerId);
     this.taskManager.deregisterHandler(this.checkRediscoveryHandlerId);
     this.logger.info(`Stopped ${this.constructor.name}`);

src/nodes/NodeConnection.ts

@@ -308,7 +308,7 @@ class NodeConnection {
       logger: logger.getChild(RPCClient.name),
     });
     if (validatedNodeId == null) {
-      never(`connection validated but no valid NodeId was returned`);
+      never('connection validated but no valid NodeId was returned');
     }
     // Obtaining remote node ID from certificate chain. It should always exist in the chain if validated.
     //  This may de different from the NodeId we validated it as if it renewed at some point.