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
Correct. Regardless: Making unrelated changes in a PR is ineffective use of all our time, whether that's whitespace or anything else.