From c8fe72456073204c59fd527448dc2539c70f1939 Mon Sep 17 00:00:00 2001 From: Benjamin Drung Date: Wed, 3 Dec 2025 13:51:50 +0100 Subject: [PATCH] Fix pylint in ubuntutools/test/test_requestsync.py pylint complains: ``` ubuntutools/test/test_requestsync.py:31:12: C0415: Import outside toplevel (keyring) (import-outside-toplevel) ubuntutools/test/test_requestsync.py:33:12: W0707: Consider explicitly re-raising using 'except ModuleNotFoundError as exc' and 'raise ModuleNotFoundError('package python3-keyring is not installed') from exc' (raise-missing-from) ubuntutools/test/test_requestsync.py:31:12: W0611: Unused import keyring (unused-import) ``` --- ubuntutools/test/test_requestsync.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ubuntutools/test/test_requestsync.py b/ubuntutools/test/test_requestsync.py index d30dad0..b4c26f3 100644 --- a/ubuntutools/test/test_requestsync.py +++ b/ubuntutools/test/test_requestsync.py @@ -28,6 +28,7 @@ class BinaryTests(unittest.TestCase): def test_keyring_installed(self): """Smoke test for required lp api dependencies""" try: + # pylint: disable-next=import-outside-toplevel,unused-import import keyring # noqa: F401 - except ModuleNotFoundError: - raise ModuleNotFoundError("package python3-keyring is not installed") + except ModuleNotFoundError as error: + raise ModuleNotFoundError("package python3-keyring is not installed") from error