search

tianocore / edk2

EDK II
https://github.com/tianocore/tianocore.github.io/wiki/EDK-II
Other
4.38k stars 2.4k forks source link

BaseTools\Plugin\HostBasedUnitTestRunner: Reduce the number of unit t… #5865

Open GeorgeLiao012 opened 5 days ago

GeorgeLiao012 commented 5 days ago

…est executions

REF : https://bugzilla.tianocore.org/show_bug.cgi?id=4801

Because of currently HostBasedUnitTestRunner.py triggered unit test twice, first for run host test, second for generate code coverage. Remove first time host test for saving time.

Description

<_Include a description of the change and why this change was made._> <_For each item, place an "x" in between `[` and `]` if true. Example: `[x]` (you can also check items in GitHub UI)_> <_Create the PR as a Draft PR if it is only created to run CI checks._> <_Delete lines in \<\> tags before creating the PR._> - [ ] Breaking change? - **Breaking change** - Will this cause a break in build or boot behavior? - Examples: Add a new library class or move a module to a different repo. - [ ] Impacts security? - **Security** - Does the change have a direct security impact? - Examples: Crypto algorithm change or buffer overflow fix. - [ ] Includes tests? - **Tests** - Does the change include any explicit test code? - Examples: Unit tests or integration tests. ## How This Was Tested <_Describe the test(s) that were run to verify the changes._> ## Integration Instructions <_Describe how these changes should be integrated. Use N/A if nothing is required._>
lgao4 commented 1 day ago

The change is good. But, need to resolve CI failure first.