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

refactor: 선물을 열 수 있는지 판단하는 로직 리팩토링 #130

Merged
merged 3 commits into from
Feb 9, 2024

Conversation

leeeeeyeon
Copy link
Collaborator

@leeeeeyeon leeeeeyeon commented Feb 9, 2024

🛰️ Issue Number

X

🪐 작업 내용

  • 기존에 선물박스를 열 수 있는지 판단하는 로직은 if문과 논리연산자가 중첩되어 있어 가독성이 좋지 않았습니다. 해당 로직을 canOpenGiftBox 메서드로 분리하였고, if문마다 조건을 하나씩만 작성하도록 하여 가독성을 높였습니다.
  • getSentGiftBoxesBeforeLastGiftBoxDate 테스트가 로컬에서는 통과함에도 불구하고 CI에서는 처음에 실패하고 re-run을 몇 번 하면 통과하는 이상한 현상이 발생하고 있습니다. 무의미한 CI 실패를 줄이고, 테스트 코드를 수정하기 위해 임시로 해당 테스트를 제거합니다.

📚 Reference

X

✅ Check List

  • DB 스키마가 일치하는지 확인했나요?
  • SonarLint를 반영하여 코드를 수정했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@leeeeeyeon leeeeeyeon added the 🔨 fix Modifying previously implemented features label Feb 9, 2024
@leeeeeyeon leeeeeyeon self-assigned this Feb 9, 2024
@leeeeeyeon leeeeeyeon added 🧼 refactor Refactoring for maintenance and removed 🔨 fix Modifying previously implemented features labels Feb 9, 2024
Copy link

sonarcloud bot commented Feb 9, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@leeeeeyeon leeeeeyeon merged commit e3c5608 into develop Feb 9, 2024
2 checks passed
@leeeeeyeon leeeeeyeon deleted the feature/fix-receiver-logic branch February 9, 2024 06:06
leeeeeyeon added a commit that referenced this pull request Feb 9, 2024
* fix: 스티커 조회가 sequence 순으로 페이징되도록 수정

* refactor: 선물을 열 수 있는지 판단하는 로직 리팩토링 (#130)

* refactor: 선물을 열 수 있는지 판단하는 로직 리팩토링

* remove: 사용하지 않는 변수 제거

* test: 받은 선물박스 페이징 쿼리 테스트 코드 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧼 refactor Refactoring for maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant