summaryrefslogtreecommitdiffstats
path: root/ipsilon/login/authtest.py
diff options
context:
space:
mode:
authorSimo Sorce <simo@redhat.com>2015-05-07 12:33:40 -0400
committerRob Crittenden <rcritten@redhat.com>2015-05-07 14:44:20 -0400
commit1bcc0d697dd37a9268641f0cbaa7e9e781552233 (patch)
treef17e5fb0f9f0a1a922bcbd80ec798396c508f56d /ipsilon/login/authtest.py
parent93d4e52712767fe955f3a44a60a6c6f0f909423b (diff)
downloadipsilon.git-1bcc0d697dd37a9268641f0cbaa7e9e781552233.tar.gz
ipsilon.git-1bcc0d697dd37a9268641f0cbaa7e9e781552233.tar.xz
ipsilon.git-1bcc0d697dd37a9268641f0cbaa7e9e781552233.zip
pylint 1.4.3 version fixes
Pylint 1.4.3 completely stopped recognizing the star-args condition. In order to avoid pylint error with > 1.4.3 stop caring for star-args and add cmdline option to ignore those errors completly so older pylint versions are happy too. Also fix type() vs isinstance() checks, isinstance is generally a more correct approach to check for classes. In some 'admin' files the type() -> isinstance() fix required to invert the order in which ComplexList and MappingList are checked as the latter is a subclass of ComplexList, so it needs to be checked first otherwise the check for isinstance(option, ComplexList) matches for both and the code stops functioning properly. Signed-off-by: Simo Sorce <simo@redhat.com> Reviewed-by: Rob Crittenden <rcritten@redhat.com>
Diffstat (limited to 'ipsilon/login/authtest.py')
-rw-r--r--ipsilon/login/authtest.py1
1 files changed, 0 insertions, 1 deletions
diff --git a/ipsilon/login/authtest.py b/ipsilon/login/authtest.py
index 7769650..002ab73 100644
--- a/ipsilon/login/authtest.py
+++ b/ipsilon/login/authtest.py
@@ -56,7 +56,6 @@ class TestAuth(LoginFormBase):
error_username=not username
)
self.lm.set_auth_error()
- # pylint: disable=star-args
return self._template('login/form.html', **context)