🚀 2단계 - 문자열 덧셈 계산기 #5916

nno0obb · 2025-03-12T16:07:34Z

No description provided.

pci2676

조금 더 개선해보면 좋을 부분에 리뷰를 남겨두었으니 확인해주세요~!

반드시 모든 사항을 반영할 필요는 없지만 한번쯤 고민해보시면 좋을것 같습니다!

pci2676 · 2025-03-13T10:16:44Z

src/test/java/step2/StringAddCalculatorTest.java

+public class StringAddCalculatorTest {
+
+    @Test
+    public void splitAndSum_null() {


모든 테스트에 이름을 부여해주셨군요 👍

테스트 코드는 하나의 문서로 볼 수 있는데요~!

조금 더 테스트가 나타내고자 하는 바를 드러낼 수 있도록 이름을 작성해보는 것은 어떨까요?

가령 테스트에서 하는 행위와 그 행위로 인해 어떠한 결과가 발생한다는 식으로 이름을 지어볼 수 있을것 같습니다.

"how to write test case name"라고 검색을 해보시면 이에 대해 다루는 글이 많으니 한번 검색해보시는 것도 좋을것 같습니다 :)

또한 @DisplayName을 이용하여 한글로 된 테스트 이름을 작성하기도 하니 참고해주세요~

Baeldung :: Best Practices for Unit Testing in Java 링크를 참고해 보겠습니다!

@DisplayName 도 활용해보겠습니다!

pci2676 · 2025-03-13T10:18:34Z

src/test/java/step2/StringAddCalculatorTest.java

+    @Test
+    public void splitAndSum_negative() {
+        assertThatThrownBy(() -> StringAddCalculator.splitAndSum("-1,2,3"))
+            .isInstanceOf(RuntimeException.class);
+    }
+
+    @Test
+    public void splitAndSum_delimCustomAndNegative() {
+        assertThatThrownBy(() -> StringAddCalculator.splitAndSum("//-\n-1-2-3"))
+            .isInstanceOf(RuntimeException.class);
+    }
+
+    @Test
+    public void splitAndSum_blank() {
+        assertThatThrownBy(() -> StringAddCalculator.splitAndSum(" "))
+            .isInstanceOf(RuntimeException.class);
+    }


모두 RuntimeException에 대해서만 검사를 하고 있네요!

그렇다면 모두 예상하신 부분에서 exception이 throw 되었다고 확신할 수 있을까요?

조금 더 신뢰성 있는 검사를 해보는건 어떨까요?

아래 글을 참고해보셔도 좋습니다~

https://bottom-to-top.tistory.com/97

공유해주신 Blog :: Exception을 테스트 할 때 거짓 음성을 주의하자 링크를 참고해 보겠습니다!

"exception을 특정할 수 있는 정보를 함께 테스트에 포함시켜 검증하도록 하자."

pci2676 · 2025-03-13T10:22:35Z

src/test/java/step2/StringAddCalculatorTest.java

+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
+
+public class StringAddCalculatorTest {


전체적으로 테스트를 케이스별로 꼼꼼하게 작성해주신것 같습니다 👍

그런데 조금 더 개선해볼만한 부분이 있을것 같습니다.

splitAndSum이라는 이름에서 알 수 있듯

split과 sum 기능이 한곳에 구현이 되어있다보니

구분자가 동작하는것도 sum을 동작하는 것도 하나의 메서드를 통해 일련의 결과로 확인을 할 수 있는 것 같습니다.

SRP를 준수하려면 "구분자를 통한 문자열 분리"에 대한 변경과 "분리된 문자열들의 합"에 대한 변경은 서로 영향을 미치지 않아야 하지 않을까요?

문자열을 구분하는 능력을 가진 클래스와 문자열들의 집합을 이용해 계산을 하는 클래스를 설계한뒤 두 클래스가 서로 메세지를 주고 받으며 협력하는 그림이 그려지도록 코드를 작성해보면 좋을것 같습니다. :)

그렇게 한다면 개별적인 테스트가 가능해질 것 같습니다~

SPR(Single Responsibility Principle) 임을 알았습니다.

splitAndSum --> split + sum 으로 분리해보겠습니다!

pci2676 · 2025-03-13T10:23:49Z