fix(test): destroy Tiptap editor everywhere at the end of the tests

Signed-off-by: Jonas <jonas@freesources.org>

Author: mejo-

src/tests/components/Editor/TableOfContents.spec.ts

@@ -30,6 +30,7 @@ test('renders nothing for editor without headings', () => {
 	const wrapper = mountWithEditor(editor)
 	expect(wrapper.text()).toEqual('')
 	expect(wrapper.vm.$data.headings).toEqual([])
+	editor.destroy()
 })
 
 test('renders initial heading', async () => {
@@ -38,6 +39,7 @@ test('renders initial heading', async () => {
 	await nextTick()
 	expect(wrapper.text()).toEqual(text)
 	expect(wrapper.vm.$data.headings).toEqual(headingsForContent)
+	editor.destroy()
 })
 
 test('updates according to editor changes', async () => {
@@ -48,6 +50,7 @@ test('updates according to editor changes', async () => {
 	await nextTick()
 	expect(wrapper.text()).toEqual(text)
 	expect(wrapper.vm.$data.headings).toEqual(headingsForContent)
+	editor.destroy()
 })
 
 test('disconnects on destroy', async () => {
@@ -58,4 +61,5 @@ test('disconnects on destroy', async () => {
 	expect(on).toHaveBeenCalledWith('update', expect.any(Function))
 	wrapper.destroy()
 	expect(off).toHaveBeenCalledWith('update', expect.any(Function))
+	editor.destroy()
 })

src/tests/components/Menu/TranslateButton.spec.ts

@@ -36,6 +36,7 @@ test('does not render without state', async () => {
 	const wrapper = mountWithEditor(editor)
 	await nextTick()
 	expect(wrapper.find('button').exists()).toBe(false)
+	editor.destroy()
 })
 
 test('does render with translations', async () => {
@@ -44,6 +45,7 @@ test('does render with translations', async () => {
 	const wrapper = mountWithEditor(editor)
 	await nextTick()
 	expect(wrapper.find('button').exists()).toBe(true)
+	editor.destroy()
 })
 
 test('emits the full content when clicked', async () => {
@@ -56,4 +58,5 @@ test('emits the full content when clicked', async () => {
 	wrapper.find('button').trigger('click')
 	await nextTick()
 	expect(listen).toHaveBeenCalledWith({ content: text })
+	editor.destroy()
 })

src/tests/extensions/Search.spec.ts

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
-import { describe, expect, it } from 'vitest'
+import { afterAll, describe, expect, it } from 'vitest'
 import Search from '../../extensions/Search'
 import HardBreak from '../../nodes/HardBreak.js'
 import { searchDecorationsPluginKey } from '../../plugins/searchDecorations.js'
@@ -13,6 +13,10 @@ import createCustomEditor from '../testHelpers/createCustomEditor'
 describe('editor search highlighting', () => {
 	const editor = createCustomEditor(lorem, [Search, HardBreak])
 
+	afterAll(() => {
+		editor.destroy()
+	})
+
 	it('can highlight a match', () => {
 		const searchQuery = 'Lorem'
 		editor.commands.setSearchQuery(searchQuery)

src/tests/markdown.spec.js

@@ -334,5 +334,6 @@ describe('Trailing nodes', () => {
 		const serializer = createMarkdownSerializer(tiptap.schema)
 		const md = serializer.serialize(tiptap.state.doc)
 		expect(md).toBe(source)
+		tiptap.destroy()
 	})
 })

src/tests/marks/Link.spec.js

@@ -13,6 +13,7 @@ describe('Link extension integrated in the editor', () => {
 			[Link],
 		)
 		expect(editor.commands).toHaveProperty('insertOrSetLink')
+		editor.destroy()
 	})
 
 	it('should update link if anchor has mark', () => {
@@ -23,6 +24,7 @@ describe('Link extension integrated in the editor', () => {
 		editor.commands.setTextSelection(3)
 		editor.commands.insertOrSetLink('updated.de', { href: 'updated.de' })
 		expect(editor.getJSON()).toMatchSnapshot()
+		editor.destroy()
 	})
 
 	it('Should only update link the anchor is on', () => {
@@ -33,6 +35,7 @@ describe('Link extension integrated in the editor', () => {
 		editor.commands.setTextSelection(3)
 		editor.commands.insertOrSetLink('updated.de', { href: 'updated.de' })
 		expect(editor.getJSON()).toMatchSnapshot()
+		editor.destroy()
 	})
 
 	it('should insert new link if none at anchor', () => {
@@ -45,5 +48,6 @@ describe('Link extension integrated in the editor', () => {
 			href: 'https://nextcloud.com',
 		})
 		expect(editor.getJSON()).toMatchSnapshot()
+		editor.destroy()
 	})
 })

src/tests/marks/Underline.spec.js

@@ -23,10 +23,12 @@ describe('Underline extension integrated in the editor', () => {
 	it('is not active by default', () => {
 		const editor = createCustomEditor('<p>Test</p>', [Underline])
 		expect(editor.isActive('underline')).toBe(false)
+		editor.destroy()
 	})
 
 	it('is active within <u> tags', () => {
 		const editor = createCustomEditor('<p><u>Test</u></p>', [Underline])
 		expect(editor.isActive('underline')).toBe(true)
+		editor.destroy()
 	})
 })

src/tests/nodes/TaskItem.spec.js

@@ -23,6 +23,7 @@ describe('TaskItem extension', () => {
 		const editor = createCustomEditor('', [Markdown, TaskList, TaskItem])
 		const taskItem = editor.schema.nodes.taskItem
 		expect(taskItem.spec.toMarkdown).toBeDefined()
+		editor.destroy()
 	})
 
 	it('markdown syntax is preserved through editor', () => {

src/tests/plaintext.spec.js

@@ -21,7 +21,9 @@ const plaintextThroughEditor = (markdown) => {
 	const content = '<pre>' + escapeHTML(markdown) + '</pre>'
 	const tiptap = createPlainEditor()
 	tiptap.commands.setContent(content)
-	return serializePlainText(tiptap.state.doc) || 'failed'
+	const plaintext = serializePlainText(tiptap.state.doc) || 'failed'
+	tiptap.destroy()
+	return plaintext
 }
 
 describe('commonmark as plaintext', () => {
@@ -93,5 +95,6 @@ describe('regression tests', () => {
 		tiptap.commands.enter()
 		tiptap.commands.enter()
 		expect(serializePlainText(tiptap.state.doc)).toEqual('\n\n\n')
+		tiptap.destroy()
 	})
 })

src/tests/plugins/extractHeadings.spec.js

@@ -64,5 +64,7 @@ describe('extractHeadings', () => {
 
 const prepareDoc = (content) => {
 	const editor = createCustomEditor(content, [Heading])
-	return editor.state.doc
+	const doc = editor.state.doc
+	editor.destroy()
+	return doc
 }

src/tests/plugins/searchDecorations.spec.js

@@ -79,10 +79,12 @@ describe('search plugin', () => {
 })
 
 const testSearch = (content, query, expectedSearchResults) => {
-	const doc = createCustomEditor(content).state.doc
+	const editor = createCustomEditor(content)
+	const doc = editor.state.doc
 	const searched = runSearch(doc, query)
 	expect(searched).toHaveProperty('results', expectedSearchResults.results)
 	expect(highlightResults(doc, searched.results)).toEqual(
 		highlightResults(doc, expectedSearchResults.results),
 	)
+	editor.destroy()
 }