commit 1b957e17a33eeed9aa5b3eed061739be65e57cb6
author Mary Ruthven <mruthven@chromium.org> Tue May 14 13:51:11 2019 -0700
committer Mary Ruthven <mruthven@chromium.org> Wed May 15 17:38:16 2019 +0000
tree 797118d43755ab4967174345eb097f6e6e5e610d
parent b83de7ab6b75e05a7229a01c5d09fc2cb98cfd47

firmware_Cr50CCDServoCap: wait until ccdstate is done debouncing

Sometimes the test sees 'debouncing' as a state and fails, because that
doesn't fit into the off and on categories. This changes get_ccdstate to
wait until the state is done debouncing

BUG=none
BRANCH=none
TEST=run firmware_Cr50CCDServoCap

Change-Id: I5495f6df9f446058722ea414a31eca5968a7389b
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1612286
Reviewed-by: Kevin Shelton <kmshelton@chromium.org>

server/site_tests/firmware_Cr50CCDServoCap/firmware_Cr50CCDServoCap.py

diff --git a/server/site_tests/firmware_Cr50CCDServoCap/firmware_Cr50CCDServoCap.py b/server/site_tests/firmware_Cr50CCDServoCap/firmware_Cr50CCDServoCap.py
index 9e1e8d2..6c61292 100644
--- a/server/site_tests/firmware_Cr50CCDServoCap/firmware_Cr50CCDServoCap.py
+++ b/server/site_tests/firmware_Cr50CCDServoCap/firmware_Cr50CCDServoCap.py
@@ -159,8 +159,13 @@
 
     def get_ccdstate(self):
         """Get the current Cr50 CCD states"""
-        rv = self.cr50.send_command_get_output('ccdstate',
-                                               ['ccdstate(.*)>'])[0][0]
+        for i in range(self.cr50.MAX_RETRY_COUNT):
+            rv = self.cr50.send_command_get_output('ccdstate',
+                                                   ['ccdstate(.*)>'])[0][0]
+            if 'debouncing' not in rv:
+                break
+            time.sleep(self.cr50.SHORT_WAIT)
+
         # I2C isn't a reliable flag, because the hardware often doesn't support
         # it. Remove any I2C flags from the ccdstate output.
         rv = rv.replace(' I2C', '')