v2.0.6 (#418)

Author: char-yb

src/main/java/com/depromeet/domain/feed/application/FeedService.java

@@ -62,9 +62,11 @@ public FeedSliceResponse findFeed(int size, Long lastId, MissionVisibility visib
 
     @Transactional(readOnly = true)
     public Slice<FeedOneResponse> findFeedV2(FeedVisibility visibility, int size, Long lastId) {
+        // 전체
         if (visibility == FeedVisibility.ALL) {
             return findAllFeedV2(size, lastId);
         }
+        // 팔로잉
         return findFollowingFeedV2(size, lastId);
     }
 

src/main/java/com/depromeet/domain/missionRecord/dao/MissionRecordRepositoryImpl.java

@@ -168,8 +168,7 @@ public void deleteByMissionRecordId(Long missionRecordId) {
     }
 
     @Override
-    public Slice<MissionRecord> findAllFetch(int size, Long lastId) {
-
+    public Slice<MissionRecord> findAllFetch(final int size, final Long lastId) {
         List<MissionRecord> missionRecords =
                 jpaQueryFactory
                         .selectFrom(missionRecord)
@@ -180,7 +179,11 @@ public Slice<MissionRecord> findAllFetch(int size, Long lastId) {
                         .leftJoin(missionRecord.reactions, reaction)
                         .fetchJoin()
                         .distinct()
-                        .where(ltMissionRecordId(lastId))
+                        .where(
+                                ltMissionRecordId(lastId) != null
+                                        ? ltMissionRecordId(lastId)
+                                                .and(checkMissionVisibilityNone())
+                                        : checkMissionVisibilityNone())
                         .orderBy(missionRecord.finishedAt.desc())
                         .limit((long) size + 1)
                         .fetch();
@@ -264,4 +267,8 @@ private boolean getHasNext(List<?> list, int size) {
         }
         return hasNext;
     }
+
+    private BooleanExpression checkMissionVisibilityNone() {
+        return mission.visibility.ne(MissionVisibility.NONE);
+    }
 }

src/test/java/com/depromeet/domain/feed/application/FeedServiceTest.java

@@ -225,7 +225,7 @@ private void setFixture() {
         Slice<FeedOneResponse> response = feedService.findFeedV2(FeedVisibility.ALL, 10, 5L);
 
         // then
-        assertThat(response.getContent()).hasSize(4);
+        assertThat(response.getContent()).hasSize(3);
     }
 
     @Nested