From c12a12518ed4af55f5e7c145821fc2ec4f5cc4d1 Mon Sep 17 00:00:00 2001 From: Lucas Clemente Date: Sun, 15 May 2016 14:42:34 +0200 Subject: [PATCH] run gofmt -s on tests --- ackhandler/received_packet_handler_test.go | 2 +- ackhandler/sent_packet_handler_test.go | 54 +++++++++---------- congestion/cubic_sender_test.go | 2 +- crypto/proof_rsa_test.go | 2 +- frames/ack_frame_test.go | 44 +++++++-------- h2quic/request_test.go | 30 +++++------ .../connection_parameters_manager_test.go | 18 +++---- handshake/data_test.go | 2 +- 8 files changed, 77 insertions(+), 77 deletions(-) diff --git a/ackhandler/received_packet_handler_test.go b/ackhandler/received_packet_handler_test.go index f7102cf9..6612ca35 100644 --- a/ackhandler/received_packet_handler_test.go +++ b/ackhandler/received_packet_handler_test.go @@ -267,7 +267,7 @@ var _ = Describe("receivedPacketHandler", func() { Expect(err).ToNot(HaveOccurred()) Expect(ack.LargestObserved).To(Equal(protocol.PacketNumber(4))) Expect(ack.Entropy).To(Equal(byte(entropy))) - Expect(ack.NackRanges).To(Equal([]frames.NackRange{frames.NackRange{FirstPacketNumber: 2, LastPacketNumber: 3}})) + Expect(ack.NackRanges).To(Equal([]frames.NackRange{{FirstPacketNumber: 2, LastPacketNumber: 3}})) }) It("does not generate an ACK if an ACK has already been sent for the largest Packet", func() { diff --git a/ackhandler/sent_packet_handler_test.go b/ackhandler/sent_packet_handler_test.go index 9f38c349..035fdc2e 100644 --- a/ackhandler/sent_packet_handler_test.go +++ b/ackhandler/sent_packet_handler_test.go @@ -114,12 +114,12 @@ var _ = Describe("SentPacketHandler", func() { BeforeEach(func() { entropy = EntropyAccumulator(0) packets = []*Packet{ - &Packet{PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, - &Packet{PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, - &Packet{PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, - &Packet{PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, - &Packet{PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, - &Packet{PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, + {PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: true, Length: 1}, } for _, packet := range packets { handler.SentPacket(packet) @@ -148,7 +148,7 @@ var _ = Describe("SentPacketHandler", func() { } ack := frames.AckFrame{ LargestObserved: protocol.PacketNumber(largestObserved), - NackRanges: []frames.NackRange{frames.NackRange{FirstPacketNumber: 3, LastPacketNumber: 4}}, + NackRanges: []frames.NackRange{{FirstPacketNumber: 3, LastPacketNumber: 4}}, } calculatedEntropy, err := handler.calculateExpectedEntropy(&ack) Expect(err).ToNot(HaveOccurred()) @@ -168,7 +168,7 @@ var _ = Describe("SentPacketHandler", func() { handler.ackPacket(5) ack := frames.AckFrame{ LargestObserved: protocol.PacketNumber(largestObserved), - NackRanges: []frames.NackRange{frames.NackRange{FirstPacketNumber: 3, LastPacketNumber: 4}}, + NackRanges: []frames.NackRange{{FirstPacketNumber: 3, LastPacketNumber: 4}}, } calculatedEntropy, err := handler.calculateExpectedEntropy(&ack) Expect(err).ToNot(HaveOccurred()) @@ -186,8 +186,8 @@ var _ = Describe("SentPacketHandler", func() { ack := frames.AckFrame{ LargestObserved: protocol.PacketNumber(largestObserved), NackRanges: []frames.NackRange{ - frames.NackRange{FirstPacketNumber: 4, LastPacketNumber: 4}, - frames.NackRange{FirstPacketNumber: 2, LastPacketNumber: 2}, + {FirstPacketNumber: 4, LastPacketNumber: 4}, + {FirstPacketNumber: 2, LastPacketNumber: 2}, }, } calculatedEntropy, err := handler.calculateExpectedEntropy(&ack) @@ -244,8 +244,8 @@ var _ = Describe("SentPacketHandler", func() { LargestObserved: protocol.PacketNumber(largestObserved), Entropy: byte(entropy), NackRanges: []frames.NackRange{ - frames.NackRange{FirstPacketNumber: 5, LastPacketNumber: 5}, - frames.NackRange{FirstPacketNumber: 3, LastPacketNumber: 3}, + {FirstPacketNumber: 5, LastPacketNumber: 5}, + {FirstPacketNumber: 3, LastPacketNumber: 3}, }, } _, acked, lost, err := handler.ReceivedAck(&ack) @@ -272,12 +272,12 @@ var _ = Describe("SentPacketHandler", func() { BeforeEach(func() { packets = []*Packet{ - &Packet{PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, } for _, packet := range packets { handler.SentPacket(packet) @@ -352,12 +352,12 @@ var _ = Describe("SentPacketHandler", func() { BeforeEach(func() { packets = []*Packet{ - &Packet{PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, - &Packet{PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 1, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 2, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 3, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 4, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 5, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, + {PacketNumber: 6, Frames: []frames.Frame{&streamFrame}, EntropyBit: false, Length: 1}, } for _, packet := range packets { handler.SentPacket(packet) @@ -413,7 +413,7 @@ var _ = Describe("SentPacketHandler", func() { It("does not change the highestInOrderAckedPacketNumber after queueing a retransmission", func() { ack := frames.AckFrame{ LargestObserved: 4, - NackRanges: []frames.NackRange{frames.NackRange{FirstPacketNumber: 3, LastPacketNumber: 3}}, + NackRanges: []frames.NackRange{{FirstPacketNumber: 3, LastPacketNumber: 3}}, } _, _, _, err := handler.ReceivedAck(&ack) Expect(err).ToNot(HaveOccurred()) @@ -436,7 +436,7 @@ var _ = Describe("SentPacketHandler", func() { // ACK 2, NACK 1 ack := frames.AckFrame{ LargestObserved: 2, - NackRanges: []frames.NackRange{frames.NackRange{FirstPacketNumber: 1, LastPacketNumber: 1}}, + NackRanges: []frames.NackRange{{FirstPacketNumber: 1, LastPacketNumber: 1}}, } _, _, _, err = handler.ReceivedAck(&ack) Expect(err).NotTo(HaveOccurred()) @@ -476,7 +476,7 @@ var _ = Describe("SentPacketHandler", func() { // Now, simulate an ack frame ack := &frames.AckFrame{ LargestObserved: 2, - NackRanges: []frames.NackRange{frames.NackRange{FirstPacketNumber: 1, LastPacketNumber: 1}}, + NackRanges: []frames.NackRange{{FirstPacketNumber: 1, LastPacketNumber: 1}}, } _, acked, lost, err := handler.ReceivedAck(ack) Expect(err).NotTo(HaveOccurred()) diff --git a/congestion/cubic_sender_test.go b/congestion/cubic_sender_test.go index cbfe940e..c8e7a3d0 100644 --- a/congestion/cubic_sender_test.go +++ b/congestion/cubic_sender_test.go @@ -85,7 +85,7 @@ var _ = Describe("Cubic Sender", func() { LosePacket := func(number protocol.PacketNumber) { var ackedPackets congestion.PacketVector var lostPackets congestion.PacketVector = congestion.PacketVector([]congestion.PacketInfo{ - congestion.PacketInfo{Number: number, Length: protocol.DefaultTCPMSS}, + {Number: number, Length: protocol.DefaultTCPMSS}, }) sender.OnCongestionEvent(false, bytesInFlight, ackedPackets, lostPackets) bytesInFlight -= protocol.DefaultTCPMSS diff --git a/crypto/proof_rsa_test.go b/crypto/proof_rsa_test.go index 86818a54..19899e52 100644 --- a/crypto/proof_rsa_test.go +++ b/crypto/proof_rsa_test.go @@ -26,7 +26,7 @@ var _ = Describe("ProofRsa", func() { kd := &rsaSigner{ config: &tls.Config{ Certificates: []tls.Certificate{ - tls.Certificate{Certificate: [][]byte{cert}}, + {Certificate: [][]byte{cert}}, }, }, } diff --git a/frames/ack_frame_test.go b/frames/ack_frame_test.go index fffa3dc5..681bc9eb 100644 --- a/frames/ack_frame_test.go +++ b/frames/ack_frame_test.go @@ -157,9 +157,9 @@ var _ = Describe("AckFrame", func() { It("gets the highest in order packet number for an ACK with one NACK ranges", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 9, LastPacketNumber: 11}, - NackRange{FirstPacketNumber: 7, LastPacketNumber: 7}, - NackRange{FirstPacketNumber: 4, LastPacketNumber: 5}, + {FirstPacketNumber: 9, LastPacketNumber: 11}, + {FirstPacketNumber: 7, LastPacketNumber: 7}, + {FirstPacketNumber: 4, LastPacketNumber: 5}, } frame := &AckFrame{ LargestObserved: 15, @@ -190,8 +190,8 @@ var _ = Describe("AckFrame", func() { It("rejects NACKs with NackRanges in the wrong order", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 2, LastPacketNumber: 2}, - NackRange{FirstPacketNumber: 6, LastPacketNumber: 6}, + {FirstPacketNumber: 2, LastPacketNumber: 2}, + {FirstPacketNumber: 6, LastPacketNumber: 6}, } ack := AckFrame{ LargestObserved: 7, @@ -202,8 +202,8 @@ var _ = Describe("AckFrame", func() { It("rejects NACKs with overlapping NackRanges", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 5, LastPacketNumber: 6}, - NackRange{FirstPacketNumber: 2, LastPacketNumber: 5}, + {FirstPacketNumber: 5, LastPacketNumber: 6}, + {FirstPacketNumber: 2, LastPacketNumber: 5}, } ack := AckFrame{ LargestObserved: 7, @@ -228,8 +228,8 @@ var _ = Describe("AckFrame", func() { It("accepts an ACK with multiple NACK Ranges", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 6, LastPacketNumber: 7}, - NackRange{FirstPacketNumber: 2, LastPacketNumber: 4}, + {FirstPacketNumber: 6, LastPacketNumber: 7}, + {FirstPacketNumber: 2, LastPacketNumber: 4}, } ack := AckFrame{ LargestObserved: 10, @@ -274,7 +274,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 4, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 2}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 2}}, } err := frame.Write(b, 32) Expect(err).ToNot(HaveOccurred()) @@ -290,7 +290,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 4, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 2}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 2}}, } err := frame.Write(b, 31) Expect(err).ToNot(HaveOccurred()) @@ -330,7 +330,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 258, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 257}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 257}}, } err := frame.Write(b, 32) Expect(err).ToNot(HaveOccurred()) @@ -344,7 +344,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 302, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 301}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 301}}, } err := frame.Write(b, 32) Expect(err).ToNot(HaveOccurred()) @@ -360,7 +360,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 259, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 258}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 258}}, } err := frame.Write(b, 32) Expect(err).ToNot(HaveOccurred()) @@ -376,7 +376,7 @@ var _ = Describe("AckFrame", func() { frame := AckFrame{ Entropy: 2, LargestObserved: 603, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 601}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 601}}, } err := frame.Write(b, 32) Expect(err).ToNot(HaveOccurred()) @@ -478,7 +478,7 @@ var _ = Describe("AckFrame", func() { f := &AckFrame{ Entropy: 2, LargestObserved: 4, - NackRanges: []NackRange{NackRange{FirstPacketNumber: 2, LastPacketNumber: 2}}, + NackRanges: []NackRange{{FirstPacketNumber: 2, LastPacketNumber: 2}}, } err := f.Write(b, 31) Expect(err).ToNot(HaveOccurred()) @@ -508,9 +508,9 @@ var _ = Describe("AckFrame", func() { It("is self-consistent for ACK frames with NACK ranges", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 9, LastPacketNumber: 11}, - NackRange{FirstPacketNumber: 7, LastPacketNumber: 7}, - NackRange{FirstPacketNumber: 2, LastPacketNumber: 3}, + {FirstPacketNumber: 9, LastPacketNumber: 11}, + {FirstPacketNumber: 7, LastPacketNumber: 7}, + {FirstPacketNumber: 2, LastPacketNumber: 3}, } frameOrig := &AckFrame{ LargestObserved: 15, @@ -528,9 +528,9 @@ var _ = Describe("AckFrame", func() { It("is self-consistent for ACK frames with contiguous NACK ranges", func() { nackRanges := []NackRange{ - NackRange{FirstPacketNumber: 500, LastPacketNumber: 1500}, - NackRange{FirstPacketNumber: 350, LastPacketNumber: 351}, - NackRange{FirstPacketNumber: 2, LastPacketNumber: 306}, + {FirstPacketNumber: 500, LastPacketNumber: 1500}, + {FirstPacketNumber: 350, LastPacketNumber: 351}, + {FirstPacketNumber: 2, LastPacketNumber: 306}, } frameOrig := &AckFrame{ LargestObserved: 1600, diff --git a/h2quic/request_test.go b/h2quic/request_test.go index f609e5b2..08f42b06 100644 --- a/h2quic/request_test.go +++ b/h2quic/request_test.go @@ -12,9 +12,9 @@ import ( var _ = Describe("Request", func() { It("populates request", func() { headers := []hpack.HeaderField{ - hpack.HeaderField{":path", "/foo", false}, - hpack.HeaderField{":authority", "quic.clemente.io", false}, - hpack.HeaderField{":method", "GET", false}, + {":path", "/foo", false}, + {":authority", "quic.clemente.io", false}, + {":method", "GET", false}, } req, err := requestFromHeaders(headers) Expect(err).NotTo(HaveOccurred()) @@ -31,12 +31,12 @@ var _ = Describe("Request", func() { It("handles other headers", func() { headers := []hpack.HeaderField{ - hpack.HeaderField{":path", "/foo", false}, - hpack.HeaderField{":authority", "quic.clemente.io", false}, - hpack.HeaderField{":method", "GET", false}, - hpack.HeaderField{"content-length", "42", false}, - hpack.HeaderField{"duplicate-header", "1", false}, - hpack.HeaderField{"duplicate-header", "2", false}, + {":path", "/foo", false}, + {":authority", "quic.clemente.io", false}, + {":method", "GET", false}, + {"content-length", "42", false}, + {"duplicate-header", "1", false}, + {"duplicate-header", "2", false}, } req, err := requestFromHeaders(headers) Expect(err).NotTo(HaveOccurred()) @@ -48,8 +48,8 @@ var _ = Describe("Request", func() { It("errors with missing path", func() { headers := []hpack.HeaderField{ - hpack.HeaderField{":authority", "quic.clemente.io", false}, - hpack.HeaderField{":method", "GET", false}, + {":authority", "quic.clemente.io", false}, + {":method", "GET", false}, } _, err := requestFromHeaders(headers) Expect(err).To(MatchError(":path, :authority and :method must not be empty")) @@ -57,8 +57,8 @@ var _ = Describe("Request", func() { It("errors with missing method", func() { headers := []hpack.HeaderField{ - hpack.HeaderField{":path", "/foo", false}, - hpack.HeaderField{":authority", "quic.clemente.io", false}, + {":path", "/foo", false}, + {":authority", "quic.clemente.io", false}, } _, err := requestFromHeaders(headers) Expect(err).To(MatchError(":path, :authority and :method must not be empty")) @@ -66,8 +66,8 @@ var _ = Describe("Request", func() { It("errors with missing authority", func() { headers := []hpack.HeaderField{ - hpack.HeaderField{":path", "/foo", false}, - hpack.HeaderField{":method", "GET", false}, + {":path", "/foo", false}, + {":method", "GET", false}, } _, err := requestFromHeaders(headers) Expect(err).To(MatchError(":path, :authority and :method must not be empty")) diff --git a/handshake/connection_parameters_manager_test.go b/handshake/connection_parameters_manager_test.go index ebbb10a5..31930515 100644 --- a/handshake/connection_parameters_manager_test.go +++ b/handshake/connection_parameters_manager_test.go @@ -76,7 +76,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("reads the tag for truncated connection IDs", func() { values := map[Tag][]byte{ - TagTCID: []byte{0, 0, 0, 0}, + TagTCID: {0, 0, 0, 0}, } cpm.SetFromMap(values) Expect(cpm.TruncateConnectionID()).To(BeTrue()) @@ -102,7 +102,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("sets a new stream-level flow control window for sending", func() { values := map[Tag][]byte{ - TagSFCW: []byte{0xDE, 0xAD, 0xBE, 0xEF}, + TagSFCW: {0xDE, 0xAD, 0xBE, 0xEF}, } err := cpm.SetFromMap(values) Expect(err).ToNot(HaveOccurred()) @@ -111,7 +111,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("does not change the stream-level flow control window when given an invalid value", func() { values := map[Tag][]byte{ - TagSFCW: []byte{0xDE, 0xAD, 0xBE}, // 1 byte too short + TagSFCW: {0xDE, 0xAD, 0xBE}, // 1 byte too short } err := cpm.SetFromMap(values) Expect(err).To(HaveOccurred()) @@ -121,7 +121,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("sets a new connection-level flow control window for sending", func() { values := map[Tag][]byte{ - TagCFCW: []byte{0xDE, 0xAD, 0xBE, 0xEF}, + TagCFCW: {0xDE, 0xAD, 0xBE, 0xEF}, } err := cpm.SetFromMap(values) Expect(err).ToNot(HaveOccurred()) @@ -130,7 +130,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("does not change the connection-level flow control window when given an invalid value", func() { values := map[Tag][]byte{ - TagSFCW: []byte{0xDE, 0xAD, 0xBE}, // 1 byte too short + TagSFCW: {0xDE, 0xAD, 0xBE}, // 1 byte too short } err := cpm.SetFromMap(values) Expect(err).To(HaveOccurred()) @@ -155,7 +155,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("sets the negotiated lifetime", func() { // this test only works if the value given here is smaller than protocol.MaxIdleConnectionStateLifetime values := map[Tag][]byte{ - TagICSL: []byte{10, 0, 0, 0}, + TagICSL: {10, 0, 0, 0}, } err := cpm.SetFromMap(values) Expect(err).ToNot(HaveOccurred()) @@ -164,7 +164,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("does not change the idle connection state lifetime when given an invalid value", func() { values := map[Tag][]byte{ - TagSFCW: []byte{0xDE, 0xAD, 0xBE}, // 1 byte too short + TagSFCW: {0xDE, 0xAD, 0xBE}, // 1 byte too short } err := cpm.SetFromMap(values) Expect(err).To(HaveOccurred()) @@ -191,7 +191,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("sets the negotiated max streams per connection value", func() { // this test only works if the value given here is smaller than protocol.MaxStreamsPerConnection values := map[Tag][]byte{ - TagMSPC: []byte{2, 0, 0, 0}, + TagMSPC: {2, 0, 0, 0}, } err := cpm.SetFromMap(values) Expect(err).ToNot(HaveOccurred()) @@ -200,7 +200,7 @@ var _ = Describe("ConnectionsParameterManager", func() { It("errors when given an invalid max streams per connection value", func() { values := map[Tag][]byte{ - TagMSPC: []byte{2, 0, 0}, // 1 byte too short + TagMSPC: {2, 0, 0}, // 1 byte too short } err := cpm.SetFromMap(values) Expect(err).To(HaveOccurred()) diff --git a/handshake/data_test.go b/handshake/data_test.go index 5e94ef86..238f7033 100644 --- a/handshake/data_test.go +++ b/handshake/data_test.go @@ -16,7 +16,7 @@ var sampleCHLOMap = map[Tag][]byte{ TagICSL: []byte("\x1e\x00\x00\x00"), TagNONP: []byte("\xe1\x84T\x1b\xe3\xd6|\x1fi\xb2N\x9eF\xf4Fݫ\xe5\xdef\x94\xf6\xb2\xee\x01ĥw\xfe\xc9\v\xa3"), TagSCLS: []byte("\x01\x00\x00\x00"), - TagCSCT: []byte{}, + TagCSCT: {}, TagCOPT: []byte("FIXD"), TagSFCW: []byte("\x00\x00`\x00"), TagCFCW: []byte("\x00\x00\xf0\x00"),