summaryrefslogtreecommitdiffstats
path: root/meta/lib/patchtest/tests/test_metadata_src_uri.py
diff options
context:
space:
mode:
authorTrevor Gamblin <tgamblin@baylibre.com>2023-10-12 09:24:58 -0400
committerRichard Purdie <richard.purdie@linuxfoundation.org>2023-10-13 11:53:06 +0100
commit2fdabc368a52a9bf60f76f33e92b94de843688a8 (patch)
tree1a9ab2e980cc66a656692bf4b450dd0b3156d6a0 /meta/lib/patchtest/tests/test_metadata_src_uri.py
parent575b00dca5a9ae15b0eec7c8eea3c415c2a37bc1 (diff)
downloadpoky-2fdabc368a52a9bf60f76f33e92b94de843688a8.tar.gz
patchtest: improve test issue messages
The patchtest tests provide vague feedback to the user, and many of them also provide redundant 'fix' strings that could easily be incorporated into the issue messages themselves. Simplify them so that it is more clear what the errors are and how they can be addressed. No recommendation is given when the issue string adequately conveys the issue, e.g. with a missing "LICENSE" entry in a newly-created recipe. (From OE-Core rev: 0bfb3614244ec7aa79b6424bc63f9f2bccdabe98) Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/patchtest/tests/test_metadata_src_uri.py')
-rw-r--r--meta/lib/patchtest/tests/test_metadata_src_uri.py3
1 files changed, 1 insertions, 2 deletions
diff --git a/meta/lib/patchtest/tests/test_metadata_src_uri.py b/meta/lib/patchtest/tests/test_metadata_src_uri.py
index 718229d7ad..ce2ace17bb 100644
--- a/meta/lib/patchtest/tests/test_metadata_src_uri.py
+++ b/meta/lib/patchtest/tests/test_metadata_src_uri.py
@@ -69,7 +69,6 @@ class SrcUri(base.Metadata):
69 # TODO: we are not taking into account renames, so test may raise false positives 69 # TODO: we are not taking into account renames, so test may raise false positives
70 not_removed = filesremoved_from_usr_uri - filesremoved_from_patchset 70 not_removed = filesremoved_from_usr_uri - filesremoved_from_patchset
71 if not_removed: 71 if not_removed:
72 self.fail('Patches not removed from tree', 72 self.fail('Patches not removed from tree. Remove them and amend the submitted mbox',
73 'Amend the patch containing the software patch file removal',
74 data=[('Patch', f) for f in not_removed]) 73 data=[('Patch', f) for f in not_removed])
75 74