<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>9244</bug_id>
          
          <creation_ts>2016-03-11 17:30:05 +0000</creation_ts>
          <short_desc>patchtest-oe: Create tests to validate recipes, classes or config files</short_desc>
          <delta_ts>2016-08-04 17:32:26 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Patchwork/Patchtest</product>
          <component>Patchtest</component>
          <version>2.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.2 M2</target_milestone>
          <dependson>8648</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alex Franco">alejandro.franco</reporter>
          <assigned_to name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</assigned_to>
          <cc>leonardo.sandoval.gonzalez</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
          
          <qa_contact name="Alexandru Georgescu">alexandru.c.georgescu</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Yes (doc changes required)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>59906</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Franco">alejandro.franco</who>
    <bug_when>2016-03-11 17:30:05 +0000</bug_when>
    <thetext>Split from bug 4078

For bug 4078, pt-suites was written as a test suite to work with patchtest

The tests done there work for generic python files and patches, but not properly recipes, classes and config files. The tests we need for this bug are:

If **recipe** files modified by patch, recipe checks
  - Required for these tests:
    - list of mbox-formatted patches
    - list of files modified per patch
    - bitbake environment: use tinfoil? bitbake -e? new oe-selftest test?

  - Tests:
    1. SRC_URI - removed files, but files still around
    2. SRC_URI - points to something other than an SCM, but checksums not changed
    3. LIC_FILES_CHKSUM - missing
    4. LIC_FILES_CHKSUM - changed but commit message doesn&apos;t include &quot;lic&quot;

Additional tests should warrant additional bugs</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62961</commentid>
    <comment_count>1</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-06-14 14:13:05 +0000</bug_when>
    <thetext>This is expected to be finished early 2.2M2. These are the test cases considered right now:


patch.signed_off_by.test_signed_off_by.OEPatchSignedOffBy.test_signed_off_by_format
patch.signed_off_by.test_signed_off_by.OEPatchSignedOffBy.test_signed_off_by_presence
patch.upstream_status.test_upstream_status.OEPatchUpstreamStatus.test_upstream_status_presence
patch.upstream_status.test_upstream_status.OEPatchUpstreamStatus.test_upstream_status_valid_status
bitbake.parse.test_parse.OEBitbakeParse.test_bitbake_parse
mbox.payload.test_payload.OEPayload.test_payload_presence
mbox.signed_off_by.test_signed_off_by.OESignedOffBy.test_signed_off_by_format
mbox.signed_off_by.test_signed_off_by.OESignedOffBy.test_signed_off_by_presence
mbox.subject.test_subject.OESubject.test_subject_format
mbox.subject.test_subject.OESubject.test_subject_presence
mbox.description.test_description.OEDescription.test_presence
python.lint.test_pylint.OEPyLint.test_pylint
metadata.lic_files_chksum.test_lic_files_chksum.OELicFilesChksum.test_modified_not_mentioned
metadata.lic_files_chksum.test_lic_files_chksum.OELicFilesChksum.test_presence
metadata.src_uri.test_src_uri.OESrcUri.test_checksums_not_changed
metadata.src_uri.test_src_uri.OESrcUri.test_left_files</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63308</commentid>
    <comment_count>2</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-06-23 19:51:30 +0000</bug_when>
    <thetext>My fault, moving it to M1 again. CCB should agree on any milestone movement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>63729</commentid>
    <comment_count>3</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-07-07 15:16:23 +0000</bug_when>
    <thetext>Basically the same comment as https://bugzilla.yoctoproject.org/show_bug.cgi?id=4078#c17

Moving to 2.2 M2 so we can have this time window for testing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64883</commentid>
    <comment_count>4</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2016-08-04 17:32:26 +0000</bug_when>
    <thetext>Checks completed on repo ssh://git@git.yoctoproject.org/patchtest-oe, folder tests. Errors from this code should be reported in a separate bug.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>