Fix rebase_test.go #958

Open
kellyma2 wants to merge 2 commits from kellyma2/fix-rebase-test into main
1 changed files with 71 additions and 71 deletions

View File

@ -93,7 +93,7 @@ func TestRebaseAbort(t *testing.T) {
// Inputs // Inputs
branchName := "emile" branchName := "emile"
masterCommit := "something" mainCommit := "something"
emileCommits := []string{ emileCommits := []string{
"fou", "fou",
"barre", "barre",
@ -112,7 +112,7 @@ func TestRebaseAbort(t *testing.T) {
seedTestRepo(t, repo) seedTestRepo(t, repo)
// Setup a repo with 2 branches and a different tree // Setup a repo with 2 branches and a different tree
err := setupRepoForRebase(repo, masterCommit, branchName, commitOptions{}) err := setupRepoForRebase(repo, mainCommit, branchName, commitOptions{})
checkFatal(t, err) checkFatal(t, err)
// Create several commits in emile // Create several commits in emile
@ -126,8 +126,8 @@ func TestRebaseAbort(t *testing.T) {
checkFatal(t, err) checkFatal(t, err)
assertStringList(t, expectedHistory, actualHistory) assertStringList(t, expectedHistory, actualHistory)
// Rebase onto master // Rebase onto main
rebase, err := performRebaseOnto(repo, "master", nil) rebase, err := performRebaseOnto(repo, "main", nil)
checkFatal(t, err) checkFatal(t, err)
defer rebase.Free() defer rebase.Free()
@ -145,7 +145,7 @@ func TestRebaseNoConflicts(t *testing.T) {
// Inputs // Inputs
branchName := "emile" branchName := "emile"
masterCommit := "something" mainCommit := "something"
emileCommits := []string{ emileCommits := []string{
"fou", "fou",
"barre", "barre",
@ -157,7 +157,7 @@ func TestRebaseNoConflicts(t *testing.T) {
"Test rebase, Baby! " + emileCommits[2], "Test rebase, Baby! " + emileCommits[2],
"Test rebase, Baby! " + emileCommits[1], "Test rebase, Baby! " + emileCommits[1],
"Test rebase, Baby! " + emileCommits[0], "Test rebase, Baby! " + emileCommits[0],
"Test rebase, Baby! " + masterCommit, "Test rebase, Baby! " + mainCommit,
"This is a commit\n", "This is a commit\n",
} }
@ -173,7 +173,7 @@ func TestRebaseNoConflicts(t *testing.T) {
} }
// Setup a repo with 2 branches and a different tree // Setup a repo with 2 branches and a different tree
err = setupRepoForRebase(repo, masterCommit, branchName, commitOptions{}) err = setupRepoForRebase(repo, mainCommit, branchName, commitOptions{})
checkFatal(t, err) checkFatal(t, err)
// Create several commits in emile // Create several commits in emile
@ -182,8 +182,8 @@ func TestRebaseNoConflicts(t *testing.T) {
checkFatal(t, err) checkFatal(t, err)
} }
// Rebase onto master // Rebase onto main
rebase, err := performRebaseOnto(repo, "master", nil) rebase, err := performRebaseOnto(repo, "main", nil)
checkFatal(t, err) checkFatal(t, err)
defer rebase.Free() defer rebase.Free()
@ -237,7 +237,7 @@ func TestRebaseGpgSigned(t *testing.T) {
// Inputs // Inputs
branchName := "emile" branchName := "emile"
masterCommit := "something" mainCommit := "something"
emileCommits := []string{ emileCommits := []string{
"fou", "fou",
"barre", "barre",
@ -249,7 +249,7 @@ func TestRebaseGpgSigned(t *testing.T) {
"Test rebase, Baby! " + emileCommits[2], "Test rebase, Baby! " + emileCommits[2],
"Test rebase, Baby! " + emileCommits[1], "Test rebase, Baby! " + emileCommits[1],
"Test rebase, Baby! " + emileCommits[0], "Test rebase, Baby! " + emileCommits[0],
"Test rebase, Baby! " + masterCommit, "Test rebase, Baby! " + mainCommit,
"This is a commit\n", "This is a commit\n",
} }
@ -265,7 +265,7 @@ func TestRebaseGpgSigned(t *testing.T) {
} }
// Setup a repo with 2 branches and a different tree // Setup a repo with 2 branches and a different tree
err = setupRepoForRebase(repo, masterCommit, branchName, commitOpts) err = setupRepoForRebase(repo, mainCommit, branchName, commitOpts)
checkFatal(t, err) checkFatal(t, err)
// Create several commits in emile // Create several commits in emile
@ -274,8 +274,8 @@ func TestRebaseGpgSigned(t *testing.T) {
checkFatal(t, err) checkFatal(t, err)
} }
// Rebase onto master // Rebase onto main
rebase, err := performRebaseOnto(repo, "master", &rebaseOpts) rebase, err := performRebaseOnto(repo, "main", &rebaseOpts)
checkFatal(t, err) checkFatal(t, err)
defer rebase.Free() defer rebase.Free()
@ -329,15 +329,15 @@ func checkCommitSigned(t *testing.T, entity *openpgp.Entity, commit *Commit) err
} }
// Utils // Utils
func setupRepoForRebase(repo *Repository, masterCommit, branchName string, commitOpts commitOptions) error { func setupRepoForRebase(repo *Repository, mainCommit, branchName string, commitOpts commitOptions) error {
// Create a new branch from master // Create a new branch from main
err := createBranch(repo, branchName) err := createBranch(repo, branchName)
if err != nil { if err != nil {
return err return err
} }
// Create a commit in master // Create a commit in main
_, err = commitSomething(repo, masterCommit, masterCommit, commitOpts) _, err = commitSomething(repo, mainCommit, mainCommit, commitOpts)
if err != nil { if err != nil {
return err return err
} }
@ -348,22 +348,22 @@ func setupRepoForRebase(repo *Repository, masterCommit, branchName string, commi
return err return err
} }
// Check master commit is not in emile branch // Check main commit is not in emile branch
if entryExists(repo, masterCommit) { if entryExists(repo, mainCommit) {
return errors.New(masterCommit + " entry should not exist in " + branchName + " branch.") return errors.New(mainCommit + " entry should not exist in " + branchName + " branch.")
} }
return nil return nil
} }
func performRebaseOnto(repo *Repository, branch string, rebaseOpts *RebaseOptions) (*Rebase, error) { func performRebaseOnto(repo *Repository, branch string, rebaseOpts *RebaseOptions) (*Rebase, error) {
master, err := repo.LookupBranch(branch, BranchLocal) main, err := repo.LookupBranch(branch, BranchLocal)
if err != nil { if err != nil {
return nil, err return nil, err
} }
defer master.Free() defer main.Free()
onto, err := repo.AnnotatedCommitFromRef(master.Reference) onto, err := repo.AnnotatedCommitFromRef(main.Reference)
if err != nil { if err != nil {
return nil, err return nil, err
} }