[Concept,1/9] pickman: Fix line-too-long pylint warnings in ftest

Message ID 20260212211626.167191-2-sjg@u-boot.org
State New
Headers
Series pickman: Improve handling of large merges and add rewind |

Commit Message

Simon Glass Feb. 12, 2026, 9:16 p.m. UTC
  From: Simon Glass <simon.glass@canonical.com>

Break three lines that exceed the 80-column limit.

Co-developed-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/pickman/agent.py |  3 ++-
 tools/pickman/ftest.py | 13 ++++++++-----
 2 files changed, 10 insertions(+), 6 deletions(-)
  

Patch

diff --git a/tools/pickman/agent.py b/tools/pickman/agent.py
index 589d61b727a..129743bb329 100644
--- a/tools/pickman/agent.py
+++ b/tools/pickman/agent.py
@@ -25,7 +25,8 @@  SIGNAL_APPLIED = 'already_applied'
 SIGNAL_CONFLICT = 'conflict'
 
 # Commits that need special handling (regenerate instead of cherry-pick)
-# These run savedefconfig on all boards and depend on target branch Kconfig state
+# These run savedefconfig on all boards and depend on target branch
+# Kconfig state
 QCONFIG_SUBJECTS = [
     'configs: Resync with savedefconfig',
 ]
diff --git a/tools/pickman/ftest.py b/tools/pickman/ftest.py
index d63cfbc6758..049725475f6 100644
--- a/tools/pickman/ftest.py
+++ b/tools/pickman/ftest.py
@@ -3195,7 +3195,8 @@  class TestProcessMergedMrs(unittest.TestCase):
         with terminal.capture():
             dbs = database.Database(self.db_path)
             dbs.start()
-            dbs.source_set('us/next', 'bbb222bbb222bbb222bbb222bbb222bbb222bbb2')
+            dbs.source_set('us/next',
+                              'bbb222bbb222bbb222bbb222bbb222bbb222bbb2')
             dbs.commit()
 
             merged_mrs = [gitlab.PickmanMr(
@@ -3282,7 +3283,8 @@  Date:   Mon Jan 15 10:30:00 2024 -0600
     def test_calc_ratio_different_files(self):
         """Test delta ratio calculation for different files."""
         orig_stats = control.GitStat(2, 15, 3, {'file1.c', 'file2.h'})
-        cherry_stats = control.GitStat(3, 15, 3, {'file1.c', 'file2.h', 'file3.c'})
+        cherry_stats = control.GitStat(
+            3, 15, 3, {'file1.c', 'file2.h', 'file3.c'})
 
         ratio = control.calc_ratio(orig_stats, cherry_stats)
         self.assertGreater(ratio, 0.0)
@@ -3862,8 +3864,9 @@  class TestDoPick(unittest.TestCase):
                                    return_value=(commits, None)):
                 with mock.patch.object(control, 'run_git',
                                        return_value='main'):
-                    with mock.patch.object(control.agent, 'cherry_pick_commits',
-                                           return_value=(True, 'log')) as mock_agent:
+                    with mock.patch.object(
+                            control.agent, 'cherry_pick_commits',
+                            return_value=(True, 'log')) as mock_agent:
                         ret = control.do_pick(args, dbs)
 
             # Verify agent was called with correct branch name
@@ -3931,7 +3934,7 @@  class TestDoPick(unittest.TestCase):
 
             def run_git_handler(args):
                 if '--verify' in args:
-                    raise Exception('branch not found')
+                    raise ValueError('branch not found')
                 return 'main'
 
             with mock.patch.object(control, 'get_commits_for_pick',