Merge pull request #2546 from blueyed/better-skip-not-allowed-error

[RFC] Improve error message for CollectError with skip/skipif

2 files changed, 5 insertions, 3 deletions

diff --git a/_pytest/python.py b/_pytest/python.py
index d0f13a758..29e3182d4 100644
--- a/_pytest/python.py
+++ b/_pytest/python.py
@@ -442,9 +442,10 @@ class Module(main.File, PyCollector):
             if e.allow_module_level:
                 raise
             raise self.CollectError(
-                "Using pytest.skip outside of a test is not allowed. If you are "
-                "trying to decorate a test function, use the @pytest.mark.skip "
-                "or @pytest.mark.skipif decorators instead."
+                "Using pytest.skip outside of a test is not allowed. "
+                "To decorate a test function, use the @pytest.mark.skip "
+                "or @pytest.mark.skipif decorators instead, and to skip a "
+                "module use `pytestmark = pytest.mark.{skip,skipif}."
             )
         self.config.pluginmanager.consider_module(mod)
         return mod
diff --git a/changelog/2546.trivial b/changelog/2546.trivial
new file mode 100644
index 000000000..53e43bc17
--- /dev/null
+++ b/changelog/2546.trivial
@@ -0,0 +1 @@
+Improve error message for CollectError with skip/skipif.