Hmm. I don't really like TODOs in the code (although I sometimes add them myself as well) and in this case I'm not sure what the todo means. Either make it a full sentence that explains just what the missing test is, or better just do it