summaryrefslogtreecommitdiff
path: root/services/robotests/backup
diff options
context:
space:
mode:
author Sarp Misoglu <sarpm@google.com> 2025-01-07 17:26:25 +0000
committer Sarp Misoglu <sarpm@google.com> 2025-01-09 22:02:33 +0000
commit8a7102c166fe464256b4a1daa9684ca8b1212931 (patch)
treed678402017b07809dd6feab43b1682b0a407e062 /services/robotests/backup
parent2a5d587f7ac73b0a51efc8d907e991b9111138e6 (diff)
Fix debug logging in the backup service
The existing DEBUG and DEBUG_SCHEDULING had been set to true (maybe accidentally?) for 8+ years. Whereas MORE_DEBUG is false as expected. In this CL we: 1) Remove existing DEBUG and DEBUG_SCHEDULING checks. This is a no-op. 2) Rename MORE_DEBUG to DEBUG. Also no-op. 3) Make any Log.v log that isn't guarded by if(DEBUG) a Log.d instead. Log.v logs should always be guarded by a default false constant so the compiler strips out the Strings from the binary. Test: n/a Flag: EXEMPT no-op refactor Change-Id: I117f7fe04da1a62c3ad42d71b080fed6454f73af
Diffstat (limited to 'services/robotests/backup')
-rw-r--r--services/robotests/backup/src/com/android/server/backup/BackupManagerServiceRoboTest.java9
-rw-r--r--services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupReporterTest.java4
2 files changed, 6 insertions, 7 deletions
diff --git a/services/robotests/backup/src/com/android/server/backup/BackupManagerServiceRoboTest.java b/services/robotests/backup/src/com/android/server/backup/BackupManagerServiceRoboTest.java
index b80d68de0f2e..ba64ed44cbb5 100644
--- a/services/robotests/backup/src/com/android/server/backup/BackupManagerServiceRoboTest.java
+++ b/services/robotests/backup/src/com/android/server/backup/BackupManagerServiceRoboTest.java
@@ -1518,14 +1518,13 @@ public class BackupManagerServiceRoboTest {
}
/**
- * Test verifying that {@link BackupManagerService#MORE_DEBUG} is set to {@code false}. This is
+ * Test verifying that {@link BackupManagerService#DEBUG} is set to {@code false}. This is
* specifically to prevent overloading the logs in production.
*/
@Test
- public void testMoreDebug_isFalse() throws Exception {
- boolean moreDebug = BackupManagerService.MORE_DEBUG;
-
- assertThat(moreDebug).isFalse();
+ public void testDebug_isFalse() {
+ boolean debug = BackupManagerService.DEBUG;
+ assertThat(debug).isFalse();
}
/** Test that the constructor handles {@code null} parameters. */
diff --git a/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupReporterTest.java b/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupReporterTest.java
index 2db2438b9a21..2749b0acf193 100644
--- a/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupReporterTest.java
+++ b/services/robotests/backup/src/com/android/server/backup/keyvalue/KeyValueBackupReporterTest.java
@@ -57,8 +57,8 @@ public class KeyValueBackupReporterTest {
}
@Test
- public void testMoreDebug_isFalse() {
- assertThat(KeyValueBackupReporter.MORE_DEBUG).isFalse();
+ public void testDebug_isFalse() {
+ assertThat(KeyValueBackupReporter.DEBUG).isFalse();
}
@Test