인프런 커뮤니티 질문&답변

강의를 보고 기존 프로젝트에 작성하지 않았던 테스트 코드를 작성하면서 공부하고 있습니다.

도메인 모듈같은 경우 어떠한 의존성도 가지고 있지 않은 POJO 형태로 코드를 작성해서 단위 테스트로 검증할 수 있었습니다.

Crawling 모듈의 경우 Jsoup이라는 라이브러리를 사용하여 특정 URL의 HTML을 불러오고, 해당 HTML에서 필요한 정보를 추출하는 코드를 가지고 있습니다.

이때, 전체적으로 Mock을 사용하여 테스트를 진행하고자 하였는데 라이브러리의 특성 때문에 stubbing을 작성하는데 불편함을 느꼈습니다.

그래서 HTML을 임의로 만들고, HTML을 불러오는 메서드만 stubbing을 적용하려고 하는게 더 효율적이지 않을까 라는 생각이 들었습니다.

외부 라이브러리를 사용하는 코드에서 테스트 작성이 미숙한 것 같아서 어떤 방향이 더 좋을지 질문드리고 싶습니다.

아래 두 코드는 동일한 테스트를 진행하고 있는 코드가 아닌점 참고바랍니다.

@Mock
private lateinit var webPageLoader: WebPageLoader<Document>

@InjectMocks
private lateinit var dietParser: DietParser

fun parse() {
    // given
    val mockDocument = mock(Document::class.java)
    val mockRows = Elements()
    val row1 = mock(Element::class.java)
    val row2 = mock(Element::class.java)
    val row3 = mock(Element::class.java)
    mockRows.add(row1)
    mockRows.add(row2)
    mockRows.add(row3)
    val columns1 = Elements(
        mock(Element::class.java).apply { given(text()).willReturn("2024.10.23") },
        mock(Element::class.java),
        mock(Element::class.java),
        mock(Element::class.java).apply { given(text()).willReturn("Menu 1, Menu 2") }
    )
    val columns2 = Elements(
        mock(Element::class.java).apply { given(text()).willReturn("교직원식당") }
    )
    val columns3 = Elements(
        mock(Element::class.java).apply { given(text()).willReturn("2024.10.24") },
        mock(Element::class.java),
        mock(Element::class.java),
        mock(Element::class.java).apply { given(text()).willReturn("Menu 3, Menu 4") }
    )

    given(webPageLoader.getHTML(anyString())).willReturn(mockDocument)
    given(mockDocument.select(anyString())).willReturn(mockRows)
    given(row1.select(anyString())).willReturn(columns1)
    given(row2.select(anyString())).willReturn(columns2)
    given(row3.select(anyString())).willReturn(columns3)

    // when
    val result = dietParser.parse()

    // then
    assertThat(result).hasSize(2)
        .extracting("date", "menus")
        .containsExactly(
            tuple(LocalDate.of(2024, 10, 23), listOf("Menu 1", "Menu 2")),
            tuple(LocalDate.of(2024, 10, 24), listOf("Menu 3", "Menu 4")),
        )
}

    @Mock
    private lateinit var webPageLoader: WebPageLoader<Document>

    @InjectMocks
    private lateinit var parser: DepartmentNoticeParser

    fun parse() {
        // given
        val html = """
            <table class="board-table">
                <tbody>
                    <tr>
                        <td class="td-num">2</td>
                        <td class="td-subject"><a href="javascript:fnView('2','url','title','2')">공지 제목 2</a></td>
                        <td class="td-write">작성자 2</td>
                        <td class="td-date">2024.10.02</td>
                    </tr>
                    <tr>
                        <td class="td-num">1</td>
                        <td class="td-subject"><a href="javascript:fnView('1','url','title','1')">공지 제목 1</a></td>
                        <td class="td-write">작성자 1</td>
                        <td class="td-date">2024.10.01</td>
                    </tr>
                </tbody>
            </table>
        """.trimIndent()
        val document: Document = Jsoup.parse(html)

        given(webPageLoader.getHTML(anyString())).willReturn(document)

        parser.initialize(Major.COMPUTER_SOFTWARE_ENGINEERING)

        // when
        val notices = parser.parse()

        // then
        assertThat(notices).hasSize(2)
            .extracting("number", "title", "author", "date", "url")
            .containsExactlyInAnyOrder(
                tuple(
                    2,
                    "공지 제목 2",
                    "작성자 2",
                    LocalDate.of(2024, 10, 2),
                    "https://www.dongyang.ac.kr/combBbs/2/url/2/view.do?layout=unknown"
                ),
                tuple(
                    1,
                    "공지 제목 1",
                    "작성자 1",
                    LocalDate.of(2024, 10, 1),
                    "https://www.dongyang.ac.kr/combBbs/1/url/1/view.do?layout=unknown"
                ),

                )

    }