[edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC

Guo, Gua posted 3 patches 1 year, 3 months ago
Failed in applying to current master (apply log)
.azurepipelines/Ubuntu-GCC5.yml               |   5 +-
.azurepipelines/Windows-VS2019.yml            |   5 +
.../templates/pr-gate-build-job.yml           |  36 +++++++
.azurepipelines/templates/pr-gate-steps.yml   |   4 +
.../HostBasedUnitTestRunner.py                | 101 +++++++++++++++++-
UnitTestFrameworkPkg/ReadMe.md                |  41 +++++++
.../UnitTestFrameworkPkg.ci.yaml              |   1 +
.../UnitTestFrameworkPkgHost.dsc.inc          |   3 +-
pip-requirements.txt                          |   2 +
9 files changed, 195 insertions(+), 3 deletions(-)
[edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC
Posted by Guo, Gua 1 year, 3 months ago
From: Gua Guo <gua.guo@intel.com>

V1: Add coverage option for GCC
V2: Add ReadMe.md for how to generate coverage report
V3: Add VS2019 and GCC code coverage support
V4: Add VS2019 and GCC Azure CI/CD support
V5: Fix some typo and some flow issue
V6: Remove html coverage information
  - Due to python 3.11 install lxml will be failure,
  pycobertura need it to convert cobertura format to
  html file.
  - Add section for developer how to use OpenCppCoverage
  on IDE Visual Studio
V7: Remove redundant code and add code coverage pipeline support
  - Remove redundant code on HostBasedUnitTestRunner.py
  - Unify coding rule on HostBasedUnitTestRunner.py
  - Add CodeCoverage Azure pipeline support for GCC5 and VS2019 

Gua Guo (3):
  UnitTestFrameworkPkg: Add code coverage support for GCC
  BaseTools/Plugin: Add coverage support for Unit Test
  .azurepipelines: Install code coverage tool

 .azurepipelines/Ubuntu-GCC5.yml               |   5 +-
 .azurepipelines/Windows-VS2019.yml            |   5 +
 .../templates/pr-gate-build-job.yml           |  36 +++++++
 .azurepipelines/templates/pr-gate-steps.yml   |   4 +
 .../HostBasedUnitTestRunner.py                | 101 +++++++++++++++++-
 UnitTestFrameworkPkg/ReadMe.md                |  41 +++++++
 .../UnitTestFrameworkPkg.ci.yaml              |   1 +
 .../UnitTestFrameworkPkgHost.dsc.inc          |   3 +-
 pip-requirements.txt                          |   2 +
 9 files changed, 195 insertions(+), 3 deletions(-)

-- 
2.31.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97835): https://edk2.groups.io/g/devel/message/97835
Mute This Topic: https://groups.io/mt/96004850/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC
Posted by Sean 1 year, 3 months ago
Looks like a great first step for code coverage collection.

The container support patch series will need to be updated (since it 
hasn't yet been merged) and it will need to account for the new 
requirements for code coverage.

In the future i would like to see the local experience improved (HTML 
reports generated) and metrics tracked per package instead of build 
wide.  These topics can be discussed at the Monday tools meeting but I 
think this series as is should be merged now to start collecting this data.

Thanks for all your efforts.


Series Reviewed-by: Sean Brogan<sean.brogan@microsoft.com>



On 1/2/2023 3:24 AM, Guo, Gua wrote:
> From: Gua Guo <gua.guo@intel.com>
>
> V1: Add coverage option for GCC
> V2: Add ReadMe.md for how to generate coverage report
> V3: Add VS2019 and GCC code coverage support
> V4: Add VS2019 and GCC Azure CI/CD support
> V5: Fix some typo and some flow issue
> V6: Remove html coverage information
>    - Due to python 3.11 install lxml will be failure,
>    pycobertura need it to convert cobertura format to
>    html file.
>    - Add section for developer how to use OpenCppCoverage
>    on IDE Visual Studio
> V7: Remove redundant code and add code coverage pipeline support
>    - Remove redundant code on HostBasedUnitTestRunner.py
>    - Unify coding rule on HostBasedUnitTestRunner.py
>    - Add CodeCoverage Azure pipeline support for GCC5 and VS2019
>
> Gua Guo (3):
>    UnitTestFrameworkPkg: Add code coverage support for GCC
>    BaseTools/Plugin: Add coverage support for Unit Test
>    .azurepipelines: Install code coverage tool
>
>   .azurepipelines/Ubuntu-GCC5.yml               |   5 +-
>   .azurepipelines/Windows-VS2019.yml            |   5 +
>   .../templates/pr-gate-build-job.yml           |  36 +++++++
>   .azurepipelines/templates/pr-gate-steps.yml   |   4 +
>   .../HostBasedUnitTestRunner.py                | 101 +++++++++++++++++-
>   UnitTestFrameworkPkg/ReadMe.md                |  41 +++++++
>   .../UnitTestFrameworkPkg.ci.yaml              |   1 +
>   .../UnitTestFrameworkPkgHost.dsc.inc          |   3 +-
>   pip-requirements.txt                          |   2 +
>   9 files changed, 195 insertions(+), 3 deletions(-)
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97898): https://edk2.groups.io/g/devel/message/97898
Mute This Topic: https://groups.io/mt/96004850/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC
Posted by Michael Kubacki 1 year, 3 months ago
For the series:
Reviewed-by: Michael Kubacki <michael.kubacki@microsoft.com>

On 1/2/2023 6:24 AM, Guo, Gua wrote:
> From: Gua Guo <gua.guo@intel.com>
> 
> V1: Add coverage option for GCC
> V2: Add ReadMe.md for how to generate coverage report
> V3: Add VS2019 and GCC code coverage support
> V4: Add VS2019 and GCC Azure CI/CD support
> V5: Fix some typo and some flow issue
> V6: Remove html coverage information
>    - Due to python 3.11 install lxml will be failure,
>    pycobertura need it to convert cobertura format to
>    html file.
>    - Add section for developer how to use OpenCppCoverage
>    on IDE Visual Studio
> V7: Remove redundant code and add code coverage pipeline support
>    - Remove redundant code on HostBasedUnitTestRunner.py
>    - Unify coding rule on HostBasedUnitTestRunner.py
>    - Add CodeCoverage Azure pipeline support for GCC5 and VS2019
> 
> Gua Guo (3):
>    UnitTestFrameworkPkg: Add code coverage support for GCC
>    BaseTools/Plugin: Add coverage support for Unit Test
>    .azurepipelines: Install code coverage tool
> 
>   .azurepipelines/Ubuntu-GCC5.yml               |   5 +-
>   .azurepipelines/Windows-VS2019.yml            |   5 +
>   .../templates/pr-gate-build-job.yml           |  36 +++++++
>   .azurepipelines/templates/pr-gate-steps.yml   |   4 +
>   .../HostBasedUnitTestRunner.py                | 101 +++++++++++++++++-
>   UnitTestFrameworkPkg/ReadMe.md                |  41 +++++++
>   .../UnitTestFrameworkPkg.ci.yaml              |   1 +
>   .../UnitTestFrameworkPkgHost.dsc.inc          |   3 +-
>   pip-requirements.txt                          |   2 +
>   9 files changed, 195 insertions(+), 3 deletions(-)
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97885): https://edk2.groups.io/g/devel/message/97885
Mute This Topic: https://groups.io/mt/96004850/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC
Posted by Michael D Kinney 1 year, 3 months ago
Series Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Guo, Gua
> Sent: Monday, January 2, 2023 3:25 AM
> To: devel@edk2.groups.io
> Cc: Guo, Gua <gua.guo@intel.com>
> Subject: [edk2-devel] [PATCH v8 0/3] Add code coverage support for GCC/MSVC
> 
> From: Gua Guo <gua.guo@intel.com>
> 
> V1: Add coverage option for GCC
> V2: Add ReadMe.md for how to generate coverage report
> V3: Add VS2019 and GCC code coverage support
> V4: Add VS2019 and GCC Azure CI/CD support
> V5: Fix some typo and some flow issue
> V6: Remove html coverage information
>   - Due to python 3.11 install lxml will be failure,
>   pycobertura need it to convert cobertura format to
>   html file.
>   - Add section for developer how to use OpenCppCoverage
>   on IDE Visual Studio
> V7: Remove redundant code and add code coverage pipeline support
>   - Remove redundant code on HostBasedUnitTestRunner.py
>   - Unify coding rule on HostBasedUnitTestRunner.py
>   - Add CodeCoverage Azure pipeline support for GCC5 and VS2019
> 
> Gua Guo (3):
>   UnitTestFrameworkPkg: Add code coverage support for GCC
>   BaseTools/Plugin: Add coverage support for Unit Test
>   .azurepipelines: Install code coverage tool
> 
>  .azurepipelines/Ubuntu-GCC5.yml               |   5 +-
>  .azurepipelines/Windows-VS2019.yml            |   5 +
>  .../templates/pr-gate-build-job.yml           |  36 +++++++
>  .azurepipelines/templates/pr-gate-steps.yml   |   4 +
>  .../HostBasedUnitTestRunner.py                | 101 +++++++++++++++++-
>  UnitTestFrameworkPkg/ReadMe.md                |  41 +++++++
>  .../UnitTestFrameworkPkg.ci.yaml              |   1 +
>  .../UnitTestFrameworkPkgHost.dsc.inc          |   3 +-
>  pip-requirements.txt                          |   2 +
>  9 files changed, 195 insertions(+), 3 deletions(-)
> 
> --
> 2.31.1.windows.1
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97842): https://edk2.groups.io/g/devel/message/97842
Mute This Topic: https://groups.io/mt/96004850/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-