Test: 인터뷰 채팅 스트림 명세 검증 수정 (#323)

[IISweetHeartII]

Summary

인터뷰 채팅 스트림 테스트가 현재 AI relay payload 스펙과 일치하도록 수정했습니다.

Changes

• InterviewService.sendChatStream() 테스트 기대값을 숫자 배열에서 문자열 배열로 수정
• 현재 구현의 mentioned_insight_ids 직렬화 방식과 테스트를 일치시킴

Type of Change

해당하는 항목에 체크해주세요:

• Bug fix (기존 기능을 수정하는 변경)
• New feature (새로운 기능 추가)
• Breaking change (기존 기능에 영향을 주는 변경)
• Refactoring (기능 변경 없이 코드 개선)
• Documentation (문서 변경)
• Chore (빌드, 설정 등)

Target Environment

배포 대상 브랜치를 선택해주세요:

• Dev (dev)
• Prod (main)

Related Issues

관련 이슈를 연결해주세요:

• Closes N/A

Testing

테스트 방법을 작성해주세요:

• 단위 테스트 통과
• 로컬 빌드가 성공합니다 (pnpm run build)
• 로컬 린트가 통과합니다 (pnpm run lint)
• 실행 결과
  • pnpm test -- src/modules/interview/application/services/interview.service.spec.ts: 통과
  • pnpm run test: 통과
  • pnpm run build: 통과
  • pnpm run lint: 통과

Checklist

PR 생성 전 확인사항:

• 코드 컨벤션을 준수했습니다 (docs/development/CODE_STYLE.md)
• Git 컨벤션을 준수했습니다 (docs/development/GIT_CONVENTIONS.md)
• 네이밍 컨벤션을 준수했습니다 (docs/development/NAMING_CONVENTIONS.md)
• 로컬에서 빌드가 성공합니다 (pnpm run build)
• 로컬에서 린트가 통과합니다 (pnpm run lint)
• (API 변경 시) Swagger 문서가 업데이트되었습니다
• (필요 시) 테스트 코드를 작성했습니다

Screenshots (Optional)

N/A

Additional Notes

• 현재 open release PR #323 의 검증 상태를 맞추기 위한 선행 수정입니다.

[gemini-code-assist]

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 인터뷰 채팅 스트림 테스트가 AI 릴레이 페이로드 사양과 일치하도록 업데이트합니다. InterviewService.sendChatStream() 테스트의 기대값을 조정하고 mentioned_insight_ids 필드의 직렬화 방식에 맞춰 테스트 로직을 수정하여, 실제 구현과의 불일치를 해소하고 테스트의 정확성을 높였습니다.

Highlights

• 테스트 기대값 수정: InterviewService.sendChatStream() 테스트에서 mentioned_insight_ids의 기대값을 숫자 배열에서 문자열 배열로 변경했습니다.
• 직렬화 방식 일치: mentioned_insight_ids 필드의 현재 직렬화 방식과 일치하도록 테스트 로직을 수정했습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog

• src/modules/interview/application/services/interview.service.spec.ts
  • sendChatStream 테스트에서 mentioned_insight_ids의 값을 숫자 배열 [1]에서 문자열 배열 ['1']로 변경했습니다.

Activity

• 현재까지 기록된 활동이 없습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

PR의 테스트 수정 사항은 올바릅니다. 다만, 리뷰 중 SendInterviewChatReqDTO는 단일 insightId를, InterviewService.sendChatStream은 ID 배열을 인자로 받는 불일치를 확인했습니다. 이는 API와 서비스 계층 간의 명세 불일치로, 혼란을 유발할 수 있습니다. 향후 API에서 여러 ID를 지원할 계획이 아니라면, 서비스 메서드 시그니처를 DTO와 일치시키는 (..., insightId?: number) 형태로 변경하여 계층 간 책임을 명확히 하는 것을 제안합니다.