diff options
author | 2021-04-20 19:45:11 +0000 | |
---|---|---|
committer | 2021-04-20 19:45:11 +0000 | |
commit | 9c359a24b0c7512401c7badb808b19e3a4e3fced (patch) | |
tree | 7a4f894cd7d245a4310475ab5c45a9bbf3dbf64b | |
parent | 035c3c18c18dd2e97b833a679ab3f10f52975ac3 (diff) | |
parent | a96f87f3fea487f984f5a21e3e6b05234dc49c8b (diff) |
Merge "Change g.*Converter to fromXml."
-rw-r--r-- | cmds/lshal/ListCommand.cpp | 2 | ||||
-rw-r--r-- | cmds/lshal/test.cpp | 14 |
2 files changed, 7 insertions, 9 deletions
diff --git a/cmds/lshal/ListCommand.cpp b/cmds/lshal/ListCommand.cpp index d5110f6203..2722e214e8 100644 --- a/cmds/lshal/ListCommand.cpp +++ b/cmds/lshal/ListCommand.cpp @@ -417,7 +417,7 @@ void ListCommand::dumpVintf(const NullableOStream<std::ostream>& out) const { } } out << "-->" << std::endl; - out << vintf::gHalManifestConverter(manifest, vintf::SerializeFlags::HALS_ONLY); + out << vintf::toXml(manifest, vintf::SerializeFlags::HALS_ONLY); } std::string ListCommand::INIT_VINTF_NOTES{ diff --git a/cmds/lshal/test.cpp b/cmds/lshal/test.cpp index 7c1ca91528..6f08f74690 100644 --- a/cmds/lshal/test.cpp +++ b/cmds/lshal/test.cpp @@ -47,8 +47,6 @@ using ::android::hardware::hidl_vec; using ::android::hardware::Void; using android::vintf::Arch; using android::vintf::CompatibilityMatrix; -using android::vintf::gCompatibilityMatrixConverter; -using android::vintf::gHalManifestConverter; using android::vintf::HalManifest; using android::vintf::Transport; using android::vintf::VintfObject; @@ -510,7 +508,7 @@ TEST_F(ListTest, DumpVintf) { std::string error; vintf::HalManifest m; - EXPECT_EQ(true, vintf::gHalManifestConverter(&m, out.str(), &error)) + EXPECT_EQ(true, vintf::fromXml(&m, out.str(), &error)) << "--init-vintf does not emit valid HAL manifest: " << error; } @@ -775,10 +773,10 @@ TEST_F(ListTest, Vintf) { auto deviceMatrix = std::make_shared<CompatibilityMatrix>(); auto frameworkMatrix = std::make_shared<CompatibilityMatrix>(); - ASSERT_TRUE(gHalManifestConverter(deviceManifest.get(), deviceManifestXml)); - ASSERT_TRUE(gHalManifestConverter(frameworkManifest.get(), frameworkManifestXml)); - ASSERT_TRUE(gCompatibilityMatrixConverter(deviceMatrix.get(), deviceMatrixXml)); - ASSERT_TRUE(gCompatibilityMatrixConverter(frameworkMatrix.get(), frameworkMatrixXml)); + ASSERT_TRUE(fromXml(deviceManifest.get(), deviceManifestXml)); + ASSERT_TRUE(fromXml(frameworkManifest.get(), frameworkManifestXml)); + ASSERT_TRUE(fromXml(deviceMatrix.get(), deviceMatrixXml)); + ASSERT_TRUE(fromXml(frameworkMatrix.get(), frameworkMatrixXml)); ON_CALL(*mockList, getDeviceManifest()).WillByDefault(Return(deviceManifest)); ON_CALL(*mockList, getDeviceMatrix()).WillByDefault(Return(deviceMatrix)); @@ -964,7 +962,7 @@ public: " </hal>\n" "</manifest>"; auto manifest = std::make_shared<HalManifest>(); - EXPECT_TRUE(gHalManifestConverter(manifest.get(), mockManifestXml)); + EXPECT_TRUE(fromXml(manifest.get(), mockManifestXml)); EXPECT_CALL(*mockList, getDeviceManifest()) .Times(AnyNumber()) .WillRepeatedly(Return(manifest)); |