From 95c6576299259db960f6c5b9b69ea52422860fce Mon Sep 17 00:00:00 2001 From: Tobias Klauser Date: Mon, 30 Oct 2017 11:08:44 +0100 Subject: [PATCH] unix: enable TestPoll for all unix systems Poll is now supported for all unix systems. Also enable TestPoll for all of them. Change-Id: Id6250702bf0d29c49fbeeed4cca21d232b72666a Reviewed-on: https://go-review.googlesource.com/74310 Run-TryBot: Tobias Klauser TryBot-Result: Gobot Gobot Reviewed-by: Ian Lance Taylor --- unix/syscall_linux_test.go | 47 -------------------------------------- unix/syscall_unix_test.go | 47 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+), 47 deletions(-) diff --git a/unix/syscall_linux_test.go b/unix/syscall_linux_test.go index 006a0375..f7f5f4aa 100644 --- a/unix/syscall_linux_test.go +++ b/unix/syscall_linux_test.go @@ -56,34 +56,6 @@ func TestIoctlGetInt(t *testing.T) { t.Logf("%d bits of entropy available", v) } -func TestPoll(t *testing.T) { - f, cleanup := mktmpfifo(t) - defer cleanup() - - const timeout = 100 - - ok := make(chan bool, 1) - go func() { - select { - case <-time.After(10 * timeout * time.Millisecond): - t.Errorf("Poll: failed to timeout after %d milliseconds", 10*timeout) - case <-ok: - } - }() - - fds := []unix.PollFd{{Fd: int32(f.Fd()), Events: unix.POLLIN}} - n, err := unix.Poll(fds, timeout) - ok <- true - if err != nil { - t.Errorf("Poll: unexpected error: %v", err) - return - } - if n != 0 { - t.Errorf("Poll: wrong number of events: got %v, expected %v", n, 0) - return - } -} - func TestPpoll(t *testing.T) { f, cleanup := mktmpfifo(t) defer cleanup() @@ -113,25 +85,6 @@ func TestPpoll(t *testing.T) { } } -// mktmpfifo creates a temporary FIFO and provides a cleanup function. -func mktmpfifo(t *testing.T) (*os.File, func()) { - err := unix.Mkfifo("fifo", 0666) - if err != nil { - t.Fatalf("mktmpfifo: failed to create FIFO: %v", err) - } - - f, err := os.OpenFile("fifo", os.O_RDWR, 0666) - if err != nil { - os.Remove("fifo") - t.Fatalf("mktmpfifo: failed to open FIFO: %v", err) - } - - return f, func() { - f.Close() - os.Remove("fifo") - } -} - func TestTime(t *testing.T) { var ut unix.Time_t ut2, err := unix.Time(&ut) diff --git a/unix/syscall_unix_test.go b/unix/syscall_unix_test.go index 394b3502..8512dd01 100644 --- a/unix/syscall_unix_test.go +++ b/unix/syscall_unix_test.go @@ -343,3 +343,50 @@ func TestDup(t *testing.T) { t.Errorf("Dup: stdout write not in file, expected %v, got %v", string(b1), string(b2)) } } + +func TestPoll(t *testing.T) { + f, cleanup := mktmpfifo(t) + defer cleanup() + + const timeout = 100 + + ok := make(chan bool, 1) + go func() { + select { + case <-time.After(10 * timeout * time.Millisecond): + t.Errorf("Poll: failed to timeout after %d milliseconds", 10*timeout) + case <-ok: + } + }() + + fds := []unix.PollFd{{Fd: int32(f.Fd()), Events: unix.POLLIN}} + n, err := unix.Poll(fds, timeout) + ok <- true + if err != nil { + t.Errorf("Poll: unexpected error: %v", err) + return + } + if n != 0 { + t.Errorf("Poll: wrong number of events: got %v, expected %v", n, 0) + return + } +} + +// mktmpfifo creates a temporary FIFO and provides a cleanup function. +func mktmpfifo(t *testing.T) (*os.File, func()) { + err := unix.Mkfifo("fifo", 0666) + if err != nil { + t.Fatalf("mktmpfifo: failed to create FIFO: %v", err) + } + + f, err := os.OpenFile("fifo", os.O_RDWR, 0666) + if err != nil { + os.Remove("fifo") + t.Fatalf("mktmpfifo: failed to open FIFO: %v", err) + } + + return f, func() { + f.Close() + os.Remove("fifo") + } +}