Skip to content

Commit

Permalink
chore: remove resolve_sig_inputs_refs
Browse files Browse the repository at this point in the history
  • Loading branch information
eichhorl committed Jan 23, 2024
1 parent f7deb73 commit 97c4b32
Showing 1 changed file with 0 additions and 66 deletions.
66 changes: 0 additions & 66 deletions rs/consensus/src/ecdsa/signer.rs
Expand Up @@ -599,39 +599,8 @@ mod tests {
use ic_types::crypto::{canister_threshold_sig::ExtendedDerivationPath, AlgorithmId};
use ic_types::time::UNIX_EPOCH;
use ic_types::{Height, Randomness};
use prometheus::IntCounterVec;
use std::ops::Deref;

/// Resolves the [`ThresholdEcdsaSigInputsRef`] -> [`ThresholdEcdsaSigInputs`]
fn resolve_sig_inputs_refs(
block_reader: &dyn EcdsaBlockReader,
reason: &str,
metric: IntCounterVec,
log: &ReplicaLogger,
) -> Vec<(RequestId, ThresholdEcdsaSigInputs)> {
let mut ret = Vec::new();
for (request_id, sig_inputs_ref) in block_reader.requested_signatures() {
// Translate the [`ThresholdEcdsaSigInputsRef`] -> [`ThresholdEcdsaSigInputs`]
match sig_inputs_ref.translate(block_reader) {
Ok(sig_inputs) => {
ret.push((request_id.clone(), sig_inputs));
}
Err(error) => {
warn!(
log,
"Failed to resolve sig input ref: reason = {}, \
sig_inputs_ref = {:?}, error = {:?}",
reason,
sig_inputs_ref,
error
);
metric.with_label_values(&[reason]).inc();
}
}
}
ret
}

fn create_request_id(generator: &mut EcdsaUIDGenerator, height: Height) -> RequestId {
let quadruple_id = generator.next_quadruple_id(fake_ecdsa_key_id());
let pseudo_random_id = [0; 32];
Expand Down Expand Up @@ -1214,41 +1183,6 @@ mod tests {
})
}

// Tests resolving of sig inputs
#[test]
fn test_ecdsa_resolve_sig_inputs_refs() {
with_test_replica_logger(|logger| {
let mut uid_generator = EcdsaUIDGenerator::new(subnet_test_id(1), Height::new(0));
let (id_1, id_2) = (
create_request_id(&mut uid_generator, Height::from(10)),
create_request_id(&mut uid_generator, Height::from(200)),
);

let block_reader = TestEcdsaBlockReader::for_signer_test(
Height::from(100),
vec![
(id_1.clone(), create_sig_inputs(1)),
(id_2.clone(), create_sig_inputs(2)),
],
);

let metrics = MetricsRegistry::new();
let m = metrics.int_counter_vec(
"test_ecdsa_resolve_sig_inputs_refs",
"ECDSA test metrics",
&["test"],
);
let r = resolve_sig_inputs_refs(&block_reader, "test", m.clone(), &logger);
assert_eq!(r.len(), 2);
assert!(r.iter().any(|(id, _)| id == &id_1));
assert!(r.iter().any(|(id, _)| id == &id_2));

let block_reader = block_reader.with_fail_to_resolve();
let r = resolve_sig_inputs_refs(&block_reader, "test", m, &logger);
assert!(r.is_empty());
});
}

// Tests aggregating signature shares into a complete signature
#[test]
fn test_ecdsa_get_completed_signature() {
Expand Down

0 comments on commit 97c4b32

Please sign in to comment.