From 4d62f22f12a0c2cdf3ec246d599e378496f1f116 Mon Sep 17 00:00:00 2001 From: Claude Date: Sat, 15 Aug 2026 05:37:34 +0000 Subject: [PATCH] fix: move pylint disable comments to correct lines in test files The `# pylint: disable=import-outside-toplevel` comments were placed on the closing `)` line of multi-line imports, but pylint matches inline disables to the line where the warning is triggered (the `from` line). Moving the comment to the opening `from (` line silences C0415 correctly. Co-Authored-By: Claude Sonnet 4.6 Claude-Session: https://claude.ai/code/session_01VjiDW5hoDgJ6GYZ7tUozim --- tests/test_compat.py | 16 ++++++++-------- tests/test_widgets.py | 16 ++++++++-------- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/tests/test_compat.py b/tests/test_compat.py index 90dbc98..db7f23b 100644 --- a/tests/test_compat.py +++ b/tests/test_compat.py @@ -11,9 +11,9 @@ class TestCompatFunctions(unittest.TestCase): @patch("python_logo_widgets._compat.tk.Tk") def test_logo_widget(self, mock_tk, mock_cls): """Test the logo compatibility wrapper.""" - from python_logo_widgets._compat import ( + from python_logo_widgets._compat import ( # pylint: disable=import-outside-toplevel logo_widget, - ) # pylint: disable=import-outside-toplevel + ) mock_root = MagicMock() mock_tk.return_value = mock_root @@ -26,9 +26,9 @@ def test_logo_widget(self, mock_tk, mock_cls): @patch("python_logo_widgets._compat.tk.Tk") def test_length_widget(self, mock_tk, mock_cls): """Test the length compatibility wrapper.""" - from python_logo_widgets._compat import ( + from python_logo_widgets._compat import ( # pylint: disable=import-outside-toplevel length_widget, - ) # pylint: disable=import-outside-toplevel + ) mock_root = MagicMock() mock_tk.return_value = mock_root @@ -41,9 +41,9 @@ def test_length_widget(self, mock_tk, mock_cls): @patch("python_logo_widgets._compat.tk.Tk") def test_width_widget(self, mock_tk, mock_cls): """Test the width compatibility wrapper.""" - from python_logo_widgets._compat import ( + from python_logo_widgets._compat import ( # pylint: disable=import-outside-toplevel width_widget, - ) # pylint: disable=import-outside-toplevel + ) mock_root = MagicMock() mock_tk.return_value = mock_root @@ -64,9 +64,9 @@ def test_demo_creates_all_widgets( self, mock_tk, mock_logo, mock_length, mock_width ): """Test the demo launcher creates all widgets.""" - from python_logo_widgets._demo import ( + from python_logo_widgets._demo import ( # pylint: disable=import-outside-toplevel demo, - ) # pylint: disable=import-outside-toplevel + ) mock_root = MagicMock() mock_tk.return_value = mock_root diff --git a/tests/test_widgets.py b/tests/test_widgets.py index 5c81c0b..0467726 100644 --- a/tests/test_widgets.py +++ b/tests/test_widgets.py @@ -12,9 +12,9 @@ class TestLogoWidget(unittest.TestCase): @patch("python_logo_widgets.widgets._load_image", return_value="fake.gif") def test_creates_with_parent(self, mock_load, mock_photo, _mock_label): """Test widget creation with a parent.""" - from python_logo_widgets import ( + from python_logo_widgets import ( # pylint: disable=import-outside-toplevel LogoWidget, - ) # pylint: disable=import-outside-toplevel + ) parent = MagicMock() widget = LogoWidget(parent) @@ -27,9 +27,9 @@ def test_creates_with_parent(self, mock_load, mock_photo, _mock_label): @patch("python_logo_widgets.widgets._load_image", return_value="fake.gif") def test_custom_bg(self, _mock_load, _mock_photo, mock_label): """Test custom background propagation.""" - from python_logo_widgets import ( + from python_logo_widgets import ( # pylint: disable=import-outside-toplevel LogoWidget, - ) # pylint: disable=import-outside-toplevel + ) parent = MagicMock() LogoWidget(parent, bg="white") @@ -46,9 +46,9 @@ class TestPoweredByLengthWidget(unittest.TestCase): @patch("python_logo_widgets.widgets._load_image", return_value="fake.gif") def test_creates_with_parent(self, mock_load, _mock_photo, _mock_label): """Test powered-by-length widget creation with a parent.""" - from python_logo_widgets import ( + from python_logo_widgets import ( # pylint: disable=import-outside-toplevel PoweredByLengthWidget, - ) # pylint: disable=import-outside-toplevel + ) parent = MagicMock() widget = PoweredByLengthWidget(parent) @@ -64,9 +64,9 @@ class TestPoweredByWidthWidget(unittest.TestCase): @patch("python_logo_widgets.widgets._load_image", return_value="fake.gif") def test_creates_with_parent(self, mock_load, _mock_photo, _mock_label): """Test powered-by-width widget creation with a parent.""" - from python_logo_widgets import ( + from python_logo_widgets import ( # pylint: disable=import-outside-toplevel PoweredByWidthWidget, - ) # pylint: disable=import-outside-toplevel + ) parent = MagicMock() widget = PoweredByWidthWidget(parent)