Skip to content

Commit

Permalink
refactor: test 한글 네이밍 영어로 변경
Browse files Browse the repository at this point in the history
모든 테스트의 네이밍을 변경하였음
  • Loading branch information
Livenow14 committed Feb 22, 2021
1 parent b7fa142 commit c5bcd38
Show file tree
Hide file tree
Showing 10 changed files with 62 additions and 59 deletions.
2 changes: 1 addition & 1 deletion docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
## 기능 요구사항 정리
- [x] 구현 부분 구입금액 입력 확인
- [x] 보장된 구입금액을 입력 받는다.
- 보장된 구입금액(양수의 숫자이어야 한.)
- 보장된 구입금액(로또 티켓 구입 금액보다 큰 양수의 숫자)
- [x] 보장된 구입금액이 아니면 에러가 발생한다.
- `IllegalArgumentException`
- [x] 구입금액 따른 로또의 갯수를 구한다.
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/view/LottoGameScreen.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ public class LottoGameScreen {
public static final String RESULT = "당첨통계";
public static final String LINE = "----------";
public static final String LOTTO_RESULT_MESSAGE = "%s개 일치 (%d원)- %d개";
public static final String SECOND_PRIZE_RESULT_MESSAGE = "%d개 일치, 보너스 볼 일치 (%d)원- %d개";
public static final String SECOND_PRIZE_RESULT_MESSAGE = "%d개 일치, 보너스 볼 일치 (%d)원- %d";
public static final String REVENUE_RESULT_FORMATTER = "총 수익률은 %.2f입니다.";

public void showAllLottoStatus(final List<LottoTicket> lottoTickets) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,19 @@

import static org.assertj.core.api.Assertions.assertThatCode;

class LottoTicketBallTest {
class LottoBallTest {

@DisplayName("LottoBall 정상 생성된다.")
@DisplayName("LottoBall 정상 생성 테스트.")
@Test
void LottoBall_생성_테스트() {
void lottoBallGenerateTest() {
assertThatCode(() -> new LottoBall(3))
.doesNotThrowAnyException();
}

@DisplayName("보장되지 않은 LottoNumber 에러 발생한다. ")
@DisplayName("보장되지 않은(중복되지 않은 1~45의 숫자이며, 6개의 숫자) LottoBall이 저장될 시 에러 발생한다.")
@ParameterizedTest
@ValueSource(ints = {-1, 0, 46, 47})
void LottoBall_에러_테스트(int value) {
void lottoBallNotGuaranteedErrorTest(int value) {
Assertions.assertThatThrownBy(() -> new LottoBall(value))
.isInstanceOf(IllegalArgumentException.class);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@
import static org.assertj.core.api.Assertions.assertThatCode;
import static org.assertj.core.api.Assertions.assertThatThrownBy;

public class LottoTicketBallsTest {
public class LottoBallsTest {

@DisplayName("LottoBalls 정상 생성된다")
@DisplayName("LottoBalls 정상 생성 테스트.")
@Test
void LottoBalls_생성_테스트() {
void lottoBallsGenerateTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6);

Expand All @@ -28,9 +28,9 @@ public class LottoTicketBallsTest {
.doesNotThrowAnyException();
}

@DisplayName("LottoBalls의 인스턴스의 사이즈가 6이 아니면 에러가 발생한다. ")
@DisplayName("LottoBalls 인스턴스의 사이즈가 6이 아니면 에러가 발생한다.")
@Test
void LottoBalls_사이즈_예외_테스트() {
void lottoBallsSizeErrorTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6, 7);

Expand All @@ -43,9 +43,9 @@ public class LottoTicketBallsTest {
.isInstanceOf(IllegalArgumentException.class);
}

@DisplayName("LottoBalls에 중복된 LottoNumber가 있다면 에러가 발생한다.")
@DisplayName("LottoBalls에 중복된 LottoBall이 있다면 에러가 발생한다.")
@Test
void LottoBalls_중복_예외_테스트() {
void lottoBallsDuplicateErrorTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 6, 6);

Expand Down
10 changes: 5 additions & 5 deletions src/test/java/domain/bettingMoney/BettingMoneyTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,18 @@

class BettingMoneyTest {

@DisplayName("Budget 정상 생성된다.")
@DisplayName("BettingMoney 정상 생성테스트.")
@ParameterizedTest
@ValueSource(ints = {10000, 14000})
void Budget_생성_테스트(int value) {
void bettingMoneyGenerateTest(int value) {
assertThatCode(() -> BettingMoney.of(value))
.doesNotThrowAnyException();
}

@DisplayName("보장된 숫자가 아니면 에러가 발생한다.")
@DisplayName("보장된 숫자(로또 티켓 구입 금액보다 큰 양수의 숫자))가 아니면 에러가 발생한다.")
@ParameterizedTest
@ValueSource(ints = {-1, 0})
void Budget_생성_예외_테스트(int value) {
@ValueSource(ints = {-1, 0, 900})
void bettingMoneyNotGuaranteedErrorTest(int value) {
assertThatThrownBy(() -> BettingMoney.of(value))
.isInstanceOf(IllegalArgumentException.class);
}
Expand Down
24 changes: 0 additions & 24 deletions src/test/java/domain/lotto/LottoTicketCountTest.java

This file was deleted.

4 changes: 2 additions & 2 deletions src/test/java/domain/lotto/LottoTicketTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@

public class LottoTicketTest {

@DisplayName("Lotto 정상 생성된다.")
@DisplayName("LottoTicket 정상 생성 테스트.")
@Test
public void Lotto_생성() {
public void lottoTicketGenerateTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6);

Expand Down
27 changes: 27 additions & 0 deletions src/test/java/domain/lotto/TicketCountTest.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
package domain.lotto;

import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.ValueSource;

import static org.assertj.core.api.Assertions.assertThatCode;
import static org.assertj.core.api.Assertions.assertThatThrownBy;

class TicketCountTest {

@DisplayName("ticketCount 정상 생성 테스트.")
@Test
void ticketCountGenerateTest() {
assertThatCode(() -> TicketCount.of(1))
.doesNotThrowAnyException();
}

@DisplayName("ticketCount의 수가 0이하이면 에러가 발생한다")
@ParameterizedTest
@ValueSource(ints = {-1, 0})
void ticketCountGenerateErrorTest(int value) {
assertThatThrownBy(() -> TicketCount.of(value))
.isInstanceOf(IllegalArgumentException.class);
}
}
10 changes: 5 additions & 5 deletions src/test/java/domain/lotto/WinningLottoTicketTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@

public class WinningLottoTicketTest {

@DisplayName("WinningLotto 정상 생성된다.")
@DisplayName("WinningLotto 정상 생성 테스트.")
@Test
void winningLotto_생성_테스트() {
void winningLottoGenerateTest() {
//given
int[] lottoNumbers = {1, 2, 3, 4, 5, 6};
int bonusNumber = 7;
Expand All @@ -28,9 +28,9 @@ public class WinningLottoTicketTest {
.doesNotThrowAnyException();
}

@DisplayName("LottoNumbers와 BonusNumber 중복 검사")
@DisplayName("winningLotto LottoBalls의 수와 BonusBall의 수거 중복일 시 에러가 발생한다.")
@Test
void winningLotto_중복_테스트() {
void winningLottoDuplicateErrorTest() {
//given
int[] lottoNumbers = {1, 2, 3, 4, 5, 6};
int bonusNumber = 6;
Expand All @@ -47,7 +47,7 @@ public class WinningLottoTicketTest {

@DisplayName("WinningBalls 갯수가 5개 이하이면 에러가 발생한다.")
@Test
void winningLottoWiningBallSizeTest() {
void winningLottoBallSizeErrorTest() {
//given
int[] lottoNumbers = {1, 2, 3, 5, 5, 6};
int bonusNumber = 6;
Expand Down
18 changes: 9 additions & 9 deletions src/test/java/domain/result/ResultTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,9 @@ void initWinningLotto() {
winningLotto = new WinningLotto(winningNumbers, bonusNumber);
}

@DisplayName("Result객체 정상 생성 테스트")
@DisplayName("Result 정상 생성 테스트.")
@Test
void Result_객체_정상_생성된다() {
void resultGenerateTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6);

Expand All @@ -48,9 +48,9 @@ void initWinningLotto() {
.doesNotThrowAnyException();
}

@DisplayName("Result 결과 반환한다.")
@DisplayName("Result 결과 테스트.")
@Test
void Result_결과를_반환한다() {
void resultFindRankTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6);

Expand All @@ -67,9 +67,9 @@ void initWinningLotto() {
assertThat(results.get(LottoRank.SIX_MATCHES)).isEqualTo(1);
}

@DisplayName("Result 복수 결과 반환한다")
@DisplayName("Result 복수 결과 반환 테스트.")
@Test
void Result_로또_복수_결과_테스트() {
void resultManyRankTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 6);
List<Integer> lottoNumbers2 = Arrays.asList(1, 2, 3, 4, 5, 8);
Expand All @@ -92,7 +92,7 @@ void initWinningLotto() {

@DisplayName("5개의 볼, 보너스 볼이 맞을 때 2등 당첨된다.")
@Test
void ResultSecondPrizeTest() {
void resultSecondPrizeTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 4, 5, 7);

Expand All @@ -109,9 +109,9 @@ void ResultSecondPrizeTest() {
assertThat(results.get(LottoRank.FIVE_AND_BONUS_MATCHES)).isEqualTo(1);
}

@DisplayName("수익률 반환 테스트")
@DisplayName("수익률 반환 테스트.")
@Test
void earningsRateTest() {
void resultEarningsRateTest() {
//given
List<Integer> lottoNumbers = Arrays.asList(1, 2, 3, 7, 8, 9);
List<Integer> lottoNumbers2 = Arrays.asList(1, 2, 3, 7, 8, 9);
Expand Down

0 comments on commit c5bcd38

Please sign in to comment.