Test pass, clippy happy :)

osmosis-labs · May 13, 2024 · 4938265 · 4938265
1 parent 1009590
commit 4938265
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 5 deletions.
diff --git a/contracts/provider/native-staking-proxy/src/multitest.rs b/contracts/provider/native-staking-proxy/src/multitest.rs
@@ -293,7 +293,6 @@ fn unstaking() {
     );
 }
 
-
 #[test]
 fn burning() {
     let owner = "vault_admin";
@@ -336,7 +335,6 @@ fn burning() {
         coin(0, OSMO)
     );
 
-
     // Advance time until the unbonding period is over
     process_staking_unbondings(&app);
 
@@ -551,8 +549,8 @@ fn process_staking_unbondings(app: &App<MtApp>) {
     // This is deprecated as unneeded, but tests fail if it isn't here. What's up???
     app.app_mut()
         .sudo(cw_multi_test::SudoMsg::Staking(
+            #[allow(deprecated)]
             cw_multi_test::StakingSudo::ProcessQueue {},
         ))
         .unwrap();
 }
-
diff --git a/contracts/provider/vault/src/multitest.rs b/contracts/provider/vault/src/multitest.rs
@@ -280,6 +280,7 @@ fn process_staking_unbondings(app: &App<MtApp>) {
     // This is deprecated as unneeded, but tests fail if it isn't here. What's up???
     app.app_mut()
         .sudo(cw_multi_test::SudoMsg::Staking(
+            #[allow(deprecated)]
             cw_multi_test::StakingSudo::ProcessQueue {},
         ))
         .unwrap();

diff --git a/packages/apis/src/converter_api.rs b/packages/apis/src/converter_api.rs
@@ -1,3 +1,6 @@
+// only for valset_update but doesn't work for the autogenerated code
+#![allow(clippy::too_many_arguments)]
+
 use cosmwasm_schema::cw_serde;
 use cosmwasm_std::{Coin, CustomMsg, CustomQuery, Response, StdError, Uint128, Validator};
 use sylvia::types::ExecCtx;
@@ -37,7 +40,6 @@ pub trait ConverterApi {
     ///
     /// TODO: pubkeys need to be part of the Validator struct (requires CosmWasm support).
     #[sv::msg(exec)]
-    #[allow(clippy::too_many_arguments)]
     fn valset_update(
         &self,
         ctx: ExecCtx<Self::QueryC>,

diff --git a/packages/apis/src/virtual_staking_api.rs b/packages/apis/src/virtual_staking_api.rs
@@ -1,3 +1,7 @@
+// only for handle_valset_update but doesn't work for the autogenerated code
+#![allow(clippy::too_many_arguments)]
+
+
 use cosmwasm_schema::cw_serde;
 use cosmwasm_std::{Coin, Response, StdError, Uint128, Validator};
 use sylvia::cw_std::{CustomMsg, CustomQuery};
@@ -66,7 +70,6 @@ pub trait VirtualStakingApi {
     ///  - Temporary removal of a validator from the active set due to jailing. Implies slashing.
     ///  - Addition of an existing validator to the active validator set.
     ///  - Permanent removal (i.e. tombstoning) of a validator from the active set. Implies slashing
-    #[allow(clippy::too_many_arguments)]
     #[sv::msg(sudo)]
     fn handle_valset_update(
         &self,