make tests more reliable

kentcdodds · kentcdodds · commit 3ff6d74e435e · 2025-09-23T11:45:44.000-06:00
diff --git a/exercises/01.advanced-tools/01.problem.annotations/src/index.test.ts b/exercises/01.advanced-tools/01.problem.annotations/src/index.test.ts
@@ -36,7 +36,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/01.advanced-tools/01.solution.annotations/src/index.test.ts b/exercises/01.advanced-tools/01.solution.annotations/src/index.test.ts
@@ -36,7 +36,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/01.advanced-tools/02.problem.structured/src/index.test.ts b/exercises/01.advanced-tools/02.problem.structured/src/index.test.ts
@@ -36,7 +36,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/01.advanced-tools/02.solution.structured/src/index.test.ts b/exercises/01.advanced-tools/02.solution.structured/src/index.test.ts
@@ -36,7 +36,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/02.elicitation/01.problem/src/index.test.ts b/exercises/02.elicitation/01.problem/src/index.test.ts
@@ -37,7 +37,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/02.elicitation/01.solution/src/index.test.ts b/exercises/02.elicitation/01.solution/src/index.test.ts
@@ -37,7 +37,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.sampling/01.problem.simple/src/index.test.ts b/exercises/03.sampling/01.problem.simple/src/index.test.ts
@@ -42,7 +42,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.sampling/01.solution.simple/src/index.test.ts b/exercises/03.sampling/01.solution.simple/src/index.test.ts
@@ -42,7 +42,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.sampling/02.problem.advanced/src/index.test.ts b/exercises/03.sampling/02.problem.advanced/src/index.test.ts
@@ -42,7 +42,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.sampling/02.solution.advanced/src/index.test.ts b/exercises/03.sampling/02.solution.advanced/src/index.test.ts
@@ -42,7 +42,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.long-running-tasks/01.problem.progress/src/index.test.ts b/exercises/04.long-running-tasks/01.problem.progress/src/index.test.ts
@@ -43,7 +43,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.long-running-tasks/01.solution.progress/src/index.test.ts b/exercises/04.long-running-tasks/01.solution.progress/src/index.test.ts
@@ -43,7 +43,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.long-running-tasks/02.problem.cancellation/src/index.test.ts b/exercises/04.long-running-tasks/02.problem.cancellation/src/index.test.ts
@@ -43,7 +43,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.long-running-tasks/02.solution.cancellation/src/index.test.ts b/exercises/04.long-running-tasks/02.solution.cancellation/src/index.test.ts
@@ -43,7 +43,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/01.problem.list-changed/src/index.test.ts b/exercises/05.changes/01.problem.list-changed/src/index.test.ts
@@ -44,7 +44,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/01.solution.list-changed/src/index.test.ts b/exercises/05.changes/01.solution.list-changed/src/index.test.ts
@@ -44,7 +44,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/02.problem.resources-list-changed/src/index.test.ts b/exercises/05.changes/02.problem.resources-list-changed/src/index.test.ts
@@ -46,7 +46,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/02.solution.resources-list-changed/src/index.test.ts b/exercises/05.changes/02.solution.resources-list-changed/src/index.test.ts
@@ -46,7 +46,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/03.problem.subscriptions/src/index.test.ts b/exercises/05.changes/03.problem.subscriptions/src/index.test.ts
@@ -47,7 +47,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/05.changes/03.solution.subscriptions/src/index.test.ts b/exercises/05.changes/03.solution.subscriptions/src/index.test.ts
@@ -47,7 +47,9 @@ async function setupClient({ capabilities = {} } = {}) {
 		EPIC_ME_DB_PATH,
 		async [Symbol.asyncDispose]() {
 			await client.transport?.close()
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			// give things a moment to release locks and whatnot
+			await new Promise((r) => setTimeout(r, 100))
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,9 @@ async function setupClient({ capabilities = {} } = {}) {`
`36`	`36`	`EPIC_ME_DB_PATH,`
`37`	`37`	`async [Symbol.asyncDispose]() {`
`38`	`38`	`await client.transport?.close()`
`39`		`- await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})`
	`39`	`+ // give things a moment to release locks and whatnot`
	`40`	`+ await new Promise((r) => setTimeout(r, 100))`
	`41`	`+ await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file`
`40`	`42`	`},`
`41`	`43`	`}`
`42`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,9 @@ async function setupClient({ capabilities = {} } = {}) {`
`37`	`37`	`EPIC_ME_DB_PATH,`
`38`	`38`	`async [Symbol.asyncDispose]() {`
`39`	`39`	`await client.transport?.close()`
`40`		`- await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})`
	`40`	`+ // give things a moment to release locks and whatnot`
	`41`	`+ await new Promise((r) => setTimeout(r, 100))`
	`42`	`+ await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file`
`41`	`43`	`},`
`42`	`44`	`}`
`43`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,9 @@ async function setupClient({ capabilities = {} } = {}) {`
`42`	`42`	`EPIC_ME_DB_PATH,`
`43`	`43`	`async [Symbol.asyncDispose]() {`
`44`	`44`	`await client.transport?.close()`
`45`		`- await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})`
	`45`	`+ // give things a moment to release locks and whatnot`
	`46`	`+ await new Promise((r) => setTimeout(r, 100))`
	`47`	`+ await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file`
`46`	`48`	`},`
`47`	`49`	`}`
`48`	`50`	`}`