From 493826569b74db2765384169a4659558f9e46f85 Mon Sep 17 00:00:00 2001 From: Ethan Frey Date: Mon, 13 May 2024 16:29:43 +0200 Subject: [PATCH] Test pass, clippy happy :) --- contracts/provider/native-staking-proxy/src/multitest.rs | 4 +--- contracts/provider/vault/src/multitest.rs | 1 + packages/apis/src/converter_api.rs | 4 +++- packages/apis/src/virtual_staking_api.rs | 5 ++++- 4 files changed, 9 insertions(+), 5 deletions(-) diff --git a/contracts/provider/native-staking-proxy/src/multitest.rs b/contracts/provider/native-staking-proxy/src/multitest.rs index 410f067c..02321a7d 100644 --- 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) { // 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 index 7b448b89..558bebf4 100644 --- a/contracts/provider/vault/src/multitest.rs +++ b/contracts/provider/vault/src/multitest.rs @@ -280,6 +280,7 @@ fn process_staking_unbondings(app: &App) { // 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 index 22fb1dfe..8fc26412 100644 --- 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, diff --git a/packages/apis/src/virtual_staking_api.rs b/packages/apis/src/virtual_staking_api.rs index d820a668..d05013a2 100644 --- 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,