openstack/keystone
Code Issues Proposed changes

Run pep8 for tests.

Browse Source 
Fixes bug 1021508

Test code should be as pep8 compliant as source code, however right now
'run_tests.sh' only runs pep8 for source code.

Change-Id: I877493a8bf6f28bef71fe2aca6f188a75798225f
This commit is contained in:
Rafael Durán Castañeda 2012-07-06 00:56:23 +02:00
parent f584c06aaa
commit 04df79b64e
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
run_tests.sh
View File

@ -108,7 +108,7 @@ function run_pep8 {
ignore_dirs="*ajaxterm*" ignore_dirs="*ajaxterm*"
GLOBIGNORE="$ignore_scripts:$ignore_files:$ignore_dirs" GLOBIGNORE="$ignore_scripts:$ignore_files:$ignore_dirs"
srcfiles=`find bin -type f ! -name .*.swp` srcfiles=`find bin -type f ! -name .*.swp`
srcfiles+=" keystone" srcfiles+=" keystone tests"
# Just run PEP8 in current environment # Just run PEP8 in current environment
${wrapper} pep8 --repeat --show-pep8 --show-source \ ${wrapper} pep8 --repeat --show-pep8 --show-source \
--exclude=vcsversion.py ${srcfiles} | tee pep8.txt --exclude=vcsversion.py ${srcfiles} | tee pep8.txt

1
tests/test_backend_ldap.py
View File

@ -146,4 +146,3 @@ class LDAPIdentity(test.TestCase, test_backend.IdentityTests):
user_api = identity_ldap.UserApi(CONF) user_api = identity_ldap.UserApi(CONF)
self.assertTrue(user_api) self.assertTrue(user_api)
self.assertEquals(user_api.tree_dn, "ou=Users,%s" % CONF.ldap.suffix) self.assertEquals(user_api.tree_dn, "ou=Users,%s" % CONF.ldap.suffix)