<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"> <html><head><meta name="qrichtext" content="1" /><style type="text/css"> p, li { white-space: pre-wrap; } </style></head><body style=" font-family:'DejaVu Sans Mono'; font-size:9pt; font-weight:400; font-style:normal;"> On Wednesday, April 02, 2014 16:12:23 Thomas Wood wrote: > Signed-off-by: Thomas Wood <thomas.wood@intel.com> > --- > tests/igt.py | 41 +++++++++++++++++++++++++---------------- > 1 file changed, 25 insertions(+), 16 deletions(-) > > diff --git a/tests/igt.py b/tests/igt.py > index f80a6c4..4640814 100644 > --- a/tests/igt.py > +++ b/tests/igt.py > @@ -98,21 +98,30 @@ class IGTTest(ExecTest): > > def listTests(listname): > oldDir = os.getcwd() > + > + lines = None I have a comment below about this. > try: > - os.chdir(igtTestRoot) > - proc = subprocess.Popen( > - ['make', listname ], > - stdout=subprocess.PIPE, > - stderr=subprocess.PIPE, > - env=os.environ.copy(), > - universal_newlines=True > - ) > - out, err = proc.communicate() > - returncode = proc.returncode > - finally: > - os.chdir(oldDir) > - > - lines = out.split('\n') > + with open(path.join(igtTestRoot, listname + '.txt')) as f: you should set a mode for this. Probably 'r' > + lines = [ line.rstrip() for line in f.readlines() ] Python convention is to never have spaces on the inside of brackets. Also since you only iterate over lines once, why not use a generator: lines = (l.rstrip() for l in f.readlines()) > + except IOError: > + pass rather than setting lines to None and doing an if check why not make the block below the except IOError instead of pass, it would allow you to cut the if check and assigning lines = None above. > + > + if lines is None: > + try: > + os.chdir(igtTestRoot) > + proc = subprocess.Popen( > + ['make', 'list-' + listname ], no space inside the bracket after listname > + stdout=subprocess.PIPE, > + stderr=subprocess.PIPE, > + env=os.environ.copy(), > + universal_newlines=True > + ) > + out, err = proc.communicate() > + lines = out.split('\n') suggestion: I think it's cleaner to use .splitlines() than .split('\n') > + returncode = proc.returncode with the way you're using Popen I'd suggest using subprocess.check_output, a convenience wrapper for Popen: lines = subprocess.check_output(['make', 'list-' + listname], env=os.environ.copy(), universal_newlnies=True).splitlines() I recommend this since you don't actually use err or returncode at all. > + finally: > + os.chdir(oldDir) > + > found_header = False > progs = "" > > @@ -126,7 +135,7 @@ def listTests(listname): > > return progs > > -singleTests = listTests("list-single-tests") > +singleTests = listTests("single-tests") > > for test in singleTests: > profile.test_list[path.join('igt', test)] = IGTTest(test) > @@ -150,7 +159,7 @@ def addSubTestCases(test): > profile.test_list[path.join('igt', test, subtest)] = \ > IGTTest(test, ['--run-subtest', subtest]) > > -multiTests = listTests("list-multi-tests") > +multiTests = listTests("multi-tests") > > for test in multiTests: > addSubTestCases(test) </body></html>