Skip a Broken Test For READ_PRECISE_PHONE_STATE

Currently this test doesn't support proper checking for
methods that require multiple permissions. This can't
be fixed in RVC because it requires fixing the soft
denial logic reported in b/159082270. Instead, just
disable the test for now.

Bug: 159075096
Test: atest TelephonyRegistryTest
Change-Id: Id0e4ac402c6338909c7050343c1ba455d3a1dc1f

1 files changed, 5 insertions, 0 deletions

diff --git a/tests/telephonytests/src/com/android/internal/telephony/TelephonyRegistryTest.java b/tests/telephonytests/src/com/android/internal/telephony/TelephonyRegistryTest.java
index 657eea18d6..626dee2d86 100644
--- a/tests/telephonytests/src/com/android/internal/telephony/TelephonyRegistryTest.java
+++ b/tests/telephonytests/src/com/android/internal/telephony/TelephonyRegistryTest.java
@@ -53,6 +53,7 @@ import com.android.server.TelephonyRegistry;
 
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.Mock;
@@ -373,6 +374,10 @@ public class TelephonyRegistryTest extends TelephonyTest {
     /**
      * Test listen to events that require READ_PRECISE_PHONE_STATE permission.
      */
+    // FIXME(b/159082270) - Simply not granting location permission doesn't fix the test because
+    // Location is soft-denied to apps that aren't in the foreground, and soft-denial currently
+    // short-circuits the test.
+    @Ignore("Skip due to b/159082270")
     @Test
     public void testReadPrecisePhoneStatePermission() {
         // Clear all permission grants for test.