commit 7c92b643182dfec30f943b5f41f621e022b71580
author Andy Hung <hunga@google.com> Mon Feb 06 22:19:37 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Mon Feb 06 22:19:37 2023 +0000
tree 065ef077dbfef85ada4bf93b8192bb5da02ea37b
parent f39bc07a12c3d60d81be6d2d3f771e3a26d021ad
parent 6766ca4760937e3f81db056236b4412c7503f112

Merge "HidRawDeviceTest: Enable for device build" am: 698fe8448b am: f4871891b2 am: 6766ca4760

Original change: https://android-review.googlesource.com/c/platform/hardware/libhardware/+/2404797

Change-Id: I02d826405a4d2aa2d36d8bff8720d7639f3a8fed
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

modules/sensors/dynamic_sensor/Android.bp

diff --git a/modules/sensors/dynamic_sensor/Android.bp b/modules/sensors/dynamic_sensor/Android.bp
index bad6048..890da82 100644
--- a/modules/sensors/dynamic_sensor/Android.bp
+++ b/modules/sensors/dynamic_sensor/Android.bp
@@ -157,3 +157,28 @@
         "test/HidRawDeviceTest.cpp",
     ],
 }
+
+//
+// Android device test for HidRawDevice and HidRawSensor
+//
+// Assuming lunch target 1
+// $ cd test
+// $ mma -j .
+// $ adb push $ANDROID_BUILD_TOP/out/target/product/generic/vendor/bin/hidrawdevice_test /vendor/bin
+// $ adb shell hidrawdevice_test /dev/hidraw0
+//
+cc_binary {
+    name: "hidrawdevice_test",
+    defaults: ["dynamic_sensor_defaults"],
+
+    srcs: [
+        "test/HidRawDeviceTest.cpp",
+    ],
+
+    cflags: ["-DLOG_TO_CONSOLE=1"],
+
+    local_include_dirs: [
+        "test",
+        "HidUtils/test",
+    ],
+}

modules/sensors/dynamic_sensor/test/HidRawDeviceTest.cpp

diff --git a/modules/sensors/dynamic_sensor/test/HidRawDeviceTest.cpp b/modules/sensors/dynamic_sensor/test/HidRawDeviceTest.cpp
index 2a68e39..7522ca7 100644
--- a/modules/sensors/dynamic_sensor/test/HidRawDeviceTest.cpp
+++ b/modules/sensors/dynamic_sensor/test/HidRawDeviceTest.cpp
@@ -38,8 +38,7 @@
         std::unordered_set<unsigned int> interestedUsage{
                 ACCELEROMETER_3D, GYROMETER_3D, COMPASS_3D, CUSTOM};
 
-        SP(HidRawDevice) device =
-                std::make_shared<HidRawDevice>(std::string(devicePath), interestedUsage);
+        SP(HidRawDevice) device{new HidRawDevice(std::string(devicePath), interestedUsage)};
         const HidDevice::HidDeviceInfo &info = device->getDeviceInfo();
 
         LOG_V << "Sizeof descriptor: " << info.descriptor.size() << LOG_ENDL;
@@ -69,8 +68,8 @@
         // use HidRawSensor to operate the device, pick first digest
         //
         auto &reportDigest = device->mDigestVector[0];
-        SP(HidRawSensor) sensor = std::make_shared<HidRawSensor>(
-                device, reportDigest.fullUsage, reportDigest.packets);
+        SP(HidRawSensor) sensor{
+            new HidRawSensor(device, reportDigest.fullUsage, reportDigest.packets)};
 
         if (!sensor->isValid()) {
             LOG_E << "Sensor is not valid " << LOG_ENDL;