Another thought, though no need to change this MR: when you can, first adding the tests (with appropriate `todo` hints) and then the implementation (removing the `todo` hints) helps the reader understanding what your commit is actually fixing. It's pretty trivial in this case, but a good habit nevertheless I think.