chore(playwright): Rewrite login handler to avoid race conditions

Assisted-by: ClaudeCode:claude-opus-4.6[1M]
Signed-off-by: David Dreschner <david.dreschner@nextcloud.com>

Author: DerDreschner

tests/playwright/support/utils/password-confirmation.ts

@@ -16,18 +16,16 @@ export async function handlePasswordConfirmation(page: Page, password = 'admin')
 
 	try {
 		// Check if the dialog exists within a short timeout
-		const dialogVisible = await dialog.isVisible({ timeout: 500 }).catch(() => false)
+		await dialog.waitFor({ state: 'visible', timeout: 500 })
 
-		if (dialogVisible) {
-			// Fill the password field
-			await dialog.locator('input[type="password"]').fill(password)
+		// Fill the password field
+		await dialog.locator('input[type="password"]').fill(password)
 
-			// Click the confirm button
-			await dialog.getByRole('button', { name: 'Confirm' }).click()
+		// Click the confirm button
+		await dialog.getByRole('button', { name: 'Confirm' }).click()
 
-			// Wait for the dialog to disappear
-			await dialog.waitFor({ state: 'hidden' })
-		}
+		// Wait for the dialog to disappear
+		await dialog.waitFor({ state: 'hidden' })
 	} catch {
 		// Dialog didn't appear, which is fine - some operations might not require confirmation
 	}