| commit | 2aa353e91b8f2c51733f7bc1650d2981bda9b710 | [log] [tgz] |
|---|---|---|
| author | Author Gustav Svensson <gustav.svensson@volvo.corp-partner.google.com> | Tue Jul 07 15:06:01 2020 +0200 |
| committer | Gustav Svensson <gustav.svensson@volvo.corp-partner.google.com> | Tue Aug 04 07:25:13 2020 +0000 |
| tree | 62f79029ab5a46a41ab8622bb9357546a427d181 | |
| parent | 1b5373aeab11d828e055ded601cdaee64baa1c12 [diff] |
Handle empty results in CtsDeviceIdleHostTestCases Tests gets syntax error when trying to split an empty list which results from when cmd deviceidle sys-whitelist results is only one package. Solved by returning before trying to split if string is empty. https://partnerissuetracker.corp.google.com/issues/155116999 Test: ran tc on devices with one or many whitelisted packages. Signed-off-by: Gustav Svensson <gustav.svensson@volvo.corp-partner.google.com> Change-Id: Ia13b6bb0df65c1cca7da1632fe9a2b7448125b40
diff --git a/hostsidetests/deviceidle/src/com/android/cts/deviceidle/DeviceIdleWhitelistTest.java b/hostsidetests/deviceidle/src/com/android/cts/deviceidle/DeviceIdleWhitelistTest.java index bdba196..def9d75 100644 --- a/hostsidetests/deviceidle/src/com/android/cts/deviceidle/DeviceIdleWhitelistTest.java +++ b/hostsidetests/deviceidle/src/com/android/cts/deviceidle/DeviceIdleWhitelistTest.java
@@ -91,7 +91,9 @@ final List<String> packages = new ArrayList<>(); for (String line : output.split("\n")) { final int i = line.indexOf(','); - packages.add(line.substring(0, i)); + if (i > 0) { + packages.add(line.substring(0, i)); + } } return packages; }