From: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Date: Tue, 15 Feb 2022 07:00:59 +0000 (+0530)
Subject: test-008: make it PEP8 compliant
X-Git-Tag: v3.1.0~308^2~2^2~95
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9ef6f7fc897620ac4f9fddd07327e6cdbffca837;p=thirdparty%2Flibcgroup.git

test-008: make it PEP8 compliant

Make test 008-cgget-multiple_r_flags.py PEP-8 compliant.

Signed-off-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
---

diff --git a/ftests/008-cgget-multiple_r_flags.py b/ftests/008-cgget-multiple_r_flags.py
index b522f491..761da71e 100755
--- a/ftests/008-cgget-multiple_r_flags.py
+++ b/ftests/008-cgget-multiple_r_flags.py
@@ -23,8 +23,8 @@
 from cgroup import Cgroup, CgroupVersion
 import consts
 import ftests
-import os
 import sys
+import os
 
 CONTROLLER = 'memory'
 CGNAME = '008cgget'
@@ -37,12 +37,14 @@ SETTING2_V1 = 'memory.soft_limit_in_bytes'
 SETTING2_V2 = 'memory.high'
 VALUE2 = '1024000'
 
+
 def prereqs(config):
     result = consts.TEST_PASSED
     cause = None
 
     return result, cause
 
+
 def setup(config):
     Cgroup.create(config, CONTROLLER, CGNAME)
 
@@ -55,6 +57,7 @@ def setup(config):
         Cgroup.set(config, CGNAME, SETTING1_V2, VALUE1)
         Cgroup.set(config, CGNAME, SETTING2_V2, VALUE2)
 
+
 def test(config):
     result = consts.TEST_PASSED
     cause = None
@@ -62,34 +65,43 @@ def test(config):
     version = CgroupVersion.get_version(CONTROLLER)
 
     if version == CgroupVersion.CGROUP_V1:
-        settings=[SETTING1_V1, SETTING2_V1]
+        settings = [SETTING1_V1, SETTING2_V1]
     elif version == CgroupVersion.CGROUP_V2:
-        settings=[SETTING1_V2, SETTING2_V2]
+        settings = [SETTING1_V2, SETTING2_V2]
 
     out = Cgroup.get(config, controller=None, cgname=CGNAME, setting=settings)
 
-    if out.splitlines()[0] != "{}:".format(CGNAME):
+    if out.splitlines()[0] != '{}:'.format(CGNAME):
         result = consts.TEST_FAILED
-        cause = "cgget expected the cgroup name {} in the first line.\n" \
-                "Instead it received {}".format(CGNAME, out.splitlines()[0])
+        cause = (
+                    'cgget expected the cgroup name {} in the first line.\n'
+                    'Instead it received {}'
+                    ''.format(CGNAME, out.splitlines()[0])
+                )
 
-    if out.splitlines()[1] != "{}: {}".format(settings[0], VALUE1):
+    if out.splitlines()[1] != '{}: {}'.format(settings[0], VALUE1):
         result = consts.TEST_FAILED
-        cause = "cgget expected the following:\n\t" \
-                "{}: {}\nbut received:\n\t{}".format(
-                settings[0], VALUE1, out.splitlines()[1])
+        cause = (
+                    'cgget expected the following:\n\t{}: {}\n'
+                    'but received:\n\t{}'
+                    ''.format(settings[0], VALUE1, out.splitlines()[1])
+                )
 
-    if out.splitlines()[2] != "{}: {}".format(settings[1], VALUE2):
+    if out.splitlines()[2] != '{}: {}'.format(settings[1], VALUE2):
         result = consts.TEST_FAILED
-        cause = "cgget expected the following:\n\t" \
-                "{}: {}\nbut received:\n\t{}".format(
-                settings[1], VALUE2, out.splitlines()[2])
+        cause = (
+                    'cgget expected the following:\n\t{}: {}\n'
+                    'but received:\n\t{}'
+                    ''.format(settings[1], VALUE2, out.splitlines()[2])
+                )
 
     return result, cause
 
+
 def teardown(config):
     Cgroup.delete(config, CONTROLLER, CGNAME)
 
+
 def main(config):
     [result, cause] = prereqs(config)
     if result != consts.TEST_PASSED:
@@ -101,6 +113,7 @@ def main(config):
 
     return [result, cause]
 
+
 if __name__ == '__main__':
     config = ftests.parse_args()
     # this test was invoked directly.  run only it