Skip to content

Commit e910eae

Browse files
committed
tests: Fix warnings from clippy::neg_cmp_op_on_partial_ord
1 parent a60f640 commit e910eae

1 file changed

Lines changed: 9 additions & 16 deletions

File tree

chain/ethereum/src/network.rs

Lines changed: 9 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -329,6 +329,7 @@ mod tests {
329329
use super::{EthereumNetworkAdapter, EthereumNetworkAdapters, NodeCapabilities};
330330

331331
#[test]
332+
#[allow(clippy::neg_cmp_op_on_partial_ord)]
332333
fn ethereum_capabilities_comparison() {
333334
let archive = NodeCapabilities {
334335
archive: true,
@@ -470,9 +471,7 @@ mod tests {
470471
{
471472
let adapter = adapters.call_or_cheapest(None).unwrap();
472473
assert!(adapter.is_call_only());
473-
assert!(
474-
!adapters.call_or_cheapest(None).unwrap().is_call_only()
475-
);
474+
assert!(!adapters.call_or_cheapest(None).unwrap().is_call_only());
476475
}
477476

478477
// Check empty falls back to call only
@@ -620,9 +619,7 @@ mod tests {
620619
// one reference above and one inside adapters struct
621620
assert_eq!(Arc::strong_count(&eth_call_adapter), 2);
622621
assert_eq!(Arc::strong_count(&eth_adapter), 2);
623-
assert!(
624-
!adapters.call_or_cheapest(None).unwrap().is_call_only()
625-
);
622+
assert!(!adapters.call_or_cheapest(None).unwrap().is_call_only());
626623
}
627624

628625
#[graph::test]
@@ -665,9 +662,7 @@ mod tests {
665662
.await;
666663
// one reference above and one inside adapters struct
667664
assert_eq!(Arc::strong_count(&eth_adapter), 2);
668-
assert!(
669-
!adapters.call_or_cheapest(None).unwrap().is_call_only()
670-
);
665+
assert!(!adapters.call_or_cheapest(None).unwrap().is_call_only());
671666
}
672667

673668
#[graph::test]
@@ -687,7 +682,8 @@ mod tests {
687682
let provider_metrics = Arc::new(ProviderEthRpcMetrics::new(mock_registry.clone()));
688683
let chain_id: Word = "chain_id".into();
689684

690-
let adapters = [fake_adapter(
685+
let adapters = [
686+
fake_adapter(
691687
&logger,
692688
unavailable_provider,
693689
&provider_metrics,
@@ -703,7 +699,8 @@ mod tests {
703699
&metrics,
704700
false,
705701
)
706-
.await];
702+
.await,
703+
];
707704

708705
// Set errors
709706
metrics.report_for_test(&ProviderName::from(error_provider), false);
@@ -896,11 +893,7 @@ mod tests {
896893
});
897894
let manager = ProviderManager::new(
898895
logger,
899-
vec![(
900-
chain_id.clone(),
901-
no_available_adapter.to_vec(),
902-
)]
903-
.into_iter(),
896+
vec![(chain_id.clone(), no_available_adapter.to_vec())].into_iter(),
904897
ProviderCheckStrategy::MarkAsValid,
905898
);
906899

0 commit comments

Comments
 (0)