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: 최신순 정렬을 사용하는 ServiceTest, RepositoryTest에 아이디 기준 내림차순 옵션 추가 #77

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

70825
Copy link
Member

@70825 70825 commented Jun 7, 2024

Issue

✨ 구현한 기능

  • 최신순으로 정렬할 때, 아이디 기준 내림차순으로 정렬되는 조건도 고정적으로 추가하도록 했습니다.

📢 논의하고 싶은 내용

이것도 쿼리문은 소수점 6자리 기준으로 데이터가 나와서 id2, id3 순서로 나오게 됩니다. 근데 H2에서 가져온 데이터를 사용하는게 아니라 영속성 컨텍스트 1차 캐시 값을 재사용하고 있기 때문에 이상하게 결과값을 가져오는 것처럼 보이는 것입니다.

[Java - 영속성 컨텍스트 1차 캐시]
2번 데이터 = 11:00:21.498842569
3번 데이터 = 11:00:21.498843211

[H2 - 실제로 저장된 값]
2번 데이터 = 11:00:21.498843 (반올림)
3번 데이터 = 11:00:21.498843 (반올림)

[이전 쿼리문]

스크린샷 2024-06-07 오후 9 03 37

[변경된 쿼리문]

스크린샷 2024-06-07 오후 9 26 24

🎸 기타

  • 인수테스트에서 문제 없었던 이유

스크린샷 2024-06-07 오후 9 30 10

요청이 오면 기존 정렬 조건에 추가적으로 id 기준 내림차순을 적용해놨어서 🤣

스크린샷 2024-06-07 오후 9 34 49

⏰ 일정

  • 추정 시간 : 30분
  • 걸린 시간 : 35분

@70825 70825 added the 🐛 Bug label Jun 7, 2024
@70825 70825 self-assigned this Jun 7, 2024
Copy link

github-actions bot commented Jun 7, 2024

Test Results

333 tests  ±0   333 ✅ ±0   18s ⏱️ -3s
204 suites ±0     0 💤 ±0 
204 files   ±0     0 ❌ ±0 

Results for commit 3650499. ± Comparison against base commit 9ec1ed0.

♻️ This comment has been updated with latest results.

@70825 70825 requested review from Go-Jaecheol and wugawuga June 7, 2024 12:58
@@ -219,7 +220,7 @@ class findRecipeByMember_성공_테스트 {
final var recipeImage1_2 = 레시피이미지_생성(recipe1_2);
복수_꿀조합_이미지_저장(recipeImage1_1, recipeImage1_2);

final var page = 페이지요청_생성(0, 10, 최신순);
final var page = 페이지요청_생성(0, 10, 최신순, 아이디_내림차순);
Copy link
Member Author

Choose a reason for hiding this comment

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

최신순을 적용한 모든 ServiceTest, RepositoryTest아이디_내림차순을 추가했습니다.

@70825 70825 changed the title fix: 테스트 코드에 있는 최신순 정렬 Fixture 수정 fix: 최신순 정렬을 사용하는 ServiceTest, RepositoryTest에 아이디 기준 내림차순 옵션 추가 Jun 7, 2024
@70825 70825 merged commit 2a2bdd7 into develop Jun 7, 2024
3 checks passed
@70825 70825 deleted the fix/issue-76 branch June 7, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

테스트 정렬 조건 수정
1 participant