From df73b91a654efa6648821eab1fa249575007990f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ram=C3=B3n=20M=C3=A1rquez?= Date: Tue, 5 Oct 2021 05:53:05 -0400 Subject: [PATCH] Fix: message size check in timeout test (#1014) Fix message size check in `TestMessageTimeoutReached`. Since the message parser has parsed two packets of size 63 << 10, then the message size should be 63 << 11 --- tcp/tcp_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tcp/tcp_test.go b/tcp/tcp_test.go index 4dfd3d3..753f950 100644 --- a/tcp/tcp_test.go +++ b/tcp/tcp_test.go @@ -213,7 +213,8 @@ func TestMessageParserWithoutHint(t *testing.T) { } func TestMessageTimeoutReached(t *testing.T) { - var data [63 << 10]byte + const size = 63 << 11 + var data [size >> 1]byte packets := GetPackets(true, 1, 2, data[:]) p := NewMessageParser(nil, nil, nil, 10*time.Millisecond, true) p.processPacket(packets[0]) @@ -222,8 +223,8 @@ func TestMessageTimeoutReached(t *testing.T) { p.processPacket(packets[1]) m := p.Read() - if m.Length != 63<<10 { - t.Errorf("expected %d to equal %d", m.Length, 63<<10) + if m.Length != size { + t.Errorf("expected %d to equal %d", m.Length, size) } if !m.TimedOut { t.Error("expected message to be timeout")