What's Changed
-
patina\_dxe\_core: Improve performance unit test isolation @makubacki (#1682)
Change Details
## Description
A sporadic test issue occurred in performance unit tests recently:
failures:
---- performance::tests::test_core_performance_perf_helpers_record_when_enabled stdout ---- thread 'performance::tests::test_core_performance_perf_helpers_record_when_enabled' (24684) panicked at patina_dxe_core/src/performance.rs:873:13: assertion `left == right` failed left: 0 right: 10Several tests in the module were using hardcoded handles for testing and not cleaning the protocol database between tests.
This applies
with_clean_global_lockto tests that read and write the protocol database. With guidance on what type of test wrapper to use given in a comment at the beginning of the test module.- Impacts functionality?
- Impacts security?
- Breaking change?
- Includes tests?
- Includes documentation?
How This Was Tested
cargo make all- Run
cargo make test5 times consecutively.
Integration Instructions
- N/A
🐛 Bug Fixes
-
patina: Fix dropped re-export fix from major merge conflict @cfernald (#1681)
Change Details
## Description
The merge of the major branch SDK refactor dropped the change to re-export of Char8String and Char16String only when the
allocfeature is enabled.- Impacts functionality?
- Impacts security?
- Breaking change?
- Includes tests?
- Includes documentation?
How This Was Tested
Cargo make all
Integration Instructions
N/A
Full Changelog: patina-v23.0.0...v23.0.1