Skip to content

Commit c96e0c8

Browse files
simonjbeaumontsimonjbeaumont
committed
Add missing availability guards in tests
1 parent d2d3566 commit c96e0c8

File tree

7 files changed

+30
-59
lines changed

7 files changed

+30
-59
lines changed

‎Sources/AsyncHTTPClient/NIOTransportServices/TLSConfiguration.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ extension TLSVersion {
5757
}
5858
}
5959

60-
@available(macOS 10.14, iOS 12.0, tvOS 12.0, watchOS 6.0, *)
60+
@available(macOS 10.14, iOS 12.0, tvOS 12.0, watchOS 5.0, *)
6161
extension TLSConfiguration {
6262
/// Dispatch queue used by Network framework TLS to control certificate verification
6363
static var tlsDispatchQueue = DispatchQueue(label: "TLSDispatch")

‎Tests/AsyncHTTPClientTests/AsyncAwaitEndToEndTests.swift

Lines changed: 3 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ private func makeDefaultHTTPClient(
3333
)
3434
}
3535

36+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
3637
final class AsyncAwaitEndToEndTests: XCTestCase {
3738
var clientGroup: EventLoopGroup!
3839
var serverGroup: EventLoopGroup!
@@ -56,7 +57,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
5657
}
5758

5859
func testSimpleGet() {
59-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
6060
XCTAsyncTest {
6161
let bin = HTTPBin(.http2(compress: false))
6262
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -77,7 +77,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
7777
}
7878

7979
func testSimplePost() {
80-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
8180
XCTAsyncTest {
8281
let bin = HTTPBin(.http2(compress: false))
8382
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -98,7 +97,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
9897
}
9998

10099
func testPostWithByteBuffer() {
101-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
102100
XCTAsyncTest {
103101
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
104102
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -121,7 +119,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
121119
}
122120

123121
func testPostWithSequenceOfUInt8() {
124-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
125122
XCTAsyncTest {
126123
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
127124
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -144,7 +141,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
144141
}
145142

146143
func testPostWithCollectionOfUInt8() {
147-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
148144
XCTAsyncTest {
149145
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
150146
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -167,7 +163,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
167163
}
168164

169165
func testPostWithRandomAccessCollectionOfUInt8() {
170-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
171166
XCTAsyncTest {
172167
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
173168
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -190,7 +185,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
190185
}
191186

192187
func testPostWithAsyncSequenceOfByteBuffers() {
193-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
194188
XCTAsyncTest {
195189
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
196190
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -217,7 +211,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
217211
}
218212

219213
func testPostWithAsyncSequenceOfUInt8() {
220-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
221214
XCTAsyncTest {
222215
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
223216
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -240,7 +233,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
240233
}
241234

242235
func testPostWithFragmentedAsyncSequenceOfByteBuffers() {
243-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
244236
XCTAsyncTest {
245237
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
246238
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -280,7 +272,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
280272
}
281273

282274
func testPostWithFragmentedAsyncSequenceOfLargeByteBuffers() {
283-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
284275
XCTAsyncTest {
285276
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
286277
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -321,7 +312,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
321312
}
322313

323314
func testCanceling() {
324-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
325315
XCTAsyncTest(timeout: 5) {
326316
let bin = HTTPBin(.http2(compress: false))
327317
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -344,7 +334,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
344334
}
345335

346336
func testCancelingResponseBody() {
347-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
348337
XCTAsyncTest(timeout: 5) {
349338
let bin = HTTPBin(.http2(compress: false)) { _ in
350339
HTTPEchoHandler()
@@ -373,7 +362,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
373362
}
374363

375364
func testDeadline() {
376-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
377365
XCTAsyncTest(timeout: 5) {
378366
let bin = HTTPBin(.http2(compress: false))
379367
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -398,7 +386,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
398386
}
399387

400388
func testImmediateDeadline() {
401-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
402389
XCTAsyncTest(timeout: 5) {
403390
let bin = HTTPBin(.http2(compress: false))
404391
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -423,7 +410,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
423410
}
424411

425412
func testConnectTimeout() {
426-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
427413
XCTAsyncTest(timeout: 60) {
428414
#if os(Linux)
429415
// 198.51.100.254 is reserved for documentation only and therefore should not accept any TCP connection
@@ -480,7 +466,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
480466
}
481467

482468
func testSelfSignedCertificateIsRejectedWithCorrectErrorIfRequestDeadlineIsExceeded() {
483-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
484469
XCTAsyncTest(timeout: 5) {
485470
/// key + cert was created with the follwing command:
486471
/// openssl req -x509 -newkey rsa:4096 -keyout self_signed_key.pem -out self_signed_cert.pem -sha256 -days 99999 -nodes -subj '/CN=localhost'
@@ -526,7 +511,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
526511
}
527512

528513
func testDnsOverride() {
529-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
530514
XCTAsyncTest(timeout: 5) {
531515
/// key + cert was created with the following code (depends on swift-certificates)
532516
/// ```
@@ -584,7 +568,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
584568
}
585569

586570
func testInvalidURL() {
587-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
588571
XCTAsyncTest(timeout: 5) {
589572
let client = makeDefaultHTTPClient()
590573
defer { XCTAssertNoThrow(try client.syncShutdown()) }
@@ -597,8 +580,8 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
597580
}
598581
}
599582

583+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
600584
func testRedirectChangesHostHeader() {
601-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
602585
XCTAsyncTest {
603586
let bin = HTTPBin(.http2(compress: false))
604587
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -625,7 +608,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
625608
}
626609

627610
func testShutdown() {
628-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
629611
XCTAsyncTest {
630612
let client = makeDefaultHTTPClient()
631613
try await client.shutdown()
@@ -636,8 +618,8 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
636618
}
637619

638620
/// Regression test for https://github.com/swift-server/async-http-client/issues/612
621+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
639622
func testCancelingBodyDoesNotCrash() {
640-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
641623
XCTAsyncTest {
642624
let client = makeDefaultHTTPClient()
643625
defer { XCTAssertNoThrow(try client.syncShutdown()) }
@@ -654,7 +636,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
654636
}
655637

656638
func testAsyncSequenceReuse() {
657-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
658639
XCTAsyncTest {
659640
let bin = HTTPBin(.http2(compress: false)) { _ in HTTPEchoHandler() }
660641
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -698,7 +679,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
698679
}
699680

700681
private func _rejectsInvalidCharactersInHeaderFieldNames(mode: HTTPBin<HTTPBinHandler>.Mode) {
701-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
702682
XCTAsyncTest {
703683
let bin = HTTPBin(mode)
704684
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -759,7 +739,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
759739
}
760740

761741
private func _rejectsInvalidCharactersInHeaderFieldValues(mode: HTTPBin<HTTPBinHandler>.Mode) {
762-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
763742
XCTAsyncTest {
764743
let bin = HTTPBin(mode)
765744
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -818,7 +797,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
818797
}
819798

820799
func testUsingGetMethodInsteadOfWait() {
821-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
822800
XCTAsyncTest {
823801
let bin = HTTPBin(.http2(compress: false))
824802
defer { XCTAssertNoThrow(try bin.shutdown()) }
@@ -838,7 +816,6 @@ final class AsyncAwaitEndToEndTests: XCTestCase {
838816
}
839817

840818
func testSimpleContentLengthErrorNoBody() {
841-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
842819
XCTAsyncTest {
843820
let bin = HTTPBin(.http2(compress: false))
844821
defer { XCTAssertNoThrow(try bin.shutdown()) }

‎Tests/AsyncHTTPClientTests/HTTPClientRequestTests.swift

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,13 @@ import Algorithms
1717
import NIOCore
1818
import XCTest
1919

20+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
2021
class HTTPClientRequestTests: XCTestCase {
21-
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
2222
private typealias Request = HTTPClientRequest
2323

24-
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
2524
private typealias PreparedRequest = HTTPClientRequest.Prepared
2625

2726
func testCustomHeadersAreRespected() {
28-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
2927
XCTAsyncTest {
3028
var request = Request(url: "https://example.com/get")
3129
request.headers = [
@@ -60,7 +58,6 @@ class HTTPClientRequestTests: XCTestCase {
6058
}
6159

6260
func testUnixScheme() {
63-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
6461
XCTAsyncTest {
6562
var request = Request(url: "unix://%2Fexample%2Ffolder.sock/some_path")
6663
request.headers = ["custom-header": "custom-value"]
@@ -90,7 +87,6 @@ class HTTPClientRequestTests: XCTestCase {
9087
}
9188

9289
func testHTTPUnixScheme() {
93-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
9490
XCTAsyncTest {
9591
var request = Request(url: "http+unix://%2Fexample%2Ffolder.sock/some_path")
9692
request.headers = ["custom-header": "custom-value"]
@@ -120,7 +116,6 @@ class HTTPClientRequestTests: XCTestCase {
120116
}
121117

122118
func testHTTPSUnixScheme() {
123-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
124119
XCTAsyncTest {
125120
var request = Request(url: "https+unix://%2Fexample%2Ffolder.sock/some_path")
126121
request.headers = ["custom-header": "custom-value"]
@@ -150,7 +145,6 @@ class HTTPClientRequestTests: XCTestCase {
150145
}
151146

152147
func testGetWithoutBody() {
153-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
154148
XCTAsyncTest {
155149
let request = Request(url: "https://example.com/get")
156150
var preparedRequest: PreparedRequest?
@@ -179,7 +173,6 @@ class HTTPClientRequestTests: XCTestCase {
179173
}
180174

181175
func testPostWithoutBody() {
182-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
183176
XCTAsyncTest {
184177
var request = Request(url: "http://example.com/post")
185178
request.method = .POST
@@ -213,7 +206,6 @@ class HTTPClientRequestTests: XCTestCase {
213206
}
214207

215208
func testPostWithEmptyByteBuffer() {
216-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
217209
XCTAsyncTest {
218210
var request = Request(url: "http://example.com/post")
219211
request.method = .POST
@@ -248,7 +240,6 @@ class HTTPClientRequestTests: XCTestCase {
248240
}
249241

250242
func testPostWithByteBuffer() {
251-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
252243
XCTAsyncTest {
253244
var request = Request(url: "http://example.com/post")
254245
request.method = .POST
@@ -282,7 +273,6 @@ class HTTPClientRequestTests: XCTestCase {
282273
}
283274

284275
func testPostWithSequenceOfUnknownLength() {
285-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
286276
XCTAsyncTest {
287277
var request = Request(url: "http://example.com/post")
288278
request.method = .POST
@@ -317,7 +307,6 @@ class HTTPClientRequestTests: XCTestCase {
317307
}
318308

319309
func testPostWithSequenceWithFixedLength() {
320-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
321310
XCTAsyncTest {
322311
var request = Request(url: "http://example.com/post")
323312
request.method = .POST
@@ -353,7 +342,6 @@ class HTTPClientRequestTests: XCTestCase {
353342
}
354343

355344
func testPostWithRandomAccessCollection() {
356-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
357345
XCTAsyncTest {
358346
var request = Request(url: "http://example.com/post")
359347
request.method = .POST
@@ -388,7 +376,6 @@ class HTTPClientRequestTests: XCTestCase {
388376
}
389377

390378
func testPostWithAsyncSequenceOfUnknownLength() {
391-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
392379
XCTAsyncTest {
393380
var request = Request(url: "http://example.com/post")
394381
request.method = .POST
@@ -428,7 +415,6 @@ class HTTPClientRequestTests: XCTestCase {
428415
}
429416

430417
func testPostWithAsyncSequenceWithKnownLength() {
431-
guard #available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *) else { return }
432418
XCTAsyncTest {
433419
var request = Request(url: "http://example.com/post")
434420
request.method = .POST
@@ -467,6 +453,7 @@ class HTTPClientRequestTests: XCTestCase {
467453
}
468454
}
469455

456+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
470457
func testChunkingRandomAccessCollection() async throws {
471458
let body = try await HTTPClientRequest.Body.bytes(
472459
Array(repeating: 0, count: bagOfBytesToByteBufferConversionChunkSize) +
@@ -483,6 +470,7 @@ class HTTPClientRequestTests: XCTestCase {
483470
XCTAssertEqual(body, expectedChunks)
484471
}
485472

473+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
486474
func testChunkingCollection() async throws {
487475
let body = try await HTTPClientRequest.Body.bytes(
488476
(
@@ -502,6 +490,7 @@ class HTTPClientRequestTests: XCTestCase {
502490
XCTAssertEqual(body, expectedChunks)
503491
}
504492

493+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
505494
func testChunkingSequenceThatDoesNotImplementWithContiguousStorageIfAvailable() async throws {
506495
let bagOfBytesToByteBufferConversionChunkSize = 8
507496
let body = try await HTTPClientRequest.Body._bytes(
@@ -522,6 +511,7 @@ class HTTPClientRequestTests: XCTestCase {
522511
XCTAssertEqual(body, expectedChunks)
523512
}
524513

514+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
525515
func testChunkingSequenceFastPath() async throws {
526516
func makeBytes() -> some Sequence<UInt8> & Sendable {
527517
Array(repeating: 0, count: bagOfBytesToByteBufferConversionChunkSize) +
@@ -543,6 +533,7 @@ class HTTPClientRequestTests: XCTestCase {
543533
XCTAssertEqual(body, expectedChunks)
544534
}
545535

536+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
546537
func testChunkingSequenceFastPathExceedingByteBufferMaxSize() async throws {
547538
let bagOfBytesToByteBufferConversionChunkSize = 8
548539
let byteBufferMaxSize = 16
@@ -602,6 +593,7 @@ class HTTPClientRequestTests: XCTestCase {
602593
}
603594
}
604595

596+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
605597
extension AsyncSequence {
606598
func collect() async throws -> [Element] {
607599
try await self.reduce(into: []) { $0 += CollectionOfOne($1) }

‎Tests/AsyncHTTPClientTests/HTTPClientResponseTests.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import Logging
1717
import NIOCore
1818
import XCTest
1919

20+
@available(macOS 10.15, iOS 13.0, watchOS 6.0, tvOS 13.0, *)
2021
final class HTTPClientResponseTests: XCTestCase {
2122
func testSimpleResponse() {
2223
let response = HTTPClientResponse.expectedContentLength(requestMethod: .GET, headers: ["content-length": "1025"], status: .ok)

0 commit comments

Comments
 (0)