edk2-basetools icon indicating copy to clipboard operation
edk2-basetools copied to clipboard

Published 20 hours ago • verified publisher icon tianocore

Enable Integration tests

Open Javagedes opened this issue 3 years ago • 6 comments
trafficstars

As mentioned in #53, this PR is waiting for an update to the PlatformBuild.py files for OvmfPkg, ArmVirtPkg, and EmulatorPkg before this PR is merged.

The integration tests currently only run platform builds (IA32/X64, DEBUG/RELEASE). Please let me know if there are tools other than build that use BaseTools that you wish integration tests to be ran on, and I will add them to this PR.

Javagedes avatar Sep 28 '22 20:09 Javagedes

Codecov Report

Base: 4.28% // Head: 4.28% // No change to project coverage :thumbsup:

Coverage data is based on head (fa1bba4) compared to base (e2dc17a). Patch has no changes to coverable lines.

Additional details and impacted files 
@@          Coverage Diff           @@
##           master     #54   +/-   ##
======================================
  Coverage    4.28%   4.28%           
======================================
  Files         181     181           
  Lines       81448   81448           
======================================
  Hits         3493    3493           
  Misses      77955   77955
Flag Coverage Δ
Linux 4.28% <ø> (ø)
Windows_NT 4.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Sep 28 '22 20:09 codecov[bot]

Request to add additional architectures on the Linux / GCC5 build side.

Javagedes avatar Oct 03 '22 23:10 Javagedes

Have all the updates been made to support this PR?

mdkinney avatar Mar 07 '23 23:03 mdkinney

No, This bugzilla bug never made it in, which allows edk2 repo to build with basetools automatically:

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

Javagedes avatar Mar 13 '23 17:03 Javagedes

@Javagedes Sorry I got swamped and am just trying to catch up on PRs in this repo. Is this PR still relevant? If so, I'll review it and try and get it merged.

bcran avatar Jun 16 '24 16:06 bcran

@bcran this is still applicable, but quite old so it may be out of date. Let me review it again. I'm out next week so it'll be the following week that I can take a look.

Javagedes avatar Jun 16 '24 19:06 Javagedes