diff options
author | Dylan Baker <baker.dylan.c@gmail.com> | 2015-01-17 21:54:22 -0800 |
---|---|---|
committer | Dylan Baker <baker.dylan.c@gmail.com> | 2015-03-09 10:23:38 -0700 |
commit | a32b93c629880e86c306f0645e80f917f299db28 (patch) | |
tree | d0eddee7e4f4cce9c07bd992f409732d94ed20e5 | |
parent | 7f1704aa49222c9f13d63e914a2b2284c1c3a5db (diff) |
igt.py: Fix whitepace warnings for pylint
v2: - Also remove stderr from subprocess.Popen
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Thomas Wood <thomas.wood@intel.com>
-rw-r--r-- | tests/igt.py | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/tests/igt.py b/tests/igt.py index bd4c70eb2..d54bb65c5 100644 --- a/tests/igt.py +++ b/tests/igt.py @@ -124,22 +124,20 @@ tests.extend(listTests("multi-tests")) def addSubTestCases(test): proc = subprocess.Popen( - [os.path.join(igtTestRoot, test), '--list-subtests'], - stdout=subprocess.PIPE, - stderr=subprocess.PIPE, - env=os.environ.copy(), - universal_newlines=True - ) - out, err = proc.communicate() + [os.path.join(igtTestRoot, test), '--list-subtests'], + stdout=subprocess.PIPE, + env=os.environ.copy(), + universal_newlines=True) + out, _ = proc.communicate() # a return code of 79 indicates there are no subtests if proc.returncode == 79: - profile.test_list[grouptools.join('igt', test)] = IGTTest(test) - return + profile.test_list[grouptools.join('igt', test)] = IGTTest(test) + return if proc.returncode != 0: - print "Error: Could not list subtests for " + test - return + print "Error: Could not list subtests for " + test + return subtests = out.split("\n") |