[test-suite] r341225 - litsupport: fix pycodestyle/pyflakes reports; NFC

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 31 09:23:08 PDT 2018


Author: matze
Date: Fri Aug 31 09:23:08 2018
New Revision: 341225

URL: http://llvm.org/viewvc/llvm-project?rev=341225&view=rev
Log:
litsupport: fix pycodestyle/pyflakes reports; NFC

Modified:
    test-suite/trunk/litsupport/modules/codesize.py
    test-suite/trunk/litsupport/modules/hash.py
    test-suite/trunk/litsupport/modules/microbenchmark.py
    test-suite/trunk/litsupport/modules/remote.py
    test-suite/trunk/litsupport/modules/timeit.py
    test-suite/trunk/litsupport/shellcommand.py
    test-suite/trunk/litsupport/test.py
    test-suite/trunk/litsupport/testfile.py
    test-suite/trunk/litsupport/testplan.py

Modified: test-suite/trunk/litsupport/modules/codesize.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/modules/codesize.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/modules/codesize.py (original)
+++ test-suite/trunk/litsupport/modules/codesize.py Fri Aug 31 09:23:08 2018
@@ -22,13 +22,13 @@ def _getCodeSize(context):
             logging.warning("Unexpected output from llvm-size on '%s'",
                             context.executable)
         else:
-            for l in lines[2:]:
-                l = l.strip()
-                if l == "":
+            for line in lines[2:]:
+                line = line.strip()
+                if line == "":
                     continue
-                values = l.split()
+                values = line.split()
                 if len(values) < 2:
-                    logging.info("Ignoring malformed output line: %s", l)
+                    logging.info("Ignoring malformed output line: %s", line)
                     continue
                 if values[0] == 'Total':
                     continue
@@ -36,8 +36,8 @@ def _getCodeSize(context):
                     name = values[0]
                     val = int(values[1])
                     metrics['size.%s' % name] = val
-                except ValueError as e:
-                    logging.info("Ignoring malformed output line: %s", l)
+                except ValueError:
+                    logging.info("Ignoring malformed output line: %s", line)
 
     return metrics
 

Modified: test-suite/trunk/litsupport/modules/hash.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/modules/hash.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/modules/hash.py (original)
+++ test-suite/trunk/litsupport/modules/hash.py Fri Aug 31 09:23:08 2018
@@ -1,5 +1,4 @@
 """Test module to collect test executable hashsum."""
-from litsupport import shellcommand
 from litsupport import testplan
 import hashlib
 import logging
@@ -24,7 +23,7 @@ def compute(context):
         h = hashlib.md5()
         h.update(open(executable, 'rb').read())
         context.executable_hash = h.hexdigest()
-    except:
+    except Exception:
         logging.info('Could not calculate hash for %s' % executable)
         context.executable_hash = ''
 

Modified: test-suite/trunk/litsupport/modules/microbenchmark.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/modules/microbenchmark.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/modules/microbenchmark.py (original)
+++ test-suite/trunk/litsupport/modules/microbenchmark.py Fri Aug 31 09:23:08 2018
@@ -36,13 +36,17 @@ def _collectMicrobenchmarkTime(context,
                 microBenchmark = lit.Test.Result(lit.Test.PASS)
 
                 # Index 3 is cpu_time
-                microBenchmark.addMetric('exec_time', lit.Test.toMetricValue(float(line[3])))
-              
-                # Add Micro Result 
+                exec_time_metric = lit.Test.toMetricValue(float(line[3]))
+                microBenchmark.addMetric('exec_time', exec_time_metric)
+
+                # Add Micro Result
                 context.micro_results[name] = microBenchmark
 
-    # returning the number of microbenchmarks collected as a metric for the base test
-    return ({'MicroBenchmarks': lit.Test.toMetricValue(len(context.micro_results))})
+    # returning the number of microbenchmarks collected as a metric for the
+    # base test
+    return ({
+        'MicroBenchmarks': lit.Test.toMetricValue(len(context.micro_results))
+    })
 
 
 def mutatePlan(context, plan):

Modified: test-suite/trunk/litsupport/modules/remote.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/modules/remote.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/modules/remote.py (original)
+++ test-suite/trunk/litsupport/modules/remote.py Fri Aug 31 09:23:08 2018
@@ -20,7 +20,8 @@ def _mutateCommandline(context, commandl
 
 
 def _mutateScript(context, script, suffix=""):
-    mutate = lambda c, cmd: _mutateCommandline(c, cmd, suffix)
+    def mutate(context, command):
+        return _mutateCommandline(context, command, suffix)
     return testplan.mutateScript(context, script, mutate)
 
 

Modified: test-suite/trunk/litsupport/modules/timeit.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/modules/timeit.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/modules/timeit.py (original)
+++ test-suite/trunk/litsupport/modules/timeit.py Fri Aug 31 09:23:08 2018
@@ -81,9 +81,8 @@ def mutatePlan(context, plan):
 def getUserTime(filename):
     """Extract the user time form a .time file produced by timeit"""
     with open(filename) as fd:
-        l = [l for l in fd.readlines()
-             if l.startswith('user')]
-    assert len(l) == 1
+        line = [line for line in fd.readlines() if line.startswith('user')]
+    assert len(line) == 1
 
-    m = re.match(r'user\s+([0-9.]+)', l[0])
+    m = re.match(r'user\s+([0-9.]+)', line[0])
     return float(m.group(1))

Modified: test-suite/trunk/litsupport/shellcommand.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/shellcommand.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/shellcommand.py (original)
+++ test-suite/trunk/litsupport/shellcommand.py Fri Aug 31 09:23:08 2018
@@ -5,7 +5,7 @@ import re
 import os
 try:
         from shlex import quote  # python 3.3 and above
-except:
+except ImportError:
         from pipes import quote  # python 3.2 and earlier
 
 
@@ -35,7 +35,6 @@ class ShellCommand:
         res_list = [self.executable] + self.arguments
         result += " ".join(map(quote, res_list))
 
-        envlist = []
         for key, value in self.envvars.items():
             result += "%s=%s " % (key, quote(value))
 
@@ -71,7 +70,6 @@ def parse(commandline):
     and switching directories upfront. It does not support full posix shell
     and will throw an exception if the commandline uses unsupported features.
     """
-    previous_commands = []
     result = ShellCommand()
     tokens = shlex.split(commandline)
     i = 0
@@ -139,7 +137,7 @@ def getMainExecutable(context):
         return context.executable
 
     executable = None
-    cwd = '.';
+    cwd = '.'
     for line in context.parsed_runscript:
         cmd = parse(line)
         if cmd.workdir is not None:

Modified: test-suite/trunk/litsupport/test.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/test.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/test.py (original)
+++ test-suite/trunk/litsupport/test.py Fri Aug 31 09:23:08 2018
@@ -10,7 +10,6 @@ import litsupport.modules
 import litsupport.modules.hash
 import litsupport.testfile
 import litsupport.testplan
-import logging
 import os
 
 
@@ -38,9 +37,9 @@ class TestSuiteTest(lit.formats.ShTest):
     def execute(self, test, litConfig):
         config = test.config
         if config.unsupported:
-            return lit.Test.Result(Test.UNSUPPORTED, 'Test is unsupported')
+            return lit.Test.Result(lit.Test.UNSUPPORTED, 'Test is unsupported')
         if litConfig.noExecute:
-            return lit.Test.Result(Test.PASS)
+            return lit.Test.Result(lit.Test.PASS)
 
         # Parse .test file and initialize context
         tmpDir, tmpBase = lit.TestRunner.getTempPaths(test)

Modified: test-suite/trunk/litsupport/testfile.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/testfile.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/testfile.py (original)
+++ test-suite/trunk/litsupport/testfile.py Fri Aug 31 09:23:08 2018
@@ -2,6 +2,7 @@
 from lit.TestRunner import parseIntegratedTestScriptCommands, \
         getDefaultSubstitutions, applySubstitutions
 from litsupport import shellcommand
+import logging
 
 
 def _parseShellCommand(script, ln):

Modified: test-suite/trunk/litsupport/testplan.py
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/litsupport/testplan.py?rev=341225&r1=341224&r2=341225&view=diff
==============================================================================
--- test-suite/trunk/litsupport/testplan.py (original)
+++ test-suite/trunk/litsupport/testplan.py Fri Aug 31 09:23:08 2018
@@ -1,7 +1,6 @@
 """
 Datastructures for test plans; Parsing of .test files; Executing test plans.
 """
-from litsupport import shellcommand
 import lit.Test
 import lit.TestRunner
 import logging




More information about the llvm-commits mailing list