diff options
| -rw-r--r-- | ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift b/ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift index e17f48c330..34a6fe3478 100644 --- 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") |