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>
98 lines
2.1 KiB
Go
98 lines
2.1 KiB
Go
package command
|
|
|
|
import (
|
|
"io/ioutil"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/hashicorp/nomad/api"
|
|
"github.com/hashicorp/nomad/ci"
|
|
"github.com/hashicorp/nomad/command/agent"
|
|
"github.com/mitchellh/cli"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestOperatorSnapshotInspect_Works(t *testing.T) {
|
|
ci.Parallel(t)
|
|
|
|
snapPath := generateSnapshotFile(t, nil)
|
|
|
|
ui := cli.NewMockUi()
|
|
cmd := &OperatorSnapshotInspectCommand{Meta: Meta{Ui: ui}}
|
|
|
|
code := cmd.Run([]string{snapPath})
|
|
require.Zero(t, code)
|
|
|
|
output := ui.OutputWriter.String()
|
|
for _, key := range []string{
|
|
"ID",
|
|
"Size",
|
|
"Index",
|
|
"Term",
|
|
"Version",
|
|
} {
|
|
require.Contains(t, output, key)
|
|
}
|
|
}
|
|
|
|
func TestOperatorSnapshotInspect_HandlesFailure(t *testing.T) {
|
|
ci.Parallel(t)
|
|
|
|
tmpDir := t.TempDir()
|
|
|
|
err := ioutil.WriteFile(
|
|
filepath.Join(tmpDir, "invalid.snap"),
|
|
[]byte("invalid data"),
|
|
0600)
|
|
require.NoError(t, err)
|
|
|
|
t.Run("not found", func(t *testing.T) {
|
|
ui := cli.NewMockUi()
|
|
cmd := &OperatorSnapshotInspectCommand{Meta: Meta{Ui: ui}}
|
|
|
|
code := cmd.Run([]string{filepath.Join(tmpDir, "foo")})
|
|
require.NotZero(t, code)
|
|
require.Contains(t, ui.ErrorWriter.String(), "no such file")
|
|
})
|
|
|
|
t.Run("invalid file", func(t *testing.T) {
|
|
ui := cli.NewMockUi()
|
|
cmd := &OperatorSnapshotInspectCommand{Meta: Meta{Ui: ui}}
|
|
|
|
code := cmd.Run([]string{filepath.Join(tmpDir, "invalid.snap")})
|
|
require.NotZero(t, code)
|
|
require.Contains(t, ui.ErrorWriter.String(), "Error verifying snapshot")
|
|
})
|
|
}
|
|
|
|
func generateSnapshotFile(t *testing.T, prepare func(srv *agent.TestAgent, client *api.Client, url string)) string {
|
|
tmpDir := t.TempDir()
|
|
|
|
srv, api, url := testServer(t, false, func(c *agent.Config) {
|
|
c.DevMode = false
|
|
c.DataDir = filepath.Join(tmpDir, "server")
|
|
|
|
c.AdvertiseAddrs.HTTP = "127.0.0.1"
|
|
c.AdvertiseAddrs.RPC = "127.0.0.1"
|
|
c.AdvertiseAddrs.Serf = "127.0.0.1"
|
|
})
|
|
|
|
defer srv.Shutdown()
|
|
|
|
if prepare != nil {
|
|
prepare(srv, api, url)
|
|
}
|
|
|
|
ui := cli.NewMockUi()
|
|
cmd := &OperatorSnapshotSaveCommand{Meta: Meta{Ui: ui}}
|
|
|
|
dest := filepath.Join(tmpDir, "backup.snap")
|
|
code := cmd.Run([]string{
|
|
"--address=" + url,
|
|
dest,
|
|
})
|
|
require.Zero(t, code)
|
|
|
|
return dest
|
|
}
|