delete wait_for_animation_end() and fix tests to use playwright auto-…

…waiting correctly

.github/workflows/test.yml

@@ -12,4 +12,3 @@ on:
 jobs:
   tests:
     uses: janosh/workflows/.github/workflows/npm-test-release.yml@main
-    secrets: inherit

package.json

@@ -23,37 +23,37 @@
     "update-coverage": "vitest tests/unit --run --coverage && npx istanbul-badges-readme"
   },
   "dependencies": {
-    "svelte": "^3.57.0"
+    "svelte": "^3.58.0"
   },
   "devDependencies": {
-    "@iconify/svelte": "^3.1.0",
-    "@playwright/test": "^1.31.2",
+    "@iconify/svelte": "^3.1.1",
+    "@playwright/test": "^1.32.2",
     "@sveltejs/adapter-static": "^2.0.1",
-    "@sveltejs/kit": "^1.12.0",
+    "@sveltejs/kit": "^1.15.2",
     "@sveltejs/package": "2.0.2",
-    "@sveltejs/vite-plugin-svelte": "^2.0.3",
-    "@typescript-eslint/eslint-plugin": "^5.55.0",
-    "@typescript-eslint/parser": "^5.55.0",
-    "@vitest/coverage-c8": "^0.29.3",
-    "eslint": "^8.36.0",
+    "@sveltejs/vite-plugin-svelte": "^2.0.4",
+    "@typescript-eslint/eslint-plugin": "^5.58.0",
+    "@typescript-eslint/parser": "^5.58.0",
+    "@vitest/coverage-c8": "^0.30.0",
+    "eslint": "^8.38.0",
     "eslint-plugin-svelte3": "^4.0.0",
     "hastscript": "^7.2.0",
     "highlight.js": "^11.7.0",
     "jsdom": "^21.1.1",
     "mdsvex": "^0.10.6",
     "mdsvexamples": "^0.3.3",
-    "prettier": "^2.8.4",
-    "prettier-plugin-svelte": "^2.9.0",
+    "prettier": "^2.8.7",
+    "prettier-plugin-svelte": "^2.10.0",
     "rehype-autolink-headings": "^6.1.1",
     "rehype-slug": "^5.1.0",
-    "svelte-check": "^3.1.4",
+    "svelte-check": "^3.2.0",
     "svelte-preprocess": "^5.0.3",
     "svelte-toc": "^0.5.4",
     "svelte-zoo": "^0.4.3",
-    "svelte2tsx": "^0.6.10",
-    "typescript": "5.0.2",
-    "vite": "^4.2.0",
-    "vitest": "^0.29.3"
+    "svelte2tsx": "^0.6.11",
+    "typescript": "5.0.4",
+    "vite": "^4.2.1",
+    "vitest": "^0.30.0"
   },
   "keywords": [
     "svelte",

src/lib/CmdPalette.svelte

@@ -12,8 +12,8 @@
   // for span in option slot, has no effect when passing slot="option"
   export let span_style: string = ``
   export let open: boolean = false
-  export let dialog: HTMLDialogElement
-  export let input: HTMLInputElement
+  export let dialog: HTMLDialogElement | null = null
+  export let input: HTMLInputElement | null = null
   export let placeholder: string = `Filter actions...`
 
   type Action = { label: string; action: () => void }

tests/MultiSelect.test.ts

@@ -1,6 +1,5 @@
 import { expect, test } from '@playwright/test'
 import { foods } from '../src/site/options.ts'
-import { wait_for_animation_end } from './index.ts'
 
 // to run tests in this file, use `npm run test:e2e`
 
@@ -466,9 +465,8 @@ test.describe(`maxSelect`, async () => {
     page,
   }) => {
     const ul_selector = `#languages ul.options`
-    await wait_for_animation_end(page, ul_selector)
 
-    expect(await page.locator(ul_selector)).toBeHidden()
+    await expect(page.locator(ul_selector)).toBeHidden()
     expect(await page.getAttribute(ul_selector, `class`)).toContain(`hidden`)
   })
 
@@ -504,7 +502,6 @@ test.describe(`slots`, async () => {
 
     // then click on the expand icon to open the dropdown and change open to true
     await page.click(`#languages-1 input + svg`)
-    await wait_for_animation_end(page, `#languages-1 ul.options`)
 
     // assert that the collapse icon path differs from expand icon path
     const collapse_icon_path = await page.$eval(