Commit 4276efd8 authored by Haiyang Pan's avatar Haiyang Pan Committed by Commit Bot

android: Add logging for local instrumentation test run to help debugging

Currently the builder test-n-tablet has flaky test failure due to timeout in
apk installation.
Adding logging to see which apk spent the most time.

This CL can be reverted after the issue 1004322 is resolved

Bug: 1004322
Change-Id: Ic51694add3666aa0e0442cb6f8c11131e04ae13d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1859904Reviewed-by: default avatarJohn Budorick <jbudorick@chromium.org>
Commit-Queue: Haiyang Pan <hypan@google.com>
Cr-Commit-Position: refs/heads/master@{#706212}
parent 3d75e8ac
......@@ -197,11 +197,13 @@ class LocalDeviceInstrumentationTestRun(
@trace_event.traced
def install_helper_internal(d, apk_path=None):
# pylint: disable=unused-argument
logging.info('Start Installing %s', apk.path)
d.Install(
apk,
modules=modules,
fake_modules=fake_modules,
permissions=permissions)
logging.info('Finished Installing %s', apk.path)
return install_helper_internal
......@@ -210,7 +212,10 @@ class LocalDeviceInstrumentationTestRun(
@trace_event.traced
def incremental_install_helper_internal(d, apk_path=None):
# pylint: disable=unused-argument
logging.info('Start Incremental Installing %s', apk.path)
installer.Install(d, json_path, apk=apk, permissions=permissions)
logging.info('Finished Incremental Installing %s', apk.path)
return incremental_install_helper_internal
permissions = self._test_instance.test_apk.GetPermissions()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment