fix #510 by adding a describing skip marker on empty parameterize

[RonnyPfannschmidt]

this might need to go to features

also we might want to consider using a xfail marker with run=False instead of skip

[RonnyPfannschmidt]

also fml for formatting fixes

[nicoddemus]

LGTM

[RonnyPfannschmidt]

while working on the features branch, i noticed a correct skip that is checking for an apparently missing attribute

[nicoddemus]

@RonnyPfannschmidt could you take a look at the failing AppVeyor build?

[RonnyPfannschmidt]

@nicoddemus it seems that one is related to the config filename on non-posix fix

[nicoddemus]

@RonnyPfannschmidt could you rebase this and try again? This is fixed on master already.

[RonnyPfannschmidt]

@nicoddemus i figured the underlying issue, the one on master was not composable

[coveralls]

Coverage decreased (-0.02%) to 92.204% when pulling afc5c7e on RonnyPfannschmidt:fix-510 into 436e13a on pytest-dev:master.

[RonnyPfannschmidt]

i removed the old hack and always use the marker

this has the upside of always composing and the downside of hiding one of the error in case of a double empty parametrize

[nicoddemus]

Thanks, the new code looks better indeed. 😁