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

[fix] 누락된 필드값 및 엔티티 수정 #148

Merged
merged 4 commits into from
Feb 1, 2025

Conversation

ehs208
Copy link
Member

@ehs208 ehs208 commented Jan 31, 2025

개요

누락된 필드들을 모두 추가하였습니다.
또한 10명의 사용자가 10번씩 동시에 모임참여를 시도할시 발생할수 있는 상황에대한(비현실적이지만.. 극단적으로) 테스트를 진행하였습니다.

2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 총 요청 수: 100
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 성공한 요청 수: 1
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 실패한 요청 수: 99
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 일반 실패 (인원초과): 99
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 타임아웃: 0
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 데드락/기타: 0
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 타임아웃 발생여부: false
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 총 소요 시간: 525ms
2025-01-31T22:57:51.152+09:00  INFO 66480 --- [    Test worker] c.e.e.a.d.m.service.MeetingServiceTest   : 초당 처리량: 190.47619047619048 requests/second

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

@ehs208 ehs208 added add feat 이외의 부수적인 코드, 파일, 라이브러리 추가 현수 labels Jan 31, 2025
@ehs208 ehs208 requested a review from a team January 31, 2025 13:59
@ehs208 ehs208 self-assigned this Jan 31, 2025
@ehs208 ehs208 linked an issue Jan 31, 2025 that may be closed by this pull request
4 tasks
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

수정 사항과 테스트 코드까지 모두 잘 확인했습니다! 특이 사항은 없는 것 같습니다. 고생하셨습니다!

Copy link
Contributor

@seokjun01 seokjun01 left a comment

Choose a reason for hiding this comment

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

코드확인하였습니다! 테스트 코드 작성하시느라 고생하셨습니다. 이상없는 것 같아 승인하겠습니다!

@ehs208 ehs208 merged commit 8641756 into dev Feb 1, 2025
2 checks passed
@ehs208 ehs208 deleted the fix/#147-누락된-필드값-및-엔티티-수정 branch February 4, 2025 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add feat 이외의 부수적인 코드, 파일, 라이브러리 추가 현수
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[add] 누락된 필드값 및 엔티티 수정
3 participants