mirror of
https://github.com/jesseduffield/lazygit.git
synced 2025-08-07 22:02:56 +03:00
Add test demonstrating the desired behavior
It has two modified hunks, one for a master commit and one for a branch commit. Currently we get an error mentioning those two commits, but we would like to silently select the branch commit.
This commit is contained in:
@@ -0,0 +1,56 @@
|
|||||||
|
package commit
|
||||||
|
|
||||||
|
import (
|
||||||
|
"github.com/jesseduffield/lazygit/pkg/config"
|
||||||
|
. "github.com/jesseduffield/lazygit/pkg/integration/components"
|
||||||
|
)
|
||||||
|
|
||||||
|
var FindBaseCommitForFixupDisregardMainBranch = NewIntegrationTest(NewIntegrationTestArgs{
|
||||||
|
Description: "Finds the base commit to create a fixup for, disregarding changes to a commit that is already on master",
|
||||||
|
ExtraCmdArgs: []string{},
|
||||||
|
Skip: false,
|
||||||
|
SetupConfig: func(config *config.AppConfig) {},
|
||||||
|
SetupRepo: func(shell *Shell) {
|
||||||
|
shell.
|
||||||
|
EmptyCommit("1st commit").
|
||||||
|
CreateFileAndAdd("file1", "file1 content\n").
|
||||||
|
Commit("2nd commit").
|
||||||
|
NewBranch("mybranch").
|
||||||
|
CreateFileAndAdd("file2", "file2 content\n").
|
||||||
|
Commit("3rd commit").
|
||||||
|
EmptyCommit("4th commit").
|
||||||
|
UpdateFile("file1", "file1 changed content").
|
||||||
|
UpdateFile("file2", "file2 changed content")
|
||||||
|
},
|
||||||
|
Run: func(t *TestDriver, keys config.KeybindingConfig) {
|
||||||
|
t.Views().Commits().
|
||||||
|
Lines(
|
||||||
|
Contains("4th commit").IsSelected(),
|
||||||
|
Contains("3rd commit"),
|
||||||
|
Contains("2nd commit"),
|
||||||
|
Contains("1st commit"),
|
||||||
|
)
|
||||||
|
|
||||||
|
t.Views().Files().
|
||||||
|
Focus().
|
||||||
|
Press(keys.Files.FindBaseCommitForFixup)
|
||||||
|
|
||||||
|
/* EXPECTED:
|
||||||
|
t.Views().Commits().
|
||||||
|
IsFocused().
|
||||||
|
Lines(
|
||||||
|
Contains("4th commit"),
|
||||||
|
Contains("3rd commit").IsSelected(),
|
||||||
|
Contains("2nd commit"),
|
||||||
|
Contains("1st commit"),
|
||||||
|
)
|
||||||
|
*/
|
||||||
|
|
||||||
|
// ACTUAL:
|
||||||
|
t.ExpectPopup().Alert().Title(Equals("Error")).Content(
|
||||||
|
Contains("Multiple base commits found.").
|
||||||
|
Contains("2nd commit").
|
||||||
|
Contains("3rd commit"),
|
||||||
|
).Confirm()
|
||||||
|
},
|
||||||
|
})
|
@@ -82,6 +82,7 @@ var tests = []*components.IntegrationTest{
|
|||||||
commit.CreateTag,
|
commit.CreateTag,
|
||||||
commit.DiscardOldFileChanges,
|
commit.DiscardOldFileChanges,
|
||||||
commit.FindBaseCommitForFixup,
|
commit.FindBaseCommitForFixup,
|
||||||
|
commit.FindBaseCommitForFixupDisregardMainBranch,
|
||||||
commit.FindBaseCommitForFixupOnlyAddedLines,
|
commit.FindBaseCommitForFixupOnlyAddedLines,
|
||||||
commit.FindBaseCommitForFixupWarningForAddedLines,
|
commit.FindBaseCommitForFixupWarningForAddedLines,
|
||||||
commit.Highlight,
|
commit.Highlight,
|
||||||
|
Reference in New Issue
Block a user