diff options
author | Karthikeyan Ramasubramanian <kramasub@chromium.org> | 2018-10-25 13:16:07 -0600 |
---|---|---|
committer | Furquan Shaikh <furquan@chromium.org> | 2018-11-30 16:00:29 +0000 |
commit | ee88e5efedd49e80606ef3007ee6b4e3fc3c9b1e (patch) | |
tree | 666d27731663bb4f180e82b039fea0f78cc677c5 | |
parent | 65d11d9dedb7b897f1fa427b385152a3bee4b744 (diff) | |
download | vboot-ee88e5efedd49e80606ef3007ee6b4e3fc3c9b1e.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>
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/1355748
Commit-Queue: Furquan Shaikh <furquan@chromium.org>
Tested-by: Furquan Shaikh <furquan@chromium.org>
Trybot-Ready: Furquan Shaikh <furquan@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"); |