3 ms·
> One thing that is different about the linux kernel, as far as I know, is how thorough the reviewing is that is going on for every patch. This is another unde
by berkes 5y ago
> One thing that is different about the linux kernel, as far as I know, is how thorough the reviewing is that is going on for every patch.
This is another underexposed feature of TDD: reviews. When I see a PR with solid test coverage, and those are green, it greatly speeds up the process.
I just need to read through tests, see if they make sense, match the requirements and test those. And then glance over the added code quickly.
It (unfortunately, I might add) is not always possible to flat out reject a PR without solid test coverage, so it often is less work for me to add some test myself to a PR, than to manually poke around the changed code or to read through it in detail. This is not TDD, but testing-after-the-fact, and those test often are of much lower quality, but at least this is less work, for me.
Testing greatly speeds up the work for a reviewer.