From f6982bb02950ba352177a70585118c818aafe144 Mon Sep 17 00:00:00 2001 From: Zhou Hao Date: Thu, 23 Jul 2020 10:35:17 +0800 Subject: [PATCH] add os.RemoveAll err verification Signed-off-by: Zhou Hao --- upup/pkg/fi/cloudup/awstasks/render_test.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/upup/pkg/fi/cloudup/awstasks/render_test.go b/upup/pkg/fi/cloudup/awstasks/render_test.go index 97f44bee7b..6bce09bda1 100644 --- a/upup/pkg/fi/cloudup/awstasks/render_test.go +++ b/upup/pkg/fi/cloudup/awstasks/render_test.go @@ -41,7 +41,12 @@ func doRenderTests(t *testing.T, method string, cases []*renderTest) { t.Errorf("failed to create local render directory: %s", err) t.FailNow() } - defer os.RemoveAll(outdir) + defer func() { + err := os.RemoveAll(outdir) + if err != nil { + t.Errorf("failed to remove temp dir %q: %v", outdir, err) + } + }() for i, c := range cases { var filename string