summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorEmīls <emils@mullvad.net>2023-11-09 14:36:19 +0100
committerEmīls <emils@mullvad.net>2023-11-10 14:49:35 +0100
commit7704e46b1c8febc7410feb498c00e9818e198043 (patch)
treeacd39dbfea1ef49bb672d1c3e776c32873efcf2d
parent7fc7726584c65461b977e1d38bb0d51a017407a4 (diff)
downloadmullvadvpn-7704e46b1c8febc7410feb498c00e9818e198043.tar.xz
mullvadvpn-7704e46b1c8febc7410feb498c00e9818e198043.zip
Remove offending test
-rw-r--r--ios/PacketTunnelCoreTests/PacketTunnelActorTests.swift7
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")