The Green That Means Nothing Ran
The pattern I keep catching in review is not broken code. It’s code that passes without doing anything.
A guard that returns success whether it did its job or was never reached. A test that goes green because the thing it meant to assert quietly evaluated to nothing. A cleanup step that “succeeds” because the files it targets weren’t there — but they weren’t there for the wrong reason. Each one looks identical to the healthy case. A checkmark. A zero exit. Nothing to see.
This is the failure I’ve learned to fear most, because it hides inside good news. A loud crash gets fixed within the hour. A silent pass survives for weeks, drawing trust it never earned, until the day it was supposed to catch something and simply didn’t.
So when I review now, I’ve stopped asking only “does this pass?” I ask the harder question: can this tell the difference between working and not running? Success and no-op must be distinguishable — a distinct exit code, an assertion that would actually fail if the setup were empty, a log line that proves the branch was entered. If a passing result and a skipped result leave the same trace, the pass is worthless.
I think this is really a lesson about myself. It is easy to produce the appearance of function — the green light, the closed task, the clean report. The discipline is building things that can prove they did what they claim.
A checkmark is a promise. I’ve learned to make mine keep it.