test: 주석 및 메서드 네이밍 수정

sosow0212 · Jun 14, 2024 · f1d8169 · f1d8169
1 parent 12e4272
commit f1d8169
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 12 deletions.
diff --git a/src/test/java/com/atwoz/report/domain/vo/ReportResultTest.java b/src/test/java/com/atwoz/report/domain/vo/ReportResultTest.java
@@ -7,7 +7,6 @@
 import org.junit.jupiter.api.Test;
 
 import static com.atwoz.report.domain.vo.ReportResult.BAN;
-import static com.atwoz.report.domain.vo.ReportResult.findByName;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
@@ -19,12 +18,12 @@ class ReportResultTest {
     class ReportResult_조회 {
 
         @Test
-        void 신고_결과_정보가_유효하지_않으면_예외가_밠생한다() {
-            //  given
+        void 신고_결과_정보가_유효하지_않으면_예외가_발생한다() {
+            // given
             String invalidReportResult = "invalid report result";
 
-            // when
-            assertThatThrownBy(() -> findByName(invalidReportResult))
+            // when & then
+            assertThatThrownBy(() -> ReportResult.findByName(invalidReportResult))
                     .isInstanceOf(InvalidReportResultException.class);
         }
 
@@ -33,8 +32,8 @@ class ReportResult_조회 {
             // given
             String validReportResult = BAN.getName();
 
-            // When
-            ReportResult foundReportResult = findByName(validReportResult);
+            // when
+            ReportResult foundReportResult = ReportResult.findByName(validReportResult);
 
             // then
             assertThat(foundReportResult.getName()).isEqualTo(validReportResult);

diff --git a/src/test/java/com/atwoz/report/domain/vo/ReportTypeTest.java b/src/test/java/com/atwoz/report/domain/vo/ReportTypeTest.java
@@ -18,11 +18,11 @@ class ReportTypeTest {
     class ReportType_조회{
 
         @Test
-        void 신고_유형_정보가_유효하지_않으면_예외가_밠생한다() {
-            //  given
+        void 신고_유형_정보가_유효하지_않으면_예외가_발생한다() {
+            // given
             String invalidReportType = "invalid report type";
 
-            // when
+            // when & then
             assertThatThrownBy(() -> ReportType.findByCode(invalidReportType))
                     .isInstanceOf(InvalidReportTypeException.class);
         }
@@ -32,7 +32,7 @@ class ReportType_조회{
             // given
             String validReportType = FAKE_PROFILE.getCode();
 
-            // When
+            // when
             ReportType foundReportType = ReportType.findByCode(validReportType);
 
             // then

diff --git a/src/test/java/com/atwoz/report/infrastructure/ReportJpaRepositoryTest.java b/src/test/java/com/atwoz/report/infrastructure/ReportJpaRepositoryTest.java
@@ -45,7 +45,9 @@ class ReportJpaRepositoryTest {
         assertSoftly(softly -> {
             softly.assertThat(foundReport).isPresent();
             Report report = foundReport.get();
-            softly.assertThat(report).usingRecursiveComparison().isEqualTo(createdReport);
+            softly.assertThat(report)
+                    .usingRecursiveComparison()
+                    .isEqualTo(createdReport);
         });
     }
 }