diff mbox series

[Concept,1/3] buildman: Fix coverage test configuration

Message ID 20251222231838.2298178-2-sjg@u-boot.org
State New
Headers
Series buildman: Enable code coverage in CI |

Commit Message

Simon Glass Dec. 22, 2025, 11:18 p.m. UTC 
  From: Simon Glass <simon.glass@canonical.com>

Exclude tools/qconfig.py from coverage since it's a separate tool with
its own tests. Add allow_failures for core buildman modules that don't
have 100% test coverage yet (builder.py, builderthread.py, cfgutil.py,
control.py, toolchain.py).

This fixes the --coverage test returning error code 1.

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

 tools/buildman/main.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/tools/buildman/main.py b/tools/buildman/main.py
index 3c9820b262a..22be946b37d 100755
--- a/tools/buildman/main.py
+++ b/tools/buildman/main.py
@@ -63,8 +63,13 @@  def run_test_coverage():
         'tools/buildman/buildman', None,
         ['tools/patman/*.py', 'tools/u_boot_pylib/*', '*test_fdt.py',
          'tools/buildman/kconfiglib.py', 'tools/buildman/*test*.py',
-         'tools/buildman/main.py'],
-        '/tmp/b', single_thread='-T1')
+         'tools/buildman/main.py', 'tools/qconfig.py'],
+        '/tmp/b', single_thread='-T1',
+        allow_failures=['tools/buildman/builder.py',
+                        'tools/buildman/builderthread.py',
+                        'tools/buildman/cfgutil.py',
+                        'tools/buildman/control.py',
+                        'tools/buildman/toolchain.py'])
 
 
 def run_buildman():