Remove offending test

mullvad · Nov 10, 2023 · 7704e46 · 7704e46
1 parent 7fc7726
commit 7704e46
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift b/ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift
@@ -302,7 +302,12 @@ final class PacketTunnelActorTests: XCTestCase {
         await fulfillment(of: [disconnectedStateExpectation, didStopObserverExpectation], timeout: 1)
     }
 
-    func testSetErrorStateGetsCancelledWhenStopping() async throws {
+    // FIXME: Reconsider if this test should exist. As it stands currently, it
+    // relies the packet tunnel process processing app message calls and a
+    // `stopTunnel()` call in a particular, deterministic order, which makes it
+    // unreliable. In reality, we cannot guarantee the order between those
+    // calls, and it fails almost reliably on low core count VMs.
+    func setErrorStateGetsCancelledWhenStopping() async throws {
         let actor = PacketTunnelActor.mock()
         let connectingStateExpectation = expectation(description: "Connecting state")
         let disconnectedStateExpectation = expectation(description: "Disconnected state")