tree e8b2c78b039d75cce53db1378e317e3d31f67e69
parent e17977cd4c1cacfb5f768a6321b6f3e647904a32
author Dana Goyette <dgoyette@chromium.org> 1580519961 +0000
committer Dana Goyette <dgoyette@chromium.org> 1580520067 +0000

Revert "faft: check power state in the test, not in servod (revised)"

This reverts commit 577af62dbf5990c6a7a495288bdb5a61f28d5ae2.

Reason for revert: Asking the EC for "powerinfo" too frequently seems to prevent the S5->G3 transition.  The extra info it provided can be provided by CL:2007341 instead.

Original change's description:
> faft: check power state in the test, not in servod (revised)
> 
> This is a redo of CL:1895163, this time unsetting the regex before
> running the command, to ensure the servod caching doesn't break it.
> It also now reports a specific error message for cases where the output
> did not match.
> 
> TEST=Run firmware_ECLidShutdown.
> BUG=b:141519692
> BUG=chromium:1031604
> 
> Change-Id: I11ab9927c363a24b793f6c0841888618a7dfd8c4
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1955479
> Tested-by: Dana Goyette <dgoyette@chromium.org>
> Commit-Queue: Dana Goyette <dgoyette@chromium.org>
> Reviewed-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>

BUG=b:148496362
TEST=Run firmware_FwScreenCloseLid

Change-Id: Ifd6af086a98782ff08057bdc1c2d37ec1bdb0185
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2033875
Reviewed-by: Dana Goyette <dgoyette@chromium.org>
Tested-by: Dana Goyette <dgoyette@chromium.org>