From 2eda166aa1c7291b7d06cd73019bb0ac610e16b7 Mon Sep 17 00:00:00 2001
From: Thomas Coratger <thomas.coratger@gmail.com>
Date: Fri, 11 Oct 2024 12:32:29 +0200
Subject: [PATCH] rm useless #[allow(deprecated)]

---
 crates/dojo-test-utils/src/sequencer.rs     | 2 --
 crates/katana/rpc/rpc/src/dev.rs            | 1 -
 crates/katana/rpc/rpc/src/starknet/read.rs  | 1 -
 crates/katana/rpc/rpc/src/starknet/trace.rs | 2 --
 4 files changed, 6 deletions(-)

diff --git a/crates/dojo-test-utils/src/sequencer.rs b/crates/dojo-test-utils/src/sequencer.rs
index d2fe8b4f75..87a795812c 100644
--- a/crates/dojo-test-utils/src/sequencer.rs
+++ b/crates/dojo-test-utils/src/sequencer.rs
@@ -35,7 +35,6 @@ pub struct TestSequencer {
 }
 
 impl TestSequencer {
-    #[allow(deprecated)]
     pub async fn start(config: Config) -> Self {
         let handle = katana_node::build(config)
             .await
@@ -81,7 +80,6 @@ impl TestSequencer {
         &self,
         index: usize,
     ) -> SingleOwnerAccount<JsonRpcClient<HttpTransport>, LocalWallet> {
-        #[allow(deprecated)]
         let accounts: Vec<_> =
             self.handle.node.backend.chain_spec.genesis.accounts().collect::<_>();
 
diff --git a/crates/katana/rpc/rpc/src/dev.rs b/crates/katana/rpc/rpc/src/dev.rs
index 8dfbe3e334..9c48eb1cf0 100644
--- a/crates/katana/rpc/rpc/src/dev.rs
+++ b/crates/katana/rpc/rpc/src/dev.rs
@@ -92,7 +92,6 @@ impl<EF: ExecutorFactory> DevApiServer for DevApi<EF> {
         Ok(())
     }
 
-    #[allow(deprecated)]
     async fn predeployed_accounts(&self) -> Result<Vec<Account>, Error> {
         Ok(self.backend.chain_spec.genesis.accounts().map(|e| Account::new(*e.0, e.1)).collect())
     }
diff --git a/crates/katana/rpc/rpc/src/starknet/read.rs b/crates/katana/rpc/rpc/src/starknet/read.rs
index 7b7642d2c2..e30715544c 100644
--- a/crates/katana/rpc/rpc/src/starknet/read.rs
+++ b/crates/katana/rpc/rpc/src/starknet/read.rs
@@ -470,7 +470,6 @@ impl<EF: ExecutorFactory> StarknetApiServer for StarknetApi<EF> {
 
             // If the node is run with transaction validation disabled, then we should not validate
             // transactions when estimating the fee even if the `SKIP_VALIDATE` flag is not set.
-            #[allow(deprecated)]
             let should_validate = !(skip_validate
                 || this.inner.backend.executor_factory.execution_flags().skip_validate);
             let flags = katana_executor::SimulationFlag {
diff --git a/crates/katana/rpc/rpc/src/starknet/trace.rs b/crates/katana/rpc/rpc/src/starknet/trace.rs
index 77426865ad..edfa21c95a 100644
--- a/crates/katana/rpc/rpc/src/starknet/trace.rs
+++ b/crates/katana/rpc/rpc/src/starknet/trace.rs
@@ -64,13 +64,11 @@ impl<EF: ExecutorFactory> StarknetApi<EF> {
 
         // If the node is run with transaction validation disabled, then we should not validate
         // even if the `SKIP_VALIDATE` flag is not set.
-        #[allow(deprecated)]
         let should_validate = !(simulation_flags.contains(&SimulationFlag::SkipValidate)
             || self.inner.backend.executor_factory.execution_flags().skip_validate);
 
         // If the node is run with fee charge disabled, then we should disable charing fees even
         // if the `SKIP_FEE_CHARGE` flag is not set.
-        #[allow(deprecated)]
         let should_skip_fee = !(simulation_flags.contains(&SimulationFlag::SkipFeeCharge)
             || self.inner.backend.executor_factory.execution_flags().skip_fee_transfer);