tree f06894d5f16ec945435c2e37d457fa75e32158c1
parent 075100f06053da97e2c726dcd12bfba07568006d
author Wai-Hong Tam <waihong@google.com> 1506464374 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> 1506977537 +0000

multimedia: Introduce a way to save and show the log messages from the client

Server tests use the multimedia facade framework to call the client
library, such that the existing client library can be reused in server
tests. The implementation is done by RPC. The Autotest logging system
is unable to know the log output from the client. It is hard for debug.

This change introduces a way to save the log by running a tail command
in background. The stdout and stderr are piped to the Autotest logging
system. On every RPC call, the log is processed, meaning showed to
the Autotest log.

BUG=chromium:766631
TEST=Ran a multimedia test cases, with --debug, saw the log, like:
16:54:36 INFO | autoserv| Waiting for calibration image to stabilize...
16:54:47 INFO | autoserv| [client] 14:54:46 INFO | Dispatching method
<bound method DisplayFacadeNative.func_retry of
<autotest_lib.client.cros.multimedia.display_facade_native.DisplayFacadeNative
object at 0x7f5cb558c2d0>> with args ()
16:54:47 INFO | autoserv| [client] 14:54:46 DEBUG| Running 'modetest -c'
16:54:47 INFO | autoserv| [client] 14:54:46 DEBUG| Running 'modetest -p'
16:54:47 INFO | autoserv| [client] 127.0.0.1 - - [26/Sep/2017 14:54:46]
"POST /RPC2 HTTP/1.1" 200 -
16:54:47 INFO | autoserv| RPC 'display.get_external_resolution'((), {})
returns [1920, 1080].
16:54:47 INFO | autoserv| Checking the resolutions of Chameleon and
CrOS...
16:54:47 INFO | autoserv| Resolutions across CrOS and Chameleon match:
1920x1080
16:54:47 INFO | autoserv| Capturing the screen on Chameleon...
16:54:52 INFO | autoserv| Capturing the external screen on CrOS...

Also tested the cases, like DUT reboot, RPC error.

Change-Id: I870052d82cc489acd71b90dc009e470005a0d4ce
Reviewed-on: https://chromium-review.googlesource.com/685310
Commit-Ready: Wai-Hong Tam <waihong@google.com>
Tested-by: Wai-Hong Tam <waihong@google.com>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Armando Miraglia <armax@chromium.org>
Reviewed-by: Harpreet Grewal <harpreet@chromium.org>
Reviewed-by: Denis Tosic <dtosic@chromium.org>
(cherry picked from commit 610b49cad27e9904bbc7f2665b7f6e2f4ed6447b)
Reviewed-on: https://chromium-review.googlesource.com/695702
Reviewed-by: Wai-Hong Tam <waihong@google.com>
Commit-Queue: Wai-Hong Tam <waihong@google.com>