97d1bc735c
* test: use `T.TempDir` to create temporary test directory
This commit replaces `ioutil.TempDir` with `t.TempDir` in tests. The
directory created by `t.TempDir` is automatically removed when the test
and all its subtests complete.
Prior to this commit, temporary directory created using `ioutil.TempDir`
needs to be removed manually by calling `os.RemoveAll`, which is omitted
in some tests. The error handling boilerplate e.g.
defer func() {
if err := os.RemoveAll(dir); err != nil {
t.Fatal(err)
}
}
is also tedious, but `t.TempDir` handles this for us nicely.
Reference: https://pkg.go.dev/testing#T.TempDir
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
* test: fix TestLogmon_Start_restart on Windows
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
* test: fix failing TestConsul_Integration
t.TempDir fails to perform the cleanup properly because the folder is
still in use
testing.go:967: TempDir RemoveAll cleanup: unlinkat /tmp/TestConsul_Integration2837567823/002/191a6f1a-5371-cf7c-da38-220fe85d10e5/web/secrets: device or resource busy
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
132 lines
2.4 KiB
Go
132 lines
2.4 KiB
Go
package fifo
|
|
|
|
import (
|
|
"bytes"
|
|
"io"
|
|
"path/filepath"
|
|
"runtime"
|
|
"sync"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/hashicorp/nomad/helper/uuid"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
// TestFIFO tests basic behavior, and that reader closes when writer closes
|
|
func TestFIFO(t *testing.T) {
|
|
require := require.New(t)
|
|
var path string
|
|
|
|
if runtime.GOOS == "windows" {
|
|
path = "//./pipe/fifo"
|
|
} else {
|
|
path = filepath.Join(t.TempDir(), "fifo")
|
|
}
|
|
|
|
readerOpenFn, err := CreateAndRead(path)
|
|
require.NoError(err)
|
|
|
|
var reader io.ReadCloser
|
|
|
|
toWrite := [][]byte{
|
|
[]byte("abc\n"),
|
|
[]byte(""),
|
|
[]byte("def\n"),
|
|
[]byte("nomad"),
|
|
[]byte("\n"),
|
|
}
|
|
|
|
var readBuf bytes.Buffer
|
|
var wait sync.WaitGroup
|
|
wait.Add(1)
|
|
go func() {
|
|
defer wait.Done()
|
|
|
|
var err error
|
|
reader, err = readerOpenFn()
|
|
assert.NoError(t, err)
|
|
if err != nil {
|
|
return
|
|
}
|
|
|
|
_, err = io.Copy(&readBuf, reader)
|
|
assert.NoError(t, err)
|
|
}()
|
|
|
|
writer, err := OpenWriter(path)
|
|
require.NoError(err)
|
|
for _, b := range toWrite {
|
|
n, err := writer.Write(b)
|
|
require.NoError(err)
|
|
require.Equal(n, len(b))
|
|
}
|
|
require.NoError(writer.Close())
|
|
time.Sleep(500 * time.Millisecond)
|
|
|
|
wait.Wait()
|
|
require.NoError(reader.Close())
|
|
|
|
expected := "abc\ndef\nnomad\n"
|
|
require.Equal(expected, readBuf.String())
|
|
|
|
require.NoError(Remove(path))
|
|
}
|
|
|
|
// TestWriteClose asserts that when writer closes, subsequent Write() fails
|
|
func TestWriteClose(t *testing.T) {
|
|
require := require.New(t)
|
|
var path string
|
|
|
|
if runtime.GOOS == "windows" {
|
|
path = "//./pipe/" + uuid.Generate()[:4]
|
|
} else {
|
|
path = filepath.Join(t.TempDir(), "fifo")
|
|
}
|
|
|
|
readerOpenFn, err := CreateAndRead(path)
|
|
require.NoError(err)
|
|
var reader io.ReadCloser
|
|
|
|
var readBuf bytes.Buffer
|
|
var wait sync.WaitGroup
|
|
wait.Add(1)
|
|
go func() {
|
|
defer wait.Done()
|
|
|
|
var err error
|
|
reader, err = readerOpenFn()
|
|
assert.NoError(t, err)
|
|
if err != nil {
|
|
return
|
|
}
|
|
|
|
_, err = io.Copy(&readBuf, reader)
|
|
assert.NoError(t, err)
|
|
}()
|
|
|
|
writer, err := OpenWriter(path)
|
|
require.NoError(err)
|
|
|
|
var count int
|
|
wait.Add(1)
|
|
go func() {
|
|
defer wait.Done()
|
|
for count = 0; count < int(^uint16(0)); count++ {
|
|
_, err := writer.Write([]byte(","))
|
|
if err != nil && IsClosedErr(err) {
|
|
break
|
|
}
|
|
require.NoError(err)
|
|
time.Sleep(5 * time.Millisecond)
|
|
}
|
|
}()
|
|
|
|
time.Sleep(500 * time.Millisecond)
|
|
require.NoError(writer.Close())
|
|
wait.Wait()
|
|
|
|
require.Equal(count, len(readBuf.String()))
|
|
}
|