Confusing pytestmark documentation

[asmeurer]

The pytestmark documentation at http://pytest.org/dev/skipping.html#skip-all-test-functions-of-a-class-or-module says:

@pytest.mark.skipif(sys.platform == 'win32',
                    reason="requires windows")
class TestPosixCalls:

    def test_function(self):
        "will not be setup or run under 'win32' platform"

The reason given is "requires windows", but if I understand the test correctly, it should be "requires POSIX".

[RonnyPfannschmidt]

thats a wrong operator, i believe it should be !=
it would be a good contribution

[asmeurer]

The class is called TestPosixCalls and the docstring says "will not be setup or run under 'win32' platform". If I understand skipif correctly, the operator is correct, but the reason is wrong.

[nicoddemus]

That link is out of date, this has been fixed sometime ago already! 👍

Latest version: http://doc.pytest.org/en/latest/skipping.html#skip-all-test-functions-of-a-class-or-module

Where did you find that link btw? It seems like we are redirecting only pytest.org/latest to readthedocs.

[asmeurer]

It's the fourth result from Googling "pytest skip". I see the first result is the right one, but I chose that one because of the "dev" in the URL. If those docs are out of date I would suggest removing them.

[nicoddemus]

Agreed, thanks for pointing it out.

@RonnyPfannschmidt @hpk42 should we just remove them or redirect them as well?