From 50b445cf35b31e2c0ea20f570fdd086e7d0fa77c Mon Sep 17 00:00:00 2001 From: Graham Whaley Date: Wed, 18 Jul 2018 11:05:02 +0100 Subject: [PATCH] cli: tests: Clarify who cleans up tmpdir Add a comment to clarify that the caller of testRunContainerSetup() cleans up the tmpdir. Signed-off-by: Graham Whaley --- cli/run_test.go | 1 + 1 file changed, 1 insertion(+) diff --git a/cli/run_test.go b/cli/run_test.go index 9377a4900b..2d051b5342 100644 --- a/cli/run_test.go +++ b/cli/run_test.go @@ -168,6 +168,7 @@ func testRunContainerSetup(t *testing.T) runContainerData { assert.NoError(err, "unable to start fake container workload %+v: %s", workload, err) // temporal dir to place container files + // Note - it is returned to the caller, who does the defer remove to clean up. tmpdir, err := ioutil.TempDir("", "") assert.NoError(err)