Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f5b6d4bd authored by Nathan Harold's avatar Nathan Harold
Browse files

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
parent 52014f02

tests/telephonytests/src/com/android/internal/telephony/TelephonyRegistryTest.java

+5 −0
Original line number Diff line number Diff line
@@ -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.