remove legacy test

Author: betegon

packages/core/test/lib/integrations/mcp-server/semanticConventions.test.ts

@@ -434,63 +434,5 @@ describe('MCP Server Semantic Conventions', () => {
       expect(setStatusSpy).not.toHaveBeenCalled();
       expect(endSpy).toHaveBeenCalled();
     });
-
-    it('should set span status to ERROR when tool result has isError: true', async () => {
-      await wrappedMcpServer.connect(mockTransport);
-
-      const setAttributesSpy = vi.fn();
-      const setStatusSpy = vi.fn();
-      const endSpy = vi.fn();
-      const mockSpan = {
-        setAttributes: setAttributesSpy,
-        setStatus: setStatusSpy,
-        end: endSpy,
-      } as unknown as ReturnType<typeof tracingModule.startInactiveSpan>;
-      startInactiveSpanSpy.mockReturnValueOnce(mockSpan);
-
-      const toolCallRequest = {
-        jsonrpc: '2.0',
-        method: 'tools/call',
-        id: 'req-tool-error',
-        params: {
-          name: 'failing-tool',
-          arguments: { input: 'test' },
-        },
-      };
-
-      // Simulate the incoming tool call request
-      mockTransport.onmessage?.(toolCallRequest, {});
-
-      // Simulate tool execution response with error
-      const toolErrorResponse = {
-        jsonrpc: '2.0',
-        id: 'req-tool-error',
-        result: {
-          content: [
-            {
-              type: 'text',
-              text: 'Tool execution failed',
-            },
-          ],
-          isError: true,
-        },
-      };
-
-      // Simulate the outgoing response (this should trigger span completion)
-      mockTransport.send?.(toolErrorResponse);
-
-      // Verify that the span was enriched with tool result attributes including error
-      expect(setAttributesSpy).toHaveBeenCalledWith(
-        expect.objectContaining({
-          'mcp.tool.result.is_error': true,
-          'mcp.tool.result.content_count': 1,
-          'mcp.tool.result.content_type': 'text',
-          'mcp.tool.result.content': 'Tool execution failed',
-        }),
-      );
-
-      expect(setStatusSpy).not.toHaveBeenCalled();
-      expect(endSpy).toHaveBeenCalled();
-    });
   });
 });