commit 015c15cd2076ceca93f759bda07591efeb8503e4
author Otabek Kasimov <otabek@google.com> Thu Aug 20 00:40:42 2020 -0700
committer Commit Bot <commit-bot@chromium.org> Fri Aug 21 20:39:20 2020 +0000
tree 32e352192965d80509b5008896535d71720b8863
parent 0fa0a387c961b52dd237dd1c78d518f64863b7a2

autotest: Make ec_board verification first.

Lid_open fail mostly based on missed ec console.

BUG=None
TEST=None

Change-Id: I3305ac8794b88dfeed1f57bf58697cc67fafb12d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2366832
Commit-Queue: Otabek Kasimov <otabek@google.com>
Tested-by: Otabek Kasimov <otabek@google.com>
Reviewed-by: Garry Wang <xianuowang@chromium.org>

server/hosts/servo_host.py

diff --git a/server/hosts/servo_host.py b/server/hosts/servo_host.py
index 21b08ad..ec18811 100644
--- a/server/hosts/servo_host.py
+++ b/server/hosts/servo_host.py
@@ -1370,12 +1370,12 @@
             or init_servo == self.VERIFY_FAILED):
             return servo_constants.SERVO_STATE_SERVOD_ISSUE
 
+        if ec_board == self.VERIFY_FAILED:
+            return servo_constants.SERVO_STATE_EC_BROKEN
         if pwr_button == self.VERIFY_FAILED:
             return servo_constants.SERVO_STATE_BAD_RIBBON_CABLE
         if lid_open == self.VERIFY_FAILED:
             return servo_constants.SERVO_STATE_LID_OPEN_FAILED
-        if ec_board == self.VERIFY_FAILED:
-            return servo_constants.SERVO_STATE_EC_BROKEN
 
         metrics.Counter(
             'chromeos/autotest/repair/unknown_servo_state'