tequilaOS/platform_hardware_interfaces
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "audio: Centralize audio configuration paths specification"

Browse source
This commit is contained in:
Treehugger Robot 2020-04-16 23:05:30 +00:00 committed by Gerrit Code Review
commit c2e1f3baef
5 changed files with 54 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
audio/common/all-versions/test/utility/include/utility/ValidateXml.h
View file

@ -51,8 +51,31 @@ namespace utility {
*/
template <bool atLeastOneRequired = true>
::testing::AssertionResult validateXmlMultipleLocations(
const char* xmlFileNameExpr, const char* xmlFileLocationsExpr, const char* xsdFilePathExpr,
const char* xmlFileName, std::vector<const char*> xmlFileLocations, const char* xsdFilePath);
const char* xmlFileNameExpr, const char* xmlFileLocationsExpr, const char* xsdFilePathExpr,
const char* xmlFileName, const std::vector<std::string>& xmlFileLocations,
const char* xsdFilePath);
template <bool atLeastOneRequired = true>
::testing::AssertionResult validateXmlMultipleLocations(
const char* xmlFileNameExpr, const char* xmlFileLocationsExpr, const char* xsdFilePathExpr,
const char* xmlFileName, std::initializer_list<const char*> xmlFileLocations,
const char* xsdFilePath) {
return validateXmlMultipleLocations<atLeastOneRequired>(
xmlFileNameExpr, xmlFileLocationsExpr, xsdFilePathExpr, xmlFileName,
std::vector<std::string>(xmlFileLocations.begin(), xmlFileLocations.end()),
xsdFilePath);
}
template <bool atLeastOneRequired = true>
::testing::AssertionResult validateXmlMultipleLocations(const char* xmlFileNameExpr,
const char* xmlFileLocationsExpr,
const char* xsdFilePathExpr,
const char* xmlFileName,
std::vector<const char*> xmlFileLocations,
const char* xsdFilePath) {
return validateXmlMultipleLocations<atLeastOneRequired>(
xmlFileNameExpr, xmlFileLocationsExpr, xsdFilePathExpr, xmlFileName,
std::vector<std::string>(xmlFileLocations.begin(), xmlFileLocations.end()),
xsdFilePath);
}
/** ASSERT that all found XML are valid according to an xsd. */
#define ASSERT_VALID_XML_MULTIPLE_LOCATIONS(xmlFileName, xmlFileLocations, xsdFilePath) \

21
audio/common/all-versions/test/utility/src/ValidateXml.cpp
View file

@ -131,14 +131,15 @@ struct Libxml2Global {
template <bool atLeastOneRequired>
::testing::AssertionResult validateXmlMultipleLocations(
const char* xmlFileNameExpr, const char* xmlFileLocationsExpr, const char* xsdFilePathExpr,
const char* xmlFileName, std::vector<const char*> xmlFileLocations, const char* xsdFilePath) {
const char* xmlFileNameExpr, const char* xmlFileLocationsExpr, const char* xsdFilePathExpr,
const char* xmlFileName, const std::vector<std::string>& xmlFileLocations,
const char* xsdFilePath) {
using namespace std::string_literals;
std::vector<std::string> errors;
std::vector<std::string> foundFiles;
for (const char* location : xmlFileLocations) {
for (const auto& location : xmlFileLocations) {
std::string xmlFilePath = location + "/"s + xmlFileName;
if (access(xmlFilePath.c_str(), F_OK) != 0) {
// If the file does not exist ignore this location and fallback on the next one
@ -166,14 +167,12 @@ template <bool atLeastOneRequired>
: "\nWhere no file might exist.");
}
template ::testing::AssertionResult validateXmlMultipleLocations<true>(const char*, const char*,
const char*, const char*,
std::vector<const char*>,
const char*);
template ::testing::AssertionResult validateXmlMultipleLocations<false>(const char*, const char*,
const char*, const char*,
std::vector<const char*>,
const char*);
template ::testing::AssertionResult validateXmlMultipleLocations<true>(
const char*, const char*, const char*, const char*, const std::vector<std::string>&,
const char*);
template ::testing::AssertionResult validateXmlMultipleLocations<false>(
const char*, const char*, const char*, const char*, const std::vector<std::string>&,
const char*);
} // namespace utility
} // namespace test

11
audio/core/all-versions/vts/functional/AudioPrimaryHidlHalTest.h
View file

@ -35,6 +35,7 @@
#include <hwbinder/IPCThreadState.h>
#include <android-base/logging.h>
#include <system/audio_config.h>
#include PATH(android/hardware/audio/FILE_VERSION/IDevice.h)
#include PATH(android/hardware/audio/FILE_VERSION/IDevicesFactory.h)
@ -133,7 +134,6 @@ class HidlTest : public ::testing::Test {
////////////////////////// Audio policy configuration ////////////////////////
//////////////////////////////////////////////////////////////////////////////
static const std::vector<const char*> kConfigLocations = {"/odm/etc", "/vendor/etc", "/system/etc"};
static constexpr char kConfigFileName[] = "audio_policy_configuration.xml";
// Stringify the argument.
@ -152,8 +152,8 @@ class PolicyConfig : private PolicyConfigData, public AudioPolicyConfig {
PolicyConfig()
: AudioPolicyConfig(hwModules, availableOutputDevices, availableInputDevices,
defaultOutputDevice) {
for (const char* location : kConfigLocations) {
std::string path = std::string(location) + '/' + kConfigFileName;
for (const auto& location : android::audio_get_configuration_paths()) {
std::string path = location + '/' + kConfigFileName;
if (access(path.c_str(), F_OK) == 0) {
mFilePath = path;
break;
@ -186,7 +186,7 @@ class PolicyConfig : private PolicyConfigData, public AudioPolicyConfig {
std::string getError() const {
if (mFilePath.empty()) {
return std::string{"Could not find "} + kConfigFileName +
" file in: " + testing::PrintToString(kConfigLocations);
" file in: " + testing::PrintToString(android::audio_get_configuration_paths());
} else {
return "Invalid config file: " + mFilePath;
}
@ -302,7 +302,8 @@ TEST(CheckConfig, audioPolicyConfigurationValidation) {
"is valid according to the schema");
const char* xsd = "/data/local/tmp/audio_policy_configuration_" STRINGIFY(CPP_VERSION) ".xsd";
EXPECT_ONE_VALID_XML_MULTIPLE_LOCATIONS(kConfigFileName, kConfigLocations, xsd);
EXPECT_ONE_VALID_XML_MULTIPLE_LOCATIONS(kConfigFileName,
android::audio_get_configuration_paths(), xsd);
}
class AudioPolicyConfigTest : public AudioHidlTestWithDeviceParameter {

8
audio/effect/all-versions/vts/functional/ValidateAudioEffectsConfiguration.cpp
View file

@ -18,6 +18,7 @@
#include <iterator>
#include <media/EffectsConfig.h>
#include <system/audio_config.h>
// clang-format off
#include PATH(android/hardware/audio/effect/FILE_VERSION/IEffectsFactory.h)
// clang-format on
@ -41,13 +42,14 @@ TEST(CheckConfig, audioEffectsConfigurationValidation) {
GTEST_SKIP() << "No Effects HAL version " STRINGIFY(CPP_VERSION) " on this device";
}
std::vector<const char*> locations(std::begin(DEFAULT_LOCATIONS), std::end(DEFAULT_LOCATIONS));
const char* xsd = "/data/local/tmp/audio_effects_conf_" STRINGIFY(CPP_VERSION) ".xsd";
#if MAJOR_VERSION == 2
// In V2, audio effect XML is not required. .conf is still allowed though deprecated
EXPECT_VALID_XML_MULTIPLE_LOCATIONS(DEFAULT_NAME, locations, xsd);
EXPECT_VALID_XML_MULTIPLE_LOCATIONS(DEFAULT_NAME, android::audio_get_configuration_paths(),
xsd);
#elif MAJOR_VERSION >= 4
// Starting with V4, audio effect XML is required
EXPECT_ONE_VALID_XML_MULTIPLE_LOCATIONS(DEFAULT_NAME, locations, xsd);
EXPECT_ONE_VALID_XML_MULTIPLE_LOCATIONS(DEFAULT_NAME, android::audio_get_configuration_paths(),
xsd);
#endif
}

12
audio/policy/1.0/vts/functional/ValidateEngineConfiguration.cpp
View file

@ -23,7 +23,8 @@
#include <string>
#include "utility/ValidateXml.h"
static const std::vector<const char*> locations = {"/odm/etc", "/vendor/etc", "/system/etc"};
#include <system/audio_config.h>
static const std::string config = "audio_policy_engine_configuration.xml";
static const std::string schema =
std::string(XSD_DIR) + "/audio_policy_engine_configuration_V1_0.xsd";
@ -42,7 +43,8 @@ TEST(ValidateConfiguration, audioPolicyEngineConfiguration) {
RecordProperty("description",
"Verify that the audio policy engine configuration file "
"is valid according to the schemas");
EXPECT_VALID_XML_MULTIPLE_LOCATIONS(config.c_str(), locations, schema.c_str());
EXPECT_VALID_XML_MULTIPLE_LOCATIONS(config.c_str(), android::audio_get_configuration_paths(),
schema.c_str());
}
/**
@ -52,9 +54,11 @@ TEST(ValidateConfiguration, audioPolicyEngineConfiguration) {
*/
static bool deviceUsesConfigurableEngine() {
return android::hardware::audio::common::test::utility::validateXmlMultipleLocations<true>(
"", "", "", config.c_str(), locations, schema.c_str()) &&
"", "", "", config.c_str(), android::audio_get_configuration_paths(),
schema.c_str()) &&
android::hardware::audio::common::test::utility::validateXmlMultipleLocations<true>(
"", "", "", configurableConfig.c_str(), locations, configurableSchemas.c_str());
"", "", "", configurableConfig.c_str(), android::audio_get_configuration_paths(),
configurableSchemas.c_str());
}
TEST(ValidateConfiguration, audioPolicyEngineConfigurable) {