Commit Graph

1121 Commits

Author SHA1 Message Date
TreeHugger Robot e5a83db490 Merge "Remove tracing for a fixed flake" into ub-launcher3-rvc-dev 2020-05-18 17:21:41 +00:00
vadimt 2bba25a08d Remove tracing for a fixed flake
Bug: 155926212
Change-Id: Ia847d735d58154bc768b69d3ae3dc0af7fcdbf11
2020-05-18 16:34:57 +00:00
vadimt bb6f5f5242 Add dumpsys input where it was missing
Bug: 156287114
Change-Id: Ie0a71e8dc261abfde4eec3ce882786d77a870c32
2020-05-15 13:12:37 -07:00
vadimt 5670d462de Avoid truncation of input trace
Bug: 156287114
Change-Id: Iac0f93d4e8082907210bd6312d1c046ff1f49dba
2020-05-14 14:25:38 -07:00
TreeHugger Robot 1462617c97 Merge "Starting investigation for non-delivered TIS events" into ub-launcher3-rvc-dev 2020-05-13 20:40:47 +00:00
vadimt 87db7187c6 Starting investigation for non-delivered TIS events
Bug: 156287114
Change-Id: Icae88318a8e5a8f6b9931913b24ad3ece5475cc3
2020-05-13 13:08:39 -07:00
TreeHugger Robot 794ca4dba5 Merge "Dumping view hierarchy before swiping to Overview" into ub-launcher3-rvc-dev 2020-05-13 04:16:18 +00:00
vadimt 0790cef41c Dumping view hierarchy before swiping to Overview
Bug: 156044202
Change-Id: I21a11a7a55103cffbc05ddeeb93ecc4f6f4063e7
2020-05-12 18:25:32 -07:00
TreeHugger Robot eb0136b7c4 Merge "Abstracting out state handler logic into base classes so that it can also be used in different activity" into ub-launcher3-rvc-dev 2020-05-12 00:29:52 +00:00
TreeHugger Robot 3d668090af Merge "Resolve Test flakes" into ub-launcher3-rvc-dev 2020-05-11 23:40:43 +00:00
Sunny Goyal 35e7d38681 Abstracting out state handler logic into base classes so that it can
also be used in different activity

Change-Id: Ic5e60b21e1429c71f1cd4e7041e70bf4fc9761b7
2020-05-11 16:38:26 -07:00
vadimt 5d59df4fd9 Don't check TIS events mismatch for successful gestures
Perhaps, due to a framework bug, events sometimes don't get delivered to
TIS; this doesn't seem to cause user-visible problems.

Later we'll need to investigate why this happens.

Bug: 154157191
Change-Id: I25f45ccab10f6c537c14610e40c2d02d2d3f28ad
2020-05-11 13:14:06 -07:00
Samuel Fufa b549819676 Resolve Test flakes
-> Synchronize calls in LoaderTask.java
issue 156041043

-> Remove non system users on setup
If a work test crashes before getting to run its teardown - we might end up with a user profile that could throw off subsequent tests.
issue 156022161

Test: Manual
Change-Id: Ife708a3de01572f7cb2187078d592d8d570dd951
2020-05-10 13:51:41 -07:00
TreeHugger Robot 4f51725717 Merge "Adding waiting for Launcher initialization" into ub-launcher3-rvc-dev 2020-05-08 21:26:30 +00:00
vadimt 2794f563a0 Adding tracing for switching to overview instead of all apps
Bug: 156095088
Change-Id: I1257617192e913d06edd0879396957b1e40524b7
2020-05-08 10:39:29 -07:00
vadimt 60d3333ff0 Adding waiting for Launcher initialization
This may fix or ease TIS sometimes not receiving events.

Bug: 154157191
Change-Id: I5db9e88397788014dbba9716d777a2b97347828c
2020-05-08 09:29:08 -07:00
Vadim Tryshev 340e68219e Merge "Restarting event reader after reaching the end of stream" into ub-launcher3-rvc-dev 2020-05-07 21:29:46 +00:00
vadimt 3e089e8a26 Restarting event reader after reaching the end of stream
This should'n happen, but happens. The best we can do is to re-run the
logcat command.

Change-Id: I471d9f0188243c81585b75f81dd03450c1f7ad9b
2020-05-07 12:29:19 -07:00
TreeHugger Robot d4f87007b6 Merge "Likely, fixing view hierarchy changing during pressHome." into ub-launcher3-rvc-dev 2020-05-07 17:41:22 +00:00
vadimt 29896284a8 Likely, fixing view hierarchy changing during pressHome.
Test RequestPinItemTest.testPinShortcut doesn't wait for the add-to-home
prompt to disappear before proceeding to pressHome. So sometimes
pressHome still sees that prompt that disappears while pressHome is
executing.

Now waiting for the prompt to disappear.

Bug: 155926212
Change-Id: I2c7bfe26839ae406da592e81de8836666c373756
2020-05-06 19:08:59 -07:00
vadimt fbfe79d8a1 Debug tracing for view hierarchy changing during pressHome
Bug: 155926212
Change-Id: Ie9e8470c23b954890ae41fc9a442422f21093855
2020-05-06 16:41:13 -07:00
TreeHugger Robot fc9ea9b0fc Merge "Speeding up TAPL by calling getVisibleStateMessage() less frequently" into ub-launcher3-rvc-dev 2020-05-04 23:18:04 +00:00
TreeHugger Robot 7bb915ba8e Merge "Remove unnecessary waitForIdle calls" into ub-launcher3-rvc-dev 2020-05-04 22:44:21 +00:00
vadimt faf4da386d Speeding up TAPL by calling getVisibleStateMessage() less frequently
The information in the message can be derived from the object dump
printed anyways.

Change-Id: I22ee136568b767c3218eaabc3920c77b3ceaa78f
2020-05-04 15:40:47 -07:00
vadimt 357e5499a2 Remove unnecessary waitForIdle calls
They were added back in times when the
code was very different, and we don't need them now.

Change-Id: Ie96b081a4f2319c7b58d22ffc410f74e9ab56d94
2020-05-04 18:57:52 +00:00
vadimt 55495201e0 Avoiding cascading "nested events checking" failure after timed-out wait
Bug: 155656943
Change-Id: I0f11552349a9f6b3b14bd1bebb37904d5dfa9c55
2020-05-04 10:52:08 -07:00
vadimt 9fd3a943ba Don't do full anomaly check after every gesture
This call is relatively expensive. Perform only cheap launcher crash
test, and do anomaly only if there are problems.

Change-Id: If45567bcedf8d177970739e9de95cbb29add744c
2020-04-30 19:31:21 -07:00
TreeHugger Robot dfd8c4806f Merge "More logging for LogEventChecker appearing to skip events" into ub-launcher3-rvc-dev 2020-04-30 20:24:21 +00:00
vadimt 232737c854 More logging for LogEventChecker appearing to skip events
Bug: 155347466
Change-Id: I6598bcf36c1f7ae76e9bb6b2eccefbd00899de2e
2020-04-30 10:19:10 -07:00
vadimt a1f0a10662 Removing tracking activity start/stop events
They were not especially useful in investigations; besides, these events
can be sent by the system asynchronously for actions that happened
before entering TAPL, causing flakes.

Change-Id: I72b5aad5521c6c0969f5b657b3db3e4d855f1d64
2020-04-29 14:23:53 -07:00
TreeHugger Robot d4066f5876 Merge "Support for counting activity creations" into ub-launcher3-rvc-dev 2020-04-28 23:26:37 +00:00
vadimt 2e6fdd7d93 Optimization: reducing number of getNavigationModeMismatchError calls
They are relatively expensive

Change-Id: I34de56884ec3f88e69c709026d3b72e62c787681
2020-04-27 13:08:50 -07:00
vadimt 921f9fd254 Support for counting activity creations
Bug: 139137636
Change-Id: Ib999fd3c82eb2997662578ec35e1d9e1e75e3cbb
2020-04-27 12:41:26 -07:00
vadimt 0703464556 Checking that there are no more than 2 live activities at any moment
This is tightening the makeshift strictmode criteria.

Starting with this moment, we will know that there is
no memory growth during tests execution, which is a big deal.

Big: 139137636

Change-Id: I5edc84524463bd1736d727496ad0fc031bb9624c
2020-04-23 16:35:09 +00:00
TreeHugger Robot d73d39b45a Merge "Makeshift analog of Strictmode leak detector" into ub-launcher3-rvc-dev 2020-04-23 03:22:31 +00:00
vadimt b3e8ae8230 Makeshift analog of Strictmode leak detector
Strictmode leak detector is still a goal, but we might not be able to
achieve it in R. Strictmode has several framework-side bugs that perhaps
hide Launcher-side strictmode violations, while the time to fix
everything is limited, and new leaks get introduced all the time.

For now, implementing a check that is slightly more relaxed than
Strictmode, but still ensures the absence of leaks. I’ll keep
eliminating Strictmode violations as well as keep strengthening the
makeshift checker conditions until we’ll be able to enable Strictmode in
continuous testing.

I’m disabling Strictmode checks for now so that they don’t generate
unnecessary hprof dumps, but leaving the code dealing with strictmode.

Bug: 139137636
Change-Id: Ib10136b0d4e9892f70a19cd052ae5a54cf0a4efb
2020-04-22 19:00:12 -07:00
Sunny Goyal 34ff8cb3c1 Adding common code for instrumentation and robolectric tests
Change-Id: Ia338f0565868bb174ef5f6721924944792866be6
2020-04-20 14:42:30 -07:00
TreeHugger Robot 11e36db88a Merge "Cleanup code related to ag/11035582" into ub-launcher3-rvc-dev 2020-04-16 22:04:45 +00:00
TreeHugger Robot 64abef8ca1 Merge "Choosing better points to output "start" events for TAPL" into ub-launcher3-rvc-dev 2020-04-16 21:58:47 +00:00
Vinit Nayak 916456bb5f Cleanup code related to ag/11035582
Fixes: 154235554
Change-Id: I114d1e1b9e446e15a774a5a51bdf988070091523
2020-04-16 14:16:56 -07:00
vadimt cf275c1bd7 Choosing better points to output "start" events for TAPL
Now doing this before branching points, thus avoiding flakes when the
execution can go to an unexpected branch and not produce an event.

Bug: 153824894
Change-Id: If117da0498eaf2d94c9610552724981be34c6569
2020-04-15 12:45:11 -07:00
Vinit Nayak 770cc42685 Revert "Cleanup code related to ag/11035582"
This reverts commit a2a1f86dff.

Reason for revert: b/154068414

Change-Id: I1db0018fa97d5a5380651cf7d04a05a0abd62857
2020-04-15 05:24:10 +00:00
Vinit Nayak a2a1f86dff Cleanup code related to ag/11035582
Bug: 153443049
Change-Id: I2a9e46fa4ffc613751c9f427ce892ddad3878b35
2020-04-14 14:02:49 -07:00
Vinit Nayak 1a09958626 Disable sensor manager for tests in platform
Bug: 153443049
Change-Id: Ieaa4b8144eba11c5d3025a1bcca42a8552d1c702
2020-04-13 15:47:02 -07:00
TreeHugger Robot 77cf9b780f Merge "Adding tracing for apparently skipping verifiable events" into ub-launcher3-rvc-dev 2020-04-10 03:01:05 +00:00
Sunny Goyal e396abf502 Moving model data structures to a separate file
Change-Id: I77ad7a5219e72d2e0d6c1803de2ac3ed6a65a8f7
2020-04-09 13:20:39 -07:00
vadimt 8cb2f9ce08 Adding tracing for apparently skipping verifiable events
Bug: 153670015
Change-Id: I481c095f1245ba4252eb07e08b4130c857dd6e72
2020-04-09 13:02:48 -07:00
TreeHugger Robot e8fade9140 Merge "Adding diags to getWidget" into ub-launcher3-rvc-dev 2020-04-09 04:47:09 +00:00
vadimt 9915a7c60f TAPL waits for widgets recycler to become scrollable
On the Launcher side, moving setLayoutFrozen from the posted action to
avoid a possible short scrollable period just after the view is shown.

Bug: 152354290
Change-Id: I7319236d8a6e49a7e017fd54d593ee131dff10a9
2020-04-08 18:56:52 -07:00
vadimt 180c2e10fc Adding diags to getWidget
Also avoiding scrolling widgets horizontally when the gesture could
happen in the lower system gesture area.

Change-Id: I80192db7e407f8c1715aad3b96178c00b5710e71
2020-04-08 21:12:20 +00:00