Skip to content

Commit

Permalink
Updated expectations to be fulfilled via a defer
Browse files Browse the repository at this point in the history
  • Loading branch information
dimitribouniol committed Jul 17, 2024
1 parent cb63bea commit c607885
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions Tests/NIOHTTPCompressionTests/HTTPResponseCompressorTest.swift
Original file line number Diff line number Diff line change
Expand Up @@ -739,7 +739,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testConditionalCompressionEnabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, true)
return true
Expand Down Expand Up @@ -767,7 +767,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testUnsupportedRequestConditionalCompressionEnabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, false)
return true
Expand All @@ -794,7 +794,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testUnsupportedStatusConditionalCompressionEnabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.status, .notModified)
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, false)
Expand Down Expand Up @@ -833,7 +833,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testConditionalCompressionDisabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, true)
return false
Expand All @@ -860,7 +860,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testUnsupportedRequestConditionalCompressionDisabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, false)
return false
Expand All @@ -887,7 +887,7 @@ class HTTPResponseCompressorTest: XCTestCase {
func testUnsupportedStatusConditionalCompressionDisabled() throws {
let predicateWasCalled = expectation(description: "Predicate was called")
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
XCTAssertEqual(responseHeaders.status, .notModified)
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json"])
XCTAssertEqual(isCompressionSupported, false)
Expand Down Expand Up @@ -927,7 +927,7 @@ class HTTPResponseCompressorTest: XCTestCase {
let predicateWasCalled = expectation(description: "Predicate was called")
predicateWasCalled.expectedFulfillmentCount = 2
let compressor = HTTPResponseCompressor { responseHeaders, isCompressionSupported in
predicateWasCalled.fulfill()
defer { predicateWasCalled.fulfill() }
let isEnabled = responseHeaders.headers[canonicalForm: "x-compression"].first == "enable"
XCTAssertEqual(responseHeaders.headers, ["Content-Type" : "json", "X-Compression" : isEnabled ? "enable" : "disable"])
responseHeaders.headers.remove(name: "X-Compression")
Expand Down

0 comments on commit c607885

Please sign in to comment.