faft: Directly install Autotest dependency, not rely on a test
FAFT and PyAuto tests need to install Autotest dependency to client.
Call the install method instead of running a client test which may
confuse the test results.
BUG=chromium-os:36163
TEST=Remove the Autotest directory on the client and run a FAFT test.
CQ-DEPEND=I34578aff8c22800120a3dbbb369e3663acf6b31c
Change-Id: I46f69671d4acd8d701bc23ae30620b3192ff602d
Reviewed-on: https://gerrit.chromium.org/gerrit/37697
Reviewed-by: Mike Truty <truty@chromium.org>
Reviewed-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Yusuf Mohsinally <mohsinally@google.com>
Commit-Ready: Tom Wai-Hong Tam <waihong@chromium.org>
Tested-by: Tom Wai-Hong Tam <waihong@chromium.org>
diff --git a/server/cros/servo_test.py b/server/cros/servo_test.py
index 469b471..1c1fa09 100644
--- a/server/cros/servo_test.py
+++ b/server/cros/servo_test.py
@@ -40,8 +40,6 @@
'ref_name': 'pyauto',
# Port number of the remote RPC.
'port': 9988,
- # Client test for installing dependency.
- 'client_test': 'desktopui_PyAutoInstall',
# The remote command to be run.
'remote_command': 'python /usr/local/autotest/cros/remote_pyauto.py'
' --no-http-server',
@@ -60,7 +58,6 @@
'used': False,
'ref_name': 'faft_client',
'port': 9990,
- 'client_test': 'firmware_FAFTClient',
'remote_command': '/usr/local/autotest/cros/faft_client.py',
'remote_command_short': 'faft_client',
'remote_log_file': '/tmp/faft_client.log',
@@ -146,7 +143,7 @@
if info['used']:
if not self._autotest_client:
self._autotest_client = autotest.Autotest(self._client)
- self._autotest_client.run_test(info['client_test'])
+ self._autotest_client.install()
self.launch_client(info)
@@ -276,7 +273,7 @@
if install_deps:
if not self._autotest_client:
self._autotest_client = autotest.Autotest(self._client)
- self._autotest_client.run_test(info['client_test'])
+ self._autotest_client.install()
self.launch_client(info)
logging.info('Server: Relaunched remote %s.', name)