should have a test