commit 1c8a5a1b6aeaba7c7143162e409bd75dd541421e
author Lais Andrade <lsandrade@google.com> Wed Nov 03 16:47:32 2021 +0000
committer Lais Andrade <lsandrade@google.com> Tue Nov 09 16:40:51 2021 +0000
tree 1c00c24693e90e62f60e085ab10828d8bd3ec891
parent 47551ec9d82c098356d022f471cc6d464db9ff05

Check primitive duration is positive for supported primitives

Fix: 203629838
Test: VtsHalVibratorTargetTest
Change-Id: I2e534f4589c9f594884385a99051c06f518103f7

vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp

diff --git a/vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp b/vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp
index 53f8c0e..09cc21b 100644
--- a/vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp
+++ b/vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp
@@ -417,6 +417,9 @@
 
             if (isPrimitiveSupported) {
                 EXPECT_EQ(Status::EX_NONE, status.exceptionCode());
+                if (primitive != CompositePrimitive::NOOP) {
+                    ASSERT_GT(duration, 0) << toString(primitive) << " " << duration;
+                }
             } else {
                 EXPECT_TRUE(isUnknownOrUnsupported(status)) << status;
             }