diff options
author | Ram Mohan M | 2017-10-11 05:08:25 -0500 |
---|---|---|
committer | Pawin Vongmasa | 2017-10-17 06:34:30 -0500 |
commit | a0698429dd47e2f029866765c6980bc8720a7acb (patch) | |
tree | 148aca952162336840b6e7d1441c49440cc4e94e /media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp | |
parent | 3021f1fafbfb1a4b464effa543aee381f42d8080 (diff) | |
download | platform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.tar.gz platform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.tar.xz platform-hardware-interfaces-a0698429dd47e2f029866765c6980bc8720a7acb.zip |
bug fix: during fatal fails bypass freeNode() calls
If test application undergoes a fatal failure bypass all the omx
api calls till the application safely returns.
Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO
Bug: 63796949
Change-Id: If5a33672a31665db913e53c6f8ba8bea6cfd3ebd
Diffstat (limited to 'media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp')
-rw-r--r-- | media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp b/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp index 9e185f27..38cdcd65 100644 --- a/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp +++ b/media/omx/1.0/vts/functional/audio/VtsHalMediaOmxV1_0TargetAudioDecTest.cpp | |||
@@ -230,6 +230,10 @@ class AudioDecHidlTest : public ::testing::VtsHalHidlTargetTestBase { | |||
230 | 230 | ||
231 | virtual void TearDown() override { | 231 | virtual void TearDown() override { |
232 | if (omxNode != nullptr) { | 232 | if (omxNode != nullptr) { |
233 | // If you have encountered a fatal failure, it is possible that | ||
234 | // freeNode() will not go through. Instead of hanging the app. | ||
235 | // let it pass through and report errors | ||
236 | if (::testing::Test::HasFatalFailure()) return; | ||
233 | EXPECT_TRUE((omxNode->freeNode()).isOk()); | 237 | EXPECT_TRUE((omxNode->freeNode()).isOk()); |
234 | omxNode = nullptr; | 238 | omxNode = nullptr; |
235 | } | 239 | } |