Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CHERRY-PICK] BoardModulePkg: Added Mock library for BiosIdLib #301

Merged

Conversation

VivianNK
Copy link
Contributor

@VivianNK VivianNK commented Oct 4, 2024

Description

Added a gmock for GoogleTests that pull in BiosIdLib.

Cherry picked from edk2_platforms
c53db4dc8bbc6cec6016792b9e08b1f0630498ae
Amended to add newline in header file description.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Consumed mock in GoogleTest

Integration Instructions

N/A

@VivianNK VivianNK requested a review from apop5 October 4, 2024 17:59
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Oct 4, 2024
@VivianNK VivianNK force-pushed the personal/vnowkakeane/biosIdLib branch from 867e462 to cb68166 Compare October 4, 2024 18:02
@VivianNK VivianNK requested a review from kenlautner October 4, 2024 18:03
@VivianNK VivianNK changed the title [CHERRY-PICK] BoardModulePkg: Added Mock library for BiosIdLib (WIP) [CHERRY-PICK] BoardModulePkg: Added Mock library for BiosIdLib Oct 4, 2024
@VivianNK VivianNK force-pushed the personal/vnowkakeane/biosIdLib branch from cb68166 to 42c8d78 Compare October 4, 2024 18:36
@VivianNK VivianNK changed the title (WIP) [CHERRY-PICK] BoardModulePkg: Added Mock library for BiosIdLib [CHERRY-PICK] BoardModulePkg: Added Mock library for BiosIdLib Oct 4, 2024
@VivianNK VivianNK requested a review from os-d October 4, 2024 20:04
@makubacki
Copy link
Member

If you don't want to fix those to keep this a straight cherry-pick that's fine. They're not that big of a deal.

Added a gmock for GoogleTests that pull in BiosIdLib.

Cc: Eric Dong <[email protected]>
Cc: Liming Gao <[email protected]>

Signed-off-by: Vivian Nowka-Keane <[email protected]>
@VivianNK VivianNK force-pushed the personal/vnowkakeane/biosIdLib branch from 42c8d78 to 53c84c4 Compare October 4, 2024 23:36
@VivianNK VivianNK merged commit e566d57 into microsoft:dev/202405 Oct 7, 2024
11 checks passed
v-bhavanisu added a commit to v-bhavanisu/mu_common_intel_min_platform that referenced this pull request Oct 18, 2024
…soft#301)

## Description
Added a gmock for GoogleTests that pull in BiosIdLib.

Cherry picked from edk2_platforms

[c53db4dc8bbc6cec6016792b9e08b1f0630498ae](tianocore/edk2-platforms@c53db4d)
Amended to add newline in header file description.

- [ ] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested
Consumed mock in GoogleTest

## Integration Instructions
N/A

Signed-off-by: Vivian Nowka-Keane <[email protected]>
Co-authored-by: Bhavani Subramanian <[email protected]>
apop5 pushed a commit that referenced this pull request Oct 18, 2024
## Description
Added a gmock for GoogleTests that pull in BiosIdLib.

Cherry picked from edk2_platforms

[c53db4dc8bbc6cec6016792b9e08b1f0630498ae](tianocore/edk2-platforms@c53db4d)
Amended to add newline in header file description.

- [ ] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested
Consumed mock in GoogleTest

## Integration Instructions
N/A

Signed-off-by: Vivian Nowka-Keane <[email protected]>
Co-authored-by: Bhavani Subramanian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:non-functional Does not have a functional impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants