improve tests

kentcdodds · kentcdodds · commit e65ef9c50c2a · 2025-07-11T16:52:06.000-06:00
diff --git a/exercises/01.ping/01.problem.connect/src/index.test.ts b/exercises/01.ping/01.problem.connect/src/index.test.ts
@@ -1,4 +1,7 @@
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
diff --git a/exercises/01.ping/01.solution.connect/src/index.test.ts b/exercises/01.ping/01.solution.connect/src/index.test.ts
@@ -1,4 +1,7 @@
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
diff --git a/exercises/02.tools/01.problem.simple/src/index.test.ts b/exercises/02.tools/01.problem.simple/src/index.test.ts
@@ -1,5 +1,8 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
diff --git a/exercises/02.tools/01.solution.simple/src/index.test.ts b/exercises/02.tools/01.solution.simple/src/index.test.ts
@@ -1,9 +1,12 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const client = new Client(
 		{
 			name: 'EpicMeTester',
diff --git a/exercises/02.tools/02.problem.args/src/index.test.ts b/exercises/02.tools/02.problem.args/src/index.test.ts
@@ -1,9 +1,12 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const client = new Client(
 		{
 			name: 'EpicMeTester',
diff --git a/exercises/02.tools/02.solution.args/src/index.test.ts b/exercises/02.tools/02.solution.args/src/index.test.ts
@@ -1,9 +1,12 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const client = new Client(
 		{
 			name: 'EpicMeTester',
diff --git a/exercises/02.tools/03.problem.errors/src/index.test.ts b/exercises/02.tools/03.problem.errors/src/index.test.ts
@@ -1,9 +1,12 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const client = new Client(
 		{
 			name: 'EpicMeTester',
diff --git a/exercises/02.tools/03.solution.errors/src/index.test.ts b/exercises/02.tools/03.solution.errors/src/index.test.ts
@@ -1,9 +1,12 @@
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const client = new Client(
 		{
 			name: 'EpicMeTester',
diff --git a/exercises/03.resources/01.problem.simple/src/index.test.ts b/exercises/03.resources/01.problem.simple/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/01.solution.simple/src/index.test.ts b/exercises/03.resources/01.solution.simple/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/02.problem.template/src/index.test.ts b/exercises/03.resources/02.problem.template/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/02.solution.template/src/index.test.ts b/exercises/03.resources/02.solution.template/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/03.problem.list/src/index.test.ts b/exercises/03.resources/03.problem.list/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/03.solution.list/src/index.test.ts b/exercises/03.resources/03.solution.list/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/04.problem.completion/src/index.test.ts b/exercises/03.resources/04.problem.completion/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/04.solution.completion/src/index.test.ts b/exercises/03.resources/04.solution.completion/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/05.problem.linked/src/index.test.ts b/exercises/03.resources/05.problem.linked/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/05.solution.linked/src/index.test.ts b/exercises/03.resources/05.solution.linked/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/03.resources/06.problem.embedded/src/index.test.ts b/exercises/03.resources/06.problem.embedded/src/index.test.ts
@@ -1,7 +1,10 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
diff --git a/exercises/03.resources/06.solution.embedded/src/index.test.ts b/exercises/03.resources/06.solution.embedded/src/index.test.ts
@@ -1,7 +1,10 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
diff --git a/exercises/04.prompts/01.problem.prompts/src/index.test.ts b/exercises/04.prompts/01.problem.prompts/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/04.prompts/01.solution.prompts/src/index.test.ts b/exercises/04.prompts/01.solution.prompts/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/04.prompts/02.problem.optimized-prompt/src/index.test.ts b/exercises/04.prompts/02.problem.optimized-prompt/src/index.test.ts
@@ -1,15 +1,18 @@
 import fs from 'node:fs/promises'
 import path from 'node:path'
 import { invariant } from '@epic-web/invariant'
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import {
+	Client,
+	type ClientOptions,
+} from '@modelcontextprotocol/sdk/client/index.js'
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
 import { test, expect } from 'vitest'
 
 function getTestDbPath() {
 	return `./test.ignored/db.${process.env.VITEST_WORKER_ID}.${Math.random().toString(36).slice(2)}.sqlite`
 }
 
-async function setupClient({ capabilities = {} } = {}) {
+async function setupClient({ capabilities }: ClientOptions = {}) {
 	const EPIC_ME_DB_PATH = getTestDbPath()
 	const dir = path.dirname(EPIC_ME_DB_PATH)
 	await fs.mkdir(dir, { recursive: true })
diff --git a/exercises/04.prompts/02.solution.optimized-prompt/src/index.test.ts b/exercises/04.prompts/02.solution.optimized-prompt/src/index.test.ts
diff --git a/exercises/04.prompts/03.problem.completion/src/index.test.ts b/exercises/04.prompts/03.problem.completion/src/index.test.ts
diff --git a/exercises/04.prompts/03.solution.completion/src/index.test.ts b/exercises/04.prompts/03.solution.completion/src/index.test.ts
