Bettertesting

Alex, incorporated your suggestions (which I greatly appreciate - as well as the info behind them)

Let me know if this looks better/ready to pull or needs more attention.   Thanks!
Hmm. Seems like maybe the empty strings will still be a problem for typed numbers. I didn't think of that before. Maybe we should do this check in `formatPhone` and return `null` if the string is empty?

The next two lines would need to be changed to something like:

```coffeescript
body.CellPhone ?= formatPhone(vars.lead.phone_2) or formatPhone(vars.lead.phone_1)
body.HomePhone ?= formatPhone(vars.lead.phone_1) or formatPhone(vars.lead.phone_2)
```