From 68f1d6d0ca3d94fe482df9aee6829f86382f2cc0 Mon Sep 17 00:00:00 2001 From: Marten Seemann Date: Sat, 5 Jan 2019 12:45:44 +0700 Subject: [PATCH] properly close session in handshake integration tests Otherwise there's a race condition when setting the supported versions. --- integrationtests/self/handshake_test.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/integrationtests/self/handshake_test.go b/integrationtests/self/handshake_test.go index 23fb9f647..9eb0ea38e 100644 --- a/integrationtests/self/handshake_test.go +++ b/integrationtests/self/handshake_test.go @@ -75,6 +75,7 @@ var _ = Describe("Handshake tests", func() { sess, err := quic.DialAddr(server.Addr().String(), &tls.Config{InsecureSkipVerify: true}, nil) Expect(err).ToNot(HaveOccurred()) Expect(sess.(versioner).GetVersion()).To(Equal(protocol.SupportedVersions[0])) + Expect(sess.Close()).To(Succeed()) }) It("when the client supports more versions than the server supports", func() { @@ -89,6 +90,7 @@ var _ = Describe("Handshake tests", func() { sess, err := quic.DialAddr(server.Addr().String(), &tls.Config{InsecureSkipVerify: true}, conf) Expect(err).ToNot(HaveOccurred()) Expect(sess.(versioner).GetVersion()).To(Equal(protocol.SupportedVersions[0])) + Expect(sess.Close()).To(Succeed()) }) })