Skip to content

[Tests] Add new UTs app inf for integration tests #1292

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

Conversation

DorLevi95
Copy link

@DorLevi95 DorLevi95 commented Mar 5, 2025

Description

The current situation is that the UTs app is getting build in mu_crypto_release/CryptoBinPkg and linked directly against basecryptlib to test functionality.
The goal here is to add additional UTs .inf file, that will be used in CryptoBinPkg to link the tests against the crypto driver and serve as integration tests for the crypto protocol.

For details on how to complete these options and their meaning refer to CONTRIBUTING.md.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?
  • Backport to release branch?

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.>

@github-actions github-actions bot added impact:non-functional Does not have a functional impact impact:testing Affects testing labels Mar 5, 2025
@Flickdm Flickdm force-pushed the tests/add_BasecryptlibIntegrationTestApp_inf branch from 16818d9 to e48a0e5 Compare March 20, 2025 16:43
Copy link

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the state:stale Has not been updated in a long time label May 19, 2025
@makubacki makubacki requested review from Flickdm and kenlautner May 22, 2025 22:19
@github-actions github-actions bot removed the state:stale Has not been updated in a long time label May 22, 2025
@Flickdm
Copy link
Member

Flickdm commented Jun 26, 2025

@Flickdm Flickdm closed this Jun 26, 2025
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 impact:testing Affects testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants