commit d20554f16eaa935d06ea2695455fc8490d2aa095
author Santiago Seifert <aquilescanta@google.com> Mon Feb 15 16:31:29 2021 +0000
committer Santiago Seifert <aquilescanta@google.com> Thu Apr 08 15:20:06 2021 +0000
tree bef6bdc8fac07226d46c67757e7bc6ef36f34d8a
parent 48ddde068dc0ebb313644450d56a74e49cae8bd6

Add tests for MediaPlayer.setOnRtpRxNoticeListener

Bug: 169965769
Test: Pending atest line.
Change-Id: Iab09bb596b82e293339bf357d9f9fe027b0c903d

tests/tests/media/src/android/media/cts/MediaPlayerTest.java

diff --git a/tests/tests/media/src/android/media/cts/MediaPlayerTest.java b/tests/tests/media/src/android/media/cts/MediaPlayerTest.java
index 656cd85..8a5c35c 100644
--- a/tests/tests/media/src/android/media/cts/MediaPlayerTest.java
+++ b/tests/tests/media/src/android/media/cts/MediaPlayerTest.java
@@ -2558,4 +2558,26 @@
         mMediaPlayer.start();
         assertTrue(mOnErrorCalled.waitForSignal());
     }
+
+    @Presubmit
+    public void testSetOnRtpRxNoticeListenerWithoutPermission() {
+        try {
+            mMediaPlayer.setOnRtpRxNoticeListener(
+                    mContext, Runnable::run, (mp, noticeType, params) -> {});
+            fail();
+        } catch (IllegalArgumentException e) {
+            // Expected. We don't have the required permission.
+        }
+    }
+
+    @Presubmit
+    public void testSetOnRtpRxNoticeListenerWithPermission() {
+        try {
+            getInstrumentation().getUiAutomation().adoptShellPermissionIdentity();
+            mMediaPlayer.setOnRtpRxNoticeListener(
+                    mContext, Runnable::run, (mp, noticeType, params) -> {});
+        } finally {
+            getInstrumentation().getUiAutomation().dropShellPermissionIdentity();
+        }
+    }
 }