From f55befbfbe9f685554a094de6f0cbd34a170e914 Mon Sep 17 00:00:00 2001 From: Kyunglyul Hyun Date: Thu, 20 Feb 2020 19:57:43 +0900 Subject: [PATCH] MediaRouter2: Fix test bug Two set of preferred features could be the same in a different order. Test: atest mediaroutertest Bug: 149887171 Change-Id: I10f6878ad42cf49f2337d2d69f884038b144df45 --- .../android/mediaroutertest/MediaRouter2ManagerTest.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/media/tests/MediaRouter/src/com/android/mediaroutertest/MediaRouter2ManagerTest.java b/media/tests/MediaRouter/src/com/android/mediaroutertest/MediaRouter2ManagerTest.java index e80562b3fd1cd..346fd04c25c15 100644 --- a/media/tests/MediaRouter/src/com/android/mediaroutertest/MediaRouter2ManagerTest.java +++ b/media/tests/MediaRouter/src/com/android/mediaroutertest/MediaRouter2ManagerTest.java @@ -55,7 +55,6 @@ import org.junit.Test; import org.junit.runner.RunWith; import java.util.ArrayList; -import java.util.Collections; import java.util.HashMap; import java.util.List; import java.util.Map; @@ -296,7 +295,7 @@ public class MediaRouter2ManagerTest { String selectedSystemRouteId = MediaRouter2Utils.getOriginalId( mManager.getActiveSessions().get(0).getSelectedRoutes().get(0)); - Map routes = waitAndGetRoutesWithManager(Collections.emptyList()); + Map routes = waitAndGetRoutesWithManager(FEATURES_ALL); MediaRoute2Info volRoute = routes.get(selectedSystemRouteId); assertNotNull(volRoute); @@ -429,10 +428,11 @@ public class MediaRouter2ManagerTest { } @Override - public void onControlCategoriesChanged(String packageName, + public void onPreferredFeaturesChanged(String packageName, List preferredFeatures) { if (TextUtils.equals(mPackageName, packageName) - && preferredFeatures.equals(routeFeatures)) { + && preferredFeatures.size() == routeFeatures.size() + && preferredFeatures.containsAll(routeFeatures)) { featuresLatch.countDown(); } }