Skip to content

Commit

Permalink
[TEST] 두개 이상의 메서드인 경우에는 assertAll 을 사용하도록 수정 #236
Browse files Browse the repository at this point in the history
  • Loading branch information
Jin409 committed Sep 25, 2024
1 parent ec306fa commit 0f3f143
Showing 1 changed file with 37 additions and 18 deletions.
55 changes: 37 additions & 18 deletions src/test/java/com/sports/server/command/game/domain/GameTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import static com.sports.server.support.fixture.FixtureMonkeyUtils.entityBuilder;
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatThrownBy;
import static org.junit.jupiter.api.Assertions.assertAll;

import com.sports.server.common.exception.CustomException;
import java.util.ArrayList;
Expand Down Expand Up @@ -43,8 +44,10 @@ class ScoreTest {
game.score(scorer);

// then
assertThat(game.getTeam1().getScore()).isEqualTo(1);
assertThat(game.getTeam2().getScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getScore()).isEqualTo(1),
() -> assertThat(game.getTeam2().getScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -58,8 +61,10 @@ class ScoreTest {
game.score(scorer);

// then
assertThat(game.getTeam1().getScore()).isEqualTo(0);
assertThat(game.getTeam2().getScore()).isEqualTo(1);
assertAll(
() -> assertThat(game.getTeam1().getScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getScore()).isEqualTo(1)
);
}

@Test
Expand Down Expand Up @@ -88,8 +93,10 @@ class ScoreTest {
game.scoreInPk(scorer);

// then
assertThat(game.getTeam1().getPkScore()).isEqualTo(1);
assertThat(game.getTeam2().getPkScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getPkScore()).isEqualTo(1),
() -> assertThat(game.getTeam2().getPkScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -103,8 +110,10 @@ class ScoreTest {
game.scoreInPk(scorer);

// then
assertThat(game.getTeam1().getPkScore()).isEqualTo(0);
assertThat(game.getTeam2().getPkScore()).isEqualTo(1);
assertAll(
() -> assertThat(game.getTeam1().getPkScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getPkScore()).isEqualTo(1)
);
}
}

Expand Down Expand Up @@ -134,8 +143,10 @@ void setUp() {
game.cancelScore(team1Player);

// then
assertThat(game.getTeam1().getScore()).isEqualTo(0);
assertThat(game.getTeam2().getScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -147,8 +158,10 @@ void setUp() {
game.cancelPkScore(team1Player);

// then
assertThat(game.getTeam1().getPkScore()).isEqualTo(0);
assertThat(game.getTeam2().getPkScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getPkScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getPkScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -160,8 +173,10 @@ void setUp() {
game.cancelPkScore(team2Player);

// then
assertThat(game.getTeam1().getPkScore()).isEqualTo(0);
assertThat(game.getTeam2().getPkScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getPkScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getPkScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -173,8 +188,10 @@ void setUp() {
game.cancelScore(team2Player);

// then
assertThat(game.getTeam1().getScore()).isEqualTo(0);
assertThat(game.getTeam2().getScore()).isEqualTo(0);
assertAll(
() -> assertThat(game.getTeam1().getScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getScore()).isEqualTo(0)
);
}

@Test
Expand All @@ -187,8 +204,10 @@ void setUp() {
game.cancelScore(team1Player);

// then
assertThat(game.getTeam1().getScore()).isEqualTo(0);
assertThat(game.getTeam2().getScore()).isEqualTo(1);
assertAll(
() -> assertThat(game.getTeam1().getScore()).isEqualTo(0),
() -> assertThat(game.getTeam2().getScore()).isEqualTo(1)
);
}

@Test
Expand Down

0 comments on commit 0f3f143

Please sign in to comment.