gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

skip the multiplex test on linux, add a better explanation

Browse Source
This commit is contained in:
Marten Seemann
2019-06-29 11:52:23 +07:00
parent 4fe0f6752c
commit 79513e81eb
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
integrationtests/self/multiplex_test.go
View File

@@ -5,7 +5,7 @@ import (
"fmt" "fmt"
"io/ioutil" "io/ioutil"
"net" "net"
"os" "runtime"
"time" "time"
quic "github.com/lucas-clemente/quic-go" quic "github.com/lucas-clemente/quic-go"
@@ -164,8 +164,8 @@ var _ = Describe("Multiplexing", func() {
}) })
It("runs a server and client on the same conn", func() { It("runs a server and client on the same conn", func() {
if os.Getenv("CI") == "true" { if runtime.GOOS == "linux" {
Skip("This test is flaky on CIs, see see https://github.com/golang/go/issues/17677.") Skip("This test would require setting of iptables rules, see https://stackoverflow.com/questions/23859164/linux-udp-socket-sendto-operation-not-permitted.")
} }
addr1, err := net.ResolveUDPAddr("udp", "localhost:0") addr1, err := net.ResolveUDPAddr("udp", "localhost:0")
Expect(err).ToNot(HaveOccurred()) Expect(err).ToNot(HaveOccurred())