net: Disable KT monitor test for now

signalapp · Jan 7, 2025 · cbb93cb · cbb93cb
1 parent 76fa0c1
commit cbb93cb
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/rust/net/src/keytrans.rs b/rust/net/src/keytrans.rs
@@ -1020,11 +1020,13 @@ mod test {
         assert_matches!(result, Ok( LocalStateUpdate {tree_head, ..}) => assert_ne!(tree_head.tree_size, 0));
     }
 
-    #[tokio::test]
-    #[test_case(false, false; "ACI")]
-    #[test_case(false, true; "ACI + E164")]
-    #[test_case(true, false; "ACI + Username Hash")]
-    #[test_case(true, true; "ACI + E164 + Username Hash")]
+    // Disabled: does not yet account for server state changing between the two requests.
+    #[allow(dead_code)]
+    // #[tokio::test]
+    // #[test_case(false, false; "ACI")]
+    // #[test_case(false, true; "ACI + E164")]
+    // #[test_case(true, false; "ACI + Username Hash")]
+    // #[test_case(true, true; "ACI + E164 + Username Hash")]
     async fn monitor_permutations_integration_test(use_e164: bool, use_username_hash: bool) {
         if std::env::var("LIBSIGNAL_TESTING_RUN_NONHERMETIC_TESTS").is_err() {
             println!("SKIPPED: running integration tests is not enabled");