Since the tests in go are run in parallel, there is a problem with creating directories.

So I made one of the tests a simple function, and then they run fine.

Author: blackmarllborooo

internal/pkg/git/tests/git_test.go

@@ -8,31 +8,19 @@ import (
 	"testing"
 )
 
-//TODO the tests run and run fine individually, but if you run them together,
-// you get confused with the test directories.
-
-func TestCreateLocalGitRepository_CurrentDir(t *testing.T) {
+// Since the tests in Go are run in parallel, there is a problem with creating
+// dirs. So i made one of the tests a simple function, and then they run fine.
+func testCreateLocalGitRepositoryNewdir(t *testing.T) {
 	t.Log("create dir for test")
 	{
 		if err := os.Mkdir(mock.TestDir, 0755); err != nil {
 			t.Fatalf("Failed to create test directory: %v", err)
 		}
-		defer os.RemoveAll(fmt.Sprintf("../%s", mock.TestDir))
-
-		if err := os.Chdir(mock.TestDir); err != nil {
-			t.Fatalf("Expected no error, but got: %s", err)
-		}
-
-		defer func() {
-			if err := os.Chdir(".."); err != nil {
-				t.Fatalf("Expected no error, but got: %s", err)
-			}
-		}()
 	}
 
-	t.Log("create repository in current dir")
+	t.Log("create repository in new dir")
 	{
-		if err := git.CreateLocalGitRepository(mock.IsCurrentDirIsTrue{}); err != nil {
+		if err := git.CreateLocalGitRepository(mock.IsCurrentDirIsFalse{}); err != nil {
 			t.Fatalf("Expected no error, but got: %s", err)
 		}
 	}
@@ -48,16 +36,28 @@ func TestCreateLocalGitRepository_CurrentDir(t *testing.T) {
 	t.Log("SUCCESS")
 }
 
-func TestCreateLocalGitRepository_NewDir(t *testing.T) {
+func TestCreateLocalGitRepository_CurrentDir(t *testing.T) {
+	testCreateLocalGitRepositoryNewdir(t)
+
 	t.Log("create dir for test")
 	{
 		if err := os.Mkdir(mock.TestDir, 0755); err != nil {
 			t.Fatalf("Failed to create test directory: %v", err)
 		}
-		defer os.RemoveAll(mock.TestDir)
+		defer os.RemoveAll(fmt.Sprintf("../%s", mock.TestDir))
+
+		if err := os.Chdir(mock.TestDir); err != nil {
+			t.Fatalf("Expected no error, but got: %s", err)
+		}
+
+		defer func() {
+			if err := os.Chdir(".."); err != nil {
+				t.Fatalf("Expected no error, but got: %s", err)
+			}
+		}()
 	}
 
-	t.Log("create repository in new dir")
+	t.Log("create repository in current dir")
 	{
 		if err := git.CreateLocalGitRepository(mock.IsCurrentDirIsTrue{}); err != nil {
 			t.Fatalf("Expected no error, but got: %s", err)

internal/pkg/git/tests/mock/name.go

@@ -1,9 +1,9 @@
 package mock
 
-type IsCurrentDirIsFalse struct{}
-
 const TestDir = "test_dir"
 
+type IsCurrentDirIsFalse struct{}
+
 func (i IsCurrentDirIsFalse) GetProjectName() (string, bool, error) {
 	return TestDir, false, nil
 }