go/lazygit
1
0
Fork 0
mirror of https://github.com/jesseduffield/lazygit.git synced 2024-12-12 11:15:00 +02:00
Code Issues Releases Activity
d2d50aedd0
lazygit/pkg/commands/git_commands/commit_loader_test.go

252 lines
9.6 KiB
Go
Raw Normal View History

Merge loaders package into git_commands package
2022-11-11 04:19:29 +02:00
package git_commands
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
import (
refactor
2021-12-30 02:22:29 +02:00
"path/filepath"
fix loaders
2022-03-26 14:55:44 +02:00
"strings"
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
"testing"
updating specs
2021-12-30 04:11:58 +02:00
"github.com/jesseduffield/lazygit/pkg/commands/models"
"github.com/jesseduffield/lazygit/pkg/commands/oscommands"
more refactoring
2021-12-30 04:35:10 +02:00
"github.com/jesseduffield/lazygit/pkg/commands/types/enums"
move OS commands into their own package
2020-09-29 11:10:57 +02:00
"github.com/jesseduffield/lazygit/pkg/utils"
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
"github.com/stretchr/testify/assert"
)
add author email to commits
2022-05-08 06:23:29 +02:00
var commitsOutput = strings.Replace(`0eea75e8c631fba6b58135697835d58ba4c18dbc|1640826609|Jesse Duffield|jessedduffield@gmail.com| (HEAD -> better-tests)|b21997d6b4cbdf84b149|better typing for rebase mode
b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164|1640824515|Jesse Duffield|jessedduffield@gmail.com| (origin/better-tests)|e94e8fc5b6fab4cb755f|fix logging
e94e8fc5b6fab4cb755f29f1bdb3ee5e001df35c|1640823749|Jesse Duffield|jessedduffield@gmail.com||d8084cd558925eb7c9c3|refactor
d8084cd558925eb7c9c38afeed5725c21653ab90|1640821426|Jesse Duffield|jessedduffield@gmail.com||65f910ebd85283b5cce9|WIP
65f910ebd85283b5cce9bf67d03d3f1a9ea3813a|1640821275|Jesse Duffield|jessedduffield@gmail.com||26c07b1ab33860a1a759|WIP
26c07b1ab33860a1a7591a0638f9925ccf497ffa|1640750752|Jesse Duffield|jessedduffield@gmail.com||3d4470a6c072208722e5|WIP
3d4470a6c072208722e5ae9a54bcb9634959a1c5|1640748818|Jesse Duffield|jessedduffield@gmail.com||053a66a7be3da43aacdc|WIP
053a66a7be3da43aacdc7aa78e1fe757b82c4dd2|1640739815|Jesse Duffield|jessedduffield@gmail.com||985fe482e806b172aea4|refactoring the config struct`, "|", "\x00", -1)
updating specs
2021-12-30 04:11:58 +02:00
func TestGetCommits(t *testing.T) {
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
type scenario struct {
updating specs
2021-12-30 04:11:58 +02:00
testName string
fix reflog failing to properly refresh
2022-01-26 01:34:56 +02:00
runner *oscommands.FakeCmdObjRunner
updating specs
2021-12-30 04:11:58 +02:00
expectedCommits []*models.Commit
expectedError error
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
logOrder string
more refactoring
2021-12-30 04:35:10 +02:00
rebaseMode enums.RebaseMode
updating specs
2021-12-30 04:11:58 +02:00
currentBranchName string
opts GetCommitsOptions
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
}
scenarios := []scenario{
{
updating specs
2021-12-30 04:11:58 +02:00
testName: "should return no commits if there are none",
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
logOrder: "topo-order",
more refactoring
2021-12-30 04:35:10 +02:00
rebaseMode: enums.REBASE_MODE_NONE,
updating specs
2021-12-30 04:11:58 +02:00
currentBranchName: "master",
opts: GetCommitsOptions{RefName: "HEAD", IncludeRebaseCommits: false},
runner: oscommands.NewFakeRunner(t).
Expect(`git merge-base "HEAD" "HEAD"@{u}`, "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164", nil).
fix(commit_loader): fix log command
2023-03-13 18:11:47 +02:00
Expect(`git log "HEAD" --topo-order --oneline --pretty=format:"%H%x00%at%x00%aN%x00%ae%x00%d%x00%p%x00%s" --abbrev=40 --no-show-signature --`, "", nil),
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
updating specs
2021-12-30 04:11:58 +02:00
expectedCommits: []*models.Commit{},
expectedError: nil,
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
},
Fix yellow/red coloring of pushed/unpushed commits in branch commits panel (#2448)
2023-02-19 01:13:46 +02:00
{
testName: "should use proper upstream name for branch",
logOrder: "topo-order",
rebaseMode: enums.REBASE_MODE_NONE,
currentBranchName: "mybranch",
opts: GetCommitsOptions{RefName: "refs/heads/mybranch", IncludeRebaseCommits: false},
runner: oscommands.NewFakeRunner(t).
Expect(`git merge-base "refs/heads/mybranch" "mybranch"@{u}`, "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164", nil).
fix(commit_loader): fix log command
2023-03-13 18:11:47 +02:00
Expect(`git log "refs/heads/mybranch" --topo-order --oneline --pretty=format:"%H%x00%at%x00%aN%x00%ae%x00%d%x00%p%x00%s" --abbrev=40 --no-show-signature --`, "", nil),
Fix yellow/red coloring of pushed/unpushed commits in branch commits panel (#2448)
2023-02-19 01:13:46 +02:00
expectedCommits: []*models.Commit{},
expectedError: nil,
},
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
{
updating specs
2021-12-30 04:11:58 +02:00
testName: "should return commits if they are present",
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
logOrder: "topo-order",
more refactoring
2021-12-30 04:35:10 +02:00
rebaseMode: enums.REBASE_MODE_NONE,
updating specs
2021-12-30 04:11:58 +02:00
currentBranchName: "master",
opts: GetCommitsOptions{RefName: "HEAD", IncludeRebaseCommits: false},
runner: oscommands.NewFakeRunner(t).
// here it's seeing which commits are yet to be pushed
Expect(`git merge-base "HEAD" "HEAD"@{u}`, "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164", nil).
// here it's actually getting all the commits in a formatted form, one per line
fix(commit_loader): fix log command
2023-03-13 18:11:47 +02:00
Expect(`git log "HEAD" --topo-order --oneline --pretty=format:"%H%x00%at%x00%aN%x00%ae%x00%d%x00%p%x00%s" --abbrev=40 --no-show-signature --`, commitsOutput, nil).
updating specs
2021-12-30 04:11:58 +02:00
// here it's seeing where our branch diverged from the master branch so that we can mark that commit and parent commits as 'merged'
Expect(`git merge-base "HEAD" "master"`, "26c07b1ab33860a1a7591a0638f9925ccf497ffa", nil),
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
updating specs
2021-12-30 04:11:58 +02:00
expectedCommits: []*models.Commit{
{
Sha: "0eea75e8c631fba6b58135697835d58ba4c18dbc",
Name: "better typing for rebase mode",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusUnpushed,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "(HEAD -> better-tests)",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640826609,
Parents: []string{
"b21997d6b4cbdf84b149",
},
},
{
Sha: "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164",
Name: "fix logging",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusPushed,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "(origin/better-tests)",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640824515,
Parents: []string{
"e94e8fc5b6fab4cb755f",
},
},
{
Sha: "e94e8fc5b6fab4cb755f29f1bdb3ee5e001df35c",
Name: "refactor",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusPushed,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640823749,
Parents: []string{
"d8084cd558925eb7c9c3",
},
},
{
Sha: "d8084cd558925eb7c9c38afeed5725c21653ab90",
Name: "WIP",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusPushed,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640821426,
Parents: []string{
"65f910ebd85283b5cce9",
},
},
{
Sha: "65f910ebd85283b5cce9bf67d03d3f1a9ea3813a",
Name: "WIP",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusPushed,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640821275,
Parents: []string{
"26c07b1ab33860a1a759",
},
},
{
Sha: "26c07b1ab33860a1a7591a0638f9925ccf497ffa",
Name: "WIP",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusMerged,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640750752,
Parents: []string{
"3d4470a6c072208722e5",
},
},
{
Sha: "3d4470a6c072208722e5ae9a54bcb9634959a1c5",
Name: "WIP",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusMerged,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640748818,
Parents: []string{
"053a66a7be3da43aacdc",
},
},
{
Sha: "053a66a7be3da43aacdc7aa78e1fe757b82c4dd2",
Name: "refactoring the config struct",
Store commit.Status as an enum instead of a string This is unrelated to the changes in this PR, but since we are doing the same thing for the commit.Action field in the next commit, it makes sense to do it for Status too for consistency. Modelling this as an enum feels more natural than modelling it as a string, since there's a finite set of possible values. And it saves a little bit of memory (not very much, since none of the strings were heap-allocated, but still).
2023-04-03 12:40:29 +02:00
Status: models.StatusMerged,
Store commit.Action as an enum instead of a string The main reason for doing this (besides the reasons given for Status in the previous commit) is that it allows us to easily convert from TodoCommand to Action and back. This will be needed later in the branch. Fortunately, TodoCommand is one-based, so this allows us to add an ActionNone constant with the value 0.
2023-04-03 12:42:29 +02:00
Action: models.ActionNone,
updating specs
2021-12-30 04:11:58 +02:00
Tags: []string{},
ExtraInfo: "",
add author email to commits
2022-05-08 06:23:29 +02:00
AuthorName: "Jesse Duffield",
AuthorEmail: "jessedduffield@gmail.com",
updating specs
2021-12-30 04:11:58 +02:00
UnixTimestamp: 1640739815,
Parents: []string{
"985fe482e806b172aea4",
},
},
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
},
updating specs
2021-12-30 04:11:58 +02:00
expectedError: nil,
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
},
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
{
testName: "should not specify order if `log.order` is `default`",
logOrder: "default",
rebaseMode: enums.REBASE_MODE_NONE,
currentBranchName: "master",
opts: GetCommitsOptions{RefName: "HEAD", IncludeRebaseCommits: false},
runner: oscommands.NewFakeRunner(t).
Expect(`git merge-base "HEAD" "HEAD"@{u}`, "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164", nil).
fix(commit_loader): fix log command
2023-03-13 18:11:47 +02:00
Expect(`git log "HEAD" --oneline --pretty=format:"%H%x00%at%x00%aN%x00%ae%x00%d%x00%p%x00%s" --abbrev=40 --no-show-signature --`, "", nil),
expectedCommits: []*models.Commit{},
expectedError: nil,
},
{
testName: "should set filter path",
logOrder: "default",
rebaseMode: enums.REBASE_MODE_NONE,
currentBranchName: "master",
opts: GetCommitsOptions{RefName: "HEAD", FilterPath: "src"},
runner: oscommands.NewFakeRunner(t).
Expect(`git merge-base "HEAD" "HEAD"@{u}`, "b21997d6b4cbdf84b149d8e6a2c4d06a8e9ec164", nil).
Expect(`git log "HEAD" --oneline --pretty=format:"%H%x00%at%x00%aN%x00%ae%x00%d%x00%p%x00%s" --abbrev=40 --follow --no-show-signature -- "src"`, "", nil),
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
expectedCommits: []*models.Commit{},
expectedError: nil,
},
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
}
updating specs
2021-12-30 04:11:58 +02:00
for _, scenario := range scenarios {
add some more linters
2022-01-08 06:46:35 +02:00
scenario := scenario
updating specs
2021-12-30 04:11:58 +02:00
t.Run(scenario.testName, func(t *testing.T) {
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
common := utils.NewDummyCommon()
common.UserConfig.Git.Log.Order = scenario.logOrder
more refactoring
2021-12-30 04:35:10 +02:00
builder := &CommitLoader{
feat(log): allow to disable `git.log.order`
2023-01-28 14:10:57 +02:00
Common: common,
WIP
2021-12-30 08:19:01 +02:00
cmd: oscommands.NewDummyCmdObjBuilder(scenario.runner),
fix: fix ambiguous branch name test: add an integration test for checkout branch by name fix: fix full ref name of detached head refactor: refactor current branch loader chore: use field name explicitly
2022-10-16 14:31:42 +02:00
getCurrentBranchInfo: func() (BranchInfo, error) {
return BranchInfo{RefName: scenario.currentBranchName, DisplayName: scenario.currentBranchName, DetachedHead: false}, nil
updating specs
2021-12-30 04:11:58 +02:00
},
more refactoring
2021-12-30 04:35:10 +02:00
getRebaseMode: func() (enums.RebaseMode, error) { return scenario.rebaseMode, nil },
updating specs
2021-12-30 04:11:58 +02:00
dotGitDir: ".git",
readFile: func(filename string) ([]byte, error) {
return []byte(""), nil
},
walkFiles: func(root string, fn filepath.WalkFunc) error {
return nil
},
}
commits, err := builder.GetCommits(scenario.opts)
assert.Equal(t, scenario.expectedCommits, commits)
assert.Equal(t, scenario.expectedError, err)
fix reflog failing to properly refresh
2022-01-26 01:34:56 +02:00
scenario.runner.CheckForMissingCalls()
fix up tests This fixes up some git and oscommand tests, and pulls some tests into commit_list_builder_test.go I've also made the NewDummyBlah functions public so that I didn't need to duplicate them across packages I've also given OSCommand a SetCommand() method for setting the command on the struct I've also created a file utils.go in the test package for creating convient 'CommandSwapper's, which basically enable you to assert a sequence of commands on the command line, and swap each one out for a different one to actually be executed
2019-03-02 04:08:09 +02:00
})
}
}
Reference in New Issue Copy Permalink