commit 881041f28dc5545605791970a83e213fe40323fe
author Andrew Jeffery <andrew@aj.id.au> Fri Mar 13 11:46:30 2020 +1030
committer Andrew Geissler <geissonator@yahoo.com> Wed Mar 18 22:57:10 2020 +0000
tree 0831c54630dbc40c483a2f55c980e7923f84959a
parent 7d4a26f9408c2c35bd940f633e3d21bc17ccd950

unit-test: Fix PEP8 E201 and E202 violations

E201: whitespace after '{'
E202: whitespace before '}'

Change-Id: Ie3cc66684ef5d30d6fdadb8fbe0cc083ba8cf7c2
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

scripts/unit-test.py

diff --git a/scripts/unit-test.py b/scripts/unit-test.py
index 432b7fb..3f5096e 100755
--- a/scripts/unit-test.py
+++ b/scripts/unit-test.py
@@ -264,7 +264,7 @@
     target              The make target we are checking
     """
     try:
-        cmd = [ 'make', '-n', target ]
+        cmd = ['make', '-n', target]
         with open(os.devnull, 'w') as devnull:
             check_call(cmd, stdout=devnull, stderr=devnull)
         return True
@@ -465,7 +465,7 @@
         return
 
     try:
-        cmd = make_parallel + [ 'check-valgrind' ]
+        cmd = make_parallel + ['check-valgrind']
         check_call_cmd(*cmd)
     except CalledProcessError:
         for root, _, files in os.walk(os.getcwd()):
@@ -486,7 +486,7 @@
 
     # Actually run code coverage
     try:
-        cmd = make_parallel + [ 'check-code-coverage' ]
+        cmd = make_parallel + ['check-code-coverage']
         check_call_cmd(*cmd)
     except CalledProcessError:
         raise Exception('Code coverage failed')
@@ -676,11 +676,11 @@
         check_call_cmd(*make_parallel)
 
     def install(self):
-        check_call_cmd('sudo', '-n', '--', *(make_parallel + [ 'install' ]))
+        check_call_cmd('sudo', '-n', '--', *(make_parallel + ['install']))
 
     def test(self):
         try:
-            cmd = make_parallel + [ 'check' ]
+            cmd = make_parallel + ['check']
             for i in range(0, args.repeat):
                 check_call_cmd(*cmd)
         except CalledProcessError:
@@ -922,20 +922,20 @@
 
 class Package(object):
     def __init__(self, name=None, path=None):
-        self.supported = [ Autotools, Meson, CMake ]
+        self.supported = [Autotools, Meson, CMake]
         self.name = name
         self.path = path
         self.test_only = False
 
     def build_systems(self):
-        instances = ( system(self.name, self.path) for system in self.supported )
-        return ( instance for instance in instances if instance.probe() )
+        instances = (system(self.name, self.path) for system in self.supported)
+        return (instance for instance in instances if instance.probe())
 
     def build_system(self, preferred=None):
         systems = self.build_systems()
 
         if preferred:
-            return { type(system): system for system in systems }[preferred]
+            return {type(system): system for system in systems}[preferred]
 
         return next(iter(systems))