Fix DPM unit tests
Bug 26911599
Change-Id: I874c9cd4f63c79bc984777ef3b1b654a414c4911
diff --git a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java
index 83aa64e..dbcad4d 100644
--- a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java
+++ b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java
@@ -1459,6 +1459,10 @@
void settingsGlobalPutString(String name, String value) {
Settings.Global.putString(mContext.getContentResolver(), name, value);
}
+
+ void securityLogSetLoggingEnabledProperty(boolean enabled) {
+ SecurityLog.setLoggingEnabledProperty(enabled);
+ }
}
/**
@@ -8317,7 +8321,7 @@
private void disableDeviceLoggingIfNotCompliant() {
if (!isDeviceOwnerManagedSingleUserDevice()) {
- SecurityLog.setLoggingEnabledProperty(false);
+ mInjector.securityLogSetLoggingEnabledProperty(false);
Slog.w(LOG_TAG, "Device logging turned off as it's no longer a single user device.");
}
}
@@ -8328,7 +8332,7 @@
ensureDeviceOwnerManagingSingleUser(admin);
synchronized (this) {
- SecurityLog.setLoggingEnabledProperty(enabled);
+ mInjector.securityLogSetLoggingEnabledProperty(enabled);
if (enabled) {
mSecurityLogMonitor.start();
} else {
diff --git a/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java b/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java
index 90e4acf..bd37f4a 100644
--- a/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java
+++ b/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java
@@ -281,5 +281,15 @@
void settingsGlobalPutString(String name, String value) {
context.settings.settingsGlobalPutString(name, value);
}
+
+ @Override
+ int settingsGlobalGetInt(String name, int def) {
+ return context.settings.settingsGlobalGetInt(name, def);
+ }
+
+ @Override
+ void securityLogSetLoggingEnabledProperty(boolean enabled) {
+ context.settings.securityLogSetLoggingEnabledProperty(enabled);
+ }
}
}
diff --git a/services/tests/servicestests/src/com/android/server/devicepolicy/DpmMockContext.java b/services/tests/servicestests/src/com/android/server/devicepolicy/DpmMockContext.java
index 56667e5..6518732 100644
--- a/services/tests/servicestests/src/com/android/server/devicepolicy/DpmMockContext.java
+++ b/services/tests/servicestests/src/com/android/server/devicepolicy/DpmMockContext.java
@@ -191,6 +191,13 @@
void settingsGlobalPutString(String name, String value) {
}
+
+ int settingsGlobalGetInt(String name, int def) {
+ return 0;
+ }
+
+ void securityLogSetLoggingEnabledProperty(boolean enabled) {
+ }
}
public final Context realTestContext;