From 96f24e799af516aea66356d7fb2dfb7a44ff90e5 Mon Sep 17 00:00:00 2001 From: funman300 Date: Mon, 24 Aug 2026 21:39:48 +0000 Subject: [PATCH] fix(test): restore #[test] on the SBC fault guard test The squad_actives default test was inserted between #[test] and the function it belonged to, which stacked a duplicate attribute on the new test and left sbc_post_commit_faults_require_all_staging_guards with none - silently disabling it while the new test ran twice. Caught by clippy (-D duplicate-macro-attributes, -D dead-code); the doubled test name in the earlier run was the tell. Lib tests go 129 -> 131 with both now executing. --- openfut-utas-host/src/config.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openfut-utas-host/src/config.rs b/openfut-utas-host/src/config.rs index 91a1f58..fe564db 100644 --- a/openfut-utas-host/src/config.rs +++ b/openfut-utas-host/src/config.rs @@ -243,7 +243,6 @@ fn default_account_path(identity_store_path: &str) -> String { mod tests { use super::*; - #[test] /// `squad.actives` is ON without any environment variable. /// /// Emitting the club's active home/away kit is normal FIFA 17 behaviour — @@ -272,6 +271,7 @@ mod tests { } } + #[test] fn sbc_post_commit_faults_require_all_staging_guards() { assert_eq!( parse_sbc_post_commit_fault(None, None, None, "0.0.0.0:8099").unwrap(),