test(client): follow the password checklist and the labelled modal close in three suites

client/src/components/Packing/PackingExportMenu.test.tsx

@@ -94,7 +94,9 @@ describe('PackingPrintPreview', () => {
     Object.defineProperty(frame, 'contentWindow', { value: { print }, configurable: true })
     await user.click(screen.getByRole('button', { name: 'Print or save as PDF' }))
     expect(print).toHaveBeenCalledTimes(1)
-    await user.click(screen.getByRole('button', { name: 'Close' }))
+    // The modal's X and the preview's own button both close; the footer one is pressed here.
+    const closeButtons = screen.getAllByRole('button', { name: 'Close' })
+    await user.click(closeButtons[closeButtons.length - 1])
     expect(onClose).toHaveBeenCalledTimes(1)
   })
 })

client/src/pages/AdminPage.test.tsx

@@ -301,7 +301,7 @@ describe('AdminPage', () => {
 
       fireEvent.change(screen.getByPlaceholderText('Username'), { target: { value: 'newuser' } });
       fireEvent.change(screen.getByPlaceholderText('Email'), { target: { value: 'newuser@example.com' } });
-      fireEvent.change(screen.getByPlaceholderText('Password'), { target: { value: 'securepassword123' } });
+      fireEvent.change(screen.getByPlaceholderText('Password'), { target: { value: 'SecurePassw0rd!23' } });
 
       // The modal footer has a second "Create User" button
       const createButtons = screen.getAllByRole('button', { name: /create user/i });

client/src/pages/LoginPage.test.tsx

@@ -388,7 +388,8 @@ describe('LoginPage', () => {
       await user.click(screen.getByRole('button', { name: /update password/i }));
 
       await waitFor(() => {
-        expect(screen.getByText(/at least 8/i)).toBeInTheDocument();
+        // The error, not the checklist line under the field that says the same rule.
+        expect(screen.getByText('Password must be at least 8 characters')).toBeInTheDocument();
       });
     });
   });
@@ -641,7 +642,8 @@ describe('LoginPage', () => {
       await user.click(screen.getByRole('button', { name: /create account/i }));
 
       await waitFor(() => {
-        expect(screen.getByText(/at least 8/i)).toBeInTheDocument();
+        // The error, not the checklist line under the field that says the same rule.
+        expect(screen.getByText('Password must be at least 8 characters')).toBeInTheDocument();
       });
     });
   });