From: Simon Glass <simon.glass@canonical.com>
Date: Fri, 6 Mar 2026 18:12:50 +0000 (-0700)
Subject: binman: Drop the allow_failures parameter from run_test_coverage()
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d6b10d2021ed772d5b1404d304db5efa3bc755f5;p=thirdparty%2Fu-boot.git

binman: Drop the allow_failures parameter from run_test_coverage()

The allow_failures parameter in run_test_coverage() is no longer used
by any caller.

Drop it.

Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

diff --git a/tools/u_boot_pylib/test_util.py b/tools/u_boot_pylib/test_util.py
index d258a1935c9..364596d7a0c 100644
--- a/tools/u_boot_pylib/test_util.py
+++ b/tools/u_boot_pylib/test_util.py
@@ -24,7 +24,7 @@ except:
 
 def run_test_coverage(prog, filter_fname, exclude_list, build_dir,
                       required=None, extra_args=None, single_thread='-P1',
-                      args=None, allow_failures=None):
+                      args=None):
     """Run tests and check that we get 100% coverage
 
     Args:
@@ -95,19 +95,6 @@ def run_test_coverage(prog, filter_fname, exclude_list, build_dir,
         print('Coverage error: %s, but should be 100%%' % coverage)
         ok = False
     if not ok:
-        if allow_failures:
-            # for line in lines:
-                # print('.', line, re.match(r'^(tools/.*py) *\d+ *(\d+) *(\d+)%$', line))
-            lines = [re.match(r'^(tools/.*py) *\d+ *(\d+) *\d+%$', line)
-                     for line in stdout.splitlines()]
-            bad = []
-            for mat in lines:
-                if mat and mat.group(2) != '0':
-                    fname = mat.group(1)
-                    if fname not in allow_failures:
-                        bad.append(fname)
-            if not bad:
-                return
         raise ValueError('Test coverage failure')