bugfix-319-setup-crash-on-repository-with-no-issues: Improve error handling and test coverage for label and issue type related operations.

Author: efraespada

src/data/model/__tests__/execution.test.ts

@@ -67,7 +67,7 @@ jest.mock('../../../usecase/steps/common/get_hotfix_version_use_case', () => ({
   })),
 }));
 
-import { INPUT_KEYS } from '../../../utils/constants';
+import { ACTIONS, INPUT_KEYS } from '../../../utils/constants';
 import { Ai } from '../ai';
 import { Branches } from '../branches';
 import { Emoji } from '../emoji';
@@ -750,5 +750,20 @@ describe('Execution', () => {
       expect(e.hotfix.baseVersion).toBeUndefined();
       expect(e.hotfix.version).toBeUndefined();
     });
+    it('setup skips internal label fetch error when ACTION is INITIAL_SETUP', async () => {
+      const singleAction = new SingleAction(ACTIONS.INITIAL_SETUP, '1', '', '', '');
+      const issue = makeIssue({ eventName: 'issues', issue: { number: 1 } } as never);
+      const e = buildExecution({ eventName: 'issues', issue: { number: 1 } } as never, { singleAction, issue });
+      mockGetLabels.mockRejectedValue(new Error('Label error'));
+      await e.setup();
+      expect(e.labels.currentIssueLabels).toEqual([]);
+    });
+
+    it('setup throws when getLabels fails and not INITIAL_SETUP', async () => {
+      const issue = makeIssue({ eventName: 'issues', issue: { number: 1 } } as never);
+      const e = buildExecution({ eventName: 'issues', issue: { number: 1 } } as never, { issue });
+      mockGetLabels.mockRejectedValue(new Error('Fatal label error'));
+      await expect(e.setup()).rejects.toThrow('Fatal label error');
+    });
   });
 });

src/data/repository/__tests__/issue_repository.test.ts

@@ -85,6 +85,20 @@ function makeLabels(overrides: { currentIssueLabels?: string[] } = {}): Labels {
   return labels;
 }
 
+function makeIssueTypes(): IssueTypes {
+  return new IssueTypes(
+    'Task', 'Task desc', 'BLUE',
+    'Bug', 'Bug desc', 'RED',
+    'Feature', 'Feature desc', 'GREEN',
+    'Docs', 'Docs desc', 'GREY',
+    'Maintenance', 'Maint desc', 'GREY',
+    'Hotfix', 'Hotfix desc', 'RED',
+    'Release', 'Release desc', 'BLUE',
+    'Question', 'Q desc', 'PURPLE',
+    'Help', 'Help desc', 'PURPLE'
+  );
+}
+
 describe('IssueRepository', () => {
   const repo = new IssueRepository();
 
@@ -470,6 +484,12 @@ describe('IssueRepository', () => {
       expect(result).toEqual([]);
       expect(mockRest.issues.listLabelsOnIssue).toHaveBeenCalled();
     });
+
+    it('returns empty array when listLabelsOnIssue returns non-404 error', async () => {
+      mockRest.issues.listLabelsOnIssue.mockRejectedValue(new Error('Some other error'));
+      const result = await repo.getLabels('owner', 'repo', 1, 'token');
+      expect(result).toEqual([]);
+    });
   });
 
   describe('setLabels', () => {
@@ -1147,6 +1167,41 @@ describe('IssueRepository', () => {
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors.some((e) => e.includes('Create failed'))).toBe(true);
     });
+
+    it('setIssueType catches error and continues (re-throws in IssueRepository)', async () => {
+      const labels = makeLabels({ currentIssueLabels: ['bug'] });
+      const issueTypes = makeIssueTypes();
+      mockGraphql.mockRejectedValue(new Error('Global error'));
+      await expect(repo.setIssueType('o', 'r', 1, labels, issueTypes, 't')).rejects.toThrow('Global error');
+    });
+
+    it('createIssueType throws when organization is null', async () => {
+      mockGraphql.mockResolvedValueOnce({ organization: null });
+      await expect(repo.createIssueType('o', 'n', 'd', 'c', 't')).rejects.toThrow('No se pudo obtener la organización o');
+    });
+
+    it('listIssueTypes throws when organization is null', async () => {
+      mockGraphql.mockResolvedValueOnce({ organization: null });
+      await expect(repo.listIssueTypes('o', 't')).rejects.toThrow('No se pudo obtener la organización o');
+    });
+
+    it('createIssueType throws when organization is missing (GraphQL error scenario)', async () => {
+      mockGraphql.mockResolvedValueOnce({ organization: null });
+      await expect(repo.createIssueType('o', 'n', 'd', 'c', 't')).rejects.toThrow('No se pudo obtener la organización o');
+    });
+
+    it('ensureLabel handles 422 error "already exists"', async () => {
+      mockRest.issues.listLabelsForRepo.mockResolvedValue({ data: [] });
+      mockRest.issues.createLabel.mockRejectedValue({ status: 422, message: 'already exists' });
+      const result = await repo.ensureLabel('o', 'r', 'new', 'c', 'd', 't');
+      expect(result).toEqual({ created: false, existed: true });
+    });
+
+    it('ensureLabel re-throws other errors', async () => {
+      mockRest.issues.listLabelsForRepo.mockResolvedValue({ data: [] });
+      mockRest.issues.createLabel.mockRejectedValue(new Error('Fatal'));
+      await expect(repo.ensureLabel('o', 'r', 'new', 'c', 'd', 't')).rejects.toThrow('Fatal');
+    });
   });
 });