Merge pull request #67 from carsongee/rc/0.12.3

Released 0.12.3

Author: carsongee

README.rst

@@ -49,6 +49,12 @@ This code is heavily based on
 Releases
 ========
 
+0.12.3
+~~~~~~
+
+- `jamur2 <https://github.com/jamur2>`_ corrected issue where file
+  paths where not being output properly on lint failures.
+
 0.12.2
 ~~~~~~
 

pytest_pylint.py

@@ -233,4 +233,4 @@ def repr_failure(self, excinfo):
 
     def reportinfo(self):
         """Generate our test report"""
-        return self.fspath, None, "[pylint] {0}".format(self.name)
+        return self.fspath, None, "[pylint] {0}".format(self.rel_path)

setup.py

@@ -13,7 +13,7 @@
     description='pytest plugin to check source code with pylint',
     long_description=open("README.rst").read(),
     license='MIT',
-    version='0.12.2',
+    version='0.12.3',
     author='Carson Gee',
     author_email='[email protected]',
     url='https://github.com/carsongee/pytest-pylint',

test_pytest_pylint.py

@@ -19,6 +19,20 @@ def test_basic(testdir):
     assert 'Linting files' in result.stdout.str()
 
 
+def test_subdirectories(testdir):
+    """Verify pylint checks files in subdirectories"""
+    subdir = testdir.mkpydir('mymodule')
+    testfile = subdir.join("test_file.py")
+    testfile.write("""import sys""")
+    result = testdir.runpytest('--pylint')
+    assert '[pylint] mymodule/test_file.py' in result.stdout.str()
+    assert 'Missing module docstring' in result.stdout.str()
+    assert 'Unused import sys' in result.stdout.str()
+    assert 'Final newline missing' in result.stdout.str()
+    assert '1 failed' in result.stdout.str()
+    assert 'Linting files' in result.stdout.str()
+
+
 def test_disable(testdir):
     """Verify basic pylint checks"""
     testdir.makepyfile("""import sys""")