Merge "Don't assert while a callback is registered" am: 6a022ad1c4

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1654108

Change-Id: Ia1cd0ac9eb42d2f1740d53a64b8c907bb4ec7b65
This commit is contained in:
Anthony Stange
2021-03-25 20:30:43 +00:00
committed by Automerger Merge Worker

View File

@@ -462,6 +462,7 @@ TEST_P(SensorsHidlTest, InjectSensorEventData) {
// Wait for events to be written back to the Event FMQ // Wait for events to be written back to the Event FMQ
callback.waitForEvents(sensors, milliseconds(1000) /* timeout */); callback.waitForEvents(sensors, milliseconds(1000) /* timeout */);
getEnvironment()->unregisterCallback();
for (const auto& s : sensors) { for (const auto& s : sensors) {
auto events = callback.getEvents(s.sensorHandle); auto events = callback.getEvents(s.sensorHandle);
@@ -485,7 +486,6 @@ TEST_P(SensorsHidlTest, InjectSensorEventData) {
ASSERT_EQ(lastEvent.u.vec3.status, injectedEvent.u.vec3.status); ASSERT_EQ(lastEvent.u.vec3.status, injectedEvent.u.vec3.status);
} }
getEnvironment()->unregisterCallback();
ASSERT_EQ(Result::OK, getSensors()->setOperationMode(OperationMode::NORMAL)); ASSERT_EQ(Result::OK, getSensors()->setOperationMode(OperationMode::NORMAL));
} }
@@ -603,7 +603,7 @@ void SensorsHidlTest::runFlushTest(const std::vector<SensorInfoType>& sensors, b
<< " type=" << static_cast<int>(sensor.type) << " name=" << sensor.name); << " type=" << static_cast<int>(sensor.type) << " name=" << sensor.name);
Result flushResult = flush(sensor.sensorHandle); Result flushResult = flush(sensor.sensorHandle);
ASSERT_EQ(flushResult, expectedResponse); EXPECT_EQ(flushResult, expectedResponse);
} }
} }