apollo_integration_tests: clean old todos

Author: ArniStarkware

crates/apollo_integration_tests/tests/common/mod.rs

@@ -1,6 +1,7 @@
 // Each test module is compiled as a separate crate, and all can declare the common module.
 // This means that any peace of code in this module that is not used by *all* test modules will be
 // identified as unused code by clippy (for one of the crates).
+// TODO(Arni): Fix this.
 #![allow(dead_code)]
 
 use std::time::Duration;
@@ -203,7 +204,7 @@ fn assert_full_blocks_flow(recorder_handle: &PrometheusHandle, expecting_full_bl
     }
     // Just because we don't expect full blocks, doesn't mean we should assert that the metric is 0.
     // It is possible that a block is filled, no need to assert that this is not the case.
-    // TODO(AlonH): In the `else` case, assert that some block closed due to time.
+    // TODO(Arni): In the `else` case, assert that some block closed due to time.
 }
 
 fn assert_on_number_of_reverted_transactions_flow(

crates/apollo_integration_tests/tests/end_to_end_flow_test.rs

@@ -15,7 +15,6 @@ use crate::common::{end_to_end_flow, test_single_tx, EndToEndFlowArgs, TestScena
 
 mod common;
 
-// TODO(Meshi): Fail the test if no class have migrated.
 /// Number of threads is 3 = Num of sequencer + 1 for the test thread.
 #[tokio::test(flavor = "multi_thread", worker_threads = 3)]
 async fn test_end_to_end_flow() {
@@ -31,7 +30,6 @@ fn create_test_scenarios() -> Vec<TestScenario> {
     vec![
         // This block should be the first to be tested, as the addition of L1 handler transaction
         // does not work smoothly with the current architecture of the test.
-        // TODO(Arni): Fix this. Move the L1 handler to be not the first block.
         TestScenario {
             create_rpc_txs_fn: |_| vec![],
             create_l1_to_l2_messages_args_fn: create_l1_to_l2_message_args,