[Concept,12/22] buildman: Fix implicit string concatenation in test.py

Message ID 20260106142834.2511220-13-sjg@u-boot.org
State New
Headers
Series buildman: Clean up test.py for pylint compliance |

Commit Message

Simon Glass Jan. 6, 2026, 2:28 p.m. UTC
  From: Simon Glass <simon.glass@canonical.com>

Add missing comma between strings in list to fix pylint W1404 warning.
This was a latent bug where 'Tester' '&' was concatenated to 'Tester&'
instead of being two separate list items.

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

 tools/buildman/test.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Patch

diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index 31458c35f57..75b10a1d219 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -550,7 +550,7 @@  class TestBuild(unittest.TestCase):
     def test_board_two_and(self):
         """Test single board selection"""
         self.assertEqual(self.brds.select_boards(['Tester', '&', 'arm',
-                                                   'Tester' '&', 'powerpc',
+                                                   'Tester', '&', 'powerpc',
                                                    'sandbox']),
                          ({'sandbox': ['board4'],
                           'all': ['board0', 'board1', 'board2', 'board3',