Private Method 테스트

[junwha]

085a9b8 에서 DataLoader에 대한 테스트를 read에 대해 수행했을 때 높은 coverage가 달성되지만,
다음과 같은 이유로 좋은 테스트가 아니라고 생각됩니다.

1. read()에서 실제값을 reading 하므로 빌드 시 소모되는 자원이 늘어남
2. private 메소드에 대한 테스트가 없으므로 오류가 발생했을 시 별도의 debuging이 필요함 (쓸모있는 unit test가 아님)

private 메소드를 테스트하는 방법으로는 reflection이 있고 static 클래스를 잘 테스트하는 방법으로는 Mock Object를 활용하는 방법이 있던데 두 가지 모두 가벼운 개념이 아닌 것으로 판단되어 우선 지금 상태로 구현해뒀습니다.

이 discussion을 통해 DataLoaderTest를 어떻게 테스트해야할 지 의논해보면 좋을 것으로 생각됩니다.

[junwha]

How do I test a private function or a class that has private methods, fields or inner classes?

The best way to test a private method is via another public method. If this cannot be done, then one of the following conditions is true:

1. The private method is dead code
2. There is a design smell near the class that you are testing
3. The method that you are trying to test should not be private

When I have private methods in a class that are sufficiently complicated that I feel the need to test the private methods directly, that is a code smell: my class is too complicated.

My usual approach to addressing such issues is to tease out a new class that contains the interesting bits. Often, this method and the fields it interacts with, and maybe another method or two can be extracted in to a new class.

I have used reflection to do this for Java in the past, and in my opinion it was a big mistake.

Strictly speaking, you should not be writing unit tests that directly test private methods. What you should be testing is the public contract that the class has with other objects

https://www.artima.com/articles/testing-private-methods-with-junit-and-suiterunner

1. Don't test private methods.
2. Give the methods package access.
3. Use a nested test class.

[queuedq]

DataLoader는 #43 에서 완전히 바뀔 예정이니 지금 테스트를 작성하지 않아도 됩니다.
어떤 경우에도 private method는 테스트하면 안 되며, infra 레이어에서 DB에 접근하는 등 바깥 세상과 소통하는 클래스는 유닛 테스트가 아닌 integration 테스트를 진행해야 합니다.
다만 DataLoader는 유닛 테스트가 가능한 유틸리티 클래스의 형태로 리팩토링할 것입니다.

[junwha]

integration 테스트는 외부와 잘 소통되는지 판단하는건가요?

[queuedq]

유닛 테스트가 각각의 모듈에 대한 테스트를 진행하는 거라면 integration 테스트는 모듈끼리 통합했을 때, 혹은 모듈과 바깥 세상을 통합했을 때 잘 작동하는지 테스트하는 거라고 볼 수 있습니다.