Bug 11328 - tests: test_lic_files_chksum_presence uses a variable not always provided by Bitbake
Summary: tests: test_lic_files_chksum_presence uses a variable not always provided by ...
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchtest Testsuite (show other bugs)
Version: 2.3
Hardware: x86 Multiple
: Medium normal
Target Milestone: 2.5 M1
Assignee: Daniela Plascencia
QA Contact:
URL:
Whiteboard:
Depends on: 10804 10805
Blocks:
  Show dependency tree
 
Reported: 2017-04-10 20:52 UTC by Daniela Plascencia
Modified: 2017-12-06 17:28 UTC (History)
1 user (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniela Plascencia 2017-04-10 20:52:12 UTC 
The test expects 'bitbake -e' to return a variable (LIC_FILES_CHKSUM) or raise an ERROR when it is not present on a new recipe. The latter doesn't happen as bitbake will build any recipe without a license file checksum specified, meaning the test is not reliable and it should be modified using tinfoil to avoid further issues.
Comment 1 Daniela Plascencia 2017-08-24 19:58:34 UTC 
I think this issue might be solved already as we are using Tinfoil now. Have to double check if this still applies or if it's already corrected.
Comment 2 Daniela Plascencia 2017-09-08 21:01:39 UTC 
This is not an issue anymore on master-next; the code was changed and this test case uses tinfoil instead of bitbake directly, meaning the variable in the description is not present anymore. 
I'll change the status to "IN PROGRESS REVIEW" as the branch that solves this (and many other issues) is not yet merged on master.
Comment 3 Daniela Plascencia 2017-10-02 15:38:38 UTC 
As mentioned before, this is not an issue anymore, but we are still testing. Moving this to 2.5 M1, it should be solved before M1 ends.
Comment 4 Daniela Plascencia 2017-10-20 20:44:37 UTC 
Latest changes on master solve this issue.

http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe/log/
Comment 5 Daniela Plascencia 2017-10-20 20:46:55 UTC 
Forgot to change the status to Resolved Fixed.