From 04df79b64e5f2296df03579700535774e158f623 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rafael=20Dur=C3=A1n=20Casta=C3=B1eda?= Date: Fri, 6 Jul 2012 00:56:23 +0200 Subject: [PATCH] Run pep8 for tests. 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 --- run_tests.sh | 2 +- tests/test_backend_ldap.py | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/run_tests.sh b/run_tests.sh index 68e028a2ac..a225d78d15 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -108,7 +108,7 @@ function run_pep8 { ignore_dirs="*ajaxterm*" GLOBIGNORE="$ignore_scripts:$ignore_files:$ignore_dirs" srcfiles=`find bin -type f ! -name .*.swp` - srcfiles+=" keystone" + srcfiles+=" keystone tests" # Just run PEP8 in current environment ${wrapper} pep8 --repeat --show-pep8 --show-source \ --exclude=vcsversion.py ${srcfiles} | tee pep8.txt diff --git a/tests/test_backend_ldap.py b/tests/test_backend_ldap.py index f81f0eb985..7dfb29a479 100644 --- a/tests/test_backend_ldap.py +++ b/tests/test_backend_ldap.py @@ -146,4 +146,3 @@ class LDAPIdentity(test.TestCase, test_backend.IdentityTests): user_api = identity_ldap.UserApi(CONF) self.assertTrue(user_api) self.assertEquals(user_api.tree_dn, "ou=Users,%s" % CONF.ldap.suffix) -