Commit Graph

883 Commits

Author SHA1 Message Date
Vadim Tryshev f1fe64eb95 Merge "Not spamming logs with "Hierarchy before swiping up to home"" into udc-dev 2023-04-11 02:04:50 +00:00
vadimt 22b1daea5e Not spamming logs with "Hierarchy before swiping up to home"
Bug: 260260325
Test: presubmit
Change-Id: I74e0729ae62656a235eb71e443a2ddf9a6cae3f9
2023-04-10 15:19:48 -07:00
TreeHugger Robot 7163918dd0 Merge "Show suggested widgets header only when there are suggested widgets" into udc-dev 2023-04-02 22:47:07 +00:00
Jeremy Sim 9a9d4ce653 Merge "App Pairs (behind flag): Add Overview menu item, icon, tests" into udc-dev 2023-04-01 02:05:30 +00:00
Jeremy Sim 9f0a6b8e82 App Pairs (behind flag): Add Overview menu item, icon, tests
This is the first of several patches implementing the App Pairs feature behind a flag.

This patch includes:
- The icon asset for the "Save app pair" menu option
- The menu option, which will appear in the Overview drop-down menu for split pairs
- Some tests ensuring that the menu item appears when appropriate

Flag: ENABLE_APP_PAIRS (set to false)
Bug: 274189428
Test: 2 tests in TaplTestsSplitscreen.java
Change-Id: I174f2c66e7eb6d83ff7fe501eb82cc48e727022c
2023-03-31 17:16:56 -07:00
fbaron ba3f77be7d Show suggested widgets header only when there are suggested widgets
Bug: 275435999
Test: Verified suggested widgets header doesn't show up unless there are suggested widgets to show
Change-Id: I2ec2ba100d7a6ddc338a36ece75633abba990b2b
2023-03-31 21:45:07 +00:00
vadimt 406a96c1e2 Add more logging to debug b/270770472
Test: presubmit
Bug: 270770472
Change-Id: Ie7c07b7095c7fba0058d9785a1fb47208d09eb21
2023-03-31 13:41:50 -07:00
Schneider Victor-tulias f86b574ef8 Merge "Add debug logs to help investigate view and activity leak test failures." into tm-qpr-dev am: afdf2f1020 am: 3bc74626d5
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/22163529

Change-Id: Iabaad3fe3a5b938bb22e3c8f8b626c510172a948
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-27 18:17:16 +00:00
Federico Baron c3ab820144 Add test for widget picker on landscape mode for tablets
When we added widget picker two pane layout, it broke some tests with the flag on. Here we fix those tests.
Bug: 256684299
Test: AddWidgetsTest and TaplTestsLauncher3#testWidget

Change-Id: I383c754016d82fba51fa0b7b0459cd7e83b274d8
2023-03-21 16:48:12 +00:00
Schneider Victor-tulias dc25fca903 Add debug logs to help investigate view and activity leak test failures.
Flag: not needed
Test: MemoryTests
Bug: 260260325
Change-Id: Ie9ed8e607c574e81696a4246f701534c18cf0b7c
2023-03-20 15:54:47 -07:00
Jeremy Sim c0d8b2dc0b Merge "Revert "Revert "Create TAPL test suite for splitscreen""" into tm-qpr-dev am: 7b286a4f68 am: 7f61f22560
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/22079982

Change-Id: Id25492e0212812d5489923fab9777c8d41f1bfe0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-15 22:19:59 +00:00
Jeremy Sim da75737d63 Revert "Revert "Create TAPL test suite for splitscreen""
This reverts commit 313a44039f.

Merging will reenable the TAPL suite TaplTestsSplitscreen.java, which was failing on phones due to a function call that only works on tablets.

Fixed by conditionally running the function only on tablet devices.

Fixes: 271852893
Test: Ran tests locally on phone and tablet.
Change-Id: If3e692924ed9c5ee113872e4f8ff0652890d1280
2023-03-14 15:36:32 -07:00
Sebastián Franco fef518da3d Merge "Making sure there is a propper Chrome icon for the test to long press" into tm-qpr-dev am: 4122868216 am: b8614e298c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21778872

Change-Id: I124c235e0dcdeab04b3fe2eb40083118778b9fa2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-10 20:32:00 +00:00
Sebastian Franco 239ae0a995 Making sure there is a propper Chrome icon for the test to long press
In some cases, the test was long pressing on a recommendation for
google chrome and not a regular chrome icon and the events didn't
match. I don't think is good to relly on chrome getting recommended
so I'm making sure there is always a chrome app icon on the hotseat

Fix: 269163620
Test: atest TaplTestsLauncher3:testWorkspace (run on pixel 7 pro)
Change-Id: Ic8ee744a9af18d60de9bbedd42043f5abbbd7356
2023-03-09 14:59:21 -08:00
TreeHugger Robot d8478f7019 Merge "Revert "Create TAPL test suite for splitscreen"" into tm-qpr-dev am: 19c0877745 am: 9fbd4b9c4e
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21806280

Change-Id: I91f799236b02142e2a9c94ef56105bbd20e669f3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-08 20:57:24 +00:00
Jeremy Sim 6d506a8622 Merge "Create TAPL test suite for splitscreen" into tm-qpr-dev am: ad9bdcfd2d am: 2a81b79a3c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21221178

Change-Id: I5dc5ccacaddfbe437ff44dd067ba8127b2fb7e40
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-03-07 03:19:44 +00:00
Vinit Nayak 313a44039f Revert "Create TAPL test suite for splitscreen"
This reverts commit 03815e6c79.

Reason for revert: b/271852893

Change-Id: I165a15b29e41ce1114f6b7d92d22cdd8e0dfb5b0
2023-03-06 23:47:20 +00:00
Jeremy Sim 03815e6c79 Create TAPL test suite for splitscreen
This patch creates a new file, TaplTestsSplitscreen.java, containing one TAPL test sequence.

testSplitAppFromHomeWithItself() tests the user flow of splitting from home using an app that is already running, and then selecting the app itself again as a split target from the Taskbar. The expected result is that Launcher shows an error message gracefully and does not crash.

Fixes: 267554450
Test: Ran test file locally and verified that it passes.
Change-Id: I98a1e3a4143d10a6572175711059b3a77aa5dcd0
2023-03-02 16:35:54 -08:00
Ling Wo 087c19a9ec Merge "Revert^2 "Expose getSearchBox from the HomeAllApps class"" into tm-qpr-dev am: 6d3658dd0c am: 4f9196c305
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21160354

Change-Id: I2d4c1b04e1f6fef0a0c075e8692a742f5a5bf2c8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-24 03:04:55 +00:00
Ling Wo 6d3658dd0c Merge "Revert^2 "Expose getSearchBox from the HomeAllApps class"" into tm-qpr-dev 2023-02-24 02:24:54 +00:00
Ling Wo d70c8a743d Revert^2 "Expose getSearchBox from the HomeAllApps class"
Reverted changes: /q/submissionid:21128652-revert-20714047-b_255390519-CAXVPNWYOY

Change-Id: I0e9680408e8b1db85600a226d808669c65ba4967
Bug: b/266169958
2023-02-23 02:42:09 +00:00
TreeHugger Robot b0321f97b6 Merge "Making ReorderWidget test more reliable" into tm-qpr-dev am: 1b7a140693 am: f351bd7c6c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21091023

Change-Id: Iaeda98eb011c28cc9e682a6fc07fa64fb4028a15
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-15 22:58:03 +00:00
Sebastian Franco 81b82720de Making ReorderWidget test more reliable
In some cases when starting to drag you can go to the next
CellLayout wich end in an error. Now the dragging functions
will move the item to the desired CellLayout no matter what
CellLayout you start in.

Fix: 266116487
Test: atest ReorderWidgets
Change-Id: Ie3ad2ff88f734856e2c66d5d51b7578482877b5a
2023-02-13 11:02:09 -08:00
TreeHugger Robot 4a2bf3b131 Merge "Update tests to support floating search." into tm-qpr-dev 2023-02-06 23:50:49 +00:00
Andy Wickham 8d004b0f49 Update tests to support floating search.
Tests need to be updated to account for the new placement of the
searchbar because it overlapped with touch events for scrolling:
 - Scroll back to top: Instead of scrolling from the top of the
   container which could overlap status bar in landscape, scroll
   from the bottom of the top-most visible app icon.
 - Scroll down: swipe up from bottom padding to top of top-most
   visible icon.
 - Close all apps: swipe down more quickly from top icon insetad
   of the search bar (more quickly helps it be detected as a
   fling on more cramped devices).

For Launcher3, the floating flag is not fully supported yet, so
there were some layout issues which are now resolved by ignoring
the flag if the searchbar is still at the top.

Fix: 268052768
Test: Ran tests, manual
Change-Id: If54717e2835c7cc4ed1368554bbc493193945c1d
Merged-In: I406fbcbe12acddb1dd4b862a380576a48cabbebc
2023-02-06 13:49:14 -08:00
TreeHugger Robot 005f29a7e2 Merge "Add instruction to put the device into test harness to error message" into tm-qpr-dev am: a3e61240b1 am: f9d6efc4fa
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21222958

Change-Id: I1349ce5537c3991d4e0381319417c3fc844f0e36
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-02-06 10:35:28 +00:00
Andy Wickham 346771550e [revert ^2] Enable Floating Search and update tests accordingly.
Tests need to be updated to account for the new placement of the
searchbar because it overlapped with touch events for scrolling:
 - Scroll back to top: Instead of scrolling from the top of the
   container which could overlap status bar in landscape, scroll
   from the bottom of the top-most visible app icon.
 - Scroll down: swipe up from bottom padding to top of top-most
   visible icon.
 - Close all apps: swipe down more quickly from top icon insetad
   of the search bar (more quickly helps it be detected as a
   fling on more cramped devices).

For Launcher3, the floating flag is not fully supported yet, so
there were some layout issues which are now resolved by ignoring
the flag if the searchbar is still at the top.

Fix: 261873937
Test: Ran tests, manual
Change-Id: I406fbcbe12acddb1dd4b862a380576a48cabbebc
2023-02-03 14:24:22 -08:00
Nicolo' Mazzucato bdb4923396 Add instruction to put the device into test harness to error message
Bug: 267601529
Test: Run a test while not in test harness mode.
Change-Id: I8089503e61831e5dde6f4e659e92a8d75ea18e2b
2023-02-02 13:10:52 +00:00
Becky Qiu 9b9209acc6 Merge "Revert "Expose getSearchBox from the HomeAllApps class"" into tm-qpr-dev am: 70c9aee083 am: 145c75eafd
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/21128653

Change-Id: Iabcdcab1f642f7e1b95b177253bea2b5602f6761
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-01-27 22:21:20 +00:00
Becky Qiu 70c9aee083 Merge "Revert "Expose getSearchBox from the HomeAllApps class"" into tm-qpr-dev 2023-01-27 20:52:26 +00:00
Becky Qiu d5311f6541 Revert "Expose getSearchBox from the HomeAllApps class"
Revert submission 20714047-b_255390519

Reason for revert: broken test b/266169958 (P0 bug) and broken UI for the EDU card

Reverted changes: /q/submissionid:20714047-b_255390519

Change-Id: I250003c1b3d07637dde596225f08a6bd3cb84365
2023-01-27 20:11:26 +00:00
Sebastián Franco 1b3d1cfa64 Merge "Adding the avility to add icons on multiple CellLayouts on tests." into tm-qpr-dev am: 68f5667338 am: 56f91702d2
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20890013

Change-Id: I3b9bad730707d9e5dca8f10acdb43991cd51c290
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-01-24 23:11:45 +00:00
Sebastián Franco 68f5667338 Merge "Adding the avility to add icons on multiple CellLayouts on tests." into tm-qpr-dev 2023-01-24 20:02:35 +00:00
Ling Wo a83e753c9b Merge "Expose getSearchBox from the HomeAllApps class" into tm-qpr-dev am: 38aa60c38f am: 6dafb0359c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20273481

Change-Id: I75db89675672c9c08a4e48f7ebbc8448aec51a6e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-01-19 22:44:20 +00:00
Ling Wo 38aa60c38f Merge "Expose getSearchBox from the HomeAllApps class" into tm-qpr-dev 2023-01-19 21:29:12 +00:00
Sebastian Franco de1fe14a47 Adding the avility to add icons on multiple CellLayouts on tests.
I needed to change the code so that you can pass the CellX/Y
position to the drag function instead of the pixel coordinates
and we can pass X position bigger than one CellLayout.

Also, I notice that getCellCenter only worked for 1x1 items
so I added the option to specify the widget size.

Fix: 262431878
Test: atest ReorderWidgets#multipleCellLayoutsSimpleReorder
Change-Id: I3b09de770e8f99aa10cae8cea32e7e6bc02c7b8b
2023-01-16 10:32:40 -06:00
Luca Zuccarini b25ea2d33f Add Tapl support for swiping from All Apps to Workspace. am: 55d828b6f5 am: 242eeea4ea
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20551691

Change-Id: Idff9df5162ef48783beb8eb6662838948b8d2d90
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-27 15:01:14 +00:00
Luca Zuccarini 55d828b6f5 Add Tapl support for swiping from All Apps to Workspace.
This mirrors the way we swipe from Workspace to All Apps, with minor
differences for selecting the start position of the swipe motion.

Bug: 260218332
Test: included
Change-Id: I9fe2284b1c920ebed87a6bab3dfc9c8c29aa2d5d
2022-12-21 15:38:51 +00:00
Vadim Tryshev e896d66fd6 Merge "Not waiting for animations for Launcher clicks" into tm-qpr-dev am: e74238d233 am: 0a2ed6c078
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20790706

Change-Id: I74eb64251c91c37746f32ff4e25920447427a395
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-20 22:28:28 +00:00
Ling Wo 8ba48c1e65 Expose getSearchBox from the HomeAllApps class
Doc: https://docs.google.com/document/d/1OlN-A3Z5_UjWk-QkxC4JaYWesFDNHkHTox94__rLmR0

Bug: b/255390519
Test: local testing
Change-Id: I72410035bbcfd2215815dbf6c3d6bd86d692b8c6
2022-12-20 04:13:59 +00:00
vadimt befcfa40b9 Not waiting for animations for Launcher clicks
Waiting for animations to finish can make the test wait for too long if there is an ongoing animation somewhere, for example, in an app we are trying to swipe from.

Test: presubmit
Bug: 256567004
Change-Id: Ida494128a18342bee3dbdae6d713c854c86912c1
2022-12-19 18:02:07 -08:00
Alex Chau 9de98280b8 Merge "Use overridden default xml for TwoPanelWorkspaceTest" into tm-qpr-dev am: 0c2d7e8da3 am: 7a7f6ce2c8
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20746546

Change-Id: I052bec3d1505574c50c583d01f4b03d890f27cea
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-16 00:11:02 +00:00
Alex Chau f8b79d4235 Use overridden default xml for TwoPanelWorkspaceTest
- Partially reverted TAPL and TwoPanelWorkspaceTest changes on http://ag/19195512

Fix: 262627208
Fix: 262632384
Test: TwoPanelWorkspaceTest
Change-Id: I8b793bfb3874cc6305ce8b0f9f7b0a116d01222b
2022-12-15 15:48:09 +00:00
Alex Chau b015054390 Merge "Remove unnecessary GridDisplayOption_defaultSplitDisplayLayoutId attribute" into tm-qpr-dev am: 8a7dc2b244 am: 4fb5e62810
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/19195512

Change-Id: If50694adfcaaf0b98987e7ac840a82740e3e1b41
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-14 22:49:20 +00:00
Alex Chau 8a7dc2b244 Merge "Remove unnecessary GridDisplayOption_defaultSplitDisplayLayoutId attribute" into tm-qpr-dev 2022-12-14 21:50:34 +00:00
Brandon Dayauon 65de436e38 Merge "Change method name as suggested" into tm-qpr-dev am: f772ba18ab am: b4b23d848a
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20646119

Change-Id: I98a81336fce29190f787f204716a4f1cba644876
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-08 01:01:31 +00:00
Brandon Dayauon f772ba18ab Merge "Change method name as suggested" into tm-qpr-dev 2022-12-08 00:18:26 +00:00
Brandon Dayauon bb3146271b Change method name as suggested
bug: 261098912
bug: 261099706

test: Manual - presubmit go/forest
Change-Id: I2e18ddaf2b9647bccaecccbfa7faf88e51b84f12
2022-12-08 00:15:06 +00:00
TreeHugger Robot 9b610cadbf Merge "Revert "Revert "Enable taskbar in overview by default""" into tm-qpr-dev am: 5e0d7ae019 am: 0a456b59ef
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/20640160

Change-Id: I81aefb87ac4dddc632884a8748fab9d0d58d9fda
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-07 18:57:44 +00:00
Alex Chau 766cd2b4b9 Remove unnecessary GridDisplayOption_defaultSplitDisplayLayoutId attribute
- Can use deviceCategory to make grid specific to splitDisplay instead
- Update TwoPanelWorkspaceTest for new workspace items

Fix: 233736588
Test: Clear launcher and check the default
Test: TwoPanelWorkspaceTest
Change-Id: I6ad3a2d866d847af281e10b802edbd4bcc892c39
2022-12-07 15:18:28 +00:00