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

CLAP-439 정책 테스트 코드를 AAA 패턴으로 개선 #577

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 16, 2025

#574

📄 요약(Summary)

정책 테스트 코드를 AAA 패턴으로 개선

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • TaskOrderCalculationPolicy 클래스는 작업 보드 순서 계산을 담당하는 정책 클래스로,
    입력값(Arrange) → 연산(Act) → 결과 검증(Assert)의 흐름이 명확하므로 AAA 패턴을 적용한 테스트 코드가 적합하여 해당 방식으로 코드를 개선하였습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

🧪 AAA 패턴: 객체 기반

  • 유닛테스트에서 사용되는 패턴으로, 모듈의 기능에 따른 시나리오를 작성합니다.
  • 구조적으로 GWT와 유사하지만, 테스트를 논리적인 단계로 구분하는 것이 특징이며, SUT의 의도와 목적을 최대한 표현하게끔 자연스레 유도
    합니다.

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #574

@joowojr joowojr added the 🧪 test 테스트 코드를 추가 및 변경합니다 label Feb 16, 2025
@joowojr joowojr self-assigned this Feb 16, 2025
@joowojr joowojr changed the base branch from develop to CLAP-440 February 16, 2025 02:20
@joowojr joowojr merged commit 12edc3e into CLAP-440 Feb 16, 2025
1 check passed
@joowojr joowojr deleted the CLAP-439 branch February 16, 2025 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧪 test 테스트 코드를 추가 및 변경합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLAP-439 단위 테스트 코드를 AAA 패턴으로 개선
1 participant