refactor: reorder test cases for clarity

Parkreiner · Parkreiner · commit f8f87a3d3bab · 2024-03-03T18:26:28.000Z
diff --git a/site/src/hooks/useSearchParamsKey.test.ts b/site/src/hooks/useSearchParamsKey.test.ts
@@ -75,6 +75,24 @@ describe(useSearchParamsKey.name, () => {
     expect(search.get(key)).toEqual(null);
   });
 
+  it("Will dispatch state changes through custom URLSearchParams value if provided", async () => {
+    const key = "love";
+    const initialValue = "dogs";
+    const customParams = new URLSearchParams({ [key]: initialValue });
+
+    const { result } = await renderHookWithAuth(
+      ({ key }) => useSearchParamsKey({ key, searchParams: customParams }),
+      {
+        routingOptions: { route: `/?=${key}=${initialValue}` },
+        renderOptions: { initialProps: { key } },
+      },
+    );
+
+    const newValue = "all animals";
+    void act(() => result.current.setValue(newValue));
+    await waitFor(() => expect(customParams.get(key)).toEqual(newValue));
+  });
+
   it("Does not have methods change previous values if 'key' argument changes during re-renders", async () => {
     const readonlyKey = "readonlyKey";
     const mutableKey = "mutableKey";
@@ -100,22 +118,4 @@ describe(useSearchParamsKey.name, () => {
     expect(search.get(readonlyKey)).toEqual(initialReadonlyValue);
     expect(search.get(mutableKey)).toEqual(swapValue);
   });
-
-  it("Will dispatch state changes through custom URLSearchParams value if provided", async () => {
-    const key = "love";
-    const initialValue = "dogs";
-    const customParams = new URLSearchParams({ [key]: initialValue });
-
-    const { result } = await renderHookWithAuth(
-      ({ key }) => useSearchParamsKey({ key, searchParams: customParams }),
-      {
-        routingOptions: { route: `/?=${key}=${initialValue}` },
-        renderOptions: { initialProps: { key } },
-      },
-    );
-
-    const newValue = "all animals";
-    void act(() => result.current.setValue(newValue));
-    await waitFor(() => expect(customParams.get(key)).toEqual(newValue));
-  });
 });
