diff --git a/pkg/commands/commit_list_builder.go b/pkg/commands/commit_list_builder.go index 1c22ff711..73d52e4f7 100644 --- a/pkg/commands/commit_list_builder.go +++ b/pkg/commands/commit_list_builder.go @@ -288,7 +288,6 @@ func (c *CommitListBuilder) getLog(limit bool) string { limitFlag = "-30" } - c.Log.Warn(fmt.Sprintf("git log --oneline %s --abbrev=%d", limitFlag, 20)) result, err := c.OSCommand.RunCommandWithOutput(fmt.Sprintf("git log --oneline %s --abbrev=%d", limitFlag, 20)) if err != nil { // assume if there is an error there are no commits yet for this branch diff --git a/pkg/commands/commit_list_builder_test.go b/pkg/commands/commit_list_builder_test.go index 8b843d849..e23bc5308 100644 --- a/pkg/commands/commit_list_builder_test.go +++ b/pkg/commands/commit_list_builder_test.go @@ -163,7 +163,7 @@ func TestCommitListBuilderGetLog(t *testing.T) { "Retrieves logs", func(cmd string, args ...string) *exec.Cmd { assert.EqualValues(t, "git", cmd) - assert.EqualValues(t, []string{"log", "--oneline", "-30"}, args) + assert.EqualValues(t, []string{"log", "--oneline", "-30", "--abbrev=20"}, args) return exec.Command("echo", "6f0b32f commands/git : add GetCommits tests refactor\n9d9d775 circle : remove new line") }, @@ -175,7 +175,7 @@ func TestCommitListBuilderGetLog(t *testing.T) { "An error occurred when retrieving logs", func(cmd string, args ...string) *exec.Cmd { assert.EqualValues(t, "git", cmd) - assert.EqualValues(t, []string{"log", "--oneline", "-30"}, args) + assert.EqualValues(t, []string{"log", "--oneline", "-30", "--abbrev=20"}, args) return exec.Command("test") }, func(output string) { @@ -212,7 +212,7 @@ func TestCommitListBuilderGetCommits(t *testing.T) { assert.EqualValues(t, []string{"rev-list", "@{u}..HEAD", "--abbrev-commit"}, args) return exec.Command("echo") case "log": - assert.EqualValues(t, []string{"log", "--oneline", "-30"}, args) + assert.EqualValues(t, []string{"log", "--oneline", "-30", "--abbrev=20"}, args) return exec.Command("echo") case "merge-base": assert.EqualValues(t, []string{"merge-base", "HEAD", "master"}, args) @@ -239,7 +239,7 @@ func TestCommitListBuilderGetCommits(t *testing.T) { assert.EqualValues(t, []string{"rev-list", "@{u}..HEAD", "--abbrev-commit"}, args) return exec.Command("echo", "8a2bb0e") case "log": - assert.EqualValues(t, []string{"log", "--oneline", "-30"}, args) + assert.EqualValues(t, []string{"log", "--oneline", "-30", "--abbrev=20"}, args) return exec.Command("echo", "8a2bb0e commit 1\n78976bc commit 2") case "merge-base": assert.EqualValues(t, []string{"merge-base", "HEAD", "master"}, args) @@ -280,7 +280,7 @@ func TestCommitListBuilderGetCommits(t *testing.T) { assert.EqualValues(t, []string{"rev-list", "@{u}..HEAD", "--abbrev-commit"}, args) return exec.Command("echo", "8a2bb0e") case "log": - assert.EqualValues(t, []string{"log", "--oneline", "-30"}, args) + assert.EqualValues(t, []string{"log", "--oneline", "-30", "--abbrev=20"}, args) return exec.Command("echo", "8a2bb0e commit 1\n78976bc commit 2") case "merge-base": assert.EqualValues(t, []string{"merge-base", "HEAD", "master"}, args)