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

@@ -15,20 +15,11 @@ export async function handlePasswordConfirmation(page: Page, password = 'admin')
 	const dialog = page.locator('.modal-container:has-text("Authentication required")')
 
 	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 })
+		await dialog.locator('input[type="password"]').fill(password)
+		await dialog.getByRole('button', { name: 'Confirm' }).click()
+		await dialog.waitFor({ state: 'hidden' })
 	} catch {
-		// Dialog didn't appear, which is fine - some operations might not require confirmation
+		// Dialog didn't appear — some operations don't require confirmation.
 	}
 }