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
I'm a fan of no trailing whitespace, but it really clutters up diffs and makes them more difficult to review. Please don't make cosmetic changes that are unrelated to the change at hand.