From 41a4675556fbbca75d76627e1f9fe4bf3787012f Mon Sep 17 00:00:00 2001
From: Itay Tsabary <itayt@starkware.co>
Date: Wed, 5 Jun 2024 13:12:42 +0300
Subject: [PATCH] chore: cleanup and extend todo comments

commit-id:16da86a2
---
 crates/mempool_infra/tests/common/mod.rs                   | 2 ++
 crates/mempool_infra/tests/component_server_client_test.rs | 6 +-----
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/crates/mempool_infra/tests/common/mod.rs b/crates/mempool_infra/tests/common/mod.rs
index 442cde9e2..2efe4958e 100644
--- a/crates/mempool_infra/tests/common/mod.rs
+++ b/crates/mempool_infra/tests/common/mod.rs
@@ -3,6 +3,8 @@ use async_trait::async_trait;
 type ValueA = u32;
 type ValueB = u8;
 
+// TODO(Tsabary): add more messages / functions to the components.
+
 #[async_trait]
 pub(crate) trait ComponentATrait: Send + Sync {
     async fn a_get_value(&self) -> ValueA;
diff --git a/crates/mempool_infra/tests/component_server_client_test.rs b/crates/mempool_infra/tests/component_server_client_test.rs
index ce42aea66..276eb0bb2 100644
--- a/crates/mempool_infra/tests/component_server_client_test.rs
+++ b/crates/mempool_infra/tests/component_server_client_test.rs
@@ -32,16 +32,12 @@ impl ComponentA {
     }
 }
 
-// todo find which of these derives is needed
-// todo add more messages
-// todo send messages from b to a
+// TODO(Tsabary): send messages from component b to component a.
 
-#[derive(Copy, Clone, Debug, PartialEq, Eq)]
 pub enum ComponentARequest {
     AGetValue,
 }
 
-#[derive(Copy, Clone, Debug, PartialEq, Eq)]
 pub enum ComponentAResponse {
     Value(ValueA),
 }