commit 5e7ef1d06c5419933afff57e61a25d68ffcad345
author harpreet <harpreet@chromium.org> Fri Mar 21 15:58:42 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat Mar 22 06:25:08 2014 +0000
tree 7fab0680d8e1d5e93bba4f50d6c06a88ce549461
parent 8ae3b126a8f246dab68a315790fab761300de5f2

Changing label for cellular repair devices

Created a new label (cellular_repair) to be used to run the
cellular_StaleModemReboot test. Added this label to the appropirate
devices and changing the label in the suite_scheduler to reflect this
change. also added a logging message to the test as well as another
modem state string.

BUG=chromium:349638
TEST=local test

Change-Id: I7541caf6bdb64d2c707284cb5e1d61cc36215bd0
Reviewed-on: https://chromium-review.googlesource.com/191118
Reviewed-by: Kris Rambish <krisr@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Queue: harpreet Grewal <harpreet@chromium.org>
Tested-by: harpreet Grewal <harpreet@chromium.org>

server/site_tests/cellular_StaleModemReboot/cellular_StaleModemReboot.py

diff --git a/server/site_tests/cellular_StaleModemReboot/cellular_StaleModemReboot.py b/server/site_tests/cellular_StaleModemReboot/cellular_StaleModemReboot.py
index f6c5768..e9b517e 100644
--- a/server/site_tests/cellular_StaleModemReboot/cellular_StaleModemReboot.py
+++ b/server/site_tests/cellular_StaleModemReboot/cellular_StaleModemReboot.py
@@ -113,6 +113,7 @@
             'ENABLED',
             'SEARCHING',
             'REGISTERED',
+            'DISCONNECTING',
             'CONNECTING',
             'CONNECTED'
         ]
@@ -215,7 +216,7 @@
         self._servo = host.servo
         original_modem_state = self._get_modem_state()
 
-        logging.info('Modem state for host %s: %s',
+        logging.info('Modem state before reboot on host %s: %s',
                      host.hostname,
                      self._modem_state_to_string(original_modem_state))
 
@@ -230,6 +231,8 @@
             # will show as 'orange' color code in the test results, instead of
             # green, which is reserved for when the modem was in a bad state
             # that was fixed by rebooting via this test.
+            logging.info('Modem state after default reboot: %s',
+                         self._modem_state_to_string(self._get_modem_state()))
             raise error.TestError('Modem was in stable state at the start of '
                                   'this test and is still in stable state '
                                   'after one reboot.')

suite_scheduler.ini

diff --git a/suite_scheduler.ini b/suite_scheduler.ini
index 5e6bf8e..6a5d78b 100644
--- a/suite_scheduler.ini
+++ b/suite_scheduler.ini
@@ -151,7 +151,7 @@
 run_on: new_build
 suite: cellular_modem_repair
 branch_specs: >=R35
-pool: cellular
+pool: cellular_repair
 
 [NetworkUI]
 run_on: new_build