use pending and fix unnecessary skips

pending will alert when a skip is fixed, increasing the test coverage.
Afaik skip should only be used when the spec causes infinite loops etc evils.

@bf4
you can also review without whitespace changes so you won't be distracted by it: https://github.com/mikel/mail/pull/749/files?w=1