Addressed comments

Author: m-sz

crates/e2e/tests/e2e/submission.rs

@@ -310,17 +310,6 @@ async fn test_execute_same_sell_and_buy_token(web3: Web3) {
     let initial_balance = token.balanceOf(trader.address()).call().await.unwrap();
     assert_eq!(initial_balance, eth(10));
 
-    // let quote_sell_amount = to_wei(2);
-    // let order_buy_amount = to_wei(1);
-    // let order = OrderCreation {
-    //     sell_token: token.address().into_legacy(),
-    //     sell_amount: order_sell_amount,
-    //     buy_token: token.address().into_legacy(),
-    //     buy_amount: order_buy_amount,
-    //     valid_to: model::time::now_in_epoch_seconds() + 300,
-    //     kind: OrderKind::Sell,
-    //     ..Default::default()
-    // }
     let order = OrderCreation {
         kind: OrderKind::Sell,
         sell_token: token.address().into_legacy(),

crates/solvers/src/boundary/baseline.rs

@@ -48,7 +48,6 @@ impl<'a> Solver<'a> {
         max_hops: usize,
     ) -> Option<solver::Route<'a>> {
         if request.sell.token == request.buy.token {
-            tracing::info!("Sell=Buy, returning empty route");
             return Some(solver::Route::new(vec![]));
         }
         let candidates = self.base_tokens.path_candidates_with_hops(

crates/solvers/src/domain/solver.rs

@@ -222,6 +222,11 @@ impl Inner {
                     input = route.input().expect("route is not empty");
                     output = route.output().expect("route is not empty");
                 } else {
+                    // Route is empty in case of sell and buy tokens being the same, as there
+                    // is no need to figure out the liquidity for such pair.
+                    //
+                    // The input and output of the solution can be set directly to the
+                    // respective sell and buy tokens 1 to 1.
                     interactions = Vec::default();
                     gas = eth::Gas(U256::zero()) + self.solution_gas_offset;