frontend: fix tests

Author: ffreddow

frontend/src/components/RecoveryDataComponent.tsx

@@ -69,7 +69,7 @@ export function RecoveryDataComponent(props: RecoveryDataProps) {
                         id="recovery-confirmation"
                         label={t("save_recovery_data_confirmation")}
                         checked={confirmed.value}
-                        onChange={(e: any) => { confirmed.value = e.target.checked; }}
+                        onChange={(e: React.ChangeEvent<HTMLInputElement>) => { confirmed.value = e.currentTarget.checked; }}
                         className="mt-3"
                     />
                 )}

frontend/src/components/__tests__/RecoveryDataComponent.test.tsx

@@ -18,6 +18,13 @@ describe('RecoveryDataComponent', () => {
     const email = '[email protected]';
     const secret = new Uint8Array([1, 2, 3]);
 
+    // Mock the anchor click to avoid interference with other tests
+    const clickSpy = vi
+      .spyOn(HTMLAnchorElement.prototype as unknown as { click: () => void }, 'click')
+      .mockImplementation(() => {
+        // Mock implementation for testing. This is comment is to silence lint warnings
+      });
+
     render(<RecoveryDataComponent email={email} secret={secret} show={show} />);
 
     expect(screen.getByTestId('modal-title').textContent).toBe('save_recovery_data');
@@ -51,13 +58,22 @@ describe('RecoveryDataComponent', () => {
 
     fireEvent.click(closeButton);
     expect(show.value).toBe(false);
+
+    clickSpy.mockRestore();
   });
 
   it('prevents closing modal until file is saved and confirmed', async () => {
     const { RecoveryDataComponent } = await importComponent();
 
     const show = signal(true);
 
+    // Mock the anchor click to avoid interference with other tests
+    const clickSpy = vi
+      .spyOn(HTMLAnchorElement.prototype as unknown as { click: () => void }, 'click')
+      .mockImplementation(() => {
+        // Mock implementation for testing. This is comment is to silence lint warnings
+      });
+
     render(<RecoveryDataComponent email={'[email protected]'} secret={new Uint8Array()} show={show} />);
 
     // Try to close the modal without saving/confirming - should not close
@@ -81,9 +97,11 @@ describe('RecoveryDataComponent', () => {
     await waitFor(() => {
       expect(closeButton).not.toBeDisabled();
     });
-    
+
     fireEvent.click(closeButton);
     expect(show.value).toBe(false);
+
+    clickSpy.mockRestore();
   });
 });
 
@@ -117,6 +135,11 @@ describe('saveRecoveryData', () => {
     expect(clickSpy).toHaveBeenCalledTimes(1);
     expect(revokeUrl).toHaveBeenCalledWith('blob:test-url');
 
-  expect(window.crypto.subtle.digest).toHaveBeenCalledWith('SHA-256', expect.anything());
+    expect(window.crypto.subtle.digest).toHaveBeenCalledWith('SHA-256', expect.anything());
+
+    // Clean up spies
+    createUrl.mockRestore();
+    clickSpy.mockRestore();
+    revokeUrl.mockRestore();
   });
 });

frontend/src/pages/Recovery.tsx

@@ -148,10 +148,10 @@ export function Recovery() {
                 <p>{t("recovery.no_file_warning_body")}</p>
                 <Form.Check
                     type="checkbox"
-                    id="no-file-ack"
+                    id="acknowledge-no-file"
                     label={t("recovery.no_file_warning_ack")}
                     checked={acknowledgeNoFile.value}
-                    onChange={(e: any) => { acknowledgeNoFile.value = e.target.checked; }}
+                    onChange={(e: React.ChangeEvent<HTMLInputElement>) => { acknowledgeNoFile.value = e.currentTarget.checked; }}
                 />
             </Modal.Body>
             <Modal.Footer>