diff mbox series

[PATCH/next] test_lxc.py: fix code style

Message ID 1574713802-20461-1-git-send-email-pjtexier@koncepto.io
State Accepted
Headers show
Series [PATCH/next] test_lxc.py: fix code style | expand

Commit Message

Pierre-Jean Texier Nov. 25, 2019, 8:30 p.m. UTC 
Fix these warnings:

E122 continuation line missing indentation or outdented
E127 continuation line over-indented for visual indent
E265 block comment should start with '# '
E302 expected 2 blank lines, found 1
F401 'pexpect' imported but unused

Fixes:
 - https://gitlab.com/buildroot.org/buildroot/-/jobs/360824861

Signed-off-by: Pierre-Jean Texier <pjtexier@koncepto.io>
---
 support/testing/tests/package/test_lxc.py | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Thomas Petazzoni Nov. 25, 2019, 8:50 p.m. UTC | #1 
On Mon, 25 Nov 2019 21:30:01 +0100
Pierre-Jean Texier <pjtexier@koncepto.io> wrote:

> Fix these warnings:
> 
> E122 continuation line missing indentation or outdented
> E127 continuation line over-indented for visual indent
> E265 block comment should start with '# '
> E302 expected 2 blank lines, found 1
> F401 'pexpect' imported but unused
> 
> Fixes:
>  - https://gitlab.com/buildroot.org/buildroot/-/jobs/360824861
> 
> Signed-off-by: Pierre-Jean Texier <pjtexier@koncepto.io>
> ---
>  support/testing/tests/package/test_lxc.py | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)

Applied to next, thanks.

Thomas
diff mbox series

Patch

diff --git a/support/testing/tests/package/test_lxc.py b/support/testing/tests/package/test_lxc.py
index 151bcf1..346247d 100644
--- a/support/testing/tests/package/test_lxc.py
+++ b/support/testing/tests/package/test_lxc.py
@@ -1,7 +1,7 @@ 
 import os
 
 import infra.basetest
-import pexpect
+
 
 class TestLxc(infra.basetest.BRTest):
     config = \
@@ -23,24 +23,24 @@  class TestLxc(infra.basetest.BRTest):
             BR2_ROOTFS_OVERLAY="{}"
             BR2_TARGET_ROOTFS_CPIO=y
             """.format(
-            infra.filepath("tests/package/test_lxc/lxc-kernel.config"),
-            infra.filepath("tests/package/test_lxc/rootfs-overlay"))
+              infra.filepath("tests/package/test_lxc/lxc-kernel.config"),
+              infra.filepath("tests/package/test_lxc/rootfs-overlay"))
 
     def run_ok(self, cmd):
         self.assertRunOk(cmd, 120)
 
     def wait_boot(self):
-        #the complete boot with systemd takes more time than what the default multipler permits
+        # the complete boot with systemd takes more time than what the default multipler permits
         self.emulator.timeout_multiplier *= 10
         self.emulator.login()
 
     def setup_run_test_container(self):
         self.run_ok("lxc-create -n lxc_iperf3 -t none -f /usr/share/lxc/config/minimal-iperf3.conf")
         self.run_ok("lxc-start -l trace -n lxc_iperf3 -o /tmp/lxc.log -L /tmp/lxc.console.log")
-        #need to wait for the container to be fully started
+        # need to wait for the container to be fully started
         self.run_ok("sleep 2")
         self.run_ok("iperf3 -c 192.168.1.2 -t 2")
-        #if the test fails, just cat /tmp/*.log
+        # if the test fails, just cat /tmp/*.log
 
     def test_run(self):
         cpio_file = os.path.join(self.builddir, "images", "rootfs.cpio")
@@ -50,7 +50,7 @@  class TestLxc(infra.basetest.BRTest):
                            kernel_cmdline=[
                                         "console=ttyAMA0,115200"],
                            options=["-initrd", cpio_file,
-                                        "-dtb", dtb_file,
-                                        "-M", "vexpress-a9"])
+                                    "-dtb", dtb_file,
+                                    "-M", "vexpress-a9"])
         self.wait_boot()
         self.setup_run_test_container()