ref: Update TestGetGitBranch test

It makes more sense to validate this functionality using a proper repository, rather than initializing an empty repository.
ErickKramer · Jul 2, 2024 · c552406 · c552406
1 parent 572c3aa
commit c552406
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/test/git_helpers_test.go b/test/git_helpers_test.go
@@ -81,12 +81,16 @@ func TestGitStatus(t *testing.T) {
 }
 
 func TestGetGitBranch(t *testing.T) {
-	if utils.GetGitBranch("/tmp/testdata/valid_repo") != "main" {
+	testingBranch := "jazzy"
+	repoPath := "/tmp/testdata/demos_branch"
+	if utils.GitClone("https://github.com/ros2/demos.git", testingBranch, repoPath, true, false, false) != utils.SuccessfullClone {
+		t.Errorf("Expected to successfully clone git repository")
+	}
+	if utils.GetGitBranch(repoPath) != testingBranch {
 		t.Errorf("Failed to get main branch for valid git repository")
 	}
 	testingTag := "0.34.0"
-	repoPath := "/tmp/testdata/demos_branch"
-	if utils.GitClone("https://github.com/ros2/demos.git", testingTag, repoPath, false, false, false) != utils.SuccessfullClone {
+	if utils.GitClone("https://github.com/ros2/demos.git", testingTag, repoPath, true, false, false) != utils.SuccessfullClone {
 		t.Errorf("Expected to successfully clone git repository")
 	}
 	obtainedTag := utils.GetGitBranch(repoPath)