patch(vest): Use the same test mutation function everywhere

packages/vest/src/core/test/lib/useTestAtCursor.ts

@@ -55,20 +55,16 @@ export function useTestAtCursor(newTestObject: VestTest): VestTest {
 }
 
 function removeAllNextTestsInIsolate() {
-  const [testObjects, setTestObjects] = useTestObjects();
-
-  const prevTests = testObjects.prev;
-  const current = nestedArray.getCurrent(prevTests, useCurrentPath());
   const cursorAt = useCursor().current();
 
-  current.splice(cursorAt);
   // We actually don't mind mutating the state directly (as can be seen above). There is no harm in it
   // since we're only touching the "prev" state. The reason we still use the setter function is
   // to prevent future headaches if we ever do need to rely on prev-state immutability.
-  setTestObjects(({ current }) => ({
-    prev: prevTests,
-    current,
-  }));
+
+  useSetTests(current => {
+    current.splice(cursorAt);
+    return current;
+  });
 }
 
 export function useSetTestAtCursor(testObject: VestTest): void {

packages/vest/testUtils/testObjects.ts

@@ -1,21 +1,14 @@
 import VestTest from 'VestTest';
-import { useTestObjects } from 'stateHooks';
+import { useSetTests } from 'stateHooks';
 
 export function emptyTestObjects(): void {
-  const [, setTestObjects] = useTestObjects();
-  setTestObjects(({ prev }) => ({ prev, current: [] }));
+  useSetTests(() => []);
 }
 
 export function setTestObjects(...args: VestTest[]): void {
-  const [, setTestObjects] = useTestObjects();
-  setTestObjects(({ prev }) => ({ prev, current: [...args] }));
+  useSetTests(() => [...args]);
 }
 
 export function addTestObject(addedTests: VestTest[] | VestTest): void {
-  const [, setTestObjects] = useTestObjects();
-
-  setTestObjects(({ prev, current }) => ({
-    prev,
-    current: current.concat(addedTests),
-  }));
+  useSetTests(tests => tests.concat(addedTests));
 }