kylefang/stacks-puppet-node
1
0
Fork 0
mirror of https://github.com/alexgo-io/stacks-puppet-node.git synced 2026-04-29 04:05:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: disable relevant unit tests that assume 14 stacker signer slots

Browse Source
This commit is contained in:
Jude Nelson
2024-06-03 18:07:19 -04:00
parent 631138fca3
commit c13e521c4c
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
libsigner/src/v1/messages.rs
View File

@@ -1448,7 +1448,8 @@ mod test {
use super::{StacksMessageCodecExtensions, *};
#[test]
// This test is disabled because the last signer slot can't be introduced until after Nakamoto
// #[test]
fn signer_slots_count_is_sane() {
let slot_identifiers_len = MessageSlotID::ALL.len();
assert!(

6
stacks-signer/src/client/stacks_client.rs
View File

@@ -738,7 +738,7 @@ mod tests {
};
use rand::thread_rng;
use rand_core::RngCore;
use stacks_common::consts::{CHAIN_ID_TESTNET, SIGNER_SLOTS_PER_USER};
use stacks_common::consts::CHAIN_ID_TESTNET;
use wsts::curve::scalar::Scalar;
use super::*;
@@ -1121,9 +1121,13 @@ mod tests {
let signer_slots = mock.client.parse_signer_slots(value).unwrap();
assert_eq!(signer_slots.len(), 5);
/*
* This is disabled until the number of slots is again 14
signer_slots
.into_iter()
.for_each(|(_address, slots)| assert_eq!(slots, SIGNER_SLOTS_PER_USER as u128));
*/
}
#[test]