refactor: fix descending on single condition

woowacourse-teams · Sep 27, 2024 · 500c257 · 500c257
1 parent 9b02264
commit 500c257
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/backend/src/main/java/net/pengcook/recipe/repository/RecipeRepository.java b/backend/src/main/java/net/pengcook/recipe/repository/RecipeRepository.java
@@ -16,6 +16,7 @@ public interface RecipeRepository extends JpaRepository<Recipe, Long> {
             SELECT DISTINCT recipe.id
             FROM CategoryRecipe
             WHERE category.name = :category
+            ORDER BY recipe.id DESC
             """)
     List<Long> findRecipeIdsByCategory(Pageable pageable, String category);
 
@@ -24,6 +25,7 @@ public interface RecipeRepository extends JpaRepository<Recipe, Long> {
             FROM Recipe
             WHERE title LIKE CONCAT('%', :keyword, '%')
             OR description LIKE CONCAT('%', :keyword, '%')
+            ORDER BY id DESC
             """)
     List<Long> findRecipeIdsByKeyword(Pageable pageable, String keyword);
 

diff --git a/backend/src/test/java/net/pengcook/recipe/repository/RecipeRepositoryTest.java b/backend/src/test/java/net/pengcook/recipe/repository/RecipeRepositoryTest.java
@@ -33,7 +33,7 @@ void findRecipeIdsByCategory() {
         List<Long> recipeIds = repository.findRecipeIdsByCategory(pageable, category);
 
         // then
-        assertThat(recipeIds).containsExactly(2L, 3L, 7L);
+        assertThat(recipeIds).contains(15L, 14L, 9L);
     }
 
     @Test
@@ -47,7 +47,7 @@ void findRecipeIdsByKeyword() {
         List<Long> recipeIds = repository.findRecipeIdsByKeyword(pageable, keyword);
 
         // then
-        assertThat(recipeIds).containsExactly(11L, 12L);
+        assertThat(recipeIds).containsExactly(12L, 11L);
     }
 
     @Test