diff --git a/tests/unit/java/com/android/server/net/NetworkStatsServiceTest.java b/tests/unit/java/com/android/server/net/NetworkStatsServiceTest.java index ea35c31613..45f033c3b9 100644 --- a/tests/unit/java/com/android/server/net/NetworkStatsServiceTest.java +++ b/tests/unit/java/com/android/server/net/NetworkStatsServiceTest.java @@ -73,7 +73,9 @@ import static org.mockito.ArgumentMatchers.anyBoolean; import static org.mockito.ArgumentMatchers.anyInt; import static org.mockito.ArgumentMatchers.anyLong; import static org.mockito.ArgumentMatchers.anyString; +import static org.mockito.Matchers.eq; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.never; import static org.mockito.Mockito.reset; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -113,7 +115,10 @@ import androidx.test.InstrumentationRegistry; import androidx.test.filters.SmallTest; import com.android.internal.util.test.BroadcastInterceptingContext; +import com.android.net.module.util.IBpfMap; import com.android.net.module.util.LocationPermissionChecker; +import com.android.net.module.util.Struct.U32; +import com.android.net.module.util.Struct.U8; import com.android.server.net.NetworkStatsService.AlertObserver; import com.android.server.net.NetworkStatsService.NetworkStatsSettings; import com.android.server.net.NetworkStatsService.NetworkStatsSettings.Config; @@ -192,6 +197,7 @@ public class NetworkStatsServiceTest extends NetworkStatsBaseTest { private HandlerThread mHandlerThread; @Mock private LocationPermissionChecker mLocationPermissionChecker; + private @Mock IBpfMap mUidCounterSetMap; private NetworkStatsService mService; private INetworkStatsSession mSession; @@ -347,6 +353,11 @@ public class NetworkStatsServiceTest extends NetworkStatsBaseTest { @NonNull Context ctx, @NonNull Handler handler) { return mBpfInterfaceMapUpdater; } + + @Override + public IBpfMap getUidCounterSetMap() { + return mUidCounterSetMap; + } }; } @@ -472,8 +483,11 @@ public class NetworkStatsServiceTest extends NetworkStatsBaseTest { .insertEntry(TEST_IFACE, UID_RED, SET_FOREGROUND, 0xFAAD, 256L, 2L, 128L, 1L, 0L) .insertEntry(TEST_IFACE, UID_BLUE, SET_DEFAULT, TAG_NONE, 128L, 1L, 128L, 1L, 0L)); mService.setUidForeground(UID_RED, false); + verify(mUidCounterSetMap, never()).deleteEntry(any()); mService.incrementOperationCount(UID_RED, 0xFAAD, 4); mService.setUidForeground(UID_RED, true); + verify(mUidCounterSetMap).updateEntry( + eq(new U32(UID_RED)), eq(new U8((short) SET_FOREGROUND))); mService.incrementOperationCount(UID_RED, 0xFAAD, 6); forcePollAndWaitForIdle(); @@ -1070,6 +1084,8 @@ public class NetworkStatsServiceTest extends NetworkStatsBaseTest { .insertEntry(TEST_IFACE, UID_RED, SET_FOREGROUND, TAG_NONE, 32L, 2L, 32L, 2L, 0L) .insertEntry(TEST_IFACE, UID_RED, SET_FOREGROUND, 0xFAAD, 1L, 1L, 1L, 1L, 0L)); mService.setUidForeground(UID_RED, true); + verify(mUidCounterSetMap).updateEntry( + eq(new U32(UID_RED)), eq(new U8((short) SET_FOREGROUND))); mService.incrementOperationCount(UID_RED, 0xFAAD, 1); forcePollAndWaitForIdle();