Commit Graph
1654 Commits
Author SHA1 Message Date
András KlöczlandAutomerger Merge Worker 2d1cfb899b Merge "Migrate workspace item adding tests to kotlin" into sc-v2-dev am: 283798dbd5
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16321893

Change-Id: I1fd1ed39633fdc4c48a8d4a6c0b568faa8a39e09
2021-11-24 10:37:26 +00:00
András KlöczlandAndroid (Google) Code Review 283798dbd5 Merge "Migrate workspace item adding tests to kotlin" into sc-v2-dev 2021-11-24 10:21:39 +00:00
Andras Kloczl e77eafc263 Migrate workspace item adding tests to kotlin
In a former CL (http://ag/16064580) I've tried to
do the same but presubmit was constantly failing,
probably because the test is in kotlin.

Test: AddWorkspaceItemsTaskTest.kt
Bug: 199160559
Change-Id: Ie1bc4fcd4f94cd7cb0601c21bbdf273452b9dd1f
2021-11-23 18:32:12 +00:00
TreeHugger RobotandAutomerger Merge Worker 1bad3382fc Merge "Add timeout to AbstractLauncherUiTest.getOnUiThread" into sc-v2-dev am: 953641e28d
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16321879

Change-Id: I763c42250a2793647122c3c12423457bc2a140c5
2021-11-22 21:13:14 +00:00
TreeHugger RobotandAndroid (Google) Code Review 953641e28d Merge "Add timeout to AbstractLauncherUiTest.getOnUiThread" into sc-v2-dev 2021-11-22 20:59:16 +00:00
vadimtandAutomerger Merge Worker edb0527dbd Logging the stack trace for system anomaly messages am: 499e2055a6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16322469

Change-Id: I86b2c6379ce08d42fe727a6c63679136ffe428de
2021-11-22 20:37:38 +00:00
Alex Chau 31b46953cb Add timeout to AbstractLauncherUiTest.getOnUiThread
- Send signal 6 to Launcher process when this happens to get a thread dump

Bug: 203416429
Test: labtest
Change-Id: If22b58418be33131e7f585e718a2197cda2e7365
2021-11-22 18:36:11 +00:00
vadimt 499e2055a6 Logging the stack trace for system anomaly messages
With these system anomalies, you are never sure that the system/tests will be able to normally bubble up the error.

Bug: 203416429
Test: presubmit

Change-Id: Ie1d3d183cf6e56778d5c052ef7b8e0541bbc6255
(cherry picked from commit 37542c8c0e)
2021-11-22 18:08:00 +00:00
TreeHugger RobotandAutomerger Merge Worker 29eee5f1fe Merge "Revert "Adding screen record for a flake"" into sc-v2-dev am: 8c8e2a22da
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16283892

Change-Id: I7e9973c5600c996bd26b242f3e8b7c82d513ea37
2021-11-19 01:04:50 +00:00
TreeHugger RobotandAndroid (Google) Code Review 8c8e2a22da Merge "Revert "Adding screen record for a flake"" into sc-v2-dev 2021-11-19 00:51:07 +00:00
Alex ChauandAutomerger Merge Worker 3bf362fe30 Merge "Recognizing Launcher builds consisting only of digits" into sc-v2-dev am: 90a9982b36
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16285317

Change-Id: Id1503027367fdc2c4e4f20fcf12de9863e012667
2021-11-18 10:33:48 +00:00
vadimt 60ba2ac084 Recognizing Launcher builds consisting only of digits
Bug: 206687633
Test: online regex tools
Change-Id: Ia6155f22fb9a0eba3beb6edb607f44f901629330
2021-11-17 19:07:20 +00:00
Jonathan Miranda ae1bcc9ba8 Revert "Adding screen record for a flake"
This reverts commit da02250c29.

Reason for revert: Flake resolved
Bug: 202735477
Test: presubmit

Change-Id: I86856110ca9b8745c194a716a587d3bbe7f0137e
2021-11-17 16:24:26 +00:00
TreeHugger RobotandAutomerger Merge Worker 506927514d Merge "Fix LauncherProvider newScreenId issue" into sc-v2-dev am: 82fc85f9b8
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16064580

Change-Id: I8f25b31835fc64a7f76ae7267e7bcb2a6a4d8434
2021-11-16 19:15:25 +00:00
TreeHugger RobotandAndroid (Google) Code Review 82fc85f9b8 Merge "Fix LauncherProvider newScreenId issue" into sc-v2-dev 2021-11-16 19:00:47 +00:00
TreeHugger RobotandAutomerger Merge Worker a8a8c75bd7 Merge "Revert "Add ScreenRecord to methods affected by a flaky problem"" into sc-v2-dev am: a6c4b66b7f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16272580

Change-Id: Ic3f37f1c1803ac4fe19dc048932d77772e0d3105
2021-11-16 13:13:34 +00:00
Alex Chau 033f2afe38 Revert "Add ScreenRecord to methods affected by a flaky problem"
This reverts commit ea6ee1c824.

Reason for revert: Seems to increase the severity of the flaky problem
Bug: 204807156

Change-Id: I7b87f2abf45c9da21981f9361e320163bd7834bd
2021-11-16 11:19:09 +00:00
TreeHugger RobotandAutomerger Merge Worker ce57748e66 Merge "Add ScreenRecord to methods affected by a flaky problem" into sc-v2-dev am: fffea98399
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16261407

Change-Id: Iad0bec3854b6ccd77c4ab223b2476c619e123098
2021-11-15 21:45:24 +00:00
Alex Chau ea6ee1c824 Add ScreenRecord to methods affected by a flaky problem
Bug: 204807156
Test: none
Change-Id: I554c032b2ea43b8ee813f694d9c995f4ebc4a90f
2021-11-15 15:48:54 +00:00
TreeHugger RobotandAutomerger Merge Worker 239d9dfb4a Merge "Removing a dead test" into sc-v2-dev am: d5884f1c7b
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16249737

Change-Id: I730a4f5e8ead0daf32b257f1810e1aba32ef1380
2021-11-13 03:39:28 +00:00
vadimt 581fbae506 Removing a dead test
Test: presubmit
Bug: 147235759
Change-Id: I1412360316c87813d309cfbc1deefad00e7dd2ce
2021-11-12 20:15:24 +00:00
Andras KloczlandAlex Chau 953eb8041e Fix LauncherProvider newScreenId issue
Remove maxScreenId from LauncherProvider and whenever we need
a new screenId, query the database to calculate a new screenId.
Also converted and refactored AddWorkspaceItemsTaskTest
and added some extra test cases.

Test: manual & AddWorkspaceItemsTaskTest.kt
Bug: 199160559
Change-Id: I185f6823fed171d778af0130497f5ffaf89c0a70
2021-11-12 19:56:07 +00:00
Brian IsganitisandAutomerger Merge Worker 7e5e234668 Merge "Allow overriding getOptionsPopup test method." into sc-v2-dev am: bc1b36344f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16215049

Change-Id: I5634da8649178166d17a63d04787e5079df83dc0
2021-11-10 20:17:45 +00:00
Brian IsganitisandAndroid (Google) Code Review bc1b36344f Merge "Allow overriding getOptionsPopup test method." into sc-v2-dev 2021-11-10 19:56:07 +00:00
Jon MirandaandAutomerger Merge Worker 6dbfce731a Add back waitForStateTransitionToEnd to fix flakiness. am: 8568602aaf
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16225468

Change-Id: Ide0870aab3b7d595417ae4e34502118eb951c32c
2021-11-10 09:19:08 +00:00
Jon Miranda 8568602aaf Add back waitForStateTransitionToEnd to fix flakiness.
Bug: 202735477
Test: presubmit
Change-Id: I03c7e8b0c40177f4e6e38a7d9aa3e01b029effd6
2021-11-09 17:02:31 -08:00
Jon MirandaandAutomerger Merge Worker cbe0f5fa63 Add resume check to other WorkProfileTests to fix flakiness. am: f37d7a21f2
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16222650

Change-Id: Ibb9c82bdab00dae3953a33b4e53f785528fdf2f3
2021-11-09 21:01:35 +00:00
Jon Miranda f37d7a21f2 Add resume check to other WorkProfileTests to fix flakiness.
- Tests were flaky with waitForStateTransitionToEnd
- Tests stopped flaking with both waitForResume and waitForStateTransitionToEnd
- waitForResume may be enough to fix the flakiness, so
  removing waitForStateTransitionToEnd for now, if flakiness
  returns we can add it back

Bug: 202735477
Test: presubmit
Change-Id: Ib9cc16ea7815387582486517a87249d0053475f6
2021-11-09 10:14:01 -08:00
Alex ChauandAutomerger Merge Worker 2ccecc1614 Merge "Remove gap between screenIds when restoring from single display device" into sc-v2-dev am: 048a8246fa
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16191338

Change-Id: I7d37cd871cf8f6b506154458ba09bc4fe5e9b2e0
2021-11-09 17:23:05 +00:00
Alex Chau ac038434f0 Remove gap between screenIds when restoring from single display device
- In RestoreDbTask.sanitizeDb, add extra step to fill all screenIds gaps so that no empty pages are left in between
  e.g. [0, 3, 4, 6, 7] -> [0, 1, 2, 3, 4]

Fix: 201085524
Test: RestoreDbTaskTest
Change-Id: If0b577bf789347cc490ada0b22f2cd39ee1a33bb
2021-11-09 17:01:01 +00:00
TreeHugger RobotandAutomerger Merge Worker 55bef3bdd4 Merge "Testing to see if waiting for resume will fix flaky tests." into sc-v2-dev am: 1c79327f6e
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16214207

Change-Id: I62876c198b063445041fa67bb75b3a9b90056762
2021-11-09 00:12:23 +00:00
Brian Isganitis 149dbcb137 Allow overriding getOptionsPopup test method.
Test: TAPL
Bug: 205491724
Change-Id: Ib4adfa85df6aa9e52922f76479487c9459535fcd
2021-11-08 18:17:36 -05:00
Jon Miranda b9f5602d96 Testing to see if waiting for resume will fix flaky tests.
Similar to how AbstractLauncherUITest waits for resume after
switching state to Normal.

Bug: 202735477
Test: presubmit
Change-Id: I61834d32f8005491a4e14a354dc0097b05267f7b
2021-11-08 11:58:10 -08:00
TreeHugger RobotandAutomerger Merge Worker 8ce34e46fb Merge "Add to tapl quickswitch test for testing intermediate carousel." into sc-v2-dev am: 3555ef3b31
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16118650

Change-Id: I8f216d791f5b39552591f34db06966921db9b7e7
2021-11-08 13:30:34 +00:00
TreeHugger RobotandAndroid (Google) Code Review 3555ef3b31 Merge "Add to tapl quickswitch test for testing intermediate carousel." into sc-v2-dev 2021-11-08 13:15:44 +00:00
Pat Manning 04486f19bb Add to tapl quickswitch test for testing intermediate carousel.
Test: TaplTestsQuickstep.java
Fix: 197630182
Change-Id: I5f325995e22b76eee710154ca54a050eaf70b150
2021-11-08 11:32:56 +00:00
Jonathan MirandaandAutomerger Merge Worker 45c6ef8f8d Merge "Testing to see if waiting for transition to end will fix flaky tests" into sc-v2-dev am: 58763e47d6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16200260

Change-Id: Ied2373aac635bd248aa77a45e7a99713f63e0d52
2021-11-05 23:07:23 +00:00
Jonathan MirandaandAndroid (Google) Code Review 58763e47d6 Merge "Testing to see if waiting for transition to end will fix flaky tests" into sc-v2-dev 2021-11-05 22:53:17 +00:00
TreeHugger RobotandAutomerger Merge Worker a0d76174a7 Merge "Allowing OOP tests to enable home screen rotation." into sc-v2-dev am: d25e0fb3c3
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16198841

Change-Id: I490863b4254784b501edbe34e4933243b270bfad
2021-11-05 20:17:21 +00:00
Jon Miranda a234a8ea85 Testing to see if waiting for transition to end will fix flaky tests
Test: presubmit
Bug: 202735477
Change-Id: Icf62003b9f422cf0e327365260c36f6d57b71aef
2021-11-05 12:31:15 -07:00
vadimt 41f99abeb5 Allowing OOP tests to enable home screen rotation.
Bug: 202567877
Test: local runs
Change-Id: Ifb711a3071a26e2d8e690e0378b6e74146ac722c
(cherry picked from commit 462fe7c007)
2021-11-05 18:19:01 +00:00
TreeHugger RobotandAutomerger Merge Worker 67cace3558 Merge "Make Launcher activity restart on density change" into sc-v2-dev am: e641701c9f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16183372

Change-Id: I8e26c801c530d4f2851fe7d41cc58ed83ff5d51b
2021-11-05 12:02:23 +00:00
TreeHugger RobotandAndroid (Google) Code Review e641701c9f Merge "Make Launcher activity restart on density change" into sc-v2-dev 2021-11-05 11:48:20 +00:00
Alex ChauandAutomerger Merge Worker eeef3b637a Merge "Testing to see if using isInStableState is better condition vs getCurrentStableState" into sc-v2-dev am: a917d24d2b
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16184044

Change-Id: I03c254acdfc75a256365cc220c6f41189033db65
2021-11-04 18:10:58 +00:00
Alex ChauandAndroid (Google) Code Review a917d24d2b Merge "Testing to see if using isInStableState is better condition vs getCurrentStableState" into sc-v2-dev 2021-11-04 17:43:35 +00:00
Alex ChauandAutomerger Merge Worker 2b2e9e701a Merge "Disable Tapl pressBack and dragToFolder Tests" into sc-v2-dev am: a8c19019c6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16182791

Change-Id: I84fcfaa55e1ca492e241fe6519e6470e0e90ee2c
2021-11-04 14:39:47 +00:00
Alex Chau 84fe75d72e Make Launcher activity restart on density change
- Launcher used to restart when density changes before http://ag/14234761, which added smallestScreenSize and density in configChange
- http://ag/14234761 aims to avoid restart when switching display in split dispalys, for which only size changes, therefore density is unnecessary in configChange
- Keeping density in configChange make Launcher acitvity to stay after changing display size, making all buttons stays in old size in previous denstiy, which is a bug

Bug: 204157738
Test: manual
Change-Id: Ib85326078c67c35d90dc5e9ae5c20eeb5286a610
2021-11-04 11:33:16 +00:00
Benno Lin d80ed19ba6 Disable Tapl pressBack and dragToFolder Tests
The tests break on R3 and S5 devices.
Therefore the CL disable them temporarily until the issue being
resolved.

Bug: 205027405
Bug: 205014516
Test: disable tests that break ATP tests on R3 and S5.
Change-Id: Ib71c2f334d98445aeafa211d9bba2910d191b6fb
2021-11-04 02:46:48 +00:00
Jon Miranda f3fbc935cd Testing to see if using isInStableState is better condition vs getCurrentStableState
Test: presubmit
Bug: 202735477
Change-Id: I06c964cc92b7731e73628210688817e2d9cf1bac
2021-11-03 11:42:01 -07:00
Benno LinandAutomerger Merge Worker 1cfa05ad08 Merge "Implement pressing back button and swiping back" into sc-v2-dev am: 97d749833c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16013509

Change-Id: I4b60b6e1fa2c7a14d837ef74cd2d3f4c0f6f9f25
2021-11-03 06:20:17 +00:00