Skip to content

Commit 49df908

Browse files
authored
Fix cherrypick demo (#3287)
Cherrypick selections are now cleared after pasting (#3240), so the demo needs a tiny change to reflect that. - **PR Description** The cherry pick demo is failing after the changes in #3240. This is just a small update to that demo to reflect the new (and more convenient) behavior from #3240. - **Please check if the PR fulfills these requirements** * [x] Cheatsheets are up-to-date (run `go generate ./...`) * [x] Code has been formatted (see [here](https://github.com/jesseduffield/lazygit/blob/master/CONTRIBUTING.md#code-formatting)) * [x] Tests have been added/updated (see [here](https://github.com/jesseduffield/lazygit/blob/master/pkg/integration/README.md) for the integration test guide) * [ ] Text is internationalised (see [here](https://github.com/jesseduffield/lazygit/blob/master/CONTRIBUTING.md#internationalisation)) * [ ] Docs (specifically `docs/Config.md`) have been updated if necessary * [x] You've read through your own file changes for silly mistakes etc <!-- Be sure to name your PR with an imperative e.g. 'Add worktrees view' see https://github.com/jesseduffield/lazygit/releases/tag/v0.40.0 for examples -->
2 parents 607034a + fdc54b7 commit 49df908

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

pkg/integration/tests/demo/cherry_pick.go

-5
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,6 @@ var CherryPick = NewIntegrationTest(NewIntegrationTestArgs{
8181
Contains("Integrate support for markdown in user posts"),
8282
Contains("Fix bug in timezone conversion."),
8383
).
84-
Tap(func() {
85-
// we need to manually exit out of cherry pick mode
86-
t.Views().Information().Content(Contains("2 commits copied"))
87-
}).
88-
PressEscape().
8984
Tap(func() {
9085
t.Views().Information().Content(DoesNotContain("commits copied"))
9186
})

0 commit comments

Comments
 (0)