Skip to content

Fix: Add Test for Validating ECC Default State Values #170

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.

Sign up for GitHub

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix: Add Test for Validating ECC Default State Values #170

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
67f9118
Fix: Add Test for Validating ECC Default State Values
kalyanalle Mar 13, 2025
d2ded55
Fix: Add test case for setting the default settings and check the get…
kalyanalle Mar 19, 2025
0c26a5d
Merge branch 'oneapi-src:master' into new_ecc_test_default_ecc_values
kalyanalle Apr 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
57 changes: 57 additions & 0 deletions conformance_tests/sysman/test_sysman_ecc/src/test_sysman_ecc.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -185,4 +185,61 @@ TEST_F(
}
}

TEST_F(
ECC_TEST,
GIvenValidDeviceHandleWhenEccGetDefaultStateIsQueriedThenValidValuesAreReturned) {
for (const auto &device : devices) {
auto available = lzt::get_ecc_available(device);
auto configurable = lzt::get_ecc_configurable(device);
if (available == static_cast<ze_bool_t>(true) && configurable == static_cast<ze_bool_t>(true)) {
zes_device_ecc_properties_t eccState = {
ZES_STRUCTURE_TYPE_DEVICE_ECC_PROPERTIES, nullptr};
EXPECT_EQ(ZE_RESULT_SUCCESS, zesDeviceGetEccState(device, &eccState));
EXPECT_GE(eccState.currentState, ZES_DEVICE_ECC_STATE_UNAVAILABLE);
EXPECT_LE(eccState.currentState, ZES_DEVICE_ECC_STATE_DISABLED);
EXPECT_GE(eccState.pendingState, ZES_DEVICE_ECC_STATE_UNAVAILABLE);
EXPECT_LE(eccState.pendingState, ZES_DEVICE_ECC_STATE_DISABLED);
EXPECT_GE(eccState.pendingAction, ZES_DEVICE_ACTION_NONE);
EXPECT_LE(eccState.pendingAction, ZES_DEVICE_ACTION_COLD_SYSTEM_REBOOT);
} else {
LOG_INFO << "ECC is not available or not configurable";
}
}
}

TEST_F(
ECC_TEST,
GIvenValidDeviceHandleAndECCAvailableWhenEccGetDefaultStateIsQueriedAndSetThenDefaultValuesAreReturnedFromGetEccAPI) {
for (const auto &device : devices) {
//Check if ECC is available and configurable
auto available = lzt::get_ecc_available(device);
auto configurable = lzt::get_ecc_configurable(device);
if (available == static_cast<ze_bool_t>(true) && configurable == static_cast<ze_bool_t>(true)) {
//Get ECC state with defaults
zes_device_ecc_properties_t eccState = {
ZES_STRUCTURE_TYPE_DEVICE_ECC_PROPERTIES, nullptr};
EXPECT_EQ(ZE_RESULT_SUCCESS, zesDeviceGetEccState(device, &eccState));

//Set ECC state with default settings
zes_device_ecc_desc_t newState = {ZES_STRUCTURE_TYPE_DEVICE_ECC_DESC, nullptr};
newState.state = eccState.currentState;
zes_device_ecc_properties_t setState = {
ZES_STRUCTURE_TYPE_DEVICE_ECC_PROPERTIES, nullptr};
EXPECT_EQ(ZE_RESULT_SUCCESS, zesDeviceSetEccState(device, &newState, &setState));

//Get ECC state without defaults
zes_device_ecc_properties_t eccStateAfterSet = {
ZES_STRUCTURE_TYPE_DEVICE_ECC_PROPERTIES, nullptr};
EXPECT_EQ(ZE_RESULT_SUCCESS, zesDeviceGetEccState(device, &eccStateAfterSet));

//Check that returned values match defaults
EXPECT_EQ(eccStateAfterSet.currentState, eccState.currentState);
EXPECT_EQ(eccStateAfterSet.pendingState, eccState.pendingState);
EXPECT_EQ(eccStateAfterSet.pendingAction, eccState.pendingAction);
} else {
LOG_INFO << "ECC is not available or not configurable";
}
}
}

} // namespace
Loading