@uarlouski/testrail-mcp-server 1.1.0 → 1.3.0

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');
     });
 });

package/test/tools/get_statuses.test.ts

@@ -0,0 +1,56 @@
+import { jest, describe, test, expect, beforeEach } from '@jest/globals';
+import { getStatusesTool } from '../../src/tools/get_statuses.js';
+import { TestRailClient } from '../../src/client/testrail.js';
+import { Status } from '../../src/types/testrail.js';
+
+describe('get_statuses tool', () => {
+    let mockClient: jest.Mocked<TestRailClient>;
+    let getStatusesMock: jest.Mock<() => Promise<Status[]>>;
+
+    const mockStatuses: Status[] = [
+        { id: 1, name: 'passed', label: 'Passed' },
+        { id: 2, name: 'blocked', label: 'Blocked' },
+        { id: 3, name: 'untested', label: 'Untested' },
+        { id: 4, name: 'retest', label: 'Retest' },
+        { id: 5, name: 'failed', label: 'Failed' },
+    ];
+
+    beforeEach(() => {
+        getStatusesMock = jest.fn<() => Promise<Status[]>>().mockResolvedValue(mockStatuses);
+
+        mockClient = {
+            getStatuses: getStatusesMock
+        } as unknown as jest.Mocked<TestRailClient>;
+    });
+
+    test('exports correct tool definition', () => {
+        expect(getStatusesTool.name).toBe('get_statuses');
+        expect(getStatusesTool.description).toContain('statuses');
+        expect(getStatusesTool.parameters).toBeDefined();
+    });
+
+    test('handler fetches and returns statuses', async () => {
+        const result = await getStatusesTool.handler({}, mockClient);
+
+        expect(result).toBeDefined();
+        expect(result.statuses).toBeDefined();
+        expect(result.statuses).toHaveLength(5);
+        expect(mockClient.getStatuses).toHaveBeenCalled();
+    });
+
+    test('returns correct status structure', async () => {
+        const result = await getStatusesTool.handler({}, mockClient);
+
+        expect(result.statuses[0]).toEqual({
+            id: 1,
+            name: 'passed',
+            label: 'Passed'
+        });
+    });
+
+    test('handler returns error on failure', async () => {
+        getStatusesMock.mockRejectedValue(new Error('API Error'));
+
+        await expect(getStatusesTool.handler({}, mockClient)).rejects.toThrow('API Error');
+    });
+});

package/test/tools/get_templates.test.ts

@@ -32,36 +32,7 @@ describe('get_templates tool', () => {
         const result = await getTemplatesTool.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.templates).toHaveLength(3);
+        expect(result.templates).toHaveLength(3);
         expect(mockClient.getTemplates).toHaveBeenCalledWith('1');
     });
-
-    test('returns correct template structure', async () => {
-        const result = await getTemplatesTool.handler({ project_id: '1' }, mockClient);
-        const parsed = JSON.parse(result.content[0].text);
-
-        expect(parsed.templates[0]).toEqual({
-            id: 1,
-            name: 'Test Case (Text)',
-        });
-
-        expect(parsed.templates[1]).toEqual({
-            id: 2,
-            name: 'Test Case (Steps)',
-        });
-    });
-
-    test('handler returns error on failure', async () => {
-        getTemplatesMock.mockRejectedValue(new Error('API Error'));
-
-        const result = await getTemplatesTool.handler({ project_id: '1' }, mockClient);
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
-    });
 });

package/test/tools/get_tests.test.ts

@@ -0,0 +1,47 @@
+import { jest, describe, test, expect, beforeEach } from '@jest/globals';
+import { getTestsTool } from '../../src/tools/get_tests.js';
+import { TestRailClient } from '../../src/client/testrail.js';
+import { Test } from '../../src/types/testrail.js';
+
+describe('get_tests tool', () => {
+    let mockClient: jest.Mocked<TestRailClient>;
+    let getTestsMock: jest.Mock<(runId: number, statusId?: number[]) => Promise<Test[]>>;
+
+    const mockTests: Test[] = [
+        { id: 1, case_id: 101, status_id: 1, title: 'Test 1', run_id: 1 },
+        { id: 2, case_id: 102, status_id: 2, title: 'Test 2', run_id: 1 },
+    ];
+
+    beforeEach(() => {
+        getTestsMock = jest.fn<(runId: number, statusId?: number[]) => Promise<Test[]>>().mockResolvedValue(mockTests);
+
+        mockClient = {
+            getTests: getTestsMock
+        } as unknown as jest.Mocked<TestRailClient>;
+    });
+
+    test('exports correct tool definition', () => {
+        expect(getTestsTool.name).toBe('get_tests');
+        expect(getTestsTool.description).toBeDefined();
+        expect(getTestsTool.parameters).toBeDefined();
+    });
+
+    test('handler fetches and returns tests', async () => {
+        const result = await getTestsTool.handler({ run_id: 1 }, mockClient);
+
+        expect(result).toBeDefined();
+        expect(result.tests).toBeDefined();
+        expect(result.tests).toHaveLength(2);
+        expect(mockClient.getTests).toHaveBeenCalledWith(1, undefined);
+    });
+
+    test('handler passes status_id filter', async () => {
+        const result = await getTestsTool.handler({ run_id: 1, status_id: [1, 5] }, mockClient);
+        expect(mockClient.getTests).toHaveBeenCalledWith(1, [1, 5]);
+    });
+
+    test('handler returns error on failure', async () => {
+        getTestsMock.mockRejectedValue(new Error('API Error'));
+        await expect(getTestsTool.handler({ run_id: 1 }, mockClient)).rejects.toThrow('API Error');
+    });
+});

package/test/tools/update_case.test.ts

@@ -52,12 +52,9 @@ describe('update_case tool', () => {
         );
 
         expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.success).toBe(true);
-        expect(parsed.case_id).toBe(123);
-        expect(parsed.message).toContain('C123');
+        expect(result.success).toBe(true);
+        expect(result.case_id).toBe(123);
+        expect(result.message).toContain('C123');
         expect(mockClient.updateCase).toHaveBeenCalledWith('123', { title: 'Updated Title' });
     });
 
@@ -92,17 +89,14 @@ describe('update_case tool', () => {
         expect(mockClient.updateCase).toHaveBeenCalledWith('123', fields);
     });
 
-    test('handler returns error on failure', async () => {
+    test('handler throws error on failure', async () => {
         updateCaseMock.mockRejectedValue(new Error('API Error'));
 
-        const result = await updateCaseTool.handler(
-            { case_id: '123', fields: { title: 'Test' } },
-            mockClient
-        );
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
+        await expect(
+            updateCaseTool.handler(
+                { case_id: '123', fields: { title: 'Test' } },
+                mockClient
+            )
+        ).rejects.toThrow('API Error');
     });
 });

package/test/tools/update_cases.test.ts

@@ -10,21 +10,21 @@ describe('update_cases tool', () => {
     const mockUpdatedCases: Case[] = [
         {
             id: 1, title: 'Test Case 1', 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: 1700000001, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 1, is_deleted: 0, labels: []
+            priority_id: 3, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000001, estimate: null,
+            suite_id: 1, labels: []
         },
         {
             id: 2, title: 'Test Case 2', 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: 1700000001, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 2, is_deleted: 0, labels: []
+            priority_id: 3, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000001, estimate: null,
+            suite_id: 1, labels: []
         },
         {
             id: 3, title: 'Test Case 3', 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: 1700000001, estimate: null, estimate_forecast: null,
-            suite_id: 1, display_order: 3, is_deleted: 0, labels: []
+            priority_id: 3, milestone_id: null, refs: null, created_on: 1700000000,
+            updated_by: 1, updated_on: 1700000001, estimate: null,
+            suite_id: 1, labels: []
         },
     ];
 
@@ -33,8 +33,12 @@ describe('update_cases tool', () => {
             .mockResolvedValue(mockUpdatedCases);
 
         mockClient = {
-            updateCases: updateCasesMock
+            updateCases: updateCasesMock,
+            getCase: jest.fn(),
         } as unknown as jest.Mocked<TestRailClient>;
+
+        // Default mock for getCase to avoid failures in unrelated tests
+        (mockClient.getCase as jest.Mock<any>).mockResolvedValue({ suite_id: 1 });
     });
 
     test('exports correct tool definition', () => {
@@ -52,37 +56,34 @@ describe('update_cases tool', () => {
         );
 
         expect(result).toBeDefined();
-        expect(result.content[0].type).toBe('text');
-
-        const parsed = JSON.parse(result.content[0].text);
-        expect(parsed.success).toBe(true);
-        expect(parsed.updated_count).toBe(3);
-        expect(parsed.case_ids).toEqual([1, 2, 3]);
-        expect(parsed.message).toContain('3 test cases');
+        expect(result.success).toBe(true);
+        expect(result.updated_count).toBe(3);
+        expect(result.case_ids).toEqual([1, 2, 3]);
+        expect(result.message).toContain('3 test cases');
     });
 
     test('passes case_ids and fields correctly', async () => {
         const fields = { priority_id: 2, type_id: 1 };
+        const mockCase = { suite_id: 5 } as Case;
+        (mockClient.getCase as jest.Mock<any>).mockResolvedValue(mockCase);
 
         await updateCasesTool.handler(
             { case_ids: [10, 20, 30], fields },
             mockClient
         );
 
-        expect(mockClient.updateCases).toHaveBeenCalledWith([10, 20, 30], fields);
+        expect(mockClient.getCase).toHaveBeenCalledWith('10');
+        expect(mockClient.updateCases).toHaveBeenCalledWith(5, [10, 20, 30], fields);
     });
 
-    test('handler returns error on failure', async () => {
+    test('handler throws error on failure', async () => {
         updateCasesMock.mockRejectedValue(new Error('API Error'));
 
-        const result = await updateCasesTool.handler(
-            { case_ids: [1, 2], fields: { priority_id: 2 } },
-            mockClient
-        );
-
-        expect(result).toEqual({
-            content: [{ type: 'text', text: 'Error: API Error' }],
-            isError: true
-        });
+        await expect(
+            updateCasesTool.handler(
+                { case_ids: [1, 2], fields: { priority_id: 2 } },
+                mockClient
+            )
+        ).rejects.toThrow('API Error');
     });
 });

package/test/utils/mapper.test.ts

@@ -6,13 +6,13 @@ describe('processCustomFields', () => {
 
     // Mock CaseField array simulating TestRail API response
     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: [] },
     ];
 
     test('maps custom fields using field metadata', () => {
@@ -21,7 +21,7 @@ describe('processCustomFields', () => {
             section_id: 0, type_id: 0, priority_id: 0,
             custom_is_automated: true,
             custom_refs: 'DEF-123',
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;
 
@@ -35,7 +35,7 @@ describe('processCustomFields', () => {
             id: 1, title: 'Foo', template_id: 1,
             section_id: 0, type_id: 0, priority_id: 0,
             custom_case_automation_priority: 2,
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;
 
@@ -50,7 +50,7 @@ describe('processCustomFields', () => {
             custom_expected: 'Expectation...',
             custom_steps_separated: [{ content: 'Should not appear' }], // Template 2 field
             section_id: 0, type_id: 0, priority_id: 0,
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;
 
@@ -66,7 +66,7 @@ describe('processCustomFields', () => {
             custom_steps: 'Should not appear', // Template 1 field
             custom_steps_separated: [{ content: 'Step 1' }],
             section_id: 0, type_id: 0, priority_id: 0,
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;
 
@@ -80,7 +80,7 @@ describe('processCustomFields', () => {
             id: 1, title: 'Foo', template_id: 999,
             custom_unknown_field: 'some value',
             section_id: 0, type_id: 0, priority_id: 0,
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;
 
@@ -101,7 +101,7 @@ describe('processCustomFields', () => {
             id: 1, title: 'Foo', template_id: 1,
             section_id: 0, type_id: 0, priority_id: 0,
             custom_case_automation_priority: 999, // Unknown option ID
-            display_order: 1, suite_id: 1, created_by: 1, created_on: 1, updated_by: 1, updated_on: 1,
+            display_order: 1, suite_id: 1, created_on: 1, updated_on: 1,
             is_deleted: 0, refs: null, labels: [],
         } as unknown as Case;