From 3555240db39186cd4c47fc54417614252ededb06 Mon Sep 17 00:00:00 2001 From: Kui Shi Date: Thu, 12 Sep 2013 01:04:09 +0800 Subject: [PATCH] Don't need to init testr in run_tests.sh In run_tests.sh, function init_testr will initialize testr if the directory .testrepository is not existed. Actually, testr will do the check before run the test: In Python package testrepository, setuptools_command.py:Testr.run 68 def run(self): 69 """Set up testr repo, then run testr""" 70 if not os.path.isdir(".testrepository"): 71 self._run_testr("init") So, init_testr can be removed safely. Fixes Bug #1224285 Change-Id: Ia74d9fff92b8efba9b72e5c7387f761144daa193 --- run_tests.sh | 6 ------ 1 file changed, 6 deletions(-) diff --git a/run_tests.sh b/run_tests.sh index 78eb99a6c..cc171246c 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -102,11 +102,6 @@ if [ $no_site_packages -eq 1 ]; then installvenvopts="--no-site-packages" fi -function init_testr { - if [ ! -d .testrepository ]; then - ${wrapper} testr init - fi -} function run_tests { # Cleanup *pyc @@ -213,7 +208,6 @@ if [ $recreate_db -eq 1 ]; then rm -f tests.sqlite fi -init_testr run_tests # NOTE(sirp): we only want to run pep8 when we're running the full-test suite, -- 2.45.2