handle errors in tests better

kentcdodds · kentcdodds · commit 833d5dcf6d83 · 2025-09-23T12:02:58.000-06:00
diff --git a/exercises/03.resources/01.problem.simple/src/index.test.ts b/exercises/03.resources/01.problem.simple/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/01.solution.simple/src/index.test.ts b/exercises/03.resources/01.solution.simple/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/02.problem.template/src/index.test.ts b/exercises/03.resources/02.problem.template/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/02.solution.template/src/index.test.ts b/exercises/03.resources/02.solution.template/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/03.problem.list/src/index.test.ts b/exercises/03.resources/03.problem.list/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/03.solution.list/src/index.test.ts b/exercises/03.resources/03.solution.list/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/04.problem.completion/src/index.test.ts b/exercises/03.resources/04.problem.completion/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/03.resources/04.solution.completion/src/index.test.ts b/exercises/03.resources/04.solution.completion/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.resource-tools/01.problem.embedded/src/index.test.ts b/exercises/04.resource-tools/01.problem.embedded/src/index.test.ts
@@ -37,7 +37,7 @@ async function setupClient() {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.resource-tools/01.solution.embedded/src/index.test.ts b/exercises/04.resource-tools/01.solution.embedded/src/index.test.ts
@@ -37,7 +37,7 @@ async function setupClient() {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH)
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file
 		},
 	}
 }
diff --git a/exercises/04.resource-tools/02.problem.linked/src/index.test.ts b/exercises/04.resource-tools/02.problem.linked/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/04.resource-tools/02.solution.linked/src/index.test.ts b/exercises/04.resource-tools/02.solution.linked/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/01.problem.prompts/src/index.test.ts b/exercises/05.prompts/01.problem.prompts/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/01.solution.prompts/src/index.test.ts b/exercises/05.prompts/01.solution.prompts/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/02.problem.optimized-prompt/src/index.test.ts b/exercises/05.prompts/02.problem.optimized-prompt/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/02.solution.optimized-prompt/src/index.test.ts b/exercises/05.prompts/02.solution.optimized-prompt/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/03.problem.completion/src/index.test.ts b/exercises/05.prompts/03.problem.completion/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }
diff --git a/exercises/05.prompts/03.solution.completion/src/index.test.ts b/exercises/05.prompts/03.solution.completion/src/index.test.ts
@@ -40,7 +40,7 @@ async function setupClient({ capabilities }: ClientOptions = {}) {
 			await client.transport?.close()
 			// give things a moment to release locks and whatnot
 			await new Promise((r) => setTimeout(r, 100))
-			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {})
+			await fs.unlink(EPIC_ME_DB_PATH).catch(() => {}) // ignore missing file.catch(() => {})
 		},
 	}
 }

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