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,19 +16,18 @@ 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)
-
-		if (dialogVisible) {
-			// Fill the password field
-			await dialog.locator('input[type="password"]').fill(password)
-
-			// Click the confirm button
-			await dialog.getByRole('button', { name: 'Confirm' }).click()
-
-			// Wait for the dialog to disappear
-			await dialog.waitFor({ state: 'hidden' })
-		}
+		await dialog.waitFor({ state: 'visible', timeout: 500 })
 	} catch {
 		// Dialog didn't appear, which is fine - some operations might not require confirmation
+		return
 	}
+
+	// Fill the password field
+	await dialog.locator('input[type="password"]').fill(password)
+
+	// Click the confirm button
+	await dialog.getByRole('button', { name: 'Confirm' }).click()
+
+	// Wait for the dialog to disappear
+	await dialog.waitFor({ state: 'hidden' })
 }