Discovered while testing NamespaceString::kEmpty.

It seems today that there is a flaw if we call the NamespaceString constructor that takes in a DatabaseName.

This is because if we have an empty `collection` parameter we do not check for it and add a `.` period symbol when we should not.