Skip to content

Commit

Permalink
Fix more warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
glbrntt committed Jan 14, 2025
1 parent 0cdc406 commit 2f0cfe1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Tests/NIOSOCKSTests/SocksClientHandler+Tests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,7 @@ class SocksClientHandlerTests: XCTestCase {
self.assertOutputBuffer([])

// add the handler, there should be outbound data immediately
XCTAssertNoThrow(self.channel.pipeline.addHandler(handler))
XCTAssertNoThrow(try self.channel.pipeline.syncOperations.addHandler(handler))
self.assertOutputBuffer([0x05, 0x01, 0x00])
}

Expand Down Expand Up @@ -263,7 +263,7 @@ class SocksClientHandlerTests: XCTestCase {
let establishPromise = self.channel.eventLoop.makePromise(of: Void.self)
let removalPromise = self.channel.eventLoop.makePromise(of: Void.self)
establishPromise.futureResult.whenSuccess { _ in
self.channel.pipeline.removeHandler(self.handler).cascade(to: removalPromise)
self.channel.pipeline.syncOperations.removeHandler(self.handler).cascade(to: removalPromise)
}

XCTAssertNoThrow(
Expand Down Expand Up @@ -309,7 +309,7 @@ class SocksClientHandlerTests: XCTestCase {

// we try to remove the handler before the connection is established.
let removalPromise = self.channel.eventLoop.makePromise(of: Void.self)
self.channel.pipeline.removeHandler(self.handler, promise: removalPromise)
self.channel.pipeline.syncOperations.removeHandler(self.handler, promise: removalPromise)

// establishes the connection
self.writeInbound([0x05, 0x00, 0x00, 0x01, 192, 168, 1, 1, 0x00, 0x50])
Expand Down

0 comments on commit 2f0cfe1

Please sign in to comment.