refactor(settings): switch wipe dialog to confirm

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>

Author: pringelmann

apps/settings/src/components/AuthToken.spec.ts

@@ -16,10 +16,16 @@ vi.hoisted(() => {
 
 import type { IToken } from '../store/authtoken.ts'
 
+// Mock @nextcloud/dialogs so the wipe action's showConfirmation call resolves
+// synchronously in tests. Hoisted so it's installed before AuthToken.vue imports.
+const showConfirmationMock = vi.hoisted(() => vi.fn())
+vi.mock('@nextcloud/dialogs', () => ({
+	showConfirmation: showConfirmationMock,
+}))
+
 import NcNoteCard from '@nextcloud/vue/components/NcNoteCard'
 import AuthToken from './AuthToken.vue'
 import AuthTokenDeleteDialog from './AuthTokenDeleteDialog.vue'
-import AuthTokenWipeDialog from './AuthTokenWipeDialog.vue'
 import { TokenType, useAuthTokenStore } from '../store/authtoken.ts'
 import { detect } from '../utils/userAgentDetect.ts'
 
@@ -143,33 +149,27 @@ describe('AuthToken wipe flow', () => {
 		vi.clearAllMocks()
 	})
 
-	it('does not call wipeToken when the wipe action is triggered (dialog opens first)', async () => {
+	it('does not call wipeToken when the user rejects the confirmation', async () => {
+		showConfirmationMock.mockResolvedValueOnce(false)
 		const token = makeToken()
 		const wrapper = mountAuthToken(token)
 		const store = useAuthTokenStore()
 
-		;(wrapper.vm as unknown as { wipe: () => void }).wipe()
-		await wrapper.vm.$nextTick()
+		await (wrapper.vm as unknown as { wipe: () => Promise<void> }).wipe()
 
-		const dialog = wrapper.findComponent(AuthTokenWipeDialog)
-		expect(dialog.exists()).toBe(true)
-		expect(dialog.props('open')).toBe(true)
+		expect(showConfirmationMock).toHaveBeenCalledTimes(1)
 		expect(store.wipeToken).not.toHaveBeenCalled()
 	})
 
-	it('calls wipeToken only after the dialog emits confirm', async () => {
+	it('calls wipeToken when the user accepts the confirmation', async () => {
+		showConfirmationMock.mockResolvedValueOnce(true)
 		const token = makeToken()
 		const wrapper = mountAuthToken(token)
 		const store = useAuthTokenStore()
 
-		;(wrapper.vm as unknown as { wipe: () => void }).wipe()
-		await wrapper.vm.$nextTick()
-
-		const dialog = wrapper.findComponent(AuthTokenWipeDialog)
-		dialog.vm.$emit('confirm')
-		dialog.vm.$emit('update:open', false)
-		await wrapper.vm.$nextTick()
+		await (wrapper.vm as unknown as { wipe: () => Promise<void> }).wipe()
 
+		expect(showConfirmationMock).toHaveBeenCalledTimes(1)
 		expect(store.wipeToken).toHaveBeenCalledTimes(1)
 		expect(store.wipeToken).toHaveBeenCalledWith(token)
 	})

apps/settings/src/components/AuthToken.vue

@@ -78,11 +78,6 @@
 			:token="token"
 			:open.sync="deleteDialogOpen"
 			@confirm="confirmDelete" />
-		<AuthTokenWipeDialog
-			v-if="wipeDialogOpen"
-			:token="token"
-			:open.sync="wipeDialogOpen"
-			@confirm="confirmWipe" />
 	</tr>
 </template>
 
@@ -91,6 +86,7 @@ import type { PropType } from 'vue'
 import type { IToken } from '../store/authtoken'
 
 import { mdiCheck, mdiCellphone, mdiTablet, mdiMonitor, mdiWeb, mdiKeyOutline, mdiMicrosoftEdge, mdiFirefox, mdiGoogleChrome, mdiAppleSafari, mdiAndroid, mdiAppleIos } from '@mdi/js'
+import { showConfirmation } from '@nextcloud/dialogs'
 import { translate as t } from '@nextcloud/l10n'
 import { defineComponent } from 'vue'
 import { TokenType, useAuthTokenStore } from '../store/authtoken.ts'
@@ -103,7 +99,6 @@ import NcDateTime from '@nextcloud/vue/components/NcDateTime'
 import NcIconSvgWrapper from '@nextcloud/vue/components/NcIconSvgWrapper'
 import NcTextField from '@nextcloud/vue/components/NcTextField'
 import AuthTokenDeleteDialog from './AuthTokenDeleteDialog.vue'
-import AuthTokenWipeDialog from './AuthTokenWipeDialog.vue'
 
 // When using capture groups the following parts are extracted the first is used as the version number, the second as the OS
 const userAgentMap = {
@@ -156,7 +151,6 @@ export default defineComponent({
 	name: 'AuthToken',
 	components: {
 		AuthTokenDeleteDialog,
-		AuthTokenWipeDialog,
 		NcActions,
 		NcActionButton,
 		NcActionCheckbox,
@@ -182,7 +176,6 @@ export default defineComponent({
 			newName: '',
 			oldName: '',
 			deleteDialogOpen: false,
-			wipeDialogOpen: false,
 			mdiCheck,
 			TokenType,
 		}
@@ -321,13 +314,19 @@ export default defineComponent({
 			this.renaming = false
 			this.authTokenStore.renameToken(this.token, this.newName)
 		},
-		wipe() {
-			this.actionOpen = false
-			this.wipeDialogOpen = true
-		},
 
-		confirmWipe() {
-			this.authTokenStore.wipeToken(this.token)
+		async wipe() {
+			this.actionOpen = false
+			const confirmed = await showConfirmation({
+				name: t('settings', 'Confirm wipe'),
+				text: t('settings', 'Do you really want to wipe your data from this device?'),
+				labelConfirm: t('settings', 'Wipe device'),
+				labelReject: t('settings', 'Cancel'),
+				severity: 'warning',
+			})
+			if (confirmed) {
+				this.authTokenStore.wipeToken(this.token)
+			}
 		},
 	},
 })