Adding todos

examples/chip-tool/commands/common/Commands.cpp

@@ -38,6 +38,7 @@ void Commands::Register(const char * clusterName, commands_list commandsList)
     }
 }
 
+// TODO: Remove this work around due to crashes when storage is accessed from multiple threads and passed around
 static PersistentStorage gStorage;
 int Commands::Run(NodeId localId, NodeId remoteId, int argc, char ** argv)
 {

examples/chip-tool/commands/tests/TestCommand.cpp

@@ -55,6 +55,7 @@ CHIP_ERROR TestCommand::Run(PersistentStorage & storage, NodeId localId, NodeId
 
     // Give some time for all the pending messages to flush before shutting down
     // Note: This is working around racy code in message queues during shutdown
+    // TODO: Remove this workaround once we understand the message queue and shutdown race
     test_os_sleep_ms(1000);
     mCommissioner.Shutdown();