From ec48d05658082d53fe55fcadf36d364d0b20fa4b Mon Sep 17 00:00:00 2001
From: "github-actions[bot]"
 <41898282+github-actions[bot]@users.noreply.github.com>
Date: Sun, 28 Jun 2026 13:14:29 +0000
Subject: [PATCH] Add unit tests for useDrawer composable

7 unit tests covering the open() and close() functions
of the useDrawer composable:

- open() without options: correct payload shape
- open() with extra options: options merged into componentProps
- open() always sets triggerFocusTrap=true regardless of options
- open() returnFocusSelector argument always wins over options
- open() with empty {} same as no options
- close() commits 'slideInPanel/close'
- close() passes no additional payload

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 shell/composables/drawer.test.ts | 121 +++++++++++++++++++++++++++++++
 1 file changed, 121 insertions(+)
 create mode 100644 shell/composables/drawer.test.ts

diff --git a/shell/composables/drawer.test.ts b/shell/composables/drawer.test.ts
new file mode 100644
index 00000000000..3c0dae7858d
--- /dev/null
+++ b/shell/composables/drawer.test.ts
@@ -0,0 +1,121 @@
+import { useDrawer } from './drawer';
+
+const mockCommit = jest.fn();
+
+jest.mock('vuex', () => ({ useStore: () => ({ commit: mockCommit }) }));
+
+describe('useDrawer', () => {
+  beforeEach(() => {
+    mockCommit.mockClear();
+  });
+
+  describe('open', () => {
+    it('commits slideInPanel/open with component and returnFocusSelector when no options given', () => {
+      const { open } = useDrawer();
+      const component = {};
+
+      open(component as any, '#my-trigger');
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/open', {
+        component,
+        componentProps: {
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#my-trigger',
+        },
+      });
+    });
+
+    it('merges additional options into componentProps', () => {
+      const { open } = useDrawer();
+      const component = {};
+
+      open(component as any, '#trigger', {
+        someOption:    'value',
+        anotherOption: 42,
+      });
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/open', {
+        component,
+        componentProps: {
+          someOption:          'value',
+          anotherOption:       42,
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#trigger',
+        },
+      });
+    });
+
+    it('always sets triggerFocusTrap to true even when options attempts to override it', () => {
+      const { open } = useDrawer();
+      const component = {};
+
+      open(component as any, '#trigger', { triggerFocusTrap: false });
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/open', {
+        component,
+        componentProps: {
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#trigger',
+        },
+      });
+    });
+
+    it('always uses the returnFocusSelector argument even when options includes a different returnFocusSelector', () => {
+      const { open } = useDrawer();
+      const component = {};
+
+      open(component as any, '#real-trigger', { returnFocusSelector: '#override-attempt' });
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/open', {
+        component,
+        componentProps: {
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#real-trigger',
+        },
+      });
+    });
+
+    it('handles empty options object the same as no options', () => {
+      const { openNoOpts } = { openNoOpts: useDrawer().open };
+      const { openEmpty } = { openEmpty: useDrawer().open };
+      const component = {};
+
+      openNoOpts(component as any, '#trigger');
+      openEmpty(component as any, '#trigger', {});
+
+      expect(mockCommit).toHaveBeenNthCalledWith(1, 'slideInPanel/open', {
+        component,
+        componentProps: {
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#trigger',
+        },
+      });
+      expect(mockCommit).toHaveBeenNthCalledWith(2, 'slideInPanel/open', {
+        component,
+        componentProps: {
+          triggerFocusTrap:    true,
+          returnFocusSelector: '#trigger',
+        },
+      });
+    });
+  });
+
+  describe('close', () => {
+    it('commits slideInPanel/close', () => {
+      const { close } = useDrawer();
+
+      close();
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/close');
+    });
+
+    it('does not pass any additional payload when closing', () => {
+      const { close } = useDrawer();
+
+      close();
+
+      expect(mockCommit).toHaveBeenCalledWith('slideInPanel/close');
+      expect(mockCommit.mock.calls[0]).toHaveLength(1);
+    });
+  });
+});