mbox series

[Concept,00/22] buildman: Clean up test.py for pylint compliance

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

Message

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

This series addresses pylint warnings and improves code quality in
buildman's test.py file.

Changes include:
- Fix formatting issues (indentation, long lines, f-strings)
- Add missing docstrings and fix existing ones
- Convert CamelCase to snake_case for consistency
- Rename constants to UPPER_CASE
- Split large test classes into smaller, focused classes
- Refactor complex methods to reduce branch count
- Disable unavoidable pylint warnings with descriptive comments


Simon Glass (22):
  buildman: Fix indentation and semicolon in test.py
  buildman: Add __init__() to Options class in test.py
  buildman: Convert to f-strings in test.py
  buildman: Fix import order in test.py
  buildman: Convert CamelCase to snake_case in test.py
  buildman: Add missing docstrings in test.py
  buildman: Fix singleton comparisons in test.py
  buildman: Use tools.write_file() in test.py
  buildman: Use set comprehension in test.py
  buildman: Fix docstring in add_line_prefix()
  buildman: Remove unused variables in test.py
  buildman: Fix implicit string concatenation in test.py
  buildman: Mark unused arguments in test.py
  buildman: Disable protected-access warning in test.py
  buildman: Rename commit parameter to avoid shadowing in test.py
  buildman: Rename constants to UPPER_CASE in test.py
  buildman: Shorten long lines in test.py
  buildman: Disable pylint warnings for Options in test.py
  buildman: Split TestBuild into multiple classes in test.py
  buildman: Refactor _check_output() in test.py
  buildman: Split test_process_limit() into two tests in test.py
  buildman: Disable some final pylint warnings in test.py

 tools/buildman/main.py |   6 +-
 tools/buildman/test.py | 505 ++++++++++++++++++++++++-----------------
 2 files changed, 303 insertions(+), 208 deletions(-)