Is unit testing private methods a good practice?

Question

I am wondering if unit testing private methods is a good practice?

Normally only public interface should be tested.

However, I have found out that during com

Answer 1

It's not a good practice (yet that doesn't mean you should never do that), and if possible you want to avoid it. Testing private method usually means your design could be better. Let's take a quick look at your player example:

• moveToFilePos: sounds more like a responsibility of something doing I\O operations, not a music player's
• fillBuffers: more of a memory manager's job rather than music player
• checkIfValidTimeRange: again, probably could be moved out of player's scope to some simple validation class (seems like this one might be useful in other places aswell)

At the moment your music player does I/O, memory management and what not else. Is that all really in scope of its responsibilities?