# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@ens-lyon.org>
# Date 1482964970 -3600
# Node ID aa797bd54f44e98939417beffdb81a951829e32b
# Parent  ecc87acb4d100a80c1e3ff18fbf581a8cd0c8cce
test: add a basic 'test-check-pylint.t'

We add a minimal check using pylint for one case we knows we care about:
"mutable default" argument.

We'll likely extend this over time to cover other useful checks but this is a
good starting point.

diff -r ecc87acb4d10 -r aa797bd54f44 tests/hghave.py
--- a/tests/hghave.py	Tue Mar 14 23:50:07 2017 -0700
+++ b/tests/hghave.py	Wed Dec 28 23:42:50 2016 +0100
@@ -413,6 +413,12 @@
                        br"<stdin>:1: 're' imported but unused",
                        True)
 
+@check("pylint", "Pylint python linter")
+def has_pylint():
+    return matchoutput("pylint --help",
+                       br"Usage:  pylint",
+                       True)
+
 @check("pygments", "Pygments source highlighting library")
 def has_pygments():
     try:
diff -r ecc87acb4d10 -r aa797bd54f44 tests/test-check-pylint.t
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/test-check-pylint.t	Wed Dec 28 23:42:50 2016 +0100
@@ -0,0 +1,15 @@
+#require test-repo pylint hg10
+
+Run pylint for known rules we care about.
+-----------------------------------------
+
+There should be no recorded failures; fix the codebase before introducing a
+new check.
+
+Current checks:
+- W0102: no mutable default argument
+
+  $ touch $TESTTMP/fakerc
+  $ pylint --rcfile=$TESTTMP/fakerc --disable=all \
+  >   --enable=W0102 --reports=no \
+  >   mercurial hgext hgext3rd