commit 826f0b01d282b35d07ce43319c492d1b03990f56
author chiachangwang <chiachangwang@google.com> Tue Jul 05 09:25:50 2022 +0000
committer chiachangwang <chiachangwang@google.com> Thu Jul 07 08:54:52 2022 +0000
tree 1f0d0fb7c6d5080c9e3bed105a1f451788de285b
parent 26de58ea790041b5c1c2d38471622c7376ee648d

Verify calling thread for intent receiver

Bug: 230548427
Test: atest VpnManagerServiceTest
Change-Id: Ibe5f7aba8e31dfecfb2eaac3fb109eb442879ede

tests/unit/java/com/android/server/VpnManagerServiceTest.java

diff --git a/tests/unit/java/com/android/server/VpnManagerServiceTest.java b/tests/unit/java/com/android/server/VpnManagerServiceTest.java
index 164f825..c814cc5 100644
--- a/tests/unit/java/com/android/server/VpnManagerServiceTest.java
+++ b/tests/unit/java/com/android/server/VpnManagerServiceTest.java
@@ -231,4 +231,14 @@
     private void onPackageRemoved(String packageName, int uid, boolean isReplacing) {
         onPackageRemoved(packageName, UserHandle.USER_SYSTEM, uid, isReplacing);
     }
+
+    @Test
+    public void testReceiveIntentFromNonHandlerThread() {
+        assertThrows(IllegalStateException.class, () ->
+                mIntentReceiver.onReceive(mContext, buildIntent(Intent.ACTION_PACKAGE_REMOVED,
+                        PKGS[0], UserHandle.USER_SYSTEM, PKG_UIDS[0], true /* isReplacing */)));
+
+        assertThrows(IllegalStateException.class, () ->
+                mUserPresentReceiver.onReceive(mContext, new Intent(Intent.ACTION_USER_PRESENT)));
+    }
 }