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
git blame -w also ignores it


On Thu, Jul 24, 2014 at 2:47 PM, Jeremy Kemper <notifications@github.com>
wrote:

> In spec/mail/fields/sender_field_spec.rb:
>
> > @@ -12,7 +12,7 @@
> >      end
> >
> >      it "should mix in the CommonAddress module" do
> > -      expect(Mail::SenderField.included_modules).to include(Mail::CommonAddress)
> > +      expect(Mail::SenderField.included_modules).to include(Mail::CommonAddress)
>
> Nice tip - thanks! However, it still mucks up git-blame history. Worth
> avoiding in shared repositories.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikel/mail/pull/749/files#r15375213>.
>