True. I thought you meant the other test. Yes, sounds like a bad idea to fall back to the normal case. I would say better raise an exception indicating that null values are not supported as feature ids