Improve test reliability.

Author: kitschpatrol

test/utilities/anki-connect.ts

@@ -4,6 +4,22 @@ import { yankiModels } from '../../src/lib/model/model'
 import { requestPermission } from '../../src/lib/utilities/anki-connect'
 import { PLATFORM } from '../../src/lib/utilities/platform'
 
+/**
+ * Launches Anki
+ */
+export async function openAnki(): Promise<void> {
+	if (PLATFORM !== 'mac') {
+		throw new Error('This function only works on Mac')
+	}
+
+	const client = new YankiConnect({
+		autoLaunch: 'immediately',
+	})
+
+	// Sanity
+	await client.miscellaneous.version()
+}
+
 /**
  * Closes Anki by sending a quit command to the application.
  * This function will wait until Anki is unreachable.
@@ -30,14 +46,16 @@ export async function closeAnki(): Promise<void> {
 			await execa('sh', [
 				'-c',
 				"launchctl stop $(launchctl list | grep ankiweb | awk '{print $3}')",
-			])
+			]).catch(() => {
+				// Ignore
+			})
 		})
 	}
 
 	// Spin until it's done...
 	while (permissionStatus !== 'ankiUnreachable') {
 		await new Promise((resolve) => {
-			setTimeout(resolve, 1000)
+			setTimeout(resolve, 250)
 		})
 		permissionStatus = await requestPermission(client)
 	}

test/utilities/global-setup.ts

@@ -0,0 +1,15 @@
+import { closeAnki, openAnki } from './anki-connect'
+
+/**
+ * Ensure Anki is running before tests run
+ */
+export async function setup() {
+	await openAnki()
+}
+
+/**
+ * Clean up
+ */
+export async function teardown() {
+	await closeAnki()
+}

vitest.config.ts

@@ -15,6 +15,7 @@ export default defineConfig({
 		// Running tests concurrently across files can create race conditions in
 		// total note counts that will cause assertions to fail.
 		fileParallelism: false,
+		globalSetup: './test/utilities/global-setup.ts',
 		maxConcurrency: 1,
 		maxWorkers: 1,
 		projects: [