public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH 1/1] test: fix some pylint errors in test_efi_secboot
@ 2022-10-01 18:57 Heinrich Schuchardt
  2022-10-05  7:02 ` Ilias Apalodimas
  0 siblings, 1 reply; 2+ messages in thread
From: Heinrich Schuchardt @ 2022-10-01 18:57 UTC (permalink / raw)
  To: Ilias Apalodimas; +Cc: u-boot, Heinrich Schuchardt

* Remove unused import
* Provide module docstring

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 test/py/tests/test_efi_secboot/conftest.py | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/test/py/tests/test_efi_secboot/conftest.py b/test/py/tests/test_efi_secboot/conftest.py
index db6b8d301f..406131cb45 100644
--- a/test/py/tests/test_efi_secboot/conftest.py
+++ b/test/py/tests/test_efi_secboot/conftest.py
@@ -2,18 +2,12 @@
 # Copyright (c) 2019, Linaro Limited
 # Author: AKASHI Takahiro <takahiro.akashi@linaro.org>
 
-import os
-import os.path
-from subprocess import call, check_call, check_output, CalledProcessError
+""" Fixture for UEFI secure boot test """
+
+from subprocess import call, check_call, CalledProcessError
 import pytest
 from defs import *
 
-
-#
-# Fixture for UEFI secure boot test
-#
-
-
 @pytest.fixture(scope='session')
 def efi_boot_env(request, u_boot_config):
     """Set up a file system to be used in UEFI secure boot test.
-- 
2.37.2


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/1] test: fix some pylint errors in test_efi_secboot
  2022-10-01 18:57 [PATCH 1/1] test: fix some pylint errors in test_efi_secboot Heinrich Schuchardt
@ 2022-10-05  7:02 ` Ilias Apalodimas
  0 siblings, 0 replies; 2+ messages in thread
From: Ilias Apalodimas @ 2022-10-05  7:02 UTC (permalink / raw)
  To: Heinrich Schuchardt; +Cc: u-boot

On Sat, Oct 01, 2022 at 08:57:09PM +0200, Heinrich Schuchardt wrote:
> * Remove unused import
> * Provide module docstring
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  test/py/tests/test_efi_secboot/conftest.py | 12 +++---------
>  1 file changed, 3 insertions(+), 9 deletions(-)
> 
> diff --git a/test/py/tests/test_efi_secboot/conftest.py b/test/py/tests/test_efi_secboot/conftest.py
> index db6b8d301f..406131cb45 100644
> --- a/test/py/tests/test_efi_secboot/conftest.py
> +++ b/test/py/tests/test_efi_secboot/conftest.py
> @@ -2,18 +2,12 @@
>  # Copyright (c) 2019, Linaro Limited
>  # Author: AKASHI Takahiro <takahiro.akashi@linaro.org>
>  
> -import os
> -import os.path
> -from subprocess import call, check_call, check_output, CalledProcessError
> +""" Fixture for UEFI secure boot test """
> +
> +from subprocess import call, check_call, CalledProcessError
>  import pytest
>  from defs import *
>  
> -
> -#
> -# Fixture for UEFI secure boot test
> -#
> -
> -
>  @pytest.fixture(scope='session')
>  def efi_boot_env(request, u_boot_config):
>      """Set up a file system to be used in UEFI secure boot test.
> -- 
> 2.37.2
> 

Acked-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-10-05  7:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-01 18:57 [PATCH 1/1] test: fix some pylint errors in test_efi_secboot Heinrich Schuchardt
2022-10-05  7:02 ` Ilias Apalodimas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox