random-mirrors/smaeul-u-boot
1
0
Fork 0
mirror of https://github.com/smaeul/u-boot.git synced 2025-11-26 22:01:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

patman: hide the 'test' command unless test data is available

Browse Source 
Some tests would fail when the test data is not available, so it
doesn't make much sense to expose the action when patman is running
outside of the u-boot git checkout.

Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Maxim Cournoyer <maxim.cournoyer@savoirfairelinux.com>
This commit is contained in:
Maxim Cournoyer 2022-12-19 17:32:43 -05:00 committed by Simon Glass
parent a3997a0a00
commit 52c1c333ea
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tools/patman/__main__.py
View File

@ -20,6 +20,7 @@ if __name__ == "__main__":
# Our modules
from patman import control
from patman import func_test
from patman import gitutil
from patman import project
from patman import settings
@ -95,9 +96,11 @@ send.add_argument('--smtp-server', type=str,
send.add_argument('patchfiles', nargs='*')
test_parser = subparsers.add_parser('test', help='Run tests')
test_parser.add_argument('testname', type=str, default=None, nargs='?',
help="Specify the test to run")
# Only add the 'test' action if the test data files are available.
if os.path.exists(func_test.TEST_DATA_DIR):
test_parser = subparsers.add_parser('test', help='Run tests')
test_parser.add_argument('testname', type=str, default=None, nargs='?',
help="Specify the test to run")
status = subparsers.add_parser('status',
help='Check status of patches in patchwork')

4
tools/patman/patman.rst
View File

@ -680,6 +680,10 @@ them:
$ tools/patman/patman test
Note that since the test suite depends on data files only available in
the git checkout, the `test` command is hidden unless `patman` is
invoked from the U-Boot git repository.
Error handling doesn't always produce friendly error messages - e.g.
putting an incorrect tag in a commit may provide a confusing message.