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
because this tests passes


On Thu, Jul 24, 2014 at 2:48 PM, Jeremy Kemper <notifications@github.com>
wrote:

> In spec/mail/elements/address_list_spec.rb:
>
> >      it "should handle a really nasty obsolete address list" do
> > -      skip
>
> Why (here and elsewhere) remove a skip entirely?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikel/mail/pull/749/files#r15375272>.
>