Skip to content

Commit 8d1c106

Browse files
docs(probe): trim the startup probe test comments
Keep only the claim the fixture does not make, and leave the Given and When markers bare.
1 parent 4363bf0 commit 8d1c106

2 files changed

Lines changed: 4 additions & 6 deletions

File tree

crates/e2e-tests/tests/foreign_chain_probe.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,7 @@ fn base_config(rpc_url: &str, expected_network: u64, provider: &str) -> ForeignC
3030
}
3131
}
3232

33-
/// The startup probe asks each configured provider for its chain id and publishes the per-chain
34-
/// counts. Node 0 is pointed at a mock serving [`MOCK_EVM_CHAIN_ID`] and told to expect it, so its
35-
/// provider is healthy; node 1 is told to expect a different network, so its provider is not.
36-
/// Both configure one provider, so `configured` is 1 either way and only `healthy` separates them.
33+
/// Both nodes configure one provider, so only `healthy` separates them.
3734
#[tokio::test]
3835
#[expect(non_snake_case)]
3936
async fn foreign_chain_probe__should_publish_provider_health_on_startup() {
@@ -42,7 +39,7 @@ async fn foreign_chain_probe__should_publish_provider_health_on_startup() {
4239
setup_evm_mock(&server, MockAuthExpectation::None);
4340
let url = server.url("/");
4441

45-
// when cluster probing is ran as part of start up
42+
// when
4643
let (cluster, _running) = common::must_setup_cluster(
4744
common::FOREIGN_CHAIN_PROBE_PORT_SEED,
4845
|c: &mut e2e_tests::MpcClusterConfig| {

crates/node-config/src/foreign_chains.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -708,9 +708,10 @@ ckd:
708708
assert!(error.contains("support only header auth"), "{error}");
709709
}
710710

711+
/// Every chain is set, so a chain added later has to be listed here too.
711712
#[test]
712713
fn foreign_chains_config__should_key_every_chain_by_its_label() {
713-
// Given — every chain set, so a chain added later has to be listed here too.
714+
// Given
714715
let section = || ForeignChainConfig {
715716
timeout_sec: NonZeroU64::new(30).unwrap(),
716717
max_retries: NonZeroU64::new(1).unwrap(),

0 commit comments

Comments
 (0)