From 379486b5ea4d4ab2e2faae5a96c79276e374234f Mon Sep 17 00:00:00 2001
From: Ola Hungerford <olahungerford@gmail.com>
Date: Mon, 24 Mar 2025 09:43:48 -0700
Subject: [PATCH] Add failing test for pull/206

---
 .../src/components/__tests__/Sidebar.test.tsx | 25 +++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/client/src/components/__tests__/Sidebar.test.tsx b/client/src/components/__tests__/Sidebar.test.tsx
index 8c0b313..fc8a5f4 100644
--- a/client/src/components/__tests__/Sidebar.test.tsx
+++ b/client/src/components/__tests__/Sidebar.test.tsx
@@ -161,6 +161,31 @@ describe("Sidebar Environment Variables", () => {
         NEW_THIRD_KEY: "third_value",
       });
     });
+
+    it("should maintain order during key editing", () => {
+      const setEnv = jest.fn();
+      const initialEnv = {
+        KEY1: "value1",
+        KEY2: "value2"
+      };
+      renderSidebar({ env: initialEnv, setEnv });
+
+      openEnvVarsSection();
+
+      // Type "NEW_" one character at a time
+      const key1Input = screen.getByDisplayValue("KEY1");
+      "NEW_".split("").forEach((char) => {
+        fireEvent.change(key1Input, { target: { value: char + "KEY1".slice(1) } });
+      });
+
+      // Verify the last setEnv call maintains the order
+      const lastCall = setEnv.mock.calls[setEnv.mock.calls.length - 1][0] as Record<string, string>;
+      const entries = Object.entries(lastCall);
+      
+      // The values should stay with their original keys
+      expect(entries[0][1]).toBe("value1"); // First entry should still have value1
+      expect(entries[1][1]).toBe("value2"); // Second entry should still have value2
+    });
   });
 
   describe("Multiple Operations", () => {