forked from openlp/openlp
Changes to pylint test
This commit is contained in:
parent
8aa917c89c
commit
9294aadae9
@ -24,11 +24,14 @@ Package to test for proper bzr tags.
|
||||
"""
|
||||
import os
|
||||
import logging
|
||||
from unittest import TestCase
|
||||
|
||||
from pylint import epylint as lint
|
||||
from pylint.__pkginfo__ import pylint_version
|
||||
import platform
|
||||
from unittest import TestCase, SkipTest
|
||||
|
||||
try:
|
||||
from pylint import epylint as lint
|
||||
from pylint.__pkginfo__ import version
|
||||
except ImportError:
|
||||
raise SkipTest('pylint not installed - skipping tests using pylint.')
|
||||
|
||||
class TestPylint(TestCase):
|
||||
|
||||
@ -36,17 +39,20 @@ class TestPylint(TestCase):
|
||||
"""
|
||||
Test for pylint errors
|
||||
"""
|
||||
# GIVEN: The openlp base folder
|
||||
# GIVEN: Some checks to disable and enable, and the pylint script
|
||||
disabled_checks = 'no-member,import-error,no-name-in-module'
|
||||
enabled_checks = 'missing-format-argument-key,unused-format-string-argument'
|
||||
#disabled_checks = 'all'
|
||||
disabled_checks = ''
|
||||
if 'arch' in platform.dist()[0].lower():
|
||||
pylint_script = 'pylint'
|
||||
else:
|
||||
pylint_script = 'pylint3'
|
||||
|
||||
# WHEN: Running pylint
|
||||
(pylint_stdout, pylint_stderr) = \
|
||||
lint.py_run('{path} --disable={disabled} --enable={enabled} --reports=no'.format(path='openlp',
|
||||
disabled=disabled_checks,
|
||||
enabled=enabled_checks),
|
||||
return_std=True, script='pylint3')
|
||||
lint.py_run('openlp --errors-only --disable={disabled} --enable={enabled} --reports=no --output-format=parseable'.format(
|
||||
disabled=disabled_checks,
|
||||
enabled=enabled_checks),
|
||||
return_std=True, script=pylint_script)
|
||||
stdout = pylint_stdout.read()
|
||||
stderr = pylint_stderr.read()
|
||||
print(stdout)
|
||||
|
Loading…
Reference in New Issue
Block a user