Skip to content
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

feat: Event 단위 테스트 추가 #10

Merged

Conversation

gominnam
Copy link
Collaborator

📌 변경 사항

  • Event 컨트롤러와 서비스 단위테스트를 추가하였습니다.
  • EventController와 Service 의 디렉토리를 변경하였습니다.

@gominnam gominnam closed this Feb 17, 2025
@gominnam gominnam requested a review from f-lab-troy February 17, 2025 12:19
@gominnam gominnam reopened this Feb 17, 2025
Copy link

@f-lab-troy f-lab-troy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

단위테스트의 커버리지가 어느정도인지 수치화할 수 있는 방법을 생각해보면 좋습니다

}

@Test
public void getEventById_shouldReturnEvent() throws Exception {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

테스트 메서드명도 자바 컨벤션을 따르는게 좋습니다

@gominnam gominnam merged commit e6e3c99 into feature/3-ticket-api-event Feb 18, 2025
2 checks passed
@gominnam gominnam deleted the feature/3-ticket-api-event-unit-tests branch February 18, 2025 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants