From 6121f00e1bb703fce92707d45e8346106555e8e8 Mon Sep 17 00:00:00 2001 From: Joshua Trask Date: Tue, 15 Mar 2022 15:31:50 -0400 Subject: [PATCH] Remove two timeouts in ResolverActivityTest. These don't seem to be necessary for the respective tests to pass, and they were hampering my ability to make changes to the chooser refresh delay (since they reference the same symbol). If we do end up needing to reinstate any delay like this, we still shouldn't have to take a dependency on Chooser from Resolver. Bug: 224816815 Test: atest --iterations 30 ResolverActivityTest Change-Id: Icf806e2babca048726fe33843df4f9f62e4bd138 Merged-In: Icf806e2babca048726fe33843df4f9f62e4bd138 --- .../src/com/android/internal/app/ResolverActivityTest.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/core/tests/coretests/src/com/android/internal/app/ResolverActivityTest.java b/core/tests/coretests/src/com/android/internal/app/ResolverActivityTest.java index e7a23f2627532..43fba529182eb 100644 --- a/core/tests/coretests/src/com/android/internal/app/ResolverActivityTest.java +++ b/core/tests/coretests/src/com/android/internal/app/ResolverActivityTest.java @@ -516,8 +516,6 @@ public class ResolverActivityTest { onView(withText(R.string.resolver_work_tab)) .perform(click()); waitForIdle(); - // wait for the share sheet to expand - Thread.sleep(ChooserActivity.LIST_VIEW_UPDATE_INTERVAL_IN_MILLIS); onView(first(allOf(withText(workResolvedComponentInfos.get(0) .getResolveInfoAt(0).activityInfo.applicationInfo.name), isCompletelyDisplayed()))) .perform(click()); @@ -616,8 +614,6 @@ public class ResolverActivityTest { onView(withText(R.string.resolver_work_tab)) .perform(click()); waitForIdle(); - // wait for the share sheet to expand - Thread.sleep(ChooserActivity.LIST_VIEW_UPDATE_INTERVAL_IN_MILLIS); onView(first(allOf( withText(workResolvedComponentInfos.get(0) .getResolveInfoAt(0).activityInfo.applicationInfo.name),