fix(test): update the zot tests not to use test/data as rootDir (use a temporary folder instead) (#1162)

Signed-off-by: Nicol Draghici <idraghic@cisco.com>

pkg/common/common_test.go

@@ -42,8 +42,9 @@ func TestCommon(t *testing.T) {
 	Convey("test dirExists()", t, func() {
 		exists := common.DirExists("testdir")
 		So(exists, ShouldBeFalse)
+		tempDir := t.TempDir()
 
-		file, err := os.Create("file.txt")
+		file, err := os.Create(path.Join(tempDir, "file.txt"))
 		So(err, ShouldBeNil)
 		isDir := common.DirExists(file.Name())
 		So(isDir, ShouldBeFalse)

pkg/extensions/search/cve/cve_test.go

@@ -42,7 +42,6 @@ import (
 const (
 	username   = "test"
 	passphrase = "test"
-	testDir    = "../../../../test/data"
 )
 
 type CveResult struct {
@@ -373,8 +372,6 @@ func TestImageFormat(t *testing.T) {
 
 func TestCVESearchDisabled(t *testing.T) {
 	Convey("Test with CVE search disabled", t, func() {
-		dbDir := testDir
-
 		port := GetFreePort()
 		baseURL := GetBaseURL(port)
 		conf := config.New()
@@ -388,6 +385,8 @@ func TestCVESearchDisabled(t *testing.T) {
 			},
 		}
 
+		dbDir, err := testSetup(t)
+		So(err, ShouldBeNil)
 		conf.Storage.RootDirectory = dbDir
 		defaultVal := true
 		searchConfig := &extconf.SearchConfig{
@@ -748,15 +747,9 @@ func TestHTTPOptionsResponse(t *testing.T) {
 
 		ctlr := api.NewController(conf)
 
-		firstDir, err := os.MkdirTemp("", "oci-repo-test")
+		firstDir := t.TempDir()
-		if err != nil {
-			panic(err)
-		}
 
-		secondDir, err := os.MkdirTemp("", "oci-repo-test")
+		secondDir := t.TempDir()
-		if err != nil {
-			panic(err)
-		}
 		defer os.RemoveAll(firstDir)
 		defer os.RemoveAll(secondDir)
 

pkg/extensions/search/digest/digest_test.go

@@ -273,10 +273,7 @@ func TestDigestSearchHTTPSubPaths(t *testing.T) {
 
 		ctlr := api.NewController(conf)
 
-		globalDir, err := os.MkdirTemp("", "digest_test")
+		globalDir := t.TempDir()
-		if err != nil {
-			panic(err)
-		}
 		defer os.RemoveAll(globalDir)
 
 		ctlr.Config.Storage.RootDirectory = globalDir

pkg/extensions/sync/sync_test.go

@@ -742,10 +742,8 @@ func TestConfigReloader(t *testing.T) {
 
 		destConfig.HTTP.Port = destPort
 
-		destDir, err := os.MkdirTemp("", "oci-dest-repo-test")
+		// change
-		if err != nil {
+		destDir := t.TempDir()
-			panic(err)
-		}
 
 		defer os.RemoveAll(destDir)
 
@@ -3928,8 +3926,7 @@ func TestSyncSignaturesDiff(t *testing.T) {
 		So(err, ShouldBeNil)
 
 		defer func() { _ = os.Chdir(cwd) }()
-		tdir, err := os.MkdirTemp("", "sigs")
+		tdir := t.TempDir()
-		So(err, ShouldBeNil)
 
 		_ = os.Chdir(tdir)
 		generateKeyPairs(tdir)
@@ -4021,8 +4018,7 @@ func TestSyncSignaturesDiff(t *testing.T) {
 
 		// now add new signatures to upstream and let sync detect that upstream signatures changed and pull them
 		So(os.RemoveAll(tdir), ShouldBeNil)
-		tdir, err = os.MkdirTemp("", "sigs")
+		tdir = t.TempDir()
-		So(err, ShouldBeNil)
 		defer os.RemoveAll(tdir)
 		_ = os.Chdir(tdir)
 		generateKeyPairs(tdir)