Merge pull request #11 from asteriogonzalez/master

show the name of the skipped test

Author: RKrahl

doc/examples/basic.out

@@ -0,0 +1,15 @@
+$ py.test -rsx basic.py
+============================= test session starts ==============================
+platform linux -- Python 3.4.6, pytest-2.8.7, py-1.4.30, pluggy-0.3.1
+rootdir: /home/user/pytest-dependency-0.2, inifile: 
+plugins: dependency-0.2
+collected 5 items 
+
+basic.py x.s.s
+=========================== short test summary info ============================
+SKIP [1] /usr/lib/python3.4/site-packages/pytest_dependency.py:65: test_e depends on test_c
+SKIP [1] /usr/lib/python3.4/site-packages/pytest_dependency.py:65: test_c depends on test_a
+XFAIL basic.py::test_a
+  deliberate fail
+
+================ 2 passed, 2 skipped, 1 xfailed in 0.02 seconds ================

doc/src/usage.rst

@@ -13,9 +13,13 @@ Consider the following example test module:
 All the tests are decorated with :func:`pytest.mark.dependency`.  This
 will cause the test results to be registered internally and thus other
 tests may depend on them.  The list of dependencies of a test may be
-set in the optional `depends` argument to the marker.  The first test
-has deliberately been set to fail to illustrate the effect.  We will
-get the following resuts:
+set in the optional `depends` argument to the marker.  Running this
+test, we will get the following result:
+
+.. literalinclude:: ../examples/basic.out
+
+The first test has deliberately been set to fail to illustrate the
+effect.  We will get the following resuts:
 
 `test_a`
   deliberatly fails.

pytest_dependency.py

@@ -59,10 +59,10 @@ def addResult(self, item, marker, rep):
         status = self.results.setdefault(name, DependencyItemStatus())
         status.addResult(rep)
 
-    def checkDepend(self, depends):
+    def checkDepend(self, depends, item):
         for i in depends:
             if not(i in self.results and self.results[i].isSuccess()):
-                pytest.skip("depends on %s" % i)
+                pytest.skip("%s depends on %s" % (item.name, i))
 
 
 def depends(request, other):
@@ -84,7 +84,7 @@ def depends(request, other):
     """
     item = request.node
     manager = DependencyManager.getManager(item)
-    manager.checkDepend(other)
+    manager.checkDepend(other, item)
 
 
 @pytest.hookimpl(tryfirst=True, hookwrapper=True)
@@ -108,4 +108,4 @@ def pytest_runtest_setup(item):
         depends = marker.kwargs.get('depends')
         if depends:
             manager = DependencyManager.getManager(item)
-            manager.checkDepend(depends)
+            manager.checkDepend(depends, item)

tests/test_03_skipmsgs.py

@@ -0,0 +1,41 @@
+"""Verify the messages issued when a dependent test is skipped.
+"""
+
+import pytest
+
+
+def test_simple(ctestdir):
+    """One test fails, other dependent tests are skipped.
+    This also includes indirect dependencies.
+    """
+    ctestdir.makepyfile("""
+        import pytest
+
+        @pytest.mark.dependency()
+        def test_a():
+            pass
+
+        @pytest.mark.dependency()
+        def test_b():
+            assert False
+
+        @pytest.mark.dependency(depends=["test_b"])
+        def test_c():
+            pass
+
+        @pytest.mark.dependency(depends=["test_c"])
+        def test_d():
+            pass
+    """)
+    result = ctestdir.runpytest("--verbose", "-rs")
+    result.assert_outcomes(passed=1, skipped=2, failed=1)
+    result.stdout.fnmatch_lines("""
+        *::test_a PASSED
+        *::test_b FAILED
+        *::test_c SKIPPED
+        *::test_d SKIPPED
+    """)
+    result.stdout.fnmatch_lines_random("""
+        SKIP * test_c depends on test_b
+        SKIP * test_d depends on test_c
+    """)