03/04: remove waitFor from assertion

Author: kettanaito

exercises/03.best-practices/04.solution.inverse-assertions/README.mdx

@@ -2,6 +2,8 @@
 
 Problem: testing things that must _not_ appear on the page can lead to false-positive tests in no time. Let's learn about _inverse assertions_ that help you write such tests reliably.
 
+> Note: Vitest's `expect.element()` already has a built-in retry mechanism.
+
 ## Materials
 
 - [Inverse assertions](https://www.epicweb.dev/inverse-assertions)

exercises/03.best-practices/04.solution.inverse-assertions/src/discount-code-form.browser.test.tsx

@@ -97,7 +97,5 @@ test('removes the applied discount code', async () => {
 	})
 	await removeDiscountButton.click()
 
-	await vi.waitFor(async () => {
-		await expect.element(discountText).not.toBeInTheDocument()
-	})
+	await expect.element(discountText).not.toBeInTheDocument()
 })

exercises/03.best-practices/04.solution.inverse-assertions/src/mocks/handlers.ts

@@ -1,4 +1,4 @@
-import { http, HttpResponse } from 'msw'
+import { delay, http, HttpResponse } from 'msw'
 import type { Discount } from '../discount-code-form.js'
 
 export const handlers = [
@@ -14,6 +14,7 @@ export const handlers = [
 		},
 	),
 	http.delete('https://api.example.com/discount/code', async () => {
+		await delay(500)
 		return new HttpResponse()
 	}),
 ]