“No Duh,” say senior developers everywhere.
The article explains that vibe code often is close, but not quite, functional, requiring developers to go in and find where the problems are - resulting in a net slowdown of development rather than productivity gains.
It shouldn’t, but it does. The person who writes the code cares more about its correctness, so I trust them to write better tests.
I absolutely don’t. Since we’re talking about bad cases anyway, I don’t trust a developer to be diligent in finding bugs in their code more than I believe they will try to make all the tests pass. And it’s easier and better for the ego to achieve that if you write shit tests that only cover cases that you know will work.
Unit tests aren’t intended to find bugs, they’re intended to prove correctness. There should be a separate QA process for finding bugs, which involves integration testing. When QA inevitably finds a bug, the unit tests get updated with that case (and any similar cases).
And that’s what code reviews are for. If your tests don’t sufficiently cover the logic, the change should be rejected until they do. It’s a lot easier to verify the tests cover the logic if the tests are submitted w/ the logic changes.