diff options
| author | Kevin Rocard <krocard@google.com> | 2018-06-28 18:00:05 -0700 |
|---|---|---|
| committer | Keun Soo Yim <yim@google.com> | 2018-08-28 17:46:41 +0000 |
| commit | 8073d8e41c175ccd15129a280275fd8ac6efc635 (patch) | |
| tree | 4fe693b64fe912c434b80468524db21bdb973f8a /audio | |
| parent | dd06d73cabd7ca0df542e7384978c4c6b5087632 (diff) | |
| download | android_hardware_interfaces-8073d8e41c175ccd15129a280275fd8ac6efc635.tar.gz android_hardware_interfaces-8073d8e41c175ccd15129a280275fd8ac6efc635.tar.bz2 android_hardware_interfaces-8073d8e41c175ccd15129a280275fd8ac6efc635.zip | |
Audio VTS was testing wrong uninitialized variable due to typo
The test was testing a uninitialized variable instead of testing
the result of the tested method.
Bug: 110963314
Test: vts-tradefed run commandAndExit vts --module VtsHalAudioV2_0Target
Merged-In: I9a212eef690ae627fd7f7dbfaf4a0b4047c491b3
Change-Id: I9a212eef690ae627fd7f7dbfaf4a0b4047c491b3
Signed-off-by: Kevin Rocard <krocard@google.com>
(cherry picked from commit 72641e7ad014a011562c3fd16444b919d32aeb63)
Diffstat (limited to 'audio')
| -rw-r--r-- | audio/core/4.0/vts/functional/AudioPrimaryHidlHalTest.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/audio/core/4.0/vts/functional/AudioPrimaryHidlHalTest.cpp b/audio/core/4.0/vts/functional/AudioPrimaryHidlHalTest.cpp index c764ea625..761330f2b 100644 --- a/audio/core/4.0/vts/functional/AudioPrimaryHidlHalTest.cpp +++ b/audio/core/4.0/vts/functional/AudioPrimaryHidlHalTest.cpp @@ -555,11 +555,11 @@ TEST_F(AudioPrimaryHidlTest, SetConnectedState) { address.device = deviceType; auto ret = device->setConnectedState(address, state); ASSERT_TRUE(ret.isOk()); - if (res == Result::NOT_SUPPORTED) { + if (ret == Result::NOT_SUPPORTED) { doc::partialTest("setConnectedState is not supported"); return; } - ASSERT_OK(res); + ASSERT_OK(ret); } } } |