Merge "Unregister all the receivers during test teardown." into tm-qpr-dev am: c1fee021cc
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20255704 Change-Id: Idd1c964a9f37f462b3a566b29b16fb845138e825 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -113,8 +113,10 @@ import android.app.NotificationManager;
|
|||||||
import android.app.usage.NetworkStats;
|
import android.app.usage.NetworkStats;
|
||||||
import android.app.usage.NetworkStatsManager;
|
import android.app.usage.NetworkStatsManager;
|
||||||
import android.app.usage.UsageStatsManagerInternal;
|
import android.app.usage.UsageStatsManagerInternal;
|
||||||
|
import android.content.BroadcastReceiver;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.content.Intent;
|
import android.content.Intent;
|
||||||
|
import android.content.IntentFilter;
|
||||||
import android.content.pm.ApplicationInfo;
|
import android.content.pm.ApplicationInfo;
|
||||||
import android.content.pm.IPackageManager;
|
import android.content.pm.IPackageManager;
|
||||||
import android.content.pm.PackageInfo;
|
import android.content.pm.PackageInfo;
|
||||||
@@ -134,6 +136,7 @@ import android.net.NetworkTemplate;
|
|||||||
import android.net.TelephonyNetworkSpecifier;
|
import android.net.TelephonyNetworkSpecifier;
|
||||||
import android.net.wifi.WifiInfo;
|
import android.net.wifi.WifiInfo;
|
||||||
import android.os.Binder;
|
import android.os.Binder;
|
||||||
|
import android.os.Build;
|
||||||
import android.os.Handler;
|
import android.os.Handler;
|
||||||
import android.os.INetworkManagementService;
|
import android.os.INetworkManagementService;
|
||||||
import android.os.PersistableBundle;
|
import android.os.PersistableBundle;
|
||||||
@@ -152,6 +155,7 @@ import android.telephony.TelephonyManager;
|
|||||||
import android.test.suitebuilder.annotation.MediumTest;
|
import android.test.suitebuilder.annotation.MediumTest;
|
||||||
import android.text.TextUtils;
|
import android.text.TextUtils;
|
||||||
import android.util.ArrayMap;
|
import android.util.ArrayMap;
|
||||||
|
import android.util.ArraySet;
|
||||||
import android.util.DataUnit;
|
import android.util.DataUnit;
|
||||||
import android.util.Log;
|
import android.util.Log;
|
||||||
import android.util.Pair;
|
import android.util.Pair;
|
||||||
@@ -176,6 +180,7 @@ import com.google.common.util.concurrent.AbstractFuture;
|
|||||||
import libcore.io.Streams;
|
import libcore.io.Streams;
|
||||||
|
|
||||||
import org.junit.After;
|
import org.junit.After;
|
||||||
|
import org.junit.Assume;
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
import org.junit.Rule;
|
import org.junit.Rule;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
@@ -286,6 +291,8 @@ public class NetworkPolicyManagerServiceTest {
|
|||||||
private NetworkPolicyListenerAnswer mPolicyListener;
|
private NetworkPolicyListenerAnswer mPolicyListener;
|
||||||
private NetworkPolicyManagerService mService;
|
private NetworkPolicyManagerService mService;
|
||||||
|
|
||||||
|
private final ArraySet<BroadcastReceiver> mRegisteredReceivers = new ArraySet<>();
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* In some of the tests while initializing NetworkPolicyManagerService,
|
* In some of the tests while initializing NetworkPolicyManagerService,
|
||||||
* ACTION_RESTRICT_BACKGROUND_CHANGED is broadcasted. This is for capturing that broadcast.
|
* ACTION_RESTRICT_BACKGROUND_CHANGED is broadcasted. This is for capturing that broadcast.
|
||||||
@@ -437,6 +444,21 @@ public class NetworkPolicyManagerServiceTest {
|
|||||||
public void enforceCallingOrSelfPermission(String permission, String message) {
|
public void enforceCallingOrSelfPermission(String permission, String message) {
|
||||||
// Assume that we're AID_SYSTEM
|
// Assume that we're AID_SYSTEM
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Intent registerReceiver(BroadcastReceiver receiver,
|
||||||
|
IntentFilter filter, String broadcastPermission, Handler scheduler) {
|
||||||
|
mRegisteredReceivers.add(receiver);
|
||||||
|
return super.registerReceiver(receiver, filter, broadcastPermission, scheduler);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Intent registerReceiverForAllUsers(BroadcastReceiver receiver,
|
||||||
|
IntentFilter filter, String broadcastPermission, Handler scheduler) {
|
||||||
|
mRegisteredReceivers.add(receiver);
|
||||||
|
return super.registerReceiverForAllUsers(receiver, filter, broadcastPermission,
|
||||||
|
scheduler);
|
||||||
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
setNetpolicyXml(context);
|
setNetpolicyXml(context);
|
||||||
@@ -557,6 +579,13 @@ public class NetworkPolicyManagerServiceTest {
|
|||||||
RecurrenceRule.sClock = Clock.systemDefaultZone();
|
RecurrenceRule.sClock = Clock.systemDefaultZone();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@After
|
||||||
|
public void unregisterReceivers() throws Exception {
|
||||||
|
for (BroadcastReceiver receiver : mRegisteredReceivers) {
|
||||||
|
mServiceContext.unregisterReceiver(receiver);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testTurnRestrictBackgroundOn() throws Exception {
|
public void testTurnRestrictBackgroundOn() throws Exception {
|
||||||
assertRestrictBackgroundOff();
|
assertRestrictBackgroundOff();
|
||||||
|
|||||||
Reference in New Issue
Block a user