안녕하세요, 다람님! 수강은 물론 리팩터링까지 진행하시다니, 멋집니다. 😀먼저, 질문주신 1, 2, 3번은 말씀하신 내용이 맞습니다. 추가적으로 공유해주신 내용에서 크게 3가지를 말씀드리고 싶습니다.&nbsp;<h3>Domain Service의 적용</h3>Payment 도메인에 ExRateProvider와 Clock을 직접 넘겨 구현한 것이 의도하신 도메인 모델 패턴으로 볼 수 있겠습니다.&nbsp;<h3>추가 리팩토링</h3><ol><li>Test Code에서 @BeforeEach 라는 애노테이션 붙인 메소드를 만드시면, 각 테스트 메소드가 실행되기 전에 '준비'하는 작업을 구성해둘 수 있습니다. 또한 중복 코드를 제거하는 효과도 생기겠죠? 아래는 제가 만든 예시 입니다.</li></ol><pre><code class="language-java">class PaymentTest {

 private Clock clock;
 private ExRateProviderStub exRateProvider;

 @BeforeEach
 void setUp() {
 this.clock = Clock.fixed(Instant.now(), ZoneId.systemDefault());
 this.exRateProvider = new ExRateProviderStub(valueOf(1_000));
 }

 @Test
 void createPrepared() throws IOException {
 Payment payment = Payment.createPrepared(
 1L, "USD", BigDecimal.TEN, exRateProvider, clock
 );

 Assertions.assertThat(payment.getConvertedAmount()).isEqualByComparingTo(valueOf(10_000));
 Assertions.assertThat(payment.getValidUntil()).isEqualTo(LocalDateTime.now(clock).plusMinutes(30));
 }

 @Test
 void isValid() throws IOException {
 Payment payment = Payment.createPrepared(
 1L, "USD", BigDecimal.TEN, exRateProvider, clock
 );

 Assertions.assertThat(payment.isValid(clock)).isTrue();
 Assertions.assertThat(
 payment.isValid(Clock.offset(clock, Duration.of(30, ChronoUnit.MINUTES)))).isFalse();
 }
}</code></pre>해당 Test Class 자체에서 Payment가 전역에서 필요하다면 Payment 도 setUp()에 넣을 수 있겠죠?&nbsp;<ol start="2"><li>ExRateProviderStub 오브젝트를 저장하는 변수의 타입을 ExRateProvider로 지정하면 테스트하는 목적과 대상이 더욱 명확히 보일 것 같습니다. 위 예시에서 ExRateProviderStub 타입 선언만 ExRateProvider 으로 바꿔줘도, 테스트 코드를 읽으면서 ExRateProvider를 테스트에 사용한다는 것이 명확해집니다.</li></ol><pre><code class="language-java">class PaymentTest {

 private Clock clock;
 private ExRateProvider exRateProvider;

 @BeforeEach
 void setUp() {
 this.clock = Clock.fixed(Instant.now(), ZoneId.systemDefault());
 this.exRateProvider = new ExRateProviderStub(valueOf(1_000));
 }
 
 // 생략
}</code></pre>&nbsp;<h3>Lambda 활용</h3>위의 예시 같이 new 생성자로 ExRateProviderStub을 사용할 수도 있습니다. 또 다르게는, 인터페이스에 메소드가 1개일 경우 Lambda를 통해 그 1개의 메소드를 사용하는 방법도 있습니다. 이는 테스트 목적으로 추가 Class를 정의하지 않고도 실행할 수 있는 방법이기도 합니다.<pre><code class="language-java">class PaymentTest {

 private Clock clock;
 private ExRateProvider exRateProvider;

 @BeforeEach
 void setUp() {
 this.clock = Clock.fixed(Instant.now(), ZoneId.systemDefault());
 this.exRateProvider = currency -> BigDecimal.valueOf(1_000);
 }
 
 // 생략
}</code></pre>&nbsp;제 예시 코드가 정답은 아닙니다. 다만, 왜 이렇게 구성하고 고치는지 이해하면 그게 더 나은 개발이고 코드라고 생각합니다. 😁 지금처럼 꾸준히 강의나 학습을 통해 성장하시길 응원하겠습니다!

Zin

Posts