fix tests

Author: icecrasher321

apps/sim/lib/mcp/tool-validation.test.ts

@@ -24,69 +24,69 @@ describe('hasSchemaChanged', () => {
   })
 
   it.concurrent('returns false for identical schemas', () => {
-    const schema = { type: 'object', properties: { name: { type: 'string' } } }
+    const schema = { type: 'object' as const, properties: { name: { type: 'string' } } }
     expect(hasSchemaChanged(schema, { ...schema })).toBe(false)
   })
 
   it.concurrent('returns false when only description differs', () => {
     const stored = {
-      type: 'object',
+      type: 'object' as const,
       properties: { name: { type: 'string' } },
       description: 'Old description',
     }
     const server = {
-      type: 'object',
+      type: 'object' as const,
       properties: { name: { type: 'string' } },
       description: 'New description',
     }
     expect(hasSchemaChanged(stored, server)).toBe(false)
   })
 
-  it.concurrent('returns true when type differs', () => {
-    const stored = { type: 'object', properties: {} }
-    const server = { type: 'array', properties: {} }
-    expect(hasSchemaChanged(stored, server)).toBe(true)
-  })
-
   it.concurrent('returns true when properties differ', () => {
-    const stored = { type: 'object', properties: { name: { type: 'string' } } }
-    const server = { type: 'object', properties: { id: { type: 'number' } } }
+    const stored = { type: 'object' as const, properties: { name: { type: 'string' } } }
+    const server = { type: 'object' as const, properties: { id: { type: 'number' } } }
     expect(hasSchemaChanged(stored, server)).toBe(true)
   })
 
   it.concurrent('returns true when required fields differ', () => {
-    const stored = { type: 'object', properties: {}, required: ['name'] }
-    const server = { type: 'object', properties: {}, required: ['id'] }
+    const stored = { type: 'object' as const, properties: {}, required: ['name'] }
+    const server = { type: 'object' as const, properties: {}, required: ['id'] }
     expect(hasSchemaChanged(stored, server)).toBe(true)
   })
 
   it.concurrent('returns false for deep equal schemas with different key order', () => {
-    const stored = { type: 'object', properties: { a: 1, b: 2 } }
-    const server = { properties: { b: 2, a: 1 }, type: 'object' }
+    const stored = { type: 'object' as const, properties: { a: 1, b: 2 } }
+    const server = { properties: { b: 2, a: 1 }, type: 'object' as const }
     expect(hasSchemaChanged(stored, server)).toBe(false)
   })
 
   it.concurrent('returns true when nested properties differ', () => {
     const stored = {
-      type: 'object',
+      type: 'object' as const,
       properties: { config: { type: 'object', properties: { enabled: { type: 'boolean' } } } },
     }
     const server = {
-      type: 'object',
+      type: 'object' as const,
       properties: { config: { type: 'object', properties: { enabled: { type: 'string' } } } },
     }
     expect(hasSchemaChanged(stored, server)).toBe(true)
   })
 
   it.concurrent('returns true when additional properties setting differs', () => {
-    const stored = { type: 'object', additionalProperties: true }
-    const server = { type: 'object', additionalProperties: false }
+    const stored = { type: 'object' as const, additionalProperties: true }
+    const server = { type: 'object' as const, additionalProperties: false }
     expect(hasSchemaChanged(stored, server)).toBe(true)
   })
 
   it.concurrent('ignores description at property level', () => {
-    const stored = { type: 'object', properties: { name: { type: 'string', description: 'Old' } } }
-    const server = { type: 'object', properties: { name: { type: 'string', description: 'New' } } }
+    const stored = {
+      type: 'object' as const,
+      properties: { name: { type: 'string', description: 'Old' } },
+    }
+    const server = {
+      type: 'object' as const,
+      properties: { name: { type: 'string', description: 'New' } },
+    }
     // Only top-level description is ignored, not nested ones
     expect(hasSchemaChanged(stored, server)).toBe(true)
   })
@@ -298,7 +298,7 @@ describe('getMcpToolIssue', () => {
     })
 
     it.concurrent('returns null when schemas match exactly', () => {
-      const schema = { type: 'object', properties: { name: { type: 'string' } } }
+      const schema = { type: 'object' as const, properties: { name: { type: 'string' } } }
       const storedTool = createStoredTool({ schema })
       const servers = [createServerState()]
       const tools = [createDiscoveredTool({ inputSchema: schema })]

apps/sim/lib/mcp/types.test.ts

@@ -81,7 +81,7 @@ describe('McpError', () => {
     const error = new McpError('Multiple validation errors', 400, complexData)
 
     expect(error.data).toEqual(complexData)
-    expect(error.data.errors).toHaveLength(2)
+    expect((error.data as typeof complexData).errors).toHaveLength(2)
   })
 
   it.concurrent('handles array as data', () => {