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

test-002: make it PEP8 compliant

Make test 002-cgdelete-recursive_delete.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/002-cgdelete-recursive_delete.py b/ftests/002-cgdelete-recursive_delete.py
index a9387590..31b5bc74 100755
--- a/ftests/002-cgdelete-recursive_delete.py
+++ b/ftests/002-cgdelete-recursive_delete.py
@@ -23,19 +23,20 @@
 from cgroup import Cgroup, CgroupVersion
 import consts
 import ftests
-import os
-from process import Process
 import sys
+import os
 
 CONTROLLER = 'cpuset'
 PARENT = '002cgdelete'
 CHILD = 'childcg'
 GRANDCHILD = 'grandchildcg'
 
+
 def prereqs(config):
     # This test should run on both cgroup v1 and v2
     return consts.TEST_PASSED, None
 
+
 def setup(config):
     Cgroup.create(config, CONTROLLER, PARENT)
     Cgroup.create(config, CONTROLLER, os.path.join(PARENT, CHILD))
@@ -49,17 +50,21 @@ def setup(config):
         # Moving a process from a child cgroup to its parent isn't (easily)
         # supported in cgroup v2 because of cgroup v2's restriction that
         # processes only be located in leaf cgroups
+        grandchild_cgrp_path = os.path.join(PARENT, CHILD, GRANDCHILD)
         config.process.create_process_in_cgroup(config, CONTROLLER,
-                                         os.path.join(PARENT, CHILD, GRANDCHILD))
+                                                grandchild_cgrp_path)
+
 
 def test(config):
     Cgroup.delete(config, CONTROLLER, PARENT, recursive=True)
 
     return consts.TEST_PASSED, None
 
+
 def teardown(config):
     pass
 
+
 def main(config):
     [result, cause] = prereqs(config)
     if result != consts.TEST_PASSED:
@@ -71,6 +76,7 @@ def main(config):
 
     return [result, cause]
 
+
 if __name__ == '__main__':
     config = ftests.parse_args()
     # this test was invoked directly.  run only it