test: 테스트 리팩터링

src/main/java/in/koreatech/koin/domain/activity/model/Activity.java

@@ -42,9 +42,13 @@ public class Activity extends BaseEntity {
     private boolean isDeleted = false;
 
     @Builder
-    private Activity(Integer id, String title, String description, String imageUrls, LocalDate date,
-                     boolean isDeleted) {
-        this.id = id;
+    private Activity(
+        String title,
+        String description,
+        String imageUrls,
+        LocalDate date,
+        boolean isDeleted
+    ) {
         this.title = title;
         this.description = description;
         this.imageUrls = imageUrls;

src/main/java/in/koreatech/koin/domain/admin/land/controller/AdminLandApi.java

@@ -5,7 +5,6 @@
 import org.springframework.web.bind.annotation.RequestParam;
 
 import in.koreatech.koin.domain.admin.land.dto.AdminLandsResponse;
-import in.koreatech.koin.domain.land.dto.LandsGroupResponse;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.media.Content;
 import io.swagger.v3.oas.annotations.media.Schema;

src/main/java/in/koreatech/koin/domain/admin/land/controller/AdminLandController.java

@@ -2,13 +2,11 @@
 
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.GetMapping;
-import org.springframework.web.bind.annotation.ModelAttribute;
 import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.RestController;
 
 import in.koreatech.koin.domain.admin.land.dto.AdminLandsResponse;
 import in.koreatech.koin.domain.admin.land.service.AdminLandService;
-import in.koreatech.koin.domain.land.dto.LandsGroupResponse;
 import lombok.RequiredArgsConstructor;
 
 @RestController

src/main/java/in/koreatech/koin/domain/admin/land/dto/AdminLandResponse.java

@@ -1,9 +1,8 @@
 package in.koreatech.koin.domain.admin.land.dto;
 
-import static com.fasterxml.jackson.databind.PropertyNamingStrategies.*;
+import static com.fasterxml.jackson.databind.PropertyNamingStrategies.SnakeCaseStrategy;
 import static io.swagger.v3.oas.annotations.media.Schema.RequiredMode.REQUIRED;
 
-import com.fasterxml.jackson.databind.PropertyNamingStrategies;
 import com.fasterxml.jackson.databind.annotation.JsonNaming;
 
 import in.koreatech.koin.domain.land.model.Land;
@@ -26,7 +25,7 @@ public record AdminLandResponse(
     @Schema(description = "전세", example = "3500")
     String charterFee,
 
-    @Schema(description = "삭제(soft delete) 여부", example = "false", required = true)
+    @Schema(description = "삭제(soft delete) 여부", example = "false", requiredMode = REQUIRED)
     Boolean isDeleted
 ) {
     public static AdminLandResponse from(Land land) {

src/main/java/in/koreatech/koin/domain/admin/land/dto/AdminLandsResponse.java

@@ -1,14 +1,12 @@
 package in.koreatech.koin.domain.admin.land.dto;
 
-import static com.fasterxml.jackson.databind.PropertyNamingStrategies.*;
+import static com.fasterxml.jackson.databind.PropertyNamingStrategies.SnakeCaseStrategy;
 import static io.swagger.v3.oas.annotations.media.Schema.RequiredMode.REQUIRED;
 
 import java.util.List;
-import java.util.stream.Collectors;
 
 import org.springframework.data.domain.Page;
 
-import com.fasterxml.jackson.databind.PropertyNamingStrategies;
 import com.fasterxml.jackson.databind.annotation.JsonNaming;
 
 import in.koreatech.koin.domain.land.model.Land;
@@ -42,7 +40,7 @@ public static AdminLandsResponse of(Page<Land> pagedResult, Criteria criteria) {
             pagedResult.getContent()
                 .stream()
                 .map(AdminLandResponse::from)
-                .collect(Collectors.toList())
+                .toList()
         );
     }
 }

src/main/java/in/koreatech/koin/domain/bus/dto/BusTimetableResponse.java

@@ -6,6 +6,7 @@
 import java.time.LocalDateTime;
 import java.util.List;
 
+import com.fasterxml.jackson.annotation.JsonFormat;
 import com.fasterxml.jackson.databind.annotation.JsonNaming;
 
 import in.koreatech.koin.domain.bus.model.BusTimetable;
@@ -25,6 +26,8 @@ public record BusTimetableResponse(
     List<? extends BusTimetable> busTimetable,
 
     @Schema(description = "업데이트 시각", example = "2024-04-20 18:00:00", requiredMode = NOT_REQUIRED)
+    @JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss")
     LocalDateTime updatedAt
 ) {
+
 }

src/main/java/in/koreatech/koin/domain/bus/repository/ExpressBusCacheRepository.java

@@ -20,5 +20,5 @@ default ExpressBusCache getById(String busRoute) {
 
     List<ExpressBusCache> findAll();
 
-    Boolean existsById(String id);
+    boolean existsById(String id);
 }

src/main/java/in/koreatech/koin/domain/bus/service/BusService.java

@@ -183,14 +183,14 @@ public List<? extends BusTimetable> getBusTimetable(BusType busType, String dire
     }
 
     public BusTimetableResponse getBusTimetableWithUpdatedAt(BusType busType, String direction, String region){
-        List<? extends BusTimetable> BusTimetables = getBusTimetable(busType, direction, region);
+        List<? extends BusTimetable> busTimetables = getBusTimetable(busType, direction, region);
 
         if (busType.equals(BusType.COMMUTING)){
             busType = BusType.SHUTTLE;
         }
 
         VersionResponse version = versionService.getVersion(busType.name().toLowerCase() + "_bus_timetable");
-        return new BusTimetableResponse(BusTimetables, version.updatedAt());
+        return new BusTimetableResponse(busTimetables, version.updatedAt());
     }
 
     public List<BusCourseResponse> getBusCourses() {

src/main/java/in/koreatech/koin/domain/bus/util/CityBusOpenApiClient.java

@@ -76,7 +76,7 @@ public CityBusOpenApiClient(
 
     public List<CityBusRemainTime> getBusRemainTime(String nodeId) {
         Version version = versionRepository.getByType(VersionType.CITY);
-        if (isCacheExpired(version, clock)) {
+        if (isCacheExpired(version, clock) || cityBusCacheRepository.findById(nodeId).isEmpty()) {
             storeRemainTimeByOpenApi();
         }
         return getCityBusArrivalInfoByCache(nodeId);

src/main/java/in/koreatech/koin/domain/bus/util/ExpressBusOpenApiClient.java

@@ -104,7 +104,7 @@ public List<ExpressBusRemainTime> getBusRemainTime(BusStation depart, BusStation
         if (!expressBusCacheRepository.existsById(busCacheId)) {
             storeRemainTimeByOpenApi(depart.name().toLowerCase(), arrival.name().toLowerCase());
         }
-        return getStoredRemainTime(depart.name().toLowerCase(), arrival.name().toLowerCase());
+        return getStoredRemainTime(busCacheId);
     }
 
     private void storeRemainTimeByOpenApi(String departName, String arrivalName) {
@@ -139,7 +139,7 @@ private void storeRemainTimeByOpenApi(String departName, String arrivalName) {
     private JsonObject getBusApiResponse(String departName, String arrivalName) {
         try {
             URL url = getBusApiURL(departName, arrivalName);
-            HttpURLConnection conn = (HttpURLConnection)url.openConnection();
+            HttpURLConnection conn = (HttpURLConnection) url.openConnection();
             conn.setRequestMethod("GET");
             conn.setRequestProperty("Content-type", "application/json");
             BufferedReader reader;
@@ -202,8 +202,7 @@ private List<OpenApiExpressBusArrival> extractBusArrivalInfo(JsonObject jsonObje
         }
     }
 
-    private List<ExpressBusRemainTime> getStoredRemainTime(String departName, String arrivalName) {
-        String busCacheId = ExpressBusCache.generateId(new ExpressBusRoute(departName, arrivalName));
+    private List<ExpressBusRemainTime> getStoredRemainTime(String busCacheId) {
         ExpressBusCache expressBusCache = expressBusCacheRepository.getById(busCacheId);
         if (Objects.isNull(expressBusCache)) {
             return Collections.emptyList();

src/main/java/in/koreatech/koin/domain/community/model/Board.java

@@ -69,8 +69,16 @@ public List<Board> getChildren() {
     }
 
     @Builder
-    private Board(String tag, String name, boolean isAnonymous, Integer articleCount, boolean isDeleted,
-                  boolean isNotice, Integer parentId, Integer seq) {
+    private Board(
+        String tag,
+        String name,
+        boolean isAnonymous,
+        Integer articleCount,
+        boolean isDeleted,
+        boolean isNotice,
+        Integer parentId,
+        Integer seq
+    ) {
         this.tag = tag;
         this.name = name;
         this.isAnonymous = isAnonymous;

src/main/java/in/koreatech/koin/domain/community/service/CommunityService.java

@@ -18,11 +18,11 @@
 import in.koreatech.koin.domain.community.model.ArticleViewLog;
 import in.koreatech.koin.domain.community.model.Board;
 import in.koreatech.koin.domain.community.model.BoardTag;
-import in.koreatech.koin.global.model.Criteria;
 import in.koreatech.koin.domain.community.repository.ArticleRepository;
 import in.koreatech.koin.domain.community.repository.ArticleViewLogRepository;
 import in.koreatech.koin.domain.community.repository.BoardRepository;
 import in.koreatech.koin.domain.user.repository.UserRepository;
+import in.koreatech.koin.global.model.Criteria;
 import lombok.RequiredArgsConstructor;
 
 @Service

src/main/java/in/koreatech/koin/domain/coop/dto/SoldOutRequest.java

@@ -1,14 +1,14 @@
 package in.koreatech.koin.domain.coop.dto;
 
+import static com.fasterxml.jackson.databind.PropertyNamingStrategies.SnakeCaseStrategy;
 import static io.swagger.v3.oas.annotations.media.Schema.RequiredMode.REQUIRED;
 
-import com.fasterxml.jackson.databind.PropertyNamingStrategies;
 import com.fasterxml.jackson.databind.annotation.JsonNaming;
 
 import io.swagger.v3.oas.annotations.media.Schema;
 import jakarta.validation.constraints.NotNull;
 
-@JsonNaming(value = PropertyNamingStrategies.SnakeCaseStrategy.class)
+@JsonNaming(value = SnakeCaseStrategy.class)
 public record SoldOutRequest(
     @Schema(description = "메뉴 고유 ID", example = "1", requiredMode = REQUIRED)
     @NotNull(message = "메뉴 ID는 필수입니다.")

src/main/java/in/koreatech/koin/domain/member/model/TechStack.java

@@ -45,11 +45,15 @@ public class TechStack extends BaseEntity {
     private boolean isDeleted = false;
 
     @Builder
-    private TechStack(String imageUrl, String name, String description, Integer trackId, boolean isDeleted) {
+    private TechStack(
+        String imageUrl,
+        String name,
+        String description,
+        Integer trackId
+    ) {
         this.imageUrl = imageUrl;
         this.name = name;
         this.description = description;
         this.trackId = trackId;
-        this.isDeleted = isDeleted;
     }
 }

src/main/java/in/koreatech/koin/domain/ownershop/service/OwnerShopService.java

@@ -2,7 +2,6 @@
 
 import java.time.Clock;
 import java.time.LocalDate;
-import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 
@@ -273,7 +272,6 @@ public void createEvent(Integer ownerId, Integer shopId, CreateEventRequest shop
         Shop shop = getOwnerShopById(shopId, ownerId);
         EventArticle eventArticle = EventArticle.builder()
             .shop(shop)
-            .thumbnailImages(new ArrayList<>())
             .startDate(shopEventRequest.startDate())
             .endDate(shopEventRequest.endDate())
             .title(shopEventRequest.title())

src/main/java/in/koreatech/koin/domain/shop/dto/MenuDetailResponse.java

@@ -89,7 +89,8 @@ public static MenuDetailResponse createForMultipleOption(Menu menu, List<MenuCat
             menu.isHidden(),
             false,
             null,
-            menu.getMenuOptions().stream().map(InnerOptionPriceResponse::of).toList(), menu.getDescription(),
+            menu.getMenuOptions().stream().map(InnerOptionPriceResponse::of).toList(),
+            menu.getDescription(),
             shopMenuCategories.stream().map(MenuCategory::getId).toList(),
             menu.getMenuImages().stream().map(MenuImage::getImageUrl).toList()
         );

src/main/java/in/koreatech/koin/domain/shop/model/EventArticle.java

@@ -1,7 +1,6 @@
 package in.koreatech.koin.domain.shop.model;
 
-import static jakarta.persistence.CascadeType.MERGE;
-import static jakarta.persistence.CascadeType.PERSIST;
+import static jakarta.persistence.CascadeType.ALL;
 import static jakarta.persistence.FetchType.LAZY;
 import static jakarta.persistence.GenerationType.IDENTITY;
 import static lombok.AccessLevel.PROTECTED;
@@ -51,7 +50,7 @@ public class EventArticle extends BaseEntity {
     @Column(name = "title", nullable = false)
     private String title;
 
-    @OneToMany(mappedBy = "eventArticle", cascade = {MERGE, PERSIST})
+    @OneToMany(mappedBy = "eventArticle", orphanRemoval = true, cascade = ALL)
     private List<EventArticleImage> thumbnailImages = new ArrayList<>();
 
     /**
@@ -123,7 +122,6 @@ private EventArticle(
         String title,
         String content,
         User user,
-        List<EventArticleImage> thumbnailImages,
         Integer hit,
         String ip,
         LocalDate startDate,
@@ -133,7 +131,6 @@ private EventArticle(
         this.title = title;
         this.content = content;
         this.user = user;
-        this.thumbnailImages = thumbnailImages;
         this.hit = hit;
         this.ip = ip;
         this.startDate = startDate;
@@ -152,7 +149,6 @@ public void modifyArticle(
         this.content = content;
         this.startDate = startDate;
         this.endDate = endDate;
-        this.thumbnailImages.forEach(entityManager::remove);
         this.thumbnailImages.clear();
         entityManager.flush();
         for (String imageUrl : thumbnailImages) {

src/main/java/in/koreatech/koin/domain/shop/model/EventArticleImage.java

@@ -1,12 +1,12 @@
 package in.koreatech.koin.domain.shop.model;
 
-import static jakarta.persistence.FetchType.LAZY;
 import static jakarta.persistence.GenerationType.IDENTITY;
 import static lombok.AccessLevel.PROTECTED;
 
 import in.koreatech.koin.global.domain.BaseEntity;
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
+import jakarta.persistence.FetchType;
 import jakarta.persistence.GeneratedValue;
 import jakarta.persistence.Id;
 import jakarta.persistence.JoinColumn;
@@ -28,7 +28,7 @@ public class EventArticleImage extends BaseEntity {
     @GeneratedValue(strategy = IDENTITY)
     private Integer id;
 
-    @ManyToOne(fetch = LAZY)
+    @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "event_id", nullable = false)
     private EventArticle eventArticle;
 
@@ -38,7 +38,10 @@ public class EventArticleImage extends BaseEntity {
     private String thumbnailImage;
 
     @Builder
-    private EventArticleImage(EventArticle eventArticle, String thumbnailImage) {
+    private EventArticleImage(
+        EventArticle eventArticle,
+        String thumbnailImage
+    ) {
         this.eventArticle = eventArticle;
         this.thumbnailImage = thumbnailImage;
     }

src/main/java/in/koreatech/koin/domain/shop/model/Menu.java

@@ -62,7 +62,11 @@ public class Menu extends BaseEntity {
     private List<MenuImage> menuImages = new ArrayList<>();
 
     @Builder
-    private Menu(Integer shopId, String name, String description) {
+    private Menu(
+        Integer shopId,
+        String name,
+        String description
+    ) {
         this.shopId = shopId;
         this.name = name;
         this.description = description;

src/main/java/in/koreatech/koin/domain/shop/model/Shop.java

@@ -132,11 +132,7 @@ private Shop(
         boolean isDeleted,
         boolean isEvent,
         String remarks,
-        Integer hit,
-        List<ShopCategoryMap> shopCategories,
-        List<ShopOpen> shopOpens,
-        List<ShopImage> shopImages,
-        List<MenuCategory> menuCategories
+        Integer hit
     ) {
         this.owner = owner;
         this.name = name;
@@ -153,10 +149,6 @@ private Shop(
         this.isEvent = isEvent;
         this.remarks = remarks;
         this.hit = hit;
-        this.shopCategories = shopCategories;
-        this.shopOpens = shopOpens;
-        this.shopImages = shopImages;
-        this.menuCategories = menuCategories;
     }
 
     public void modifyShop(

src/main/java/in/koreatech/koin/domain/user/model/Student.java

@@ -48,8 +48,14 @@ public class Student {
     private User user;
 
     @Builder
-    private Student(String anonymousNickname, String studentNumber, String department,
-                    UserIdentity userIdentity, boolean isGraduated, User user) {
+    private Student(
+        String anonymousNickname,
+        String studentNumber,
+        String department,
+        UserIdentity userIdentity,
+        boolean isGraduated,
+        User user
+    ) {
         this.anonymousNickname = anonymousNickname;
         this.studentNumber = studentNumber;
         this.department = department;