commit 40d9d9ee7b654546597a0a710a5ed38092926788
author Shunkai Yao <yaoshunkai@google.com> Mon Mar 25 17:52:34 2024 +0000
committer Shunkai Yao <yaoshunkai@google.com> Sat Mar 30 00:13:23 2024 +0000
tree 2dc5a253a902d902a5bc28369a2ca87176ca6c69
parent 6507376fb940198e8c39ad6d5345d132b1dd7219

Effect VTS: skip VolumeDataTest for U

Volume effect implementation was not ready on early U releases

Bug: 327497904
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: I7e909cdabbd167bd2e012dbe378ccb3402d33f53

audio/aidl/vts/VtsHalVolumeTargetTest.cpp

diff --git a/audio/aidl/vts/VtsHalVolumeTargetTest.cpp b/audio/aidl/vts/VtsHalVolumeTargetTest.cpp
index 1c1489d..4597b39 100644
--- a/audio/aidl/vts/VtsHalVolumeTargetTest.cpp
+++ b/audio/aidl/vts/VtsHalVolumeTargetTest.cpp
@@ -140,7 +140,9 @@
 class VolumeDataTest : public ::testing::TestWithParam<VolumeDataTestParam>,
                        public VolumeControlHelper {
   public:
-    VolumeDataTest() {
+    VolumeDataTest()
+        : kVsrApiLevel(
+                  android::base::GetIntProperty("ro.vendor.api_level", __ANDROID_API_FUTURE__)) {
         std::tie(mFactory, mDescriptor) = GetParam();
         mInput.resize(kBufferSize);
         mInputMag.resize(mTestFrequencies.size());
@@ -165,13 +167,17 @@
 
     void SetUp() override {
         SKIP_TEST_IF_DATA_UNSUPPORTED(mDescriptor.common.flags);
+        // Skips test fixture if api_level <= 34 (__ANDROID_API_U__).
+        if (kVsrApiLevel <= __ANDROID_API_U__) GTEST_SKIP();
         ASSERT_NO_FATAL_FAILURE(SetUpVolumeControl());
     }
     void TearDown() override {
         SKIP_TEST_IF_DATA_UNSUPPORTED(mDescriptor.common.flags);
+        if (kVsrApiLevel <= __ANDROID_API_U__) GTEST_SKIP();
         TearDownVolumeControl();
     }
 
+    const int kVsrApiLevel;
     static constexpr int kMaxAudioSample = 1;
     static constexpr int kTransitionDuration = 300;
     static constexpr int kNPointFFT = 32768;