Check MediaMetadata bundle to verify keys are skipped

Added another check in test to make sure we don't add keys to bundle for fields with `null` values. PiperOrigin-RevId: 496948705
DiceTechnology · Jan 4, 2023 · 890fd0a · 890fd0a
1 parent 8ed5158
commit 890fd0a
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/library/common/src/test/java/com/google/android/exoplayer2/MediaMetadataTest.java b/library/common/src/test/java/com/google/android/exoplayer2/MediaMetadataTest.java
@@ -107,12 +107,17 @@ public void populate_populatesEveryField() {
   }
 
   @Test
-  public void createMinimalMediaMetadata_roundTripViaBundle_yieldsEqualInstance() {
+  public void toBundleSkipsDefaultValues_fromBundleRestoresThem() {
     MediaMetadata mediaMetadata = new MediaMetadata.Builder().build();
 
-    MediaMetadata mediaMetadataFromBundle =
-        MediaMetadata.CREATOR.fromBundle(mediaMetadata.toBundle());
+    Bundle mediaMetadataBundle = mediaMetadata.toBundle();
+
+    // check Bundle created above, contains no keys.
+    assertThat(mediaMetadataBundle.keySet()).isEmpty();
+
+    MediaMetadata mediaMetadataFromBundle = MediaMetadata.CREATOR.fromBundle(mediaMetadataBundle);
 
+    // check object retrieved from mediaMetadataBundle is equal to mediaMetadata.
     assertThat(mediaMetadataFromBundle).isEqualTo(mediaMetadata);
     // Extras is not implemented in MediaMetadata.equals(Object o).
     assertThat(mediaMetadataFromBundle.extras).isNull();