Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-17 10:24:49 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

binman: Work around missing test coverage

Browse source 
The iMX8 entry-types don't have proper test coverage. Add a work-around
to skip this for now.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2025-04-10 06:43:04 -06:00 • committed by Tom Rini
parent 06a0d9eee8
commit 2911f2c1ee
2 changed files with 22 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
tools/binman/main.py
View file

@ -94,10 +94,16 @@ def RunTestCoverage(toolpath, build_dir, args):
if toolpath: if toolpath:
for path in toolpath: for path in toolpath:
extra_args += ' --toolpath %s' % path extra_args += ' --toolpath %s' % path
# Some files unfortunately don't thave the required test coverage. This will
# eventually be fixed, but exclude them for now
test_util.run_test_coverage('tools/binman/binman', None, test_util.run_test_coverage('tools/binman/binman', None,
['*test*', '*main.py', 'tools/patman/*', 'tools/dtoc/*', ['*test*', '*main.py', 'tools/patman/*', 'tools/dtoc/*',
'tools/u_boot_pylib/*'], 'tools/u_boot_pylib/*'],
build_dir, all_set, extra_args or None, args=args) build_dir, all_set, extra_args or None, args=args,
allow_failures=['tools/binman/btool/cst.py',
'tools/binman/etype/nxp_imx8mcst.py',
'tools/binman/etype/nxp_imx8mimage.py'])
def RunBinman(args): def RunBinman(args):
"""Main entry point to binman once arguments are parsed """Main entry point to binman once arguments are parsed

16
tools/u_boot_pylib/test_util.py
View file

@ -8,6 +8,7 @@ import doctest
import glob import glob
import multiprocessing import multiprocessing
import os import os
import re
import sys import sys
import unittest import unittest
@ -25,7 +26,7 @@ except:
def run_test_coverage(prog, filter_fname, exclude_list, build_dir, def run_test_coverage(prog, filter_fname, exclude_list, build_dir,
required=None, extra_args=None, single_thread='-P1', required=None, extra_args=None, single_thread='-P1',
args=None): args=None, allow_failures=None):
"""Run tests and check that we get 100% coverage """Run tests and check that we get 100% coverage
Args: Args:
@ -96,6 +97,19 @@ def run_test_coverage(prog, filter_fname, exclude_list, build_dir,
print('Coverage error: %s, but should be 100%%' % coverage) print('Coverage error: %s, but should be 100%%' % coverage)
ok = False ok = False
if not ok: 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') raise ValueError('Test coverage failure')