Retire CommitsEqual in favor of require.Equal

The latter does a better job at diffing.

1 files changed, 0 insertions, 15 deletions

diff --git a/internal/testhelper/testhelper.go b/internal/testhelper/testhelper.go
index 00404ead1..3712b4aa9 100644
--- a/internal/testhelper/testhelper.go
+++ b/internal/testhelper/testhelper.go
@@ -8,7 +8,6 @@ import (
 	"os/exec"
 	"path"
 	"path/filepath"
-	"reflect"
 	"runtime"
 	"strings"
 	"testing"
@@ -135,20 +134,6 @@ func AuthorsEqual(a *pb.CommitAuthor, b *pb.CommitAuthor) bool {
 		a.Date.Seconds == b.Date.Seconds
 }
 
-// CommitsEqual tests if two `GitCommit`s are equal
-func CommitsEqual(a *pb.GitCommit, b *pb.GitCommit) bool {
-	if a == nil || b == nil {
-		return a == b
-	}
-
-	return a.Id == b.Id &&
-		bytes.Equal(a.Subject, b.Subject) &&
-		bytes.Equal(a.Body, b.Body) &&
-		AuthorsEqual(a.Author, b.Author) &&
-		AuthorsEqual(a.Committer, b.Committer) &&
-		reflect.DeepEqual(a.ParentIds, b.ParentIds)
-}
-
 // FindLocalBranchCommitAuthorsEqual tests if two `FindLocalBranchCommitAuthor`s are equal
 func FindLocalBranchCommitAuthorsEqual(a *pb.FindLocalBranchCommitAuthor, b *pb.FindLocalBranchCommitAuthor) bool {
 	return bytes.Equal(a.Name, b.Name) &&