I would either fill out these TODO tests, or just leave these out if they are TODO or maybe comment them out? But realistically, unless you are filling out the tests now, I would just remove them. It seems very unlikely that someone would circle back to change them later, and the API of message_reader could change in the meantime.