Skip to content

Commit ea16ae0

Browse files
committed
test: 테스트 코드 내 매직 넘버 상수로 대체
1 parent 2933372 commit ea16ae0

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

src/main/java/io/suhan/racingcar/domain/Car.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
import io.suhan.racingcar.generator.RandomNumberGenerator;
55

66
public class Car {
7-
private static final int CAR_MOVE_THRESHOLD = 4;
8-
private static final int CAR_NAME_MAXIMUM_LENGTH = 5;
7+
public static final int CAR_MOVE_THRESHOLD = 4;
8+
public static final int CAR_NAME_MAXIMUM_LENGTH = 5;
99

1010
private final String name;
1111
private final NumberGenerator generator;

src/test/java/io/suhan/racingcar/domain/CarTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package io.suhan.racingcar.domain;
22

3+
import static io.suhan.racingcar.domain.Car.CAR_MOVE_THRESHOLD;
34
import static org.junit.jupiter.api.Assertions.assertEquals;
45
import static org.junit.jupiter.api.Assertions.assertNotEquals;
56
import static org.junit.jupiter.api.Assertions.assertThrows;
@@ -23,7 +24,7 @@ public class CarTest {
2324

2425
@Test
2526
void 생성된_값이_4_이상일_경우_자동차가_움직일_수_있다() {
26-
FixedNumberGenerator generator = new FixedNumberGenerator(4);
27+
FixedNumberGenerator generator = new FixedNumberGenerator(CAR_MOVE_THRESHOLD);
2728
Car car = Car.of("neo", generator);
2829

2930
car.move();
@@ -35,7 +36,7 @@ public class CarTest {
3536

3637
@Test
3738
void 생성된_값이_3_이하일_경우_자동차가_멈춘다() {
38-
FixedNumberGenerator generator = new FixedNumberGenerator(3);
39+
FixedNumberGenerator generator = new FixedNumberGenerator(CAR_MOVE_THRESHOLD - 1);
3940
Car car = Car.of("neo", generator);
4041

4142
car.move();

src/test/java/io/suhan/racingcar/domain/GameTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package io.suhan.racingcar.domain;
22

3+
import static io.suhan.racingcar.domain.Car.CAR_MOVE_THRESHOLD;
34
import static org.junit.jupiter.api.Assertions.assertIterableEquals;
45
import static org.junit.jupiter.api.Assertions.assertThrows;
56

@@ -30,8 +31,8 @@ public class GameTest {
3031
// given
3132
Game game = Game.of(5);
3233

33-
FixedNumberGenerator forwardGenerator = new FixedNumberGenerator(4);
34-
FixedNumberGenerator stopGenerator = new FixedNumberGenerator(3);
34+
FixedNumberGenerator forwardGenerator = new FixedNumberGenerator(CAR_MOVE_THRESHOLD);
35+
FixedNumberGenerator stopGenerator = new FixedNumberGenerator(CAR_MOVE_THRESHOLD - 1);
3536

3637
Car neo = Car.of("neo", forwardGenerator);
3738
Car brie = Car.of("brie", stopGenerator);

0 commit comments

Comments
 (0)