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

Feature: 기업 삭제 API 구현 #19

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Feature: 기업 삭제 API 구현 #19

wants to merge 3 commits into from

Conversation

ujamX
Copy link

@ujamX ujamX commented Nov 2, 2023


PR Type ✔️

  • Feature - 기능 개발
  • Fix - 버그 수정
  • Code Style - 코드 스타일
  • Refactoring - 리팩토링
  • Document - 문서
  • Others - 빌드, 배포 및 의존성 등 기타사항

Summary(요약) ⭐

기업 삭제 API 구현

Describe your changes(상세) 🧾

  • 로직 테스트에서 기업 인덱스로 기업 삭제 시 deleteById() 한 번 수행되는지 검증
  • 핸들러 테스트에서 기업 삭제 후 삭제된 기업 재 조회하여 해당 기업이 삭제되었는지 검증
  • Repository 테스트에서 삭제된 아이디로 기업을 조회했을 때 기업 객체가 비어있는 것을 검증

Issue Number or Link 🔗


Reference


ujamX added 3 commits November 2, 2023 21:29
CompanyService.java
CompanyServiceTest.java
 - 특정 기업 삭제 로직 구현
 - 기업 인덱스로 기업 삭제 시 deleteById() 한 번 수행되는 테스트 로직 구현
CompanyController.java
CompanyControllerTest.java
 - 기업 삭제 핸들러 구현
 - 삭제 후 삭제된 기업 재 조회하여 검증하는 테스트 구현
CompanyRepositoryTest.java
 - 삭제된 아이디로 기업을 조회했을 때 기업 객체가 비어있는 것을 검증하는 테스트 구현
@ujamX ujamX requested a review from yh0921k November 2, 2023 12:40
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