Skip to content

Commit af270f7

Browse files
committed
fix(test): update teardownSourcePacket callers for the new signature
Threading the reset sequence through teardownSourcePacket changed its signature and left the three C1 test call sites on the old one, so the package failed to build. A local compile would have caught this instantly; with no Go toolchain here, CI is the first compiler this code meets, so signature changes need every caller checked by hand before pushing.
1 parent 0dc2a6a commit af270f7

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

ip_packet_unreachable_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,15 +107,15 @@ func TestTeardownSourcePacketRespectsSetting(t *testing.T) {
107107
on := &RemoteUserNatMultiClient{settings: &MultiClientSettings{UdpTeardownSignal: true}}
108108
off := &RemoteUserNatMultiClient{settings: &MultiClientSettings{UdpTeardownSignal: false}}
109109

110-
_, ok := on.teardownSourcePacket(udpPath)
110+
_, ok := on.teardownSourcePacket(udpPath, 0)
111111
AssertEqual(t, ok, true)
112112

113-
_, ok = off.teardownSourcePacket(udpPath)
113+
_, ok = off.teardownSourcePacket(udpPath, 0)
114114
AssertEqual(t, ok, false)
115115

116116
// tcp is unaffected either way
117117
for _, c := range []*RemoteUserNatMultiClient{on, off} {
118-
_, ok = c.teardownSourcePacket(tcpPath)
118+
_, ok = c.teardownSourcePacket(tcpPath, 0)
119119
AssertEqual(t, ok, true)
120120
}
121121
}

0 commit comments

Comments
 (0)