commit bab8aede75389e993c019c3977088201c37114c6
author Tej Singh <singhtejinder@google.com> Fri Jul 10 00:08:57 2020 -0700
committer Tej Singh <singhtejinder@google.com> Fri Jul 10 00:10:53 2020 -0700
tree 965beac21aaf91a6e60fe899917ea60be158c19c
parent 3bc4098e48be0e80196c2795ae3e2f5c7b659cce

Allow empty kernel wakelock names

The kernel wakelock api does not require the client to provide a name,
so we cannot enforce that the name is not the empty string.

Test: atest android.cts.statsd.atom.HostAtomTests#testKernelWakelock
Bug: 159673282
Change-Id: Icf979ebe2417fa682d59e802999dc60e85303281

hostsidetests/statsd/src/android/cts/statsd/atom/HostAtomTests.java

diff --git a/hostsidetests/statsd/src/android/cts/statsd/atom/HostAtomTests.java b/hostsidetests/statsd/src/android/cts/statsd/atom/HostAtomTests.java
index d7f8df1..ee6f324 100644
--- a/hostsidetests/statsd/src/android/cts/statsd/atom/HostAtomTests.java
+++ b/hostsidetests/statsd/src/android/cts/statsd/atom/HostAtomTests.java
@@ -468,12 +468,14 @@
 
         List<Atom> data = getGaugeMetricDataList();
 
-        Atom atom = data.get(0);
-        assertThat(atom.getKernelWakelock().getName()).isNotEmpty();
-        assertThat(atom.getKernelWakelock().hasCount()).isTrue();
-        assertThat(atom.getKernelWakelock().hasVersion()).isTrue();
-        assertThat(atom.getKernelWakelock().getVersion()).isGreaterThan(0);
-        assertThat(atom.getKernelWakelock().hasTimeMicros()).isTrue();
+        assertThat(data).isNotEmpty();
+        for (Atom atom : data) {
+            assertThat(atom.getKernelWakelock().hasName()).isTrue();
+            assertThat(atom.getKernelWakelock().hasCount()).isTrue();
+            assertThat(atom.getKernelWakelock().hasVersion()).isTrue();
+            assertThat(atom.getKernelWakelock().getVersion()).isGreaterThan(0);
+            assertThat(atom.getKernelWakelock().hasTimeMicros()).isTrue();
+        }
     }
 
     // Returns true iff either |WAKE_LOCK_FILE| or |WAKE_SOURCES_FILE| exists.