commit c223505b3ed3e90c57c5bd4fb98799c9e8fa0078
author Matt Mallett <mattmallett@chromium.org> Tue Jan 30 16:47:08 2018 -0800
committer chrome-bot <chrome-bot@chromium.org> Wed Jan 31 13:23:51 2018 -0800
tree bd0a3f524bfa71db00537278d42603c10e1a2bb2
parent 31c1eec1ffaa0aae641db219eedfe53e0dd9bd30

[moblab] Use the is_reboot_needed value correctly

Use the return code from --is_reboot_needed correctly
to cause the moblab device to update when a new
version is downloaded and ready to install

BUG=chromium:613747
TEST=updated local moblab device from 10176.62.0 to 10176.65.0 via UI

Change-Id: I2204b3fcf2a2602b72350d380d3112a47a48a2ad
Reviewed-on: https://chromium-review.googlesource.com/895905
Commit-Ready: Matt Mallett <mattmallett@chromium.org>
Tested-by: Matt Mallett <mattmallett@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

frontend/afe/moblab_rpc_interface_unittest.py

diff --git a/frontend/afe/moblab_rpc_interface_unittest.py b/frontend/afe/moblab_rpc_interface_unittest.py
index 484a666..87dc346 100644
--- a/frontend/afe/moblab_rpc_interface_unittest.py
+++ b/frontend/afe/moblab_rpc_interface_unittest.py
@@ -496,9 +496,8 @@
         self.mox.StubOutWithMock(moblab_rpc_interface.subprocess, 'check_call')
         moblab_rpc_interface.subprocess.check_call(['sudo',
                 update_engine_client, '--update'])
-        error = moblab_rpc_interface.subprocess.CalledProcessError(2, '')
         moblab_rpc_interface.subprocess.check_call(['sudo',
-                update_engine_client, '--is_reboot_needed']).AndRaise(error)
+                update_engine_client, '--is_reboot_needed'])
 
         self.mox.StubOutWithMock(moblab_rpc_interface.subprocess, 'call')
         moblab_rpc_interface.subprocess.call(['sudo', update_engine_client,