commit 77b2bf00f5c270940516f3aee32c89165dcbf4d1
author Connor O'Brien <connoro@google.com> Mon Feb 27 22:36:18 2017 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Mon Feb 27 22:36:18 2017 +0000
tree 7435da29ee8d22b8c83d453ed702e802280ca2f8
parent 2c5e21c69c931a90dcb9b4ea893157be17eaf8d5
parent 990ec56cdb8ce00489c7994c16e839e8060e1e0e

Merge "Fix bugs in power HAL VTS test"

power/1.0/vts/functional/VtsHalPowerV1_0TargetTest.cpp

diff --git a/power/1.0/vts/functional/VtsHalPowerV1_0TargetTest.cpp b/power/1.0/vts/functional/VtsHalPowerV1_0TargetTest.cpp
index 6783961..7a34f00 100644
--- a/power/1.0/vts/functional/VtsHalPowerV1_0TargetTest.cpp
+++ b/power/1.0/vts/functional/VtsHalPowerV1_0TargetTest.cpp
@@ -19,6 +19,7 @@
 
 #include <cutils/properties.h>
 
+#include <android-base/unique_fd.h>
 #include <android/hardware/power/1.0/IPower.h>
 
 #include <gtest/gtest.h>
@@ -33,6 +34,7 @@
 using ::android::hardware::hidl_vec;
 using ::android::hardware::Return;
 using ::android::sp;
+using ::android::base::unique_fd;
 
 using std::vector;
 
@@ -69,23 +71,24 @@
 TEST_F(PowerHidlTest, TryDifferentGovernors) {
   Return<void> ret;
 
-  int fd1 = open(CPU_GOVERNOR_PATH, O_RDWR);
-  int fd2 = open(AVAILABLE_GOVERNORS_PATH, O_RDONLY);
+  unique_fd fd1(open(CPU_GOVERNOR_PATH, O_RDWR));
+  unique_fd fd2(open(AVAILABLE_GOVERNORS_PATH, O_RDONLY));
   if (fd1 < 0 || fd2 < 0) {
-      // Files don't exist, so skip the rest of the test case
-      SUCCEED();
+    // Files don't exist, so skip the rest of the test case
+    SUCCEED();
   }
 
   char old_governor[80];
   ASSERT_LE(0, read(fd1, old_governor, 80));
 
   char governors[1024];
-  ASSERT_LE(0, read(fd2, governors, 1024));
-  close(fd2);
+  unsigned len = read(fd2, governors, 1024);
+  ASSERT_LE(0, len);
+  governors[len] = '\0';
 
   char *saveptr;
-  char *name = strtok_r(governors, " ", &saveptr);
-  while (name && strlen(name) > 1) {
+  char *name = strtok_r(governors, " \n", &saveptr);
+  while (name) {
     ASSERT_LE(0, write(fd1, name, strlen(name)));
     ret = power->setInteractive(true);
     ASSERT_TRUE(ret.isOk());
@@ -99,11 +102,10 @@
     power->powerHint(PowerHint::LAUNCH, 1);
     power->powerHint(PowerHint::LAUNCH, 0);
 
-    name = strtok_r(NULL, " ", &saveptr);
+    name = strtok_r(NULL, " \n", &saveptr);
   }
 
   ASSERT_LE(0, write(fd1, old_governor, strlen(old_governor)));
-  close(fd1);
 }
 
 // Sanity check Power::powerHint on good and bad inputs.