The big problem I've noticed when reading unit tests in various projects is that they frequently test the implementation rather than the outcome:
https://softwareengineering.stackexchange.com/a/304910/24932
https://softwareengineering.stackexchange.com/a/304910/24932