@@ -23,6 +23,8 @@ func TestMain(m *testing.M) {
2323goleak .VerifyTestMain (m ,testutil .GoleakOptions ... )
2424}
2525
26+ const expectedHandshake = "codervpn tunnel 1.2\n "
27+
2628// TestSpeaker_RawPeer tests the speaker with a peer that we simulate by directly making reads and
2729// writes to the other end of the pipe. There should be at least one test that does this, rather
2830// than use 2 speakers so that we don't have a bug where we don't adhere to the stated protocol, but
@@ -48,8 +50,6 @@ func TestSpeaker_RawPeer(t *testing.T) {
4850errCh <- err
4951}()
5052
51- expectedHandshake := "codervpn tunnel 1.2\n "
52-
5353b := make ([]byte ,256 )
5454n ,err := mp .Read (b )
5555require .NoError (t ,err )
@@ -157,8 +157,6 @@ func TestSpeaker_OversizeHandshake(t *testing.T) {
157157errCh <- err
158158}()
159159
160- expectedHandshake := "codervpn tunnel 1.2\n "
161-
162160b := make ([]byte ,256 )
163161n ,err := mp .Read (b )
164162require .NoError (t ,err )
@@ -210,7 +208,6 @@ func TestSpeaker_HandshakeInvalid(t *testing.T) {
210208_ ,err = mp .Write ([]byte (tc .handshake ))
211209require .NoError (t ,err )
212210
213- expectedHandshake := "codervpn tunnel 1.2\n "
214211b := make ([]byte ,256 )
215212n ,err := mp .Read (b )
216213require .NoError (t ,err )
@@ -248,8 +245,6 @@ func TestSpeaker_CorruptMessage(t *testing.T) {
248245errCh <- err
249246}()
250247
251- expectedHandshake := "codervpn tunnel 1.2\n "
252-
253248b := make ([]byte ,256 )
254249n ,err := mp .Read (b )
255250require .NoError (t ,err )