probably good to unit test this as well