change some Cr50DeepSleep test logging statements
This changes a few of the log statements and raises TestNAError instead
of TestError if the device does not have access to the Cr50 console.
BUG=none
BRANCH=none
TEST=run the test on reef.
Change-Id: I69af2f7b1126899733a1047dd9bbc316d04ba924
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/463868
Reviewed-by: Wai-Hong Tam <waihong@google.com>
diff --git a/server/site_tests/firmware_Cr50DeepSleepStress/firmware_Cr50DeepSleepStress.py b/server/site_tests/firmware_Cr50DeepSleepStress/firmware_Cr50DeepSleepStress.py
index 5dcfdc2..11b043e 100644
--- a/server/site_tests/firmware_Cr50DeepSleepStress/firmware_Cr50DeepSleepStress.py
+++ b/server/site_tests/firmware_Cr50DeepSleepStress/firmware_Cr50DeepSleepStress.py
@@ -7,7 +7,6 @@
from autotest_lib.client.common_lib import error
from autotest_lib.server import autotest, test
-from autotest_lib.server.cros.servo import chrome_cr50
from autotest_lib.server.cros.faft.firmware_test import FirmwareTest
@@ -33,12 +32,13 @@
def initialize(self, host, cmdline_args):
super(firmware_Cr50DeepSleepStress, self).initialize(host, cmdline_args)
if not hasattr(self, "cr50"):
- raise error.TestError('Test needs to be run through CCD')
+ raise error.TestNAError('Test can only be run on devices with '
+ 'access to the Cr50 console')
def check_deep_sleep_count(self):
self.cr50.ccd_enable()
count = self.cr50.get_deep_sleep_count()
- logging.debug("Cr50 resumed from sleep %d times", count)
+ logging.debug("Cr50 resumed from deep sleep %d times", count)
return count
def cleanup(self):
@@ -58,7 +58,6 @@
logging.info("Clear Cr50 deep sleep count")
self.cr50.clear_deep_sleep_count()
# Disable CCD so Cr50 can enter deep sleep
- logging.info("Disable CCD")
self.cr50.ccd_disable()
self.client_at = autotest.Autotest(host)