refactor: systemCheck

playkuround · Sep 4, 2024 · c649ce2 · c649ce2
1 parent d4e7195
commit c649ce2
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 19 deletions.
diff --git a/.../com/playkuround/playkuroundserver/domain/systemcheck/application/SystemCheckService.java b/.../com/playkuround/playkuroundserver/domain/systemcheck/application/SystemCheckService.java
@@ -20,6 +20,7 @@ public class SystemCheckService {
     private final AppVersionRepository appVersionRepository;
     private final SystemCheckRepository systemCheckRepository;
 
+    @Transactional(readOnly = true)
     public boolean isSystemAvailable() {
         Optional<SystemCheck> optionalSystemCheck = systemCheckRepository.findTopByOrderByUpdatedAtDesc();
         return optionalSystemCheck.map(SystemCheck::isAvailable)
@@ -33,10 +34,11 @@ public void changeSystemAvailable(boolean available) {
         systemCheckRepository.save(systemCheck);
     }
 
+    @Transactional(readOnly = true)
     public HealthCheckDto healthCheck() {
         boolean systemAvailable = isSystemAvailable();
-        List<AppVersion> supportAppVersionList = appVersionRepository.findAll();
 
+        List<AppVersion> supportAppVersionList = appVersionRepository.findAll();
         List<OSAndVersion> list = supportAppVersionList.stream()
                 .map(appVersion -> new OSAndVersion(appVersion.getOs(), appVersion.getVersion()))
                 .distinct()

diff --git a/src/main/java/com/playkuround/playkuroundserver/domain/systemcheck/domain/SystemCheck.java b/src/main/java/com/playkuround/playkuroundserver/domain/systemcheck/domain/SystemCheck.java
@@ -24,7 +24,4 @@ public SystemCheck(boolean available) {
         this.available = available;
     }
 
-    public void updateAvailable(boolean available) {
-        this.available = available;
-    }
 }
diff --git a/src/main/java/com/playkuround/playkuroundserver/domain/systemcheck/dto/HealthCheckDto.java b/src/main/java/com/playkuround/playkuroundserver/domain/systemcheck/dto/HealthCheckDto.java
@@ -4,6 +4,5 @@
 
 import java.util.List;
 
-public record HealthCheckDto(boolean systemAvailable,
-                             List<OSAndVersion> supportAppVersionList) {
+public record HealthCheckDto(boolean systemAvailable, List<OSAndVersion> supportAppVersionList) {
 }
diff --git a/.../playkuround/playkuroundserver/domain/systemcheck/application/SystemCheckServiceTest.java b/.../playkuround/playkuroundserver/domain/systemcheck/application/SystemCheckServiceTest.java
@@ -100,26 +100,26 @@ void success_1() {
             // given
             when(systemCheckRepository.findTopByOrderByUpdatedAtDesc())
                     .thenReturn(Optional.of(new SystemCheck(true)));
+
+            List<AppVersion> supportedAppVersion = List.of(
+                    new AppVersion(OperationSystem.ANDROID, "1.0.0"),
+                    new AppVersion(OperationSystem.ANDROID, "1.0.1"),
+                    new AppVersion(OperationSystem.IOS, "1.0.3")
+            );
             when(appVersionRepository.findAll())
-                    .thenReturn(
-                            List.of(
-                                    new AppVersion(OperationSystem.ANDROID, "1.0.0"),
-                                    new AppVersion(OperationSystem.ANDROID, "1.0.1"),
-                                    new AppVersion(OperationSystem.IOS, "1.0.3")
-                            )
-                    );
+                    .thenReturn(supportedAppVersion);
 
             // when
             HealthCheckDto healthCheckDto = systemCheckService.healthCheck();
 
             // then
             assertThat(healthCheckDto.systemAvailable()).isTrue();
-            assertThat(healthCheckDto.supportAppVersionList())
-                    .containsExactlyInAnyOrder(
-                            new OSAndVersion(OperationSystem.ANDROID, "1.0.0"),
-                            new OSAndVersion(OperationSystem.ANDROID, "1.0.1"),
-                            new OSAndVersion(OperationSystem.IOS, "1.0.3")
-                    );
+
+            List<OSAndVersion> expected = supportedAppVersion.stream()
+                    .map(appVersion -> new OSAndVersion(appVersion.getOs(), appVersion.getVersion()))
+                    .toList();
+            assertThat(healthCheckDto.supportAppVersionList()).hasSize(3)
+                    .containsExactlyInAnyOrderElementsOf(expected);
         }
     }