summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAbe Levkoy <alevkoy@chromium.org>2022-01-10 15:27:00 -0700
committerCommit Bot <commit-bot@chromium.org>2022-01-18 17:48:29 +0000
commit0684c22d2f24b0b8ef7952eec04276a76d53899c (patch)
treee68200675762986bb2b6e4e6bcbf27b68f3e9a5a
parentbe788ddde42b62526a76011e44b4bc09e1335b0a (diff)
downloadchrome-ec-0684c22d2f24b0b8ef7952eec04276a76d53899c.tar.gz
zephyr test: Add TODO for non-hermetic test
Add TODO for order-dependent behavior in usb_integration that should be fixed. BUG=b:213909940 TEST=zmake configure --test zephyr/test/drivers BRANCH=none Change-Id: I7564a33946990a6733c34c0ff0ecfdf4fa99ad9e Signed-off-by: Abe Levkoy <alevkoy@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3379469 Reviewed-by: Wai-Hong Tam <waihong@google.com>
-rw-r--r--zephyr/test/drivers/src/integration_usb.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/zephyr/test/drivers/src/integration_usb.c b/zephyr/test/drivers/src/integration_usb.c
index 92dd6b82d6..2b512525f9 100644
--- a/zephyr/test/drivers/src/integration_usb.c
+++ b/zephyr/test/drivers/src/integration_usb.c
@@ -156,6 +156,10 @@ static void test_attach_pd_charger(void)
charge_params.cmd = CHARGE_STATE_CMD_GET_STATE;
zassert_ok(host_command_process(&args), "Failed to get charge state");
zassert_true(charge_response.get_state.ac, "USB default but AC absent");
+ /*
+ * TODO(b/213909940): This check will fail if test_suite_smart_battery
+ * has not been run previously. Figure out why and fix it.
+ */
zassert_equal(charge_response.get_state.chg_voltage, 5000,
"USB default voltage %dmV",
charge_response.get_state.chg_voltage);