String helpers are fiddly -- start to add unit tests