Bug 11755

Summary:	patchtest-oe: test checking changes in lic_files_chcksum not failing when lacking the explanation in commit message
Product:	[Yocto Project Subprojects] Patchwork/Patchtest	Reporter:	Daniela Plascencia <daniela.plascencia>
Component:	Patchtest Testsuite	Assignee:	Daniela Plascencia <daniela.plascencia>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Medium	CC:	leonardo.sandoval.gonzalez
Version:	unspecified
Target Milestone:	2.5 M1
Hardware:	x86
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)

Description Daniela Plascencia 2017-07-04 22:52:42 UTC

The current test already has a check that should send a failure message when the LIC_FILES_CHKSUM is modified and the explanation as to why is not present on the commit message. For some reason it is not detecting the patches lacking the explanation and not failing properly. The usage of Tinfoil may solve this issue.

Comment 1 Leonardo Sandoval Gonzalez 2017-07-18 13:57:43 UTC

This issue may be resolved with current master-next branches which will land to master shortly.

Comment 2 Daniela Plascencia 2017-09-08 20:47:52 UTC

(In reply to comment #1)
> This issue may be resolved with current master-next branches which will land
> to master shortly.

This issue is not present anymore considering the following:
a) Both framework and test suite are on master-next
b) Latest change on base.py: find_pn() returns PN instead of FN (PATH)
c) All metadata test cases use parse_recipe() with PN as input instead of FN

b and c are not in master-next, but certainly helped solving runtime errors and might be merged soon.

Comment 3 Daniela Plascencia 2017-09-08 20:50:32 UTC

Changing status "IN PROGRESS REVIEW".

Comment 4 Daniela Plascencia 2017-10-20 20:45:08 UTC

Latest changes on master solve this issue.

http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe/log/