Remove unused variable

mullvad · Sep 18, 2024 · e90f94d · e90f94d
1 parent dee9d4e
commit e90f94d
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/ios/PacketTunnelCore/Pinger/TunnelPinger.swift b/ios/PacketTunnelCore/Pinger/TunnelPinger.swift
@@ -32,9 +32,7 @@ public final class TunnelPinger: PingerProtocol {
         }
     }
 
-    var socketHandle: Int32?
-
-    var pingProvider: ICMPPingProvider
+    private var pingProvider: ICMPPingProvider
 
     private let logger: Logger
 
@@ -44,7 +42,7 @@ public final class TunnelPinger: PingerProtocol {
         self.pingQueue = DispatchQueue(label: "PacketTunnel.icmp")
         self.logger = Logger(label: "TunnelPinger")
     }
-    
+
     deinit {
         pingProvider.closeICMP()
     }
@@ -68,6 +66,7 @@ public final class TunnelPinger: PingerProtocol {
             let reply: PingerReply
             do {
                 try pingProvider.sendICMPPing(seqNumber: sequenceNumber)
+                // NOTE: we cheat here by returning the destination address we were passed, rather than parsing it from the packet on the other side of the FFI boundary.
                 reply = .success(destAddress, sequenceNumber)
             } catch {
                 reply = .parseError(error)
@@ -76,7 +75,6 @@ public final class TunnelPinger: PingerProtocol {
 
             replyQueue.async { [weak self] in
                 guard let self else { return }
-                // NOTE: we cheat here by returning the destination address we were passed, rather than parsing it from the packet on the other side of the FFI boundary.
                 self.onReply?(reply)
             }
         }

diff --git a/ios/PacketTunnelCore/TunnelMonitor/TunnelMonitor.swift b/ios/PacketTunnelCore/TunnelMonitor/TunnelMonitor.swift
@@ -178,7 +178,7 @@ public final class TunnelMonitor: TunnelMonitorProtocol {
         nslock.lock()
         defer { nslock.unlock() }
 
-        guard let probeAddress, let newStats = getStats(),
+        guard let newStats = getStats(),
               state.connectionState == .connecting || state.connectionState == .connected
         else { return }
 

diff --git a/ios/PacketTunnelCoreTests/Mocks/PingerMock.swift b/ios/PacketTunnelCoreTests/Mocks/PingerMock.swift
@@ -86,7 +86,7 @@ class PingerMock: PingerProtocol {
 
         networkStatsReporting.reportBytesSent(UInt64(icmpPacketSize))
 
-        return PingerSendResult(sequenceNumber: nextSequenceId, bytesSent: icmpPacketSize)
+        return PingerSendResult(sequenceNumber: nextSequenceId)
     }
 
     // MARK: - Types
@@ -96,7 +96,7 @@ class PingerMock: PingerProtocol {
         var sequenceId: UInt16 = 0
         var isSocketOpen = false
         var onReply: ((PingerReply) -> Void)?
-        var destAddress: IPv4Address? = nil
+        var destAddress: IPv4Address?
 
         mutating func incrementSequenceId() -> UInt16 {
             sequenceId += 1

diff --git a/ios/PacketTunnelCoreTests/PingerTests.swift b/ios/PacketTunnelCoreTests/PingerTests.swift
@@ -25,8 +25,8 @@ final class PingerTests: XCTestCase {
             }
         }
 
-        try pinger.openSocket(bindTo: "lo0")
-        sendResult = try pinger.send(to: .loopback)
+        try pinger.openSocket(bindTo: "lo0", destAddress: .loopback)
+        sendResult = try pinger.send()
 
         waitForExpectations(timeout: .UnitTest.timeout)
     }