Wait until all fingers leave to complete test
Previously the test suite would collect logs as long as
1. The robot hadn't completed it's gesture yet
2. There have been new reports recently
This CL adds a third option
3. There is still a finger on the pad
This option had been avoided before for fear of a "ghost
finger" causing the test to hang indefinitely. However, with
manual testing by Allion ramping up, they were running into an
issue where the test suite would cut off logging before they
were done during the "stationary" tests. Due to delta-compression
of the events, these stationary points would sometimes go for
a while without emitting an updated event. This CL means that
the suite won't stop until it sees all the fingers leave the pad.
BUG=chromium:568162
TEST=manual testing
Change-Id: I456608a7e82c85f105e17bab878b29df61f5417a
Signed-off-by: Charlie Mooney <charliemooney@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/317311
Reviewed-by: Shyh-In Hwang <josephsih@chromium.org>
1 file changed
