change skip env var name because terratest would change it's behavior when a env var named SKIP_XXX has been set

lonegunmanb · lonegunmanb · commit 1803a8c40ecb · 2023-03-16T15:08:55.000+08:00
diff --git a/e2etest.go b/e2etest.go
@@ -56,7 +56,7 @@ func initAndApplyAndIdempotentTest(t *testing.T, moduleRootPath string, exampleR
 
 	tmpDir := copyTerraformFolderToTemp(t, moduleRootPath, exampleRelativePath)
 	defer func() {
-		skip := os.Getenv("SKIP_CLEAN_TMP")
+		skip := os.Getenv("DONOT_CLEAN_TMP")
 		if skip == "" {
 			_ = os.RemoveAll(filepath.Clean(tmpDir))
 		}
diff --git a/upgradetest.go b/upgradetest.go
@@ -130,7 +130,7 @@ func moduleUpgrade(t *testing.T, owner string, repo string, moduleFolderRelative
 	}
 	tmpTestDir := test_structure.CopyTerraformFolderToTemp(t, tmpDirForTag, moduleFolderRelativeToRoot)
 	defer func() {
-		skip := os.Getenv("SKIP_CLEAN_TMP")
+		skip := os.Getenv("DONOT_CLEAN_TMP")
 		if skip == "" {
 			_ = os.RemoveAll(filepath.Clean(tmpTestDir))
 		}

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ func initAndApplyAndIdempotentTest(t *testing.T, moduleRootPath string, exampleR`
`56`	`56`
`57`	`57`	`tmpDir := copyTerraformFolderToTemp(t, moduleRootPath, exampleRelativePath)`
`58`	`58`	`defer func() {`
`59`		`- skip := os.Getenv("SKIP_CLEAN_TMP")`
	`59`	`+ skip := os.Getenv("DONOT_CLEAN_TMP")`
`60`	`60`	`if skip == "" {`
`61`	`61`	`_ = os.RemoveAll(filepath.Clean(tmpDir))`
`62`	`62`	`}`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ func moduleUpgrade(t *testing.T, owner string, repo string, moduleFolderRelative`
`130`	`130`	`}`
`131`	`131`	`tmpTestDir := test_structure.CopyTerraformFolderToTemp(t, tmpDirForTag, moduleFolderRelativeToRoot)`
`132`	`132`	`defer func() {`
`133`		`- skip := os.Getenv("SKIP_CLEAN_TMP")`
	`133`	`+ skip := os.Getenv("DONOT_CLEAN_TMP")`
`134`	`134`	`if skip == "" {`
`135`	`135`	`_ = os.RemoveAll(filepath.Clean(tmpTestDir))`
`136`	`136`	`}`