Skip to content

Commit

Permalink
Don't assume master is the default branch.
Browse files Browse the repository at this point in the history
This introduces a quick lookup of the default branch rather than
assuming "master".

Signed-off-by: Kevin McDermott <[email protected]>
  • Loading branch information
bigkevmcd committed Jan 17, 2022
1 parent 382e812 commit 4029241
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion pkg/git/libgit2/checkout_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,18 @@ func TestCheckoutBranch_Checkout(t *testing.T) {
defer repo.Free()
defer os.RemoveAll(filepath.Join(repo.Path(), ".."))

cfg, err := git2go.OpenDefault()
if err != nil {
t.Fatal(err)
}

// ignores the error here because it can be defaulted
// https://github.blog/2020-07-27-highlights-from-git-2-28/#introducing-init-defaultbranch
defaultBranch := "main"
if v, err := cfg.LookupString("init.defaultBranch"); err != nil {
defaultBranch = v
}

firstCommit, err := commitFile(repo, "branch", "init", time.Now())
if err != nil {
t.Fatal(err)
Expand All @@ -59,7 +71,7 @@ func TestCheckoutBranch_Checkout(t *testing.T) {
}{
{
name: "Default branch",
branch: "master",
branch: defaultBranch,
expectedCommit: secondCommit.String(),
},
{
Expand Down

0 comments on commit 4029241

Please sign in to comment.