From 997ee1b815c1efe5183c20af585e9af9593f2710 Mon Sep 17 00:00:00 2001 From: maclane Date: Tue, 9 Jun 2026 22:00:21 -0400 Subject: [PATCH] Pin concrete coordinator vector in order-independence test The order-independence test asserted only that two input orderings agree, not what they agree on. The Go side now pins the concrete result for the same (members, seed, attempt) tuple, so pin Some(4) here as well to keep the cross-language vector sets symmetric. Co-Authored-By: Claude Fable 5 --- pkg/tbtc/signer/src/go_math_rand.rs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pkg/tbtc/signer/src/go_math_rand.rs b/pkg/tbtc/signer/src/go_math_rand.rs index 012a481cda..3b3648983f 100644 --- a/pkg/tbtc/signer/src/go_math_rand.rs +++ b/pkg/tbtc/signer/src/go_math_rand.rs @@ -817,5 +817,11 @@ mod tests { let right = select_coordinator_identifier(&[6, 1, 5, 2, 4, 3], 333, 4); assert_eq!(left, right); + // Pin the concrete result, not just the equality: the Go side + // (keep-core pkg/frost/roast, + // TestSelectCoordinator_CrossLanguagePinnedVectors) asserts the + // same value, so either implementation drifting fails its own + // suite instead of fracturing coordinator agreement at runtime. + assert_eq!(left, Some(4)); } }