Merge "Add device info for filtered players" into rvc-dev am: 3c336c4b64
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/12052991 Change-Id: Ib48fa3824a96ec96fca80e0f2d0edc47f373438e
This commit is contained in:
@@ -62,9 +62,9 @@ class MediaDataCombineLatest @Inject constructor(
|
|||||||
*/
|
*/
|
||||||
fun getData(): Map<String, MediaData> {
|
fun getData(): Map<String, MediaData> {
|
||||||
return entries.filter {
|
return entries.filter {
|
||||||
(key, pair) -> pair.first != null
|
(key, pair) -> pair.first != null && pair.second != null
|
||||||
}.mapValues {
|
}.mapValues {
|
||||||
(key, pair) -> pair.first!!
|
(key, pair) -> pair.first!!.copy(device = pair.second)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -39,6 +39,7 @@ import org.mockito.ArgumentCaptor;
|
|||||||
import org.mockito.Mock;
|
import org.mockito.Mock;
|
||||||
|
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
|
import java.util.Map;
|
||||||
|
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(AndroidTestingRunner.class)
|
@RunWith(AndroidTestingRunner.class)
|
||||||
@@ -159,6 +160,18 @@ public class MediaDataCombineLatestTest extends SysuiTestCase {
|
|||||||
verify(mListener).onMediaDataLoaded(eq("NEW_KEY"), any(), captor.capture());
|
verify(mListener).onMediaDataLoaded(eq("NEW_KEY"), any(), captor.capture());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void getDataIncludesDevice() {
|
||||||
|
// GIVEN that device and media events have been received
|
||||||
|
mDeviceListener.onMediaDeviceChanged(KEY, mDeviceData);
|
||||||
|
mDataListener.onMediaDataLoaded(KEY, null, mMediaData);
|
||||||
|
|
||||||
|
// THEN the result of getData includes device info
|
||||||
|
Map<String, MediaData> results = mManager.getData();
|
||||||
|
assertThat(results.get(KEY)).isNotNull();
|
||||||
|
assertThat(results.get(KEY).getDevice()).isEqualTo(mDeviceData);
|
||||||
|
}
|
||||||
|
|
||||||
private MediaDataManager.Listener captureDataListener() {
|
private MediaDataManager.Listener captureDataListener() {
|
||||||
ArgumentCaptor<MediaDataManager.Listener> captor = ArgumentCaptor.forClass(
|
ArgumentCaptor<MediaDataManager.Listener> captor = ArgumentCaptor.forClass(
|
||||||
MediaDataManager.Listener.class);
|
MediaDataManager.Listener.class);
|
||||||
|
|||||||
Reference in New Issue
Block a user