refactor: 필수 값이 없을 경우의 테스트 제거

필수 값이 없을 경우 무조건 오류가 발생하게 개선하였음

Author: six-standard

src/controllers/__test__/webhook.controller.test.ts

@@ -121,163 +121,6 @@ describe('WebhookController', () => {
       );
     });
 
-    it('resolved 액션에 대해 올바른 메시지를 생성해야 한다', async () => {
-      const resolvedData = {
-        ...mockSentryData,
-        action: 'resolved' as const,
-        data: {
-          ...mockSentryData.data,
-          issue: {
-            ...mockSentryData.data.issue,
-            status: 'resolved' as const
-          }
-        }
-      };
-      mockRequest.body = resolvedData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🚨 *오류가 해결되었습니다*')
-      );
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('✅ *제목:*')
-      );
-    });
-
-    it('ignored 액션에 대해 올바른 메시지를 생성해야 한다', async () => {
-      const ignoredData = {
-        ...mockSentryData,
-        action: 'ignored' as const,
-        data: {
-          ...mockSentryData.data,
-          issue: {
-            ...mockSentryData.data.issue,
-            status: 'ignored' as const
-          }
-        }
-      };
-      mockRequest.body = ignoredData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🚨 *오류가 무시되었습니다*')
-      );
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🔇 *제목:*')
-      );
-    });
-
-    it('assigned 액션에 대해 올바른 메시지를 생성해야 한다', async () => {
-      const assignedData = {
-        ...mockSentryData,
-        action: 'assigned' as const,
-        data: {
-          ...mockSentryData.data,
-          issue: {
-            ...mockSentryData.data.issue,
-            status: 'unresolved' as const
-          }
-        }
-      };
-      mockRequest.body = assignedData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🚨 *오류가 할당되었습니다*')
-      );
-    });
-
-    it('archived 액션에 대해 올바른 메시지를 생성해야 한다', async () => {
-      const archivedData = {
-        ...mockSentryData,
-        action: 'archived' as const,
-        data: {
-          ...mockSentryData.data,
-          issue: {
-            ...mockSentryData.data.issue,
-            status: 'archived' as const
-          }
-        }
-      };
-      mockRequest.body = archivedData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🚨 *오류가 아카이브되었습니다*')
-      );
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('📦 *제목:*')
-      );
-    });
-
-    it('알 수 없는 액션에 대해 기본 메시지를 생성해야 한다', async () => {
-      const unknownActionData = {
-        ...mockSentryData,
-        action: 'unknown_action' as 'created'
-      };
-      mockRequest.body = unknownActionData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('오류 이벤트: unknown_action')
-      );
-    });
-
-    it('알 수 없는 상태에 대해 기본 이모지를 사용해야 한다', async () => {
-      const unknownStatusData = {
-        ...mockSentryData,
-        data: {
-          ...mockSentryData.data,
-          issue: {
-            ...mockSentryData.data.issue,
-            status: 'unknown_status' as 'unresolved'
-          }
-        }
-      };
-      mockRequest.body = unknownStatusData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('❓ *제목:*')
-      );
-    });
-
     it('Slack 메시지 전송 실패 시 에러를 전달해야 한다', async () => {
       mockRequest.body = mockSentryData;
       const slackError = new Error('Slack 전송 실패');
@@ -292,46 +135,6 @@ describe('WebhookController', () => {
       expect(nextFunction).toHaveBeenCalledWith(slackError);
       expect(mockResponse.json).not.toHaveBeenCalled();
     });
-
-    it('빈 body로 요청 시에도 처리해야 한다', async () => {
-      mockRequest.body = {};
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      // undefined 값들에 대해서도 처리되어야 함
-      expect(mockSendSlackMessage).toHaveBeenCalled();
-      expect(mockResponse.status).toHaveBeenCalledWith(200);
-    });
-
-    it('필수 필드가 없는 경우에도 처리해야 한다', async () => {
-      const incompleteData = {
-        action: 'created',
-        data: {
-          issue: {
-            id: 'test-123'
-            // title, culprit 등 누락
-          }
-        }
-      };
-      mockRequest.body = incompleteData;
-      mockSendSlackMessage.mockResolvedValue();
-
-      await webhookController.handleSentryWebhook(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      expect(mockSendSlackMessage).toHaveBeenCalledWith(
-        expect.stringContaining('🔗 *상세 보기:* https://velog-dashboardv2.sentry.io/issues/test-123/')
-      );
-      expect(mockResponse.status).toHaveBeenCalledWith(200);
-    });
   });
 
   describe('formatSentryMessage (private method integration test)', () => {