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.

Sign up for GitHub

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

Jump to bottom

337 badge template impl, matcher test 코드 작성 #353

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

337 badge template impl, matcher test 코드 작성 #353

wants to merge 11 commits into from

Conversation

Sooamazing
Copy link
Contributor

변경 내용

  • template impl 디렉터리와 matcher test 작성
  • interface는 자동 테스트 포함으로 미작성

특이 사항

  • 사실 impl 내용은 같아, ContributeAll을 제외하고는 test 내용 검증은 같습니다. 커버리지를 위해 작성
  • All에서 map 생성 시 처음에 없었다가 생성되는 것과 그렇게 했을 때 가져오는 속도 차이 같은 것도 확인해 보고 싶었지만... 어떻게 해야 할지 몰라 두었습니다.

체크리스트

  • PR 날리기 전에 main branch pull 받으셨나요?
  • application.properties 등 노출되지 않아야 하는 파일이 올라가지는 않았나요?
  • 다른 담당자 파일을 수정한 부분에 대해서 이야기 하셨나요?
  • 주석 "상세히" 다셨나요?
  • 제출하기 전에 테스트코드 돌려 보셨나요?

closes #337

@Sooamazing
[#337] test: test 작성 상황 초안, 1개 작성
a673427 
- 각 메서드 예외 상황 검증 꼭지 작성
@Sooamazing
[#337] test: public getBadgeWithCount 포함 각 메서드 예외 및 성공 테스트
81c6e7a 
- 2번 이후 시간 확인 및 동시성 확인은 TODO
  - [확인] 2번 이후 시간 확인, map에 저장된 배지 조건 얻기 - getBadgeCriteria
  - [성공] 동시성 확인, map에 저장된 배지 조건 얻기 - getBadgeCriteria
@Sooamazing
Merge branch 'main' of github.com:Eagle-Strike-7/stelligence-backend …
76fef78 
…into 337-badge-template-matcher-test-코드-작성
@Sooamazing
[#337] test: 각 메서드 테스트, 기본 성공, 예
9741ddc 
- supports
- getCount
- getBadgeCriteria
- getBadgeWithCount
@Sooamazing
[#337] test: verify 추가
ba7ce49 
- repository 필수라서 확
@Sooamazing
[#337] test: 기본 성공 확
bbcf591
@Sooamazing
[#337] test: 기본 성공 확인
69a32ab
@Sooamazing
[#337] test: Matcher 기본 성공 확인
e780efd 
- 생성자 list, final map 설정
- Map reflection으로 설정 완료.
@Sooamazing
Merge branch 'main' of github.com:Eagle-Strike-7/stelligence-backend …
ba26c76 
…into 337-badge-template-matcher-test-코드-작성
@Sooamazing
[#337] test: 기본 성공 확인
2b5bb23 
- 커버리지 위해 테스트 코드 작성(다른 impl과 같음)
@Sooamazing Sooamazing added the test 빠진 test 추가 혹은 존재하는 테스트 수정 label Feb 20, 2024
@Sooamazing Sooamazing added this to the 배지 milestone Feb 20, 2024
@Sooamazing Sooamazing self-assigned this Feb 20, 2024
@Sooamazing Sooamazing linked an issue Feb 20, 2024 that may be closed by this pull request
badge template, matcher test 코드 작성 #337
Open
@Sooamazing
Merge branch 'main' of github.com:Eagle-Strike-7/stelligence-backend …
9a28ef4 
…into 337-badge-template-matcher-test-코드-작성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Sooamazing Sooamazing

Labels
test 빠진 test 추가 혹은 존재하는 테스트 수정
Projects
None yet
Milestone
배지
Development

Successfully merging this pull request may close these issues.

badge template, matcher test 코드 작성
1 participant
@Sooamazing