Skip to content

Commit 76c7581

Browse files
expose isSocketConnected internally for testing
1 parent afb8759 commit 76c7581

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

Sources/ApolloWebSocket/WebSocketTransport.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ public class WebSocketTransport {
3939

4040
private final let protocols = ["graphql-ws"]
4141

42-
private var isSocketConnected = Atomic<Bool>(false)
42+
/// non-private for testing - you should not use this directly
43+
var isSocketConnected = Atomic<Bool>(false)
4344

4445
private var acked = false
4546

Tests/ApolloWebsocketTests/WebSocketTransportTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class WebSocketTransportTests: XCTestCase {
2828
let mockWebSocketDelegate = MockWebSocketDelegate()
2929

3030
let mockWebSocket = self.webSocketTransport.websocket as? MockWebSocket
31-
mockWebSocket?.isConnected = true
31+
self.webSocketTransport.isSocketConnected.mutate { $0 = true }
3232
mockWebSocket?.delegate = mockWebSocketDelegate
3333

3434
let exp = expectation(description: "Waiting for reconnect")

0 commit comments

Comments
 (0)