I don't think you meant to just construct a ValueError here?