fix tests

Author: kentcdodds

exercises/05.changes/02.problem.resources-list-changed/src/index.test.ts

@@ -676,11 +676,11 @@ test('ListChanged notification: tools', async () => {
 	const initialTools = await client.listTools()
 	const initialToolNames = initialTools.tools.map((t) => t.name)
 
-	// Should not have advanced tools initially
+	// Should not have tools requiring exisitng entries
 	expect(
 		initialToolNames,
-		'🚨 Advanced tools like create_wrapped_video should not be available initially',
-	).not.toContain('create_wrapped_video')
+		'🚨 Tools requiring entries like get_entry should not be available initially',
+	).not.toContain('get_entry')
 
 	// Trigger a DB change that should enable additional tools
 	await client.callTool({
@@ -722,8 +722,8 @@ test('ListChanged notification: tools', async () => {
 	const enabledToolNames = enabledTools.tools.map((t) => t.name)
 	expect(
 		enabledToolNames,
-		'🚨 Advanced tools like create_wrapped_video should be enabled after creating entries/tags. The server must dynamically enable/disable tools based on content.',
-	).toContain('create_wrapped_video')
+		'🚨 Tools requiring entries like get_entry should be enabled after creating entries/tags. The server must dynamically enable/disable tools based on content.',
+	).toContain('get_entry')
 
 	// Verify that tools are properly enabled with correct count
 	expect(

exercises/05.changes/02.solution.resources-list-changed/src/index.test.ts

@@ -599,6 +599,13 @@ test('ListChanged notification: resources', async () => {
 		},
 	)
 
+	// Initially resources should be disabled/empty
+	const initialResources = await client.listResources()
+	expect(
+		initialResources.resources,
+		'🚨 Resources should initially be empty when no entries/tags exist',
+	).toHaveLength(0)
+
 	// Trigger a DB change that should enable resources
 	await client.callTool({
 		name: 'create_tag',
@@ -615,6 +622,7 @@ test('ListChanged notification: resources', async () => {
 		},
 	})
 
+	// Should receive resource listChanged notification
 	let resourceNotif
 	try {
 		resourceNotif = await Promise.race([
@@ -625,13 +633,31 @@ test('ListChanged notification: resources', async () => {
 		])
 	} catch {
 		throw new Error(
-			'🚨 Did not receive resources/listChanged notification when expected. Make sure your server calls sendResourceListChanged when resources change.',
+			'🚨 Did not receive resources/listChanged notification when expected. Make sure your server calls sendResourceListChanged when resources are enabled/disabled.',
 		)
 	}
 	expect(
 		resourceNotif,
-		'🚨 Did not receive resources/listChanged notification when expected. Make sure your server calls sendResourceListChanged when resources change.',
+		'🚨 Did not receive resources/listChanged notification when expected. Make sure your server calls sendResourceListChanged when resources are enabled/disabled.',
 	).toBeDefined()
+
+	// After notification, resources should now be available
+	const enabledResources = await client.listResources()
+	expect(
+		enabledResources.resources.length,
+		'🚨 Resources should be enabled after creating entries/tags. The server must dynamically enable/disable resources based on content.',
+	).toBeGreaterThan(0)
+
+	// Verify that resources are properly available
+	const resourceUris = enabledResources.resources.map((r) => r.uri)
+	expect(
+		resourceUris.some((uri) => uri.includes('entries')),
+		'🚨 Should have entry resources available after creating entries',
+	).toBe(true)
+	expect(
+		resourceUris.some((uri) => uri.includes('tags')),
+		'🚨 Should have tag resources available after creating tags',
+	).toBe(true)
 })
 
 test('ListChanged notification: tools', async () => {
@@ -646,7 +672,17 @@ test('ListChanged notification: tools', async () => {
 		},
 	)
 
-	// Trigger a DB change that should enable tools
+	// Get initial tool list
+	const initialTools = await client.listTools()
+	const initialToolNames = initialTools.tools.map((t) => t.name)
+
+	// Should not have tools requiring exisitng entries
+	expect(
+		initialToolNames,
+		'🚨 Tools requiring entries like get_entry should not be available initially',
+	).not.toContain('get_entry')
+
+	// Trigger a DB change that should enable additional tools
 	await client.callTool({
 		name: 'create_tag',
 		arguments: {
@@ -662,6 +698,7 @@ test('ListChanged notification: tools', async () => {
 		},
 	})
 
+	// Should receive tool listChanged notification
 	let toolNotif
 	try {
 		toolNotif = await Promise.race([
@@ -679,4 +716,18 @@ test('ListChanged notification: tools', async () => {
 		toolNotif,
 		'🚨 Did not receive tools/listChanged notification when expected. Make sure your server notifies clients when tools are enabled/disabled.',
 	).toBeDefined()
+
+	// After notification, additional tools should be available
+	const enabledTools = await client.listTools()
+	const enabledToolNames = enabledTools.tools.map((t) => t.name)
+	expect(
+		enabledToolNames,
+		'🚨 Tools requiring entries like get_entry should be enabled after creating entries/tags. The server must dynamically enable/disable tools based on content.',
+	).toContain('get_entry')
+
+	// Verify that tools are properly enabled with correct count
+	expect(
+		enabledTools.tools.length,
+		'🚨 Should have more tools available after creating content',
+	).toBeGreaterThan(initialTools.tools.length)
 })