Skip to content

Feat : Validation 추가 및 테스트코드 추가 #105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Acacian
Copy link
Contributor

@Acacian Acacian commented Apr 2, 2025

Goal

Validation 추가, 테스트코드 추가

Changes

Validation을 Dto에 추가하여 잘못된 요청을 Controller에 들어오기 전에 걸러내었으며,
일부 Controller와 전역 예외 처리 로직에 대한 단위 테스트 추가했습니다.

Description

간단하게만 처음에 데이터를 구성했는데, 질문이나 응답 수가 늘어날 경우를 고려해
main에서 다시 branch를 딴 후 쿼리최적화부터 하려 합니다.

@Acacian Acacian requested a review from heli-os April 2, 2025 14:13
@Acacian Acacian self-assigned this Apr 2, 2025
@heli-os heli-os force-pushed the koodongha-onboarding-6 branch from 4ba50b0 to dba2fba Compare April 5, 2025 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant