Ecosystem/serai
Ecosystem/serai
1
0
Fork 0
mirror of https://github.com/serai-dex/serai.git synced 2024-11-16 17:07:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Remove no longer necessary async from monero SignatableTransaction::sign

Browse source
This commit is contained in:
Luke Parker 2023-08-29 16:20:21 -04:00
parent 285422f71a
commit 83c25eff03
No known key found for this signature in database
5 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
coins/monero/src/wallet/send/mod.rs
View file

@ -805,8 +805,7 @@ impl SignableTransaction {
}
/// Sign this transaction.
// TODO: Remove async
pub async fn sign<R: RngCore + CryptoRng>(
pub fn sign<R: RngCore + CryptoRng>(
mut self,
rng: &mut R,
spend: &Zeroizing<Scalar>,

2
coins/monero/tests/runner.rs
View file

@ -228,7 +228,7 @@ macro_rules! test {
let keys = keys.clone();
async move {
if !multisig {
tx.sign(&mut OsRng, &spend).await.unwrap()
tx.sign(&mut OsRng, &spend).unwrap()
} else {
#[cfg(not(feature = "multisig"))]
panic!("Multisig branch called without the multisig feature");

1
processor/src/networks/monero.rs
View file

@ -630,7 +630,6 @@ impl Network for Monero {
)
.unwrap()
.sign(&mut OsRng, &Zeroizing::new(Scalar::ONE.0))
.await
.unwrap();
let block = self.get_latest_block_number().await.unwrap() + 1;

1
tests/full-stack/src/tests/mint_and_burn.rs
View file

@ -387,7 +387,6 @@ async fn mint_and_burn_test() {
)
.unwrap()
.sign(&mut OsRng, &Zeroizing::new(Scalar::ONE))
.await
.unwrap();
rpc.publish_transaction(&tx).await.unwrap()

1
tests/processor/src/networks.rs
View file

@ -367,7 +367,6 @@ impl Wallet {
)
.unwrap()
.sign(&mut OsRng, spend_key)
.await
.unwrap();
// Push the change output