@uarlouski/testrail-mcp-server 1.1.0 → 1.2.0

package/test/tools/get_case.test.ts

@@ -12,13 +12,13 @@ describe('get_case tool', () => {
     let getCaseFieldsMock: jest.Mock<() => Promise<CaseField[]>>;
 
     const mockCaseFields: CaseField[] = [
-        { id: 1, name: 'is_automated', system_name: 'custom_is_automated', label: 'Is Automated', type_id: 1, template_ids: [], include_all: true, is_active: true, display_order: 1, description: null, configs: [] },
-        { id: 2, name: 'refs', system_name: 'custom_refs', label: 'Defects', type_id: 1, template_ids: [], include_all: true, is_active: true, display_order: 2, description: null, configs: [] },
-        { id: 3, name: 'case_automation_priority', system_name: 'custom_case_automation_priority', label: 'Automation Priority', type_id: 6, template_ids: [], include_all: true, is_active: true, display_order: 3, description: null, configs: [{ context: { is_global: true, project_ids: null }, options: { items: '2, P0\n3, P1\n4, P2\n5, P3\n6, P4' } }] },
-        { id: 4, name: 'preconds', system_name: 'custom_preconds', label: 'Preconditions', type_id: 1, template_ids: [], include_all: true, is_active: true, display_order: 4, description: null, configs: [] },
-        { id: 5, name: 'steps', system_name: 'custom_steps', label: 'Steps', type_id: 1, template_ids: [1], include_all: false, is_active: true, display_order: 5, description: null, configs: [] },
-        { id: 6, name: 'expected', system_name: 'custom_expected', label: 'Expected', type_id: 1, template_ids: [1], include_all: false, is_active: true, display_order: 6, description: null, configs: [] },
-        { id: 7, name: 'steps_separated', system_name: 'custom_steps_separated', label: 'Steps Separated', type_id: 1, template_ids: [2], include_all: false, is_active: true, display_order: 7, description: null, configs: [] },
+        { id: 1, name: 'is_automated', system_name: 'custom_is_automated', label: 'Is Automated', type_id: 1, template_ids: [], include_all: true, is_active: true, description: null, configs: [] },
+        { id: 2, name: 'refs', system_name: 'custom_refs', label: 'Defects', type_id: 1, template_ids: [], include_all: true, is_active: true, description: null, configs: [] },
+        { id: 3, name: 'case_automation_priority', system_name: 'custom_case_automation_priority', label: 'Automation Priority', type_id: 6, template_ids: [], include_all: true, is_active: true, description: null, configs: [{ options: { items: '2, P0\n3, P1\n4, P2\n5, P3\n6, P4' } }] },
+        { id: 4, name: 'preconds', system_name: 'custom_preconds', label: 'Preconditions', type_id: 1, template_ids: [], include_all: true, is_active: true, description: null, configs: [] },
+        { id: 5, name: 'steps', system_name: 'custom_steps', label: 'Steps', type_id: 1, template_ids: [1], include_all: false, is_active: true, description: null, configs: [] },
+        { id: 6, name: 'expected', system_name: 'custom_expected', label: 'Expected', type_id: 1, template_ids: [1], include_all: false, is_active: true, description: null, configs: [] },
+        { id: 7, name: 'steps_separated', system_name: 'custom_steps_separated', label: 'Steps Separated', type_id: 1, template_ids: [2], include_all: false, is_active: true, description: null, configs: [] },
     ];
 
     beforeEach(() => {
@@ -51,15 +51,10 @@ describe('get_case tool', () => {
             priority_id: 3,
             milestone_id: null,
             refs: 'JIRA-123',
-            created_by: 1,
             created_on: 1700000000,
-            updated_by: 1,
             updated_on: 1700000000,
             estimate: null,
-            estimate_forecast: null,
             suite_id: 1,
-            display_order: 1,
-            is_deleted: 0,
             custom_refs: null,
             custom_case_automation_priority: null,
             custom_is_automated: null,
@@ -79,7 +74,7 @@ describe('get_case tool', () => {
         };
 
         const mockSection: Section = {
-            id: 456, name: 'My Section', description: null, parent_id: null, depth: 0, display_order: 1, suite_id: 1
+            id: 456, name: 'My Section', description: null, parent_id: null, suite_id: 1
         };
 
         const mockCaseTypes: CaseType[] = [
@@ -88,7 +83,7 @@ describe('get_case tool', () => {
         ];
 
         const mockPriorities: Priority[] = [
-            { id: 3, name: 'High', priority: 3, short_name: 'High', is_default: false }
+            { id: 3, name: 'High', is_default: false }
         ];
 
         getCaseMock.mockResolvedValue(mockCase);
@@ -99,23 +94,16 @@ describe('get_case tool', () => {
         const result = await getCaseTool.handler({ case_id: 'C123' }, mockClient);
 
         expect(result).toBeDefined();
-        if ('content' in result) {
-            expect(result.content[0].type).toBe('text');
-            const parsed = JSON.parse(result.content[0].text);
-
-            expect(parsed).toEqual({
-                id: 123,
-                title: 'My Case',
-                section: 'My Section',
-                type: 'Functional',
-                priority: 'High',
-                labels: ['tag1'],
-                references: 'JIRA-123',
-                updated_on: 1700000000
-            });
-        } else {
-            throw new Error('Result should have content');
-        }
+        expect(result).toEqual({
+            id: 123,
+            title: 'My Case',
+            section: 'My Section',
+            type: 'Functional',
+            priority: 'High',
+            labels: ['tag1'],
+            references: 'JIRA-123',
+            updated_on: 1700000000
+        });
 
         expect(mockClient.getCase).toHaveBeenCalledWith('123');
         expect(mockClient.getSection).toHaveBeenCalledWith('456');
@@ -125,76 +113,73 @@ describe('get_case tool', () => {
 
 
 
-    test('handler returns error on failure', async () => {
+    test('handler throws error on failure', async () => {
         getCaseMock.mockRejectedValue(new Error('API Error'));
 
-        const result = await getCaseTool.handler({ case_id: '123' }, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
+        await expect(
+            getCaseTool.handler({ case_id: '123' }, mockClient)
+        ).rejects.toThrow('API Error');
     });
 
     test('handler returns "Unknown" when caseType is not found', async () => {
         const mockCase: Case = {
             id: 123, title: 'My Case', template_id: 1,
             section_id: 456, type_id: 999, priority_id: 3,
-            milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 1, is_deleted: 0, labels: []
+            milestone_id: null, refs: null, created_on: 1700000000,
+            updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: []
         };
 
         getCaseMock.mockResolvedValue(mockCase);
-        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, depth: 0, display_order: 1, suite_id: 1 });
+        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, suite_id: 1 });
         getCaseTypesMock.mockResolvedValue([{ id: 1, name: 'Automated', is_default: false }]);
-        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', priority: 3, short_name: 'High', is_default: false }]);
+        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', is_default: false }]);
 
         const result = await getCaseTool.handler({ case_id: '123' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
+        // result is now a plain object
 
-        expect(parsed.type).toBe('Unknown');
-        expect(parsed.priority).toBe('High');
+        expect(result.type).toBe('Unknown');
+        expect(result.priority).toBe('High');
     });
 
     test('handler returns "Unknown" when priority is not found', async () => {
         const mockCase: Case = {
             id: 123, title: 'My Case', template_id: 1,
             section_id: 456, type_id: 1, priority_id: 999,
-            milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 1, is_deleted: 0, labels: []
+            milestone_id: null, refs: null, created_on: 1700000000,
+            updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: []
         };
 
         getCaseMock.mockResolvedValue(mockCase);
-        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, depth: 0, display_order: 1, suite_id: 1 });
+        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, suite_id: 1 });
         getCaseTypesMock.mockResolvedValue([{ id: 1, name: 'Automated', is_default: false }]);
-        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', priority: 3, short_name: 'High', is_default: false }]);
+        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', is_default: false }]);
 
         const result = await getCaseTool.handler({ case_id: '123' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
+        // result is now a plain object
 
-        expect(parsed.type).toBe('Automated');
-        expect(parsed.priority).toBe('Unknown');
+        expect(result.type).toBe('Automated');
+        expect(result.priority).toBe('Unknown');
     });
 
     test('handler uses empty array when labels is undefined', async () => {
         const mockCase: Case = {
             id: 123, title: 'My Case', template_id: 1,
             section_id: 456, type_id: 1, priority_id: 3,
-            milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 1, is_deleted: 0, labels: undefined as any
+            milestone_id: null, refs: null, created_on: 1700000000,
+            updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: undefined as any
         };
 
         getCaseMock.mockResolvedValue(mockCase);
-        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, depth: 0, display_order: 1, suite_id: 1 });
+        getSectionMock.mockResolvedValue({ id: 456, name: 'Section', description: null, parent_id: null, suite_id: 1 });
         getCaseTypesMock.mockResolvedValue([{ id: 1, name: 'Automated', is_default: false }]);
-        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', priority: 3, short_name: 'High', is_default: false }]);
+        getPrioritiesMock.mockResolvedValue([{ id: 3, name: 'High', is_default: false }]);
 
         const result = await getCaseTool.handler({ case_id: '123' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
+        // result is now a plain object
 
-        expect(parsed.labels).toEqual([]);
+        expect(result.labels).toEqual([]);
     });
 });

package/test/tools/get_case_fields.test.ts

@@ -17,9 +17,8 @@ describe('get_case_fields tool', () => {
             template_ids: [],
             include_all: true,
             is_active: true,
-            display_order: 1,
             description: 'Whether test is automated',
-            configs: [{ context: { is_global: true, project_ids: null }, options: { is_required: false } }]
+            configs: [{ options: { is_required: false } }]
         },
         {
             id: 2,
@@ -30,9 +29,8 @@ describe('get_case_fields tool', () => {
             template_ids: [],
             include_all: true,
             is_active: true,
-            display_order: 2,
             description: 'Priority for automation',
-            configs: [{ context: { is_global: true, project_ids: null }, options: { items: '1, P0\n2, P1\n3, P2', is_required: true } }]
+            configs: [{ options: { items: '1, P0\n2, P1\n3, P2', is_required: true } }]
         },
         {
             id: 3,
@@ -43,7 +41,6 @@ describe('get_case_fields tool', () => {
             template_ids: [],
             include_all: true,
             is_active: false,
-            display_order: 3,
             description: null,
             configs: []
         },
@@ -56,9 +53,8 @@ describe('get_case_fields tool', () => {
             template_ids: [],
             include_all: true,
             is_active: true,
-            display_order: 4,
             description: 'Test components',
-            configs: [{ context: { is_global: true, project_ids: null }, options: { items: '1, UI\n2, API\n3, Database' } }]
+            configs: [{ options: { items: '1, UI\n2, API\n3, Database' } }]
         }
     ];
 
@@ -80,76 +76,11 @@ describe('get_case_fields tool', () => {
         const result = await getCaseFieldsTool.handler({}, mockClient);
 
         expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.fields).toHaveLength(11); // 8 system fields + 3 active custom fields
+        expect(result.fields).toBeDefined();
+        expect(result.fields.length).toBeGreaterThan(0);
         expect(mockClient.getCaseFields).toHaveBeenCalled();
     });
 
-    test('returns only essential schema fields', async () => {
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const field = parsed.fields[0];
-        expect(Object.keys(field).sort()).toEqual(['is_required', 'label', 'system_name', 'type']);
-    });
-
-    test('returns correct field types', async () => {
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const checkboxField = parsed.fields.find((f: any) => f.system_name === 'custom_is_automated');
-        expect(checkboxField.type).toBe('Checkbox');
-
-        const dropdownField = parsed.fields.find((f: any) => f.system_name === 'custom_automation_priority');
-        expect(dropdownField.type).toBe('Dropdown');
-
-        const multiSelectField = parsed.fields.find((f: any) => f.system_name === 'custom_components');
-        expect(multiSelectField.type).toBe('Multi-select');
-    });
-
-    test('parses dropdown options correctly', async () => {
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const dropdownField = parsed.fields.find((f: any) => f.system_name === 'custom_automation_priority');
-        expect(dropdownField.options).toEqual(['P0', 'P1', 'P2']);
-
-        const multiSelectField = parsed.fields.find((f: any) => f.system_name === 'custom_components');
-        expect(multiSelectField.options).toEqual(['UI', 'API', 'Database']);
-    });
-
-    test('identifies required fields', async () => {
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const optionalField = parsed.fields.find((f: any) => f.system_name === 'custom_is_automated');
-        expect(optionalField.is_required).toBe(false);
-
-        const requiredField = parsed.fields.find((f: any) => f.system_name === 'custom_automation_priority');
-        expect(requiredField.is_required).toBe(true);
-    });
-
-    test('filters out inactive fields', async () => {
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const inactiveField = parsed.fields.find((f: any) => f.system_name === 'custom_inactive_field');
-        expect(inactiveField).toBeUndefined();
-    });
-
-    test('handler returns error on failure', async () => {
-        getCaseFieldsMock.mockRejectedValue(new Error('API Error'));
-
-        const result = await getCaseFieldsTool.handler({}, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
-    });
-
     test('includes template_ids for template-specific fields', async () => {
         const templateSpecificFields: CaseField[] = [
             {
@@ -161,7 +92,6 @@ describe('get_case_fields tool', () => {
                 template_ids: [1, 2],
                 include_all: false,
                 is_active: true,
-                display_order: 5,
                 description: 'Test steps',
                 configs: []
             }
@@ -170,9 +100,8 @@ describe('get_case_fields tool', () => {
         getCaseFieldsMock.mockResolvedValue(templateSpecificFields);
 
         const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
 
-        const stepsField = parsed.fields.find((f: any) => f.system_name === 'custom_steps');
+        const stepsField = result.fields.find((f: any) => f.system_name === 'custom_steps');
         expect(stepsField.template_ids).toEqual([1, 2]);
     });
 
@@ -187,7 +116,6 @@ describe('get_case_fields tool', () => {
                 template_ids: [],
                 include_all: true,
                 is_active: true,
-                display_order: 6,
                 description: null,
                 configs: []
             }
@@ -196,9 +124,8 @@ describe('get_case_fields tool', () => {
         getCaseFieldsMock.mockResolvedValue(unknownTypeFields);
 
         const result = await getCaseFieldsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
 
-        const unknownField = parsed.fields.find((f: any) => f.system_name === 'custom_unknown_type');
+        const unknownField = result.fields.find((f: any) => f.system_name === 'custom_unknown_type');
         expect(unknownField.type).toBe('Unknown (999)');
     });
 });

package/test/tools/get_cases.test.ts

@@ -10,21 +10,21 @@ describe('get_cases tool', () => {
     const mockCases: Case[] = [
         {
             id: 1, title: 'Login test', section_id: 1, template_id: 1, type_id: 1,
-            priority_id: 2, milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 1, is_deleted: 0, labels: [], custom_automation_status: 1
+            priority_id: 2, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: [], custom_automation_status: 1
         },
         {
             id: 2, title: 'Logout test', section_id: 1, template_id: 1, type_id: 1,
-            priority_id: 3, milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 2, is_deleted: 0, labels: [], custom_automation_status: 2
+            priority_id: 3, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: [], custom_automation_status: 2
         },
         {
             id: 3, title: 'Registration test', section_id: 2, template_id: 1, type_id: 1,
-            priority_id: 1, milestone_id: null, refs: null, created_by: 1, created_on: 1700000000,
-            updated_by: 1, updated_on: 1700000000, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 3, is_deleted: 0, labels: [], custom_automation_status: 0
+            priority_id: 1, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000000, estimate: null,
+            suite_id: 1, labels: [], custom_automation_status: 0
         },
     ];
 
@@ -51,105 +51,19 @@ describe('get_cases tool', () => {
         const result = await getCasesTool.handler({ project_id: '1' }, mockClient);
 
         expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.cases).toHaveLength(3);
+        expect(result.cases).toHaveLength(3);
         expect(mockClient.getCases).toHaveBeenCalledWith('1', undefined, undefined);
     });
 
-    test('passes section_id when provided', async () => {
-        await getCasesTool.handler({ project_id: '1', section_id: '2' }, mockClient);
-
-        expect(mockClient.getCases).toHaveBeenCalledWith('1', '2', undefined);
-    });
-
-    test('passes filter when provided', async () => {
-        const filter = { priority_id: '1,2', type_id: '3' };
-        await getCasesTool.handler({ project_id: '1', filter }, mockClient);
-
-        expect(mockClient.getCases).toHaveBeenCalledWith('1', undefined, filter);
-    });
-
-    test('returns correct case structure', async () => {
-        const result = await getCasesTool.handler({ project_id: '1' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.cases[0]).toEqual({
-            id: 1,
-            title: 'Login test',
-            suite_id: 1,
-        });
-
-        expect(parsed.cases[2]).toEqual({
-            id: 3,
-            title: 'Registration test',
-            suite_id: 1,
-        });
-    });
-
-    test('includes additional fields when specified', async () => {
-        const result = await getCasesTool.handler(
-            { project_id: '1', fields: ['priority_id', 'type_id', 'custom_automation_status'] },
-            mockClient
-        );
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.cases[0]).toEqual({
-            id: 1,
-            title: 'Login test',
-            suite_id: 1,
-            priority_id: 2,
-            type_id: 1,
-            custom_automation_status: 1,
-        });
-
-        expect(parsed.cases[1]).toEqual({
-            id: 2,
-            title: 'Logout test',
-            suite_id: 1,
-            priority_id: 3,
-            type_id: 1,
-            custom_automation_status: 2,
-        });
-    });
-
-    test('ignores non-existent fields', async () => {
-        const result = await getCasesTool.handler(
-            { project_id: '1', fields: ['priority_id', 'nonexistent_field'] },
-            mockClient
-        );
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.cases[0]).toEqual({
-            id: 1,
-            title: 'Login test',
-            suite_id: 1,
-            priority_id: 2,
-        });
-    });
-
-    test('handler returns error on failure', async () => {
-        getCasesMock.mockRejectedValue(new Error('API Error'));
-
-        const result = await getCasesTool.handler({ project_id: '1' }, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
-    });
-
     test('filters cases using where clause with single condition', async () => {
         const result = await getCasesTool.handler(
             { project_id: '1', where: { custom_automation_status: 1 } },
             mockClient
         );
-        const parsed = JSON.parse(result.content[0].text);
 
-        expect(parsed.cases).toHaveLength(1);
-        expect(parsed.cases[0].id).toBe(1);
-        expect(parsed.cases[0].title).toBe('Login test');
+        expect(result.cases).toHaveLength(1);
+        expect(result.cases[0].id).toBe(1);
+        expect(result.cases[0].title).toBe('Login test');
     });
 
     test('filters cases using where clause with multiple conditions', async () => {
@@ -157,11 +71,10 @@ describe('get_cases tool', () => {
             { project_id: '1', where: { custom_automation_status: 2, priority_id: 3 } },
             mockClient
         );
-        const parsed = JSON.parse(result.content[0].text);
 
-        expect(parsed.cases).toHaveLength(1);
-        expect(parsed.cases[0].id).toBe(2);
-        expect(parsed.cases[0].title).toBe('Logout test');
+        expect(result.cases).toHaveLength(1);
+        expect(result.cases[0].id).toBe(2);
+        expect(result.cases[0].title).toBe('Logout test');
     });
 
     test('returns empty array when where clause matches no cases', async () => {
@@ -169,9 +82,8 @@ describe('get_cases tool', () => {
             { project_id: '1', where: { custom_automation_status: 999 } },
             mockClient
         );
-        const parsed = JSON.parse(result.content[0].text);
 
-        expect(parsed.cases).toHaveLength(0);
+        expect(result.cases).toHaveLength(0);
     });
 
     test('combines where clause with fields parameter', async () => {
@@ -179,10 +91,9 @@ describe('get_cases tool', () => {
             { project_id: '1', where: { priority_id: 2 }, fields: ['priority_id', 'custom_automation_status'] },
             mockClient
         );
-        const parsed = JSON.parse(result.content[0].text);
 
-        expect(parsed.cases).toHaveLength(1);
-        expect(parsed.cases[0]).toEqual({
+        expect(result.cases).toHaveLength(1);
+        expect(result.cases[0]).toEqual({
             id: 1,
             title: 'Login test',
             suite_id: 1,

package/test/tools/get_projects.test.ts

@@ -8,9 +8,9 @@ describe('get_projects tool', () => {
     let getProjectsMock: jest.Mock<() => Promise<Project[]>>;
 
     const mockProjects: Project[] = [
-        { id: 1, name: 'Project Alpha', announcement: null, is_completed: false, suite_mode: 1 },
-        { id: 2, name: 'Project Beta', announcement: 'Active project', is_completed: false, suite_mode: 3 },
-        { id: 3, name: 'Old Project', announcement: null, is_completed: true, suite_mode: 1 },
+        { id: 1, name: 'Project Alpha', is_completed: false, suite_mode: 1 },
+        { id: 2, name: 'Project Beta', is_completed: false, suite_mode: 3 },
+        { id: 3, name: 'Old Project', is_completed: true, suite_mode: 1 },
     ];
 
     beforeEach(() => {
@@ -32,44 +32,7 @@ describe('get_projects tool', () => {
         const result = await getProjectsTool.handler({}, mockClient);
 
         expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.projects).toHaveLength(2); // Only active projects
+        expect(result.projects).toHaveLength(2);
         expect(mockClient.getProjects).toHaveBeenCalled();
     });
-
-    test('filters out completed projects', async () => {
-        const result = await getProjectsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        const completedProject = parsed.projects.find((p: any) => p.name === 'Old Project');
-        expect(completedProject).toBeUndefined();
-    });
-
-    test('returns correct project structure', async () => {
-        const result = await getProjectsTool.handler({}, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.projects[0]).toEqual({
-            id: 1,
-            name: 'Project Alpha',
-        });
-
-        expect(parsed.projects[1]).toEqual({
-            id: 2,
-            name: 'Project Beta',
-        });
-    });
-
-    test('handler returns error on failure', async () => {
-        getProjectsMock.mockRejectedValue(new Error('API Error'));
-
-        const result = await getProjectsTool.handler({}, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
-    });
 });

package/test/tools/get_sections.test.ts

@@ -8,9 +8,9 @@ describe('get_sections tool', () => {
     let getSectionsMock: jest.Mock<(projectId: string) => Promise<Section[]>>;
 
     const mockSections: Section[] = [
-        { id: 1, name: 'Login', description: null, parent_id: null, depth: 0, display_order: 1, suite_id: 1 },
-        { id: 2, name: 'Authentication', description: 'Auth tests', parent_id: 1, depth: 1, display_order: 2, suite_id: 1 },
-        { id: 3, name: 'Registration', description: null, parent_id: null, depth: 0, display_order: 3, suite_id: 1 },
+        { id: 1, name: 'Login', description: null, parent_id: null, suite_id: 1 },
+        { id: 2, name: 'Authentication', description: 'Auth tests', parent_id: 1, suite_id: 1 },
+        { id: 3, name: 'Registration', description: null, parent_id: null, suite_id: 1 },
     ];
 
     beforeEach(() => {
@@ -29,40 +29,11 @@ describe('get_sections tool', () => {
         expect(getSectionsTool.parameters.project_id).toBeDefined();
     });
 
-    test('handler fetches and returns sections', async () => {
-        const result = await getSectionsTool.handler({ project_id: '1' }, mockClient);
-
-        expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.sections).toHaveLength(3);
-        expect(mockClient.getSections).toHaveBeenCalledWith('1');
-    });
-
-    test('returns correct section structure', async () => {
-        const result = await getSectionsTool.handler({ project_id: '1' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.sections[0]).toEqual({
-            id: 1,
-            name: 'Login',
-        });
-
-        expect(parsed.sections[1]).toEqual({
-            id: 2,
-            name: 'Authentication',
-        });
-    });
-
-    test('handler returns error on failure', async () => {
+    test('handler throws error on failure', async () => {
         getSectionsMock.mockRejectedValue(new Error('API Error'));
 
-        const result = await getSectionsTool.handler({ project_id: '1' }, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
+        await expect(
+            getSectionsTool.handler({ project_id: '1' }, mockClient)
+        ).rejects.toThrow('API Error');
     });
 });