From eb4377fadf285ab949e07655169e87a6cf8ef6bd Mon Sep 17 00:00:00 2001
From: Josh Larson <jlarson@mbta.com>
Date: Thu, 23 May 2024 17:41:09 -0400
Subject: [PATCH] fix: Remove auto-close feature from RoutePropertiesCard
 (#2619)

---
 .../mapPage/routePropertiesCard.tsx           |  1 -
 .../mapPage/routePropertiesCard.test.tsx      | 24 +++++++++++++++++--
 2 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/assets/src/components/mapPage/routePropertiesCard.tsx b/assets/src/components/mapPage/routePropertiesCard.tsx
index 6069bdfa0..f43589ece 100644
--- a/assets/src/components/mapPage/routePropertiesCard.tsx
+++ b/assets/src/components/mapPage/routePropertiesCard.tsx
@@ -291,7 +291,6 @@ const RoutePropertiesCard = ({
           selectedRoutePatternId={selectedRoutePattern.id}
           selectRoutePattern={(routePattern: RoutePattern) => {
             selectRoutePattern(routePattern)
-            setOpenedDetails(null)
           }}
         />
       </DetailSection>
diff --git a/assets/tests/components/mapPage/routePropertiesCard.test.tsx b/assets/tests/components/mapPage/routePropertiesCard.test.tsx
index ac4e5d75a..a2a4ccde8 100644
--- a/assets/tests/components/mapPage/routePropertiesCard.test.tsx
+++ b/assets/tests/components/mapPage/routePropertiesCard.test.tsx
@@ -302,7 +302,7 @@ describe("<RoutePropertiesCard/>", () => {
       )
     })
 
-    test("Clicking a different route pattern calls selectRoutePattern and closes the variants list", async () => {
+    test("Clicking a different route pattern calls selectRoutePattern", async () => {
       const mockSelectRoutePattern = jest.fn()
 
       render(
@@ -321,7 +321,27 @@ describe("<RoutePropertiesCard/>", () => {
 
       await userEvent.click(routePattern2Radio)
       expect(mockSelectRoutePattern).toHaveBeenCalledWith(routePattern2)
-      expect(routePattern2Radio).not.toBeVisible()
+    })
+
+    test("Clicking a different route pattern does not close the variants list", async () => {
+      const mockSelectRoutePattern = jest.fn()
+
+      render(
+        <RoutePropertiesCardWithDefaults
+          selectRoutePattern={mockSelectRoutePattern}
+        />
+      )
+
+      await userEvent.click(
+        screen.getByRole("button", { name: "Show variants" })
+      )
+
+      const routePattern2Radio = screen.getByRole("radio", {
+        name: new RegExp(patternDisplayName(routePattern2).name),
+      })
+
+      await userEvent.click(routePattern2Radio)
+      expect(routePattern2Radio).toBeVisible()
     })
 
     test("Clicking the close button calls onClose prop", async () => {