test: drop assertions that cannot fail

Four tests asserted nothing about the code under test:

- app/api/copilot/methods/route.test.ts was the directory's only file — it
  asserted expect(true).toBe(true) against a route that does not exist.
- tools/index.test.ts carried a block self-documented as existing "to
  maintain test count".
- mcp/storage/memory-cache.test.ts closed a delete-a-missing-key case with
  expect(true).toBe(true); it now asserts the call resolves without throwing.
- realtime/src/index.test.ts checked typeof roomManager.x === 'function' and
  typeof process.on === 'function', both of which tsc already proves.

Removing the realtime cases leaves a real gap: index.ts registers
uncaughtException, unhandledRejection, SIGINT and SIGTERM handlers with no
coverage. Better to have that gap visible than papered over by a test that
would pass with the handlers deleted.

Author: waleedlatif1

apps/realtime/src/index.test.ts

@@ -320,24 +320,9 @@ describe('Socket Server Index Integration', () => {
       expect(getWorkflowState).toBeTypeOf('function')
       expect(WorkflowOperationSchema).toBeDefined()
     })
-
-    it.concurrent('should maintain all original functionality after refactoring', async () => {
-      expect(httpServer).toBeDefined()
-      expect(io).toBeDefined()
-      expect(roomManager).toBeDefined()
-
-      expect(typeof roomManager.addUserToRoom).toBe('function')
-      expect(typeof roomManager.removeUserFromRoom).toBe('function')
-      expect(typeof roomManager.removeSocketFromAllRooms).toBe('function')
-      expect(typeof roomManager.broadcastPresenceUpdate).toBe('function')
-    })
   })
 
   describe('Error Handling', () => {
-    it('should have global error handlers configured', () => {
-      expect(typeof process.on).toBe('function')
-    })
-
     it('should handle server setup', () => {
       expect(httpServer).toBeDefined()
       expect(io).toBeDefined()

apps/sim/app/api/copilot/methods/route.test.ts

@@ -142,10 +142,7 @@ describe('MemoryMcpCache', () => {
     })
 
     it('does not throw for non-existent key', async () => {
-      // Should complete without throwing
-      await cache.delete('non-existent')
-      // If we get here, it worked
-      expect(true).toBe(true)
+      await expect(cache.delete('non-existent')).resolves.not.toThrow()
     })
 
     it('does not affect other entries', async () => {

apps/sim/lib/mcp/storage/memory-cache.test.ts

@@ -3623,12 +3623,6 @@ describe('Automatic Internal Route Detection', () => {
     Object.assign(tools, originalTools)
   })
 
-  it('PLACEHOLDER - external routes are called directly', async () => {
-    // Placeholder test to maintain test count - external URLs now go direct
-    // No proxy is used for external URLs anymore - they use secureFetchWithPinnedIP
-    expect(true).toBe(true)
-  })
-
   it('should call external URLs directly with SSRF protection', async () => {
     // External URLs now use secureFetchWithPinnedIP which uses Node's http/https modules
     // This test verifies the proxy is NOT called for external URLs