While working in the CAB war room (like the Delta Lounge but bigger), there are a number of utterances that I seem to say all the timeΓÇª
“Use inheritance to be reused, not to reuse.”
This is a bad mis-quoting from C++ Coding Standards: 101 Rules, Guidelines, and Best Practices. In it, Herb Sutter and Andrei Alexandrescu write, “Public inheritance is substitutability. Inherit, not reuse, but to be reused.” It continually amazes me how often developers will derive from a base class to reuse its functionality when they don’t actually mean for the new class to be substitutable for the other.
Additional reading: The Liskov Substitution Principle
“Nothing is better than deleting code and having your tests still pass.”
Assuming you have a nice test harness with good code coverage, nothing makes me happier than deleting code from the production system and leaving the unit tests working. After all, if the test doesn’t need it to pass, you don’t need it to be there.
Or, maybe you’re missing a testΓÇª
_place_holder;“That class? I deleted it. There weren’t any tests, so I figured someone just forgot to delete it.”
_place_holder;I did this a lot at the beginning of CAB, when I was still trying to “bend the team to my TDD will”.  _place_holder;In the end it had the desired result, and in fact, may have caused a bit of excess. But better that than no unit tests at all.