From 787f9966eca677a47ce9f87ce152fc7f601951a6 Mon Sep 17 00:00:00 2001
From: Stefan Haller <stefan@haller-berlin.de>
Date: Mon, 9 Oct 2023 14:23:29 +0200
Subject: [PATCH] Fix crash when trying to filter the list of remotes

This seems to be a left-over from an earlier iteration of the code. Removing it
fixes the crash.
---
 pkg/gui/context/remotes_context.go            |  1 -
 .../tests/filter_and_search/filter_remotes.go | 42 +++++++++++++++++++
 pkg/integration/tests/test_list.go            |  1 +
 3 files changed, 43 insertions(+), 1 deletion(-)
 create mode 100644 pkg/integration/tests/filter_and_search/filter_remotes.go

diff --git a/pkg/gui/context/remotes_context.go b/pkg/gui/context/remotes_context.go
index 38be54149..035fb2321 100644
--- a/pkg/gui/context/remotes_context.go
+++ b/pkg/gui/context/remotes_context.go
@@ -9,7 +9,6 @@ import (
 type RemotesContext struct {
 	*FilteredListViewModel[*models.Remote]
 	*ListContextTrait
-	*SearchHistory
 }
 
 var (
diff --git a/pkg/integration/tests/filter_and_search/filter_remotes.go b/pkg/integration/tests/filter_and_search/filter_remotes.go
new file mode 100644
index 000000000..b905a06ed
--- /dev/null
+++ b/pkg/integration/tests/filter_and_search/filter_remotes.go
@@ -0,0 +1,42 @@
+package filter_and_search
+
+import (
+	"github.com/jesseduffield/lazygit/pkg/config"
+	. "github.com/jesseduffield/lazygit/pkg/integration/components"
+)
+
+var FilterRemotes = NewIntegrationTest(NewIntegrationTestArgs{
+	Description:  "Filtering remotes",
+	ExtraCmdArgs: []string{},
+	Skip:         false,
+	SetupConfig:  func(config *config.AppConfig) {},
+	SetupRepo: func(shell *Shell) {
+		shell.EmptyCommit("commit-one")
+		shell.CloneIntoRemote("remote1")
+		shell.CloneIntoRemote("remote2")
+		shell.CloneIntoRemote("remote3")
+	},
+	Run: func(t *TestDriver, keys config.KeybindingConfig) {
+		t.Views().Remotes().
+			Focus().
+			Lines(
+				Contains("remote1").IsSelected(),
+				Contains("remote2"),
+				Contains("remote3"),
+			).
+			FilterOrSearch("2").
+			Lines(
+				Contains("remote2").IsSelected(),
+			).
+			// cancel the filter
+			PressEscape().
+			Tap(func() {
+				t.Views().Search().IsInvisible()
+			}).
+			Lines(
+				Contains("remote1"),
+				Contains("remote2").IsSelected(),
+				Contains("remote3"),
+			)
+	},
+})
diff --git a/pkg/integration/tests/test_list.go b/pkg/integration/tests/test_list.go
index 7759dcda7..521656d10 100644
--- a/pkg/integration/tests/test_list.go
+++ b/pkg/integration/tests/test_list.go
@@ -131,6 +131,7 @@ var tests = []*components.IntegrationTest{
 	filter_and_search.FilterMenu,
 	filter_and_search.FilterMenuCancelFilterWithEscape,
 	filter_and_search.FilterRemoteBranches,
+	filter_and_search.FilterRemotes,
 	filter_and_search.FilterSearchHistory,
 	filter_and_search.NestedFilter,
 	filter_and_search.NestedFilterTransient,