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<[email protected]>



On 1/2/2023 3:24 AM, Guo, Gua wrote:
From: Gua Guo <[email protected]>

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/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to