diff options
| author | Karthikeyan Ramasubramanian <kramasub@chromium.org> | 2018-10-25 13:16:07 -0600 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-11-27 00:33:58 -0800 |
| commit | aa7bf63978082e4e55c0e11836964d8996f22c53 (patch) | |
| tree | 666d27731663bb4f180e82b039fea0f78cc677c5 | |
| parent | 56042b642e11b7c84b7380da21833d1615483bca (diff) | |
| download | vboot-aa7bf63978082e4e55c0e11836964d8996f22c53.tar.gz | |
test/ec_sync_test: Add mock test for VB_AUX_FW_NO_DEVICE severity
Add a mock test to handle VB_AUX_FW_NO_DEVICE severity
BUG=chromium:896451
BRANCH=None
TEST=/mnt/host/source/chromite/bin/cros_run_unit_tests
--board=octopus --packages=chromeos-base/vboot_reference
Change-Id: Ifdabdf3cee1130a8c853d57c278f0e557ebbb96f
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1299994
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
| -rw-r--r-- | tests/ec_sync_tests.c | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/tests/ec_sync_tests.c b/tests/ec_sync_tests.c index 4c1775f2..4b01b64a 100644 --- a/tests/ec_sync_tests.c +++ b/tests/ec_sync_tests.c @@ -219,8 +219,10 @@ VbError_t VbExCheckAuxFw(VbAuxFwUpdateSeverity_t *severity) VbError_t VbExUpdateAuxFw() { - ec_aux_fw_update_req = ec_aux_fw_update_severity != VB_AUX_FW_NO_UPDATE; - ec_aux_fw_protected = 1; + if (ec_aux_fw_update_severity != VB_AUX_FW_NO_DEVICE && + ec_aux_fw_update_severity != VB_AUX_FW_NO_UPDATE) + ec_aux_fw_update_req = 1; + ec_aux_fw_protected = ec_aux_fw_update_severity != VB_AUX_FW_NO_DEVICE; return ec_aux_fw_retval; } @@ -405,6 +407,15 @@ static void VbSoftwareSyncTest(void) TEST_EQ(ec_aux_fw_protected, 1, " aux fw protected"); ResetMocks(); + ec_aux_fw_mock_severity = VB_AUX_FW_NO_DEVICE; + test_ssync(VBERROR_SUCCESS, 0, + "No auxiliary FW update needed"); + TEST_EQ(screens_count, 0, + " wait screen skipped"); + TEST_EQ(ec_aux_fw_update_req, 0, " no aux fw update requested"); + TEST_EQ(ec_aux_fw_protected, 0, " no aux fw protected"); + + ResetMocks(); ec_aux_fw_mock_severity = VB_AUX_FW_NO_UPDATE; test_ssync(VBERROR_SUCCESS, 0, "No auxiliary FW update needed"); |