From a27607b28b3896e5b893bcdd57d9f6ba39f17812 Mon Sep 17 00:00:00 2001
From: Erik Michelson <github@erik.michelson.eu>
Date: Tue, 21 Jul 2026 19:51:26 +0200
Subject: [PATCH] test: update snapshots and splitter tests

Signed-off-by: Erik Michelson <github@erik.michelson.eu>
---
 .../__snapshots__/splitter.spec.tsx.snap      | 232 ++++++++++++++++--
 .../editor-page/splitter/splitter.spec.tsx    |  25 +-
 2 files changed, 238 insertions(+), 19 deletions(-)

diff --git a/frontend/src/components/editor-page/splitter/__snapshots__/splitter.spec.tsx.snap b/frontend/src/components/editor-page/splitter/__snapshots__/splitter.spec.tsx.snap
index 24370ada7..1eab3006d 100644
--- a/frontend/src/components/editor-page/splitter/__snapshots__/splitter.spec.tsx.snap
+++ b/frontend/src/components/editor-page/splitter/__snapshots__/splitter.spec.tsx.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Splitter resize can change size with mouse 1`] = `
+exports[`Splitter resize can change size with mouse: mouse initial 1`] = `
 <div>
   <div
     class="flex-fill flex-row d-flex "
@@ -68,6 +68,210 @@ exports[`Splitter resize can change size with mouse 1`] = `
 </div>
 `;
 
+exports[`Splitter resize can change size with mouse: mouse move to left 1`] = `
+<div>
+  <div
+    class="flex-fill flex-row d-flex "
+  >
+    <div
+      class="left"
+      id="editor-edit-pane"
+      style="width: calc(0% - 5px);"
+    >
+      <div
+        class="inner"
+      >
+        left
+      </div>
+    </div>
+    <div
+      aria-orientation="vertical"
+      aria-valuemax="100"
+      aria-valuemin="0"
+      aria-valuenow="0"
+      class="divider"
+      data-testid="splitter-divider"
+      id="editor-splitter"
+      role="separator"
+      tabindex="0"
+    >
+      <div
+        class="middle shift-right"
+      >
+        <div
+          class="buttons"
+        >
+          <button
+            class="btn btn-secondary"
+            type="button"
+          >
+            BootstrapIconMock_ArrowLeft
+          </button>
+          <span
+            class="grabber"
+          >
+            BootstrapIconMock_ArrowLeftRight
+          </span>
+          <button
+            class="btn btn-light"
+            type="button"
+          >
+            BootstrapIconMock_ArrowRight
+          </button>
+        </div>
+      </div>
+    </div>
+    <div
+      class="right"
+      id="editor-view-pane"
+      style="width: calc(100% - 0%);"
+    >
+      <div
+        class="inner"
+      >
+        right
+      </div>
+    </div>
+  </div>
+</div>
+`;
+
+exports[`Splitter resize can change size with mouse: mouse move to middle 1`] = `
+<div>
+  <div
+    class="flex-fill flex-row d-flex "
+  >
+    <div
+      class="left"
+      id="editor-edit-pane"
+      style="width: calc(50% - 5px);"
+    >
+      <div
+        class="inner"
+      >
+        left
+      </div>
+    </div>
+    <div
+      aria-orientation="vertical"
+      aria-valuemax="100"
+      aria-valuemin="0"
+      aria-valuenow="50"
+      class="divider"
+      data-testid="splitter-divider"
+      id="editor-splitter"
+      role="separator"
+      tabindex="0"
+    >
+      <div
+        class="middle"
+      >
+        <div
+          class="buttons"
+        >
+          <button
+            class="btn btn-light"
+            type="button"
+          >
+            BootstrapIconMock_ArrowLeft
+          </button>
+          <span
+            class="grabber"
+          >
+            BootstrapIconMock_ArrowLeftRight
+          </span>
+          <button
+            class="btn btn-light"
+            type="button"
+          >
+            BootstrapIconMock_ArrowRight
+          </button>
+        </div>
+      </div>
+    </div>
+    <div
+      class="right"
+      id="editor-view-pane"
+      style="width: calc(100% - 50%);"
+    >
+      <div
+        class="inner"
+      >
+        right
+      </div>
+    </div>
+  </div>
+</div>
+`;
+
+exports[`Splitter resize can change size with mouse: mouse move to right 1`] = `
+<div>
+  <div
+    class="flex-fill flex-row d-flex "
+  >
+    <div
+      class="left"
+      id="editor-edit-pane"
+      style="width: calc(100% - 5px);"
+    >
+      <div
+        class="inner"
+      >
+        left
+      </div>
+    </div>
+    <div
+      aria-orientation="vertical"
+      aria-valuemax="100"
+      aria-valuemin="0"
+      aria-valuenow="100"
+      class="divider"
+      data-testid="splitter-divider"
+      id="editor-splitter"
+      role="separator"
+      tabindex="0"
+    >
+      <div
+        class="middle shift-left"
+      >
+        <div
+          class="buttons"
+        >
+          <button
+            class="btn btn-light"
+            type="button"
+          >
+            BootstrapIconMock_ArrowLeft
+          </button>
+          <span
+            class="grabber"
+          >
+            BootstrapIconMock_ArrowLeftRight
+          </span>
+          <button
+            class="btn btn-secondary"
+            type="button"
+          >
+            BootstrapIconMock_ArrowRight
+          </button>
+        </div>
+      </div>
+    </div>
+    <div
+      class="right"
+      id="editor-view-pane"
+      style="width: calc(100% - 100%);"
+    >
+      <div
+        class="inner"
+      >
+        right
+      </div>
+    </div>
+  </div>
+</div>
+`;
+
 exports[`Splitter resize can change size with touch: touch initial 1`] = `
 <div>
   <div
@@ -144,7 +348,7 @@ exports[`Splitter resize can change size with touch: touch move to left 1`] = `
     <div
       class="left"
       id="editor-edit-pane"
-      style="width: calc(50% - 5px);"
+      style="width: calc(0% - 5px);"
     >
       <div
         class="inner"
@@ -156,7 +360,7 @@ exports[`Splitter resize can change size with touch: touch move to left 1`] = `
       aria-orientation="vertical"
       aria-valuemax="100"
       aria-valuemin="0"
-      aria-valuenow="50"
+      aria-valuenow="0"
       class="divider"
       data-testid="splitter-divider"
       id="editor-splitter"
@@ -164,13 +368,13 @@ exports[`Splitter resize can change size with touch: touch move to left 1`] = `
       tabindex="0"
     >
       <div
-        class="middle"
+        class="middle shift-right"
       >
         <div
           class="buttons"
         >
           <button
-            class="btn btn-light"
+            class="btn btn-secondary"
             type="button"
           >
             BootstrapIconMock_ArrowLeft
@@ -192,7 +396,7 @@ exports[`Splitter resize can change size with touch: touch move to left 1`] = `
     <div
       class="right"
       id="editor-view-pane"
-      style="width: calc(100% - 50%);"
+      style="width: calc(100% - 0%);"
     >
       <div
         class="inner"
@@ -212,7 +416,7 @@ exports[`Splitter resize can change size with touch: touch move to middle 1`] =
     <div
       class="left"
       id="editor-edit-pane"
-      style="width: calc(50% - 5px);"
+      style="width: calc(26% - 5px);"
     >
       <div
         class="inner"
@@ -224,7 +428,7 @@ exports[`Splitter resize can change size with touch: touch move to middle 1`] =
       aria-orientation="vertical"
       aria-valuemax="100"
       aria-valuemin="0"
-      aria-valuenow="50"
+      aria-valuenow="26"
       class="divider"
       data-testid="splitter-divider"
       id="editor-splitter"
@@ -260,7 +464,7 @@ exports[`Splitter resize can change size with touch: touch move to middle 1`] =
     <div
       class="right"
       id="editor-view-pane"
-      style="width: calc(100% - 50%);"
+      style="width: calc(100% - 26%);"
     >
       <div
         class="inner"
@@ -280,7 +484,7 @@ exports[`Splitter resize can change size with touch: touch move to right 1`] = `
     <div
       class="left"
       id="editor-edit-pane"
-      style="width: calc(50% - 5px);"
+      style="width: calc(100% - 5px);"
     >
       <div
         class="inner"
@@ -292,7 +496,7 @@ exports[`Splitter resize can change size with touch: touch move to right 1`] = `
       aria-orientation="vertical"
       aria-valuemax="100"
       aria-valuemin="0"
-      aria-valuenow="50"
+      aria-valuenow="100"
       class="divider"
       data-testid="splitter-divider"
       id="editor-splitter"
@@ -300,7 +504,7 @@ exports[`Splitter resize can change size with touch: touch move to right 1`] = `
       tabindex="0"
     >
       <div
-        class="middle"
+        class="middle shift-left"
       >
         <div
           class="buttons"
@@ -317,7 +521,7 @@ exports[`Splitter resize can change size with touch: touch move to right 1`] = `
             BootstrapIconMock_ArrowLeftRight
           </span>
           <button
-            class="btn btn-light"
+            class="btn btn-secondary"
             type="button"
           >
             BootstrapIconMock_ArrowRight
@@ -328,7 +532,7 @@ exports[`Splitter resize can change size with touch: touch move to right 1`] = `
     <div
       class="right"
       id="editor-view-pane"
-      style="width: calc(100% - 50%);"
+      style="width: calc(100% - 100%);"
     >
       <div
         class="inner"
diff --git a/frontend/src/components/editor-page/splitter/splitter.spec.tsx b/frontend/src/components/editor-page/splitter/splitter.spec.tsx
index 508c70230..7bfa452b3 100644
--- a/frontend/src/components/editor-page/splitter/splitter.spec.tsx
+++ b/frontend/src/components/editor-page/splitter/splitter.spec.tsx
@@ -60,7 +60,7 @@ describe('Splitter', () => {
         editorConfig: { splitPosition: 50 } as EditorConfig
       })
       const view = render(<Splitter left={<>left</>} right={<>right</>} />)
-      expect(view.container).toMatchSnapshot()
+      expect(view.container).toMatchSnapshot('mouse initial')
       const divider = await screen.findByTestId('splitter-divider')
 
       fireEvent.mouseDown(divider, {})
@@ -68,12 +68,18 @@ describe('Splitter', () => {
       fireEvent.mouseMove(moveOverlay, Mock.of<MouseEvent>({ buttons: 1, clientX: 1920 }))
       fireEvent.mouseUp(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(100)
+      mockAppState({ editorConfig: { splitPosition: 100 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
+      expect(view.container).toMatchSnapshot('mouse move to right')
 
-      fireEvent.mouseDown(divider, {})
+      fireEvent.mouseDown(await screen.findByTestId('splitter-divider'), {})
       moveOverlay = findMoveOverlay(view.container)
       fireEvent.mouseMove(moveOverlay, Mock.of<MouseEvent>({ buttons: 1, clientX: 0 }))
       fireEvent.mouseUp(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(0)
+      mockAppState({ editorConfig: { splitPosition: 0 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
+      expect(view.container).toMatchSnapshot('mouse move to left')
 
       const grabber = findGrabber(view.container)
       fireEvent.mouseDown(grabber, {})
@@ -81,6 +87,9 @@ describe('Splitter', () => {
       fireEvent.mouseMove(moveOverlay, Mock.of<MouseEvent>({ buttons: 1, clientX: 960 }))
       fireEvent.mouseUp(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(50)
+      mockAppState({ editorConfig: { splitPosition: 50 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
+      expect(view.container).toMatchSnapshot('mouse move to middle')
 
       fireEvent.mouseMove(window, Mock.of<MouseEvent>({ buttons: 1, clientX: 1920 }))
       expect(setEditorSplitPosition).toHaveBeenCalledWith(100)
@@ -118,9 +127,11 @@ describe('Splitter', () => {
       )
       fireEvent.touchEnd(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(100)
-      expect(view.container).toMatchSnapshot('touch move to left')
+      mockAppState({ editorConfig: { splitPosition: 100 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
+      expect(view.container).toMatchSnapshot('touch move to right')
 
-      fireEvent.touchStart(divider, {})
+      fireEvent.touchStart(await screen.findByTestId('splitter-divider'), {})
       moveOverlay = findMoveOverlay(view.container)
       fireEvent.touchMove(
         moveOverlay,
@@ -133,7 +144,9 @@ describe('Splitter', () => {
       )
       fireEvent.touchCancel(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(0)
-      expect(view.container).toMatchSnapshot('touch move to right')
+      mockAppState({ editorConfig: { splitPosition: 0 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
+      expect(view.container).toMatchSnapshot('touch move to left')
 
       const grabber = findGrabber(view.container)
       fireEvent.touchStart(grabber, {})
@@ -149,6 +162,8 @@ describe('Splitter', () => {
       )
       fireEvent.touchEnd(moveOverlay)
       expect(setEditorSplitPosition).toHaveBeenCalledWith(expect.closeTo(26, 1))
+      mockAppState({ editorConfig: { splitPosition: 26 } as EditorConfig })
+      view.rerender(<Splitter left={<>left</>} right={<>right</>} />)
       expect(view.container).toMatchSnapshot('touch move to middle')
     })
   })