Skip to content

Commit ba97c49

Browse files
committed
fix after pulling upstream
1 parent 1dfae04 commit ba97c49

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

Tests/ConnectLibraryTests/ConnectConformance/CallbackConformance.swift

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -435,16 +435,18 @@ final class CallbackConformance: XCTestCase {
435435
proto.domain = "connect-crosstest"
436436
}
437437
let expectation = self.expectation(description: "Stream completes")
438-
var responseCount = 0
438+
let responseCount = Locked(0)
439439
let sizes = [31_415, 9, 2_653, 58_979]
440440
let stream = client.failStreamingOutputCall { result in
441441
switch result {
442442
case .headers:
443443
break
444444

445445
case .message(let output):
446-
XCTAssertEqual(output.payload.body.count, sizes[responseCount])
447-
responseCount += 1
446+
responseCount.perform { responseCount in
447+
XCTAssertEqual(output.payload.body.count, sizes[responseCount])
448+
responseCount += 1
449+
}
448450

449451
case .complete(_, let error, _):
450452
guard let connectError = error as? ConnectError else {
@@ -468,7 +470,7 @@ final class CallbackConformance: XCTestCase {
468470
})
469471

470472
XCTAssertEqual(XCTWaiter().wait(for: [expectation], timeout: kTimeout), .completed)
471-
XCTAssertEqual(responseCount, 4)
473+
XCTAssertEqual(responseCount.value, 4)
472474
}
473475
}
474476

@@ -480,9 +482,9 @@ final class CallbackConformance: XCTestCase {
480482
let cancelable = client.emptyCall(
481483
request: SwiftProtobuf.Google_Protobuf_Empty()
482484
) { response in
483-
XCTAssertEqual(response.code, .canceled)
484-
XCTAssertEqual(response.error?.code, .canceled)
485-
expectation.fulfill()
485+
XCTAssertEqual(response.code, .canceled)
486+
XCTAssertEqual(response.error?.code, .canceled)
487+
expectation.fulfill()
486488
}
487489
cancelable.cancel()
488490
XCTAssertEqual(XCTWaiter().wait(for: [expectation], timeout: kTimeout), .completed)

0 commit comments

Comments
 (0)