Merge pull request #378 from hashicorp/binary-acctests-tidy-up-tmpdirs

Binary testing: Fix cleanup of temporary directories
hashicorp · Apr 6, 2020 · b8e75fe · b8e75fe
2 parents 9d28501 + dbe9175
commit b8e75fe
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+# 1.9.1 (Unreleased)
+
+BUG FIXES:
+
+* Binary acceptance test driver: fix cleanup of temporary directories [GH-378]
+
 # 1.9.0 (March 26, 2020)
 
 DEPRECATED:

diff --git a/helper/resource/testing.go b/helper/resource/testing.go
@@ -108,10 +108,15 @@ func TestMain(m *testing.M) {
 			os.Exit(1)
 		}
 	} else {
+		exitCode := m.Run()
+
 		if acctest.TestHelper != nil {
-			defer acctest.TestHelper.Close()
+			err := acctest.TestHelper.Close()
+			if err != nil {
+				log.Printf("Error cleaning up temporary test files: %s", err)
+			}
 		}
-		os.Exit(m.Run())
+		os.Exit(exitCode)
 	}
 }