diff options
| author | Eng Zer Jun <engzerjun@gmail.com> | 2022-12-04 17:57:28 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-12-04 20:57:28 +1100 |
| commit | 59c11581b840aa92a4de6a3014e756b875c719ce (patch) | |
| tree | 57c2f063fabb3bf9255b294013d3df1a42107c17 /internal/configuration/decode_hooks.go | |
| parent | ee50e5580c69ce8355f2dba075c1ff23a5563135 (diff) | |
test: use testing tempdir tooling (#4468)
This commit replaces `os.MkdirTemp` 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 `os.MkdirTemp`
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>
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
Co-authored-by: James Elliott <james-d-elliott@users.noreply.github.com>
Diffstat (limited to 'internal/configuration/decode_hooks.go')
0 files changed, 0 insertions, 0 deletions
