Fix clippy lints

Phoenix-Protocol-Group · Jan 17, 2024 · 5e439b2 · 5e439b2
1 parent fc86e04
commit 5e439b2
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 7 deletions.
diff --git a/contracts/pool/src/tests/config.rs b/contracts/pool/src/tests/config.rs
@@ -1,5 +1,5 @@
 extern crate std;
-use soroban_sdk::{testutils::Address as _, Address, Env, String};
+use soroban_sdk::{testutils::Address as _, Address, Env};
 
 use super::setup::{deploy_liquidity_pool_contract, deploy_token_contract};
 use crate::storage::{Config, PairType};

diff --git a/contracts/pool/src/tests/liquidity.rs b/contracts/pool/src/tests/liquidity.rs
@@ -5,7 +5,7 @@ use pretty_assertions::assert_eq;
 use soroban_sdk::{
     symbol_short,
     testutils::{Address as _, AuthorizedFunction, AuthorizedInvocation},
-    Address, Env, IntoVal, String, Symbol,
+    Address, Env, IntoVal, Symbol,
 };
 
 use super::setup::{deploy_liquidity_pool_contract, deploy_token_contract};

diff --git a/contracts/pool/src/tests/setup.rs b/contracts/pool/src/tests/setup.rs
@@ -1,4 +1,4 @@
-use soroban_sdk::{testutils::Address as _, Address, BytesN, Env, String};
+use soroban_sdk::{testutils::Address as _, Address, BytesN, Env};
 
 use crate::{
     contract::{LiquidityPool, LiquidityPoolClient},
@@ -41,7 +41,7 @@ pub fn deploy_liquidity_pool_contract<'a>(
     let stake_wasm_hash = install_stake_wasm(env);
     let fee_recipient = fee_recipient
         .into()
-        .unwrap_or_else(|| Address::generate(&env));
+        .unwrap_or_else(|| Address::generate(env));
     let max_allowed_slippage = max_allowed_slippage_bps.into().unwrap_or(5_000); // 50% if not specified
     let max_allowed_spread = max_allowed_spread_bps.into().unwrap_or(500); // 5% if not specified
     let share_token_decimals = 7u32;

diff --git a/contracts/pool/src/tests/stake_deployment.rs b/contracts/pool/src/tests/stake_deployment.rs
@@ -1,6 +1,6 @@
 extern crate std;
 use phoenix::utils::{StakeInitInfo, TokenInitInfo};
-use soroban_sdk::{testutils::Address as _, Address, Env, String};
+use soroban_sdk::{testutils::Address as _, Address, Env};
 
 use super::setup::{deploy_liquidity_pool_contract, deploy_token_contract};
 use crate::{
@@ -80,7 +80,7 @@ fn second_pool_deployment_should_fail() {
 
     let mut admin1 = Address::generate(&env);
     let mut admin2 = Address::generate(&env);
-    let mut user = Address::generate(&env);
+    let user = Address::generate(&env);
 
     let mut token1 = deploy_token_contract(&env, &admin1);
     let mut token2 = deploy_token_contract(&env, &admin2);

diff --git a/contracts/pool/src/tests/swap.rs b/contracts/pool/src/tests/swap.rs
@@ -3,7 +3,7 @@ use pretty_assertions::assert_eq;
 use soroban_sdk::{
     symbol_short,
     testutils::{Address as _, AuthorizedFunction, AuthorizedInvocation},
-    Address, Env, IntoVal, String,
+    Address, Env, IntoVal,
 };
 use test_case::test_case;
 

diff --git a/contracts/token/src/contract.rs b/contracts/token/src/contract.rs
@@ -21,6 +21,7 @@ pub struct Token;
 
 #[contractimpl]
 impl Token {
+    #[allow(dead_code)]
     pub fn initialize(e: Env, admin: Address, decimal: u32, name: String, symbol: String) {
         #[allow(dead_code)]
         if has_administrator(&e) {

diff --git a/contracts/token/src/metadata.rs b/contracts/token/src/metadata.rs
@@ -16,6 +16,7 @@ pub fn read_symbol(e: &Env) -> String {
     util.metadata().get_metadata().symbol
 }
 
+#[allow(dead_code)]
 pub fn write_metadata(e: &Env, metadata: TokenMetadata) {
     let util = TokenUtils::new(e);
     util.metadata().set_metadata(&metadata);