Skip to content

Commit

Permalink
[CHERRY-PICK] UefiCpuPkg: CpuPageTableLibTestHost: Disable Random Tes…
Browse files Browse the repository at this point in the history
…t Suite

Commit 2f499c3 commented out the
RandomTestCase tests in CpuPageTableLibTestHost, but it left the
test suite being registered without any tests. This causes a failure
for tools that check to ensure tests are being registered with test
suites.

This patch comments out the test suite in addition to the tests
being added to it.

Signed-off-by: Oliver Smith-Denny <[email protected]>
  • Loading branch information
os-d committed Oct 1, 2024
1 parent 14d2901 commit ac91cfd
Showing 1 changed file with 8 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -838,7 +838,8 @@ UefiTestMain (
EFI_STATUS Status;
UNIT_TEST_FRAMEWORK_HANDLE Framework;
UNIT_TEST_SUITE_HANDLE ManualTestCase;
UNIT_TEST_SUITE_HANDLE RandomTestCase;

// UNIT_TEST_SUITE_HANDLE RandomTestCase;

Framework = NULL;

Expand Down Expand Up @@ -874,12 +875,12 @@ UefiTestMain (
//
// Populate the Random Test Cases.
//
Status = CreateUnitTestSuite (&RandomTestCase, Framework, "Random Test Cases", "CpuPageTableLib.Random", NULL, NULL);
if (EFI_ERROR (Status)) {
DEBUG ((DEBUG_ERROR, "Failed in CreateUnitTestSuite for Random Test Cases\n"));
Status = EFI_OUT_OF_RESOURCES;
goto EXIT;
}
// Status = CreateUnitTestSuite (&RandomTestCase, Framework, "Random Test Cases", "CpuPageTableLib.Random", NULL, NULL);
// if (EFI_ERROR (Status)) {
// DEBUG ((DEBUG_ERROR, "Failed in CreateUnitTestSuite for Random Test Cases\n"));
// Status = EFI_OUT_OF_RESOURCES;
// goto EXIT;
// }

// AddTestCase (RandomTestCase, "Random Test for Paging4Level", "Random Test Case1", TestCaseforRandomTest, NULL, NULL, &mTestContextPaging4Level);
// AddTestCase (RandomTestCase, "Random Test for Paging4Level1G", "Random Test Case2", TestCaseforRandomTest, NULL, NULL, &mTestContextPaging4Level1GB);
Expand Down

0 comments on commit ac91cfd

Please sign in to comment.