Don't modify string in place.
Standardize argument names for message
Fix spec