Develop backend 작업 사항 반영

.github/workflows/cicd-backend-dev.yml

@@ -48,7 +48,7 @@ jobs:
 
   deploy:
     needs: build
-    runs-on: self-hosted
+    runs-on: [self-hosted, develop]
 
     steps:
       - name: change permission

backend/src/main/java/mouda/backend/chamyo/repository/ChamyoRepository.java

@@ -20,7 +20,7 @@ public interface ChamyoRepository extends JpaRepository<Chamyo, Long> {
 
 	boolean existsByMoimIdAndDarakbangMemberId(Long moimId, Long darakbangMemberId);
 
-	List<Chamyo> findAllByDarakbangMemberId(Long darakbangMemberId);
+	List<Chamyo> findAllByDarakbangMemberIdOrderByIdDesc(Long darakbangMemberId);
 
 	void deleteAllByMoimId(Long moimId);
 

backend/src/main/java/mouda/backend/comment/repository/CommentRepository.java

@@ -10,7 +10,7 @@
 
 public interface CommentRepository extends JpaRepository<Comment, Long> {
 
-	@Query("SELECT c.darakbangMember.memberId FROM Comment c WHERE c.parentId = :parentId")
+	@Query("SELECT c.darakbangMember.memberId FROM Comment c WHERE c.id = :parentId")
 	Long findMemberIdByParentId(@Param("parentId") long parentId);
 
 	List<Comment> findAllByMoimIdOrderByCreatedAt(long id);

backend/src/main/java/mouda/backend/moim/repository/MoimRepository.java

@@ -20,7 +20,10 @@ public interface MoimRepository extends JpaRepository<Moim, Long> {
 	@Modifying
 	int updateMoimStatusById(@Param("id") Long moimId, @Param("status") MoimStatus status);
 
-	List<Moim> findAllByDarakbangId(Long darakbangId);
-
-	boolean existsByIdAndDarakbangId(Long moimId, Long darakbangId);
+	@Query("""
+			SELECT m From Moim m
+			WHERE m.darakbangId = :darakbangId AND m.moimStatus = 'MOIMING'
+			ORDER BY m.id DESC
+		""")
+	List<Moim> findAllByDarakbangIdOrderByIdDesc(@Param("darakbangId") Long darakbangId);
 }

backend/src/main/java/mouda/backend/moim/service/MoimService.java

@@ -4,6 +4,7 @@
 
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.stream.Stream;
 
 import org.springframework.beans.factory.annotation.Value;
@@ -22,6 +23,7 @@
 import mouda.backend.comment.exception.CommentException;
 import mouda.backend.comment.repository.CommentRepository;
 import mouda.backend.darakbangmember.domain.DarakbangMember;
+import mouda.backend.darakbangmember.repository.repository.DarakbangMemberRepository;
 import mouda.backend.moim.domain.FilterType;
 import mouda.backend.moim.domain.Moim;
 import mouda.backend.moim.domain.MoimStatus;
@@ -44,6 +46,7 @@
 @RequiredArgsConstructor
 public class MoimService {
 
+	private final DarakbangMemberRepository darakbangMemberRepository;
 	@Value("${url.base}")
 	private String baseUrl;
 
@@ -72,13 +75,17 @@ public Moim createMoim(Long darakbangId, DarakbangMember darakbangMember, MoimCr
 			.targetUrl(baseUrl + String.format(moimUrl, darakbangId, moim.getId()))
 			.build();
 
-		notificationService.notifyToAllExceptMember(notification, darakbangMember.getMemberId(), darakbangId);
+		List<Long> darakbangMembersExceptMe = darakbangMemberRepository.findAllByDarakbangId(darakbangId).stream()
+			.filter(member -> !Objects.equals(member.getId(), darakbangMember.getId()))
+			.map(DarakbangMember::getMemberId)
+			.toList();
+		notificationService.notifyToMembers(notification, darakbangMembersExceptMe, darakbangId);
 		return moim;
 	}
 
 	@Transactional(readOnly = true)
 	public MoimFindAllResponses findAllMoim(Long darakbangId, DarakbangMember darakbangMember) {
-		List<Moim> moims = moimRepository.findAllByDarakbangId(darakbangId);
+		List<Moim> moims = moimRepository.findAllByDarakbangIdOrderByIdDesc(darakbangId);
 		return new MoimFindAllResponses(
 			moims.stream()
 				.map(moim -> {
@@ -301,7 +308,8 @@ private void validateCanEditMoim(Moim moim, DarakbangMember darakbangMember) {
 	}
 
 	public MoimFindAllResponses findAllMyMoim(DarakbangMember darakbangMember, FilterType filter) {
-		Stream<Chamyo> chamyoStream = chamyoRepository.findAllByDarakbangMemberId(darakbangMember.getId()).stream();
+		Stream<Chamyo> chamyoStream = chamyoRepository.findAllByDarakbangMemberIdOrderByIdDesc(darakbangMember.getId())
+			.stream();
 
 		if (filter == FilterType.PAST) {
 			chamyoStream = chamyoStream.filter(chamyo -> chamyo.getMoim().isPastMoim());
@@ -324,7 +332,7 @@ public MoimFindAllResponses findAllMyMoim(DarakbangMember darakbangMember, Filte
 	}
 
 	public MoimFindAllResponses findZzimedMoim(DarakbangMember darakbangMember) {
-		List<Zzim> zzims = zzimRepository.findAllByDarakbangMemberId(darakbangMember.getId());
+		List<Zzim> zzims = zzimRepository.findAllByDarakbangMemberIdOrderByIdDesc(darakbangMember.getId());
 
 		List<MoimFindAllResponse> responses = zzims.stream()
 			.map(zzim -> {

backend/src/main/java/mouda/backend/notification/repository/MemberNotificationRepository.java

@@ -8,5 +8,5 @@
 
 public interface MemberNotificationRepository extends JpaRepository<MemberNotification, Long> {
 
-	List<MemberNotification> findAllByMemberIdAndDarakbangId(Long memberId, Long darakbangId);
+	List<MemberNotification> findAllByMemberIdAndDarakbangIdOrderByIdDesc(Long memberId, Long darakbangId);
 }

backend/src/main/java/mouda/backend/notification/service/NotificationService.java

@@ -156,7 +156,7 @@ private List<List<String>> chunkFcmTokensForMulticast(List<String> tokens) {
 	}
 
 	public NotificationFindAllResponses findAllMyNotifications(Member member, Long darakbangId) {
-		List<NotificationFindAllResponse> responses = memberNotificationRepository.findAllByMemberIdAndDarakbangId(
+		List<NotificationFindAllResponse> responses = memberNotificationRepository.findAllByMemberIdAndDarakbangIdOrderByIdDesc(
 				member.getId(), darakbangId)
 			.stream()
 			.map(MemberNotification::getMoudaNotification)

backend/src/main/java/mouda/backend/please/repository/PleaseRepository.java

@@ -8,7 +8,5 @@
 
 public interface PleaseRepository extends JpaRepository<Please, Long> {
 
-	List<Please> findAllByDarakbangId(Long darakbangId);
-
-	boolean existsByIdAndDarakbangId(Long pleaseId, Long darakbangId);
+	List<Please> findAllByDarakbangIdOrderByIdDesc(Long darakbangId);
 }

backend/src/main/java/mouda/backend/please/service/PleaseService.java

@@ -46,7 +46,7 @@ public void deletePlease(Long darakbangId, Long pleaseId, DarakbangMember darakb
 	}
 
 	public PleaseFindAllResponses findAllPlease(Long darakbangId, DarakbangMember darakbangMember) {
-		List<Please> pleases = pleaseRepository.findAllByDarakbangId(darakbangId);
+		List<Please> pleases = pleaseRepository.findAllByDarakbangIdOrderByIdDesc(darakbangId);
 
 		return new PleaseFindAllResponses(
 			pleases.stream()

backend/src/main/java/mouda/backend/zzim/repository/ZzimRepository.java

@@ -15,5 +15,5 @@ public interface ZzimRepository extends JpaRepository<Zzim, Long> {
 
 	void deleteAllByMoimId(Long moimId);
 
-	List<Zzim> findAllByDarakbangMemberId(Long darakbangMemberId);
+	List<Zzim> findAllByDarakbangMemberIdOrderByIdDesc(Long darakbangMemberId);
 }

backend/src/test/java/mouda/backend/notification/service/NotificationServiceTest.java

@@ -96,9 +96,9 @@ void findAllMyNotifications() {
 		assertThat(responses).satisfies(res -> {
 			assertThat(res).hasSize(2);
 			assertThat(res).extracting(NotificationFindAllResponse::message)
-				.containsExactly(type1.createMessage("테스트모임"), type2.createMessage("상돌"));
+				.containsExactly(type2.createMessage("상돌"), type1.createMessage("테스트모임"));
 			assertThat(res).extracting(NotificationFindAllResponse::type)
-				.containsExactly(type1.toString(), type2.toString());
+				.containsExactly(type2.toString(), type1.toString());
 		});
 	}
 }

backend/src/test/java/mouda/backend/please/service/PleaseServiceTest.java

@@ -126,8 +126,7 @@ void findAllPlease() {
 
 			PleaseFindAllResponses pleaseFindAllResponses = pleaseService.findAllPlease(
 				darakbang.getId(), darakbangHogee);
-			assertThat(pleaseFindAllResponses.pleases().get(0).isInterested()).isTrue();
-			assertThat(pleaseFindAllResponses.pleases().get(1).isInterested()).isFalse();
+			assertThat(pleaseFindAllResponses.pleases()).extracting("isInterested").containsExactly(false, true);
 		}
 	}
 }