summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHsin-Yi Chen <hsinyichen@google.com>2023-11-10 07:46:37 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2023-11-10 07:46:37 +0000
commitdfc1ecae2307dd79d982f5f0f8f65a8f26c325ce (patch)
tree94eab7b8080661c305b10fa1e6fc08bffad23d83
parente2060957a4fbbe65d52327c33002bebfca632bca (diff)
parentf9bd962baa8e9e00461b77c5c6f183bf8e489b2b (diff)
downloadvndk-dfc1ecae2307dd79d982f5f0f8f65a8f26c325ce.tar.gz
Skip VNDK tests if ro.vndk.version is empty am: f9bd962baa
Original change: https://android-review.googlesource.com/c/platform/test/vts-testcase/vndk/+/2824232 Change-Id: Id84595ed21dbf81d7ee2d4e505ca1488b57c6863 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--abi/vts_vndk_abi_test.py3
-rw-r--r--dependency/vts_vndk_dependency_test.py2
-rw-r--r--files/vts_vndk_files_test.py5
3 files changed, 9 insertions, 1 deletions
diff --git a/abi/vts_vndk_abi_test.py b/abi/vts_vndk_abi_test.py
index 85627a3..a8633bf 100644
--- a/abi/vts_vndk_abi_test.py
+++ b/abi/vts_vndk_abi_test.py
@@ -307,6 +307,9 @@ class VtsVndkAbiTest(unittest.TestCase):
Args:
bitness: 32 or 64, the bitness of the tested libraries.
"""
+ if not vndk_utils.IsVndkRequired(self._dut):
+ logging.info("Skip the test as the device does not require VNDK.")
+ return
self.assertTrue(self._dut.IsRoot(), "This test requires adb root.")
primary_abi = self._dut.GetCpuAbiList()[0]
binder_bitness = self._dut.GetBinderBitness()
diff --git a/dependency/vts_vndk_dependency_test.py b/dependency/vts_vndk_dependency_test.py
index 6383450..e829558 100644
--- a/dependency/vts_vndk_dependency_test.py
+++ b/dependency/vts_vndk_dependency_test.py
@@ -117,6 +117,8 @@ class VtsVndkDependencyTest(unittest.TestCase):
sp_hal_strings = vndk_lists[0]
self._sp_hal = [re.compile(x) for x in sp_hal_strings]
(self._ll_ndk, self._vndk, self._vndk_sp) = vndk_lists[1:]
+ if not vndk_utils.IsVndkRequired(self._dut):
+ (self._vndk, self._vndk_sp) = ([], [])
logging.debug("LL_NDK: %s", self._ll_ndk)
logging.debug("SP_HAL: %s", sp_hal_strings)
diff --git a/files/vts_vndk_files_test.py b/files/vts_vndk_files_test.py
index faa3e6a..777cd94 100644
--- a/files/vts_vndk_files_test.py
+++ b/files/vts_vndk_files_test.py
@@ -122,7 +122,7 @@ class VtsVndkFilesTest(unittest.TestCase):
if unexpected:
self._Fail(unexpected,
"%s must not contain %s libraries." %
- (vndk_dir, ", ",join(vndk_list_names)))
+ (vndk_dir, ", ".join(vndk_list_names)))
def _TestVndkCoreDirectory(self, bitness):
"""Verifies that VNDK directory doesn't contain extra files."""
@@ -130,6 +130,9 @@ class VtsVndkFilesTest(unittest.TestCase):
logging.info("Skip the test as VNDK runtime is not enforced on "
"the device.")
return
+ if not vndk_utils.IsVndkRequired(self._dut):
+ logging.info("Skip the test as the device does not require VNDK.")
+ return
self._TestVndkDirectory(
vndk_utils.GetVndkDirectory(bitness, self._vndk_version),
(vndk_data.VNDK, vndk_data.VNDK_PRIVATE, vndk_data.VNDK_SP,