diff options
| author | Emīls <emils@mullvad.net> | 2023-11-09 14:36:19 +0100 |
|---|---|---|
| committer | Emīls <emils@mullvad.net> | 2023-11-10 14:49:35 +0100 |
| commit | 7704e46b1c8febc7410feb498c00e9818e198043 (patch) | |
| tree | acd39dbfea1ef49bb672d1c3e776c32873efcf2d | |
| parent | 7fc7726584c65461b977e1d38bb0d51a017407a4 (diff) | |
| download | mullvadvpn-7704e46b1c8febc7410feb498c00e9818e198043.tar.xz mullvadvpn-7704e46b1c8febc7410feb498c00e9818e198043.zip | |
Remove offending test
| -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") |
