Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2015 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | #include "link/ManifestFixer.h" |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 18 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 19 | #include "test/Test.h" |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 20 | |
Adam Lesinski | d5083f6 | 2017-01-16 15:07:21 -0800 | [diff] [blame] | 21 | using android::StringPiece; |
| 22 | |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 23 | namespace aapt { |
| 24 | |
| 25 | struct ManifestFixerTest : public ::testing::Test { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 26 | std::unique_ptr<IAaptContext> mContext; |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 27 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 28 | void SetUp() override { |
| 29 | mContext = |
| 30 | test::ContextBuilder() |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 31 | .SetCompilationPackage("android") |
| 32 | .SetPackageId(0x01) |
| 33 | .SetNameManglerPolicy(NameManglerPolicy{"android"}) |
| 34 | .AddSymbolSource( |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 35 | test::StaticSymbolSourceBuilder() |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 36 | .AddSymbol( |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 37 | "android:attr/package", ResourceId(0x01010000), |
| 38 | test::AttributeBuilder() |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 39 | .SetTypeMask(android::ResTable_map::TYPE_STRING) |
| 40 | .Build()) |
| 41 | .AddSymbol( |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 42 | "android:attr/minSdkVersion", ResourceId(0x01010001), |
| 43 | test::AttributeBuilder() |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 44 | .SetTypeMask(android::ResTable_map::TYPE_STRING | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 45 | android::ResTable_map::TYPE_INTEGER) |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 46 | .Build()) |
| 47 | .AddSymbol( |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 48 | "android:attr/targetSdkVersion", ResourceId(0x01010002), |
| 49 | test::AttributeBuilder() |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 50 | .SetTypeMask(android::ResTable_map::TYPE_STRING | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 51 | android::ResTable_map::TYPE_INTEGER) |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 52 | .Build()) |
| 53 | .AddSymbol("android:string/str", ResourceId(0x01060000)) |
| 54 | .Build()) |
| 55 | .Build(); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 56 | } |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 57 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 58 | std::unique_ptr<xml::XmlResource> Verify(const StringPiece& str) { |
| 59 | return VerifyWithOptions(str, {}); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 60 | } |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 61 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 62 | std::unique_ptr<xml::XmlResource> VerifyWithOptions( |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 63 | const StringPiece& str, const ManifestFixerOptions& options) { |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 64 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDom(str); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 65 | ManifestFixer fixer(options); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 66 | if (fixer.Consume(mContext.get(), doc.get())) { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 67 | return doc; |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 68 | } |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 69 | return {}; |
| 70 | } |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 71 | }; |
| 72 | |
| 73 | TEST_F(ManifestFixerTest, EnsureManifestIsRootTag) { |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 74 | EXPECT_EQ(nullptr, Verify("<other-tag />")); |
| 75 | EXPECT_EQ(nullptr, Verify("<ns:manifest xmlns:ns=\"com\" />")); |
| 76 | EXPECT_NE(nullptr, Verify("<manifest package=\"android\"></manifest>")); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 77 | } |
| 78 | |
| 79 | TEST_F(ManifestFixerTest, EnsureManifestHasPackage) { |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 80 | EXPECT_NE(nullptr, Verify("<manifest package=\"android\" />")); |
| 81 | EXPECT_NE(nullptr, Verify("<manifest package=\"com.android\" />")); |
| 82 | EXPECT_NE(nullptr, Verify("<manifest package=\"com.android.google\" />")); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 83 | EXPECT_EQ(nullptr, |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 84 | Verify("<manifest package=\"com.android.google.Class$1\" />")); |
| 85 | EXPECT_EQ(nullptr, Verify("<manifest " |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 86 | "xmlns:android=\"http://schemas.android.com/apk/" |
| 87 | "res/android\" " |
| 88 | "android:package=\"com.android\" />")); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 89 | EXPECT_EQ(nullptr, Verify("<manifest package=\"@string/str\" />")); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 90 | } |
| 91 | |
Adam Lesinski | 5119e51 | 2016-12-05 19:48:20 -0800 | [diff] [blame] | 92 | TEST_F(ManifestFixerTest, AllowMetaData) { |
| 93 | auto doc = Verify(R"EOF( |
| 94 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 95 | package="android"> |
| 96 | <meta-data /> |
| 97 | <application> |
| 98 | <meta-data /> |
| 99 | <activity android:name=".Hi"><meta-data /></activity> |
| 100 | <activity-alias android:name=".Ho"><meta-data /></activity-alias> |
| 101 | <receiver android:name=".OffToWork"><meta-data /></receiver> |
| 102 | <provider android:name=".We"><meta-data /></provider> |
| 103 | <service android:name=".Go"><meta-data /></service> |
| 104 | </application> |
| 105 | </manifest>)EOF"); |
| 106 | ASSERT_NE(nullptr, doc); |
| 107 | } |
| 108 | |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 109 | TEST_F(ManifestFixerTest, UseDefaultSdkVersionsIfNonePresent) { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 110 | ManifestFixerOptions options = {std::string("8"), std::string("22")}; |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 111 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 112 | std::unique_ptr<xml::XmlResource> doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 113 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 114 | package="android"> |
| 115 | <uses-sdk android:minSdkVersion="7" android:targetSdkVersion="21" /> |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 116 | </manifest>)EOF", |
| 117 | options); |
| 118 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 119 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 120 | xml::Element* el; |
| 121 | xml::Attribute* attr; |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 122 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 123 | el = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 124 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 125 | el = el->FindChild({}, "uses-sdk"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 126 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 127 | attr = el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 128 | ASSERT_NE(nullptr, attr); |
| 129 | EXPECT_EQ("7", attr->value); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 130 | attr = el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 131 | ASSERT_NE(nullptr, attr); |
| 132 | EXPECT_EQ("21", attr->value); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 133 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 134 | doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 135 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 136 | package="android"> |
| 137 | <uses-sdk android:targetSdkVersion="21" /> |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 138 | </manifest>)EOF", |
| 139 | options); |
| 140 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 141 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 142 | el = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 143 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 144 | el = el->FindChild({}, "uses-sdk"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 145 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 146 | attr = el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 147 | ASSERT_NE(nullptr, attr); |
| 148 | EXPECT_EQ("8", attr->value); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 149 | attr = el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 150 | ASSERT_NE(nullptr, attr); |
| 151 | EXPECT_EQ("21", attr->value); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 152 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 153 | doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 154 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 155 | package="android"> |
| 156 | <uses-sdk /> |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 157 | </manifest>)EOF", |
| 158 | options); |
| 159 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 160 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 161 | el = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 162 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 163 | el = el->FindChild({}, "uses-sdk"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 164 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 165 | attr = el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 166 | ASSERT_NE(nullptr, attr); |
| 167 | EXPECT_EQ("8", attr->value); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 168 | attr = el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 169 | ASSERT_NE(nullptr, attr); |
| 170 | EXPECT_EQ("22", attr->value); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 171 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 172 | doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 173 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 174 | package="android" />)EOF", |
| 175 | options); |
| 176 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 177 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 178 | el = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 179 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 180 | el = el->FindChild({}, "uses-sdk"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 181 | ASSERT_NE(nullptr, el); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 182 | attr = el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 183 | ASSERT_NE(nullptr, attr); |
| 184 | EXPECT_EQ("8", attr->value); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 185 | attr = el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 186 | ASSERT_NE(nullptr, attr); |
| 187 | EXPECT_EQ("22", attr->value); |
Adam Lesinski | 2ae4a87 | 2015-11-02 16:10:55 -0800 | [diff] [blame] | 188 | } |
| 189 | |
Adam Lesinski | e343eb1 | 2016-10-27 16:31:58 -0700 | [diff] [blame] | 190 | TEST_F(ManifestFixerTest, UsesSdkMustComeBeforeApplication) { |
| 191 | ManifestFixerOptions options = {std::string("8"), std::string("22")}; |
| 192 | std::unique_ptr<xml::XmlResource> doc = VerifyWithOptions(R"EOF( |
| 193 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 194 | package="android"> |
| 195 | <application android:name=".MainApplication" /> |
| 196 | </manifest>)EOF", |
| 197 | options); |
| 198 | ASSERT_NE(nullptr, doc); |
| 199 | |
| 200 | xml::Element* manifest_el = xml::FindRootElement(doc.get()); |
| 201 | ASSERT_NE(nullptr, manifest_el); |
| 202 | ASSERT_EQ("manifest", manifest_el->name); |
| 203 | |
| 204 | xml::Element* application_el = manifest_el->FindChild("", "application"); |
| 205 | ASSERT_NE(nullptr, application_el); |
| 206 | |
| 207 | xml::Element* uses_sdk_el = manifest_el->FindChild("", "uses-sdk"); |
| 208 | ASSERT_NE(nullptr, uses_sdk_el); |
| 209 | |
| 210 | // Check that the uses_sdk_el comes before application_el in the children |
| 211 | // vector. |
| 212 | // Since there are no namespaces here, these children are direct descendants |
| 213 | // of manifest. |
| 214 | auto uses_sdk_iter = |
| 215 | std::find_if(manifest_el->children.begin(), manifest_el->children.end(), |
| 216 | [&](const std::unique_ptr<xml::Node>& child) { |
| 217 | return child.get() == uses_sdk_el; |
| 218 | }); |
| 219 | |
| 220 | auto application_iter = |
| 221 | std::find_if(manifest_el->children.begin(), manifest_el->children.end(), |
| 222 | [&](const std::unique_ptr<xml::Node>& child) { |
| 223 | return child.get() == application_el; |
| 224 | }); |
| 225 | |
| 226 | ASSERT_NE(manifest_el->children.end(), uses_sdk_iter); |
| 227 | ASSERT_NE(manifest_el->children.end(), application_iter); |
| 228 | |
| 229 | // The distance should be positive, meaning uses_sdk_iter comes before |
| 230 | // application_iter. |
| 231 | EXPECT_GT(std::distance(uses_sdk_iter, application_iter), 0); |
| 232 | } |
| 233 | |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 234 | TEST_F(ManifestFixerTest, RenameManifestPackageAndFullyQualifyClasses) { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 235 | ManifestFixerOptions options; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 236 | options.rename_manifest_package = std::string("com.android"); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 237 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 238 | std::unique_ptr<xml::XmlResource> doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 239 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 240 | package="android"> |
Adam Lesinski | cc5609d | 2016-04-05 12:41:07 -0700 | [diff] [blame] | 241 | <application android:name=".MainApplication" text="hello"> |
| 242 | <activity android:name=".activity.Start" /> |
| 243 | <receiver android:name="com.google.android.Receiver" /> |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 244 | </application> |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 245 | </manifest>)EOF", |
| 246 | options); |
| 247 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 248 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 249 | xml::Element* manifestEl = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 250 | ASSERT_NE(nullptr, manifestEl); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 251 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 252 | xml::Attribute* attr = nullptr; |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 253 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 254 | attr = manifestEl->FindAttribute({}, "package"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 255 | ASSERT_NE(nullptr, attr); |
| 256 | EXPECT_EQ(std::string("com.android"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 257 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 258 | xml::Element* applicationEl = manifestEl->FindChild({}, "application"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 259 | ASSERT_NE(nullptr, applicationEl); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 260 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 261 | attr = applicationEl->FindAttribute(xml::kSchemaAndroid, "name"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 262 | ASSERT_NE(nullptr, attr); |
| 263 | EXPECT_EQ(std::string("android.MainApplication"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 264 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 265 | attr = applicationEl->FindAttribute({}, "text"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 266 | ASSERT_NE(nullptr, attr); |
| 267 | EXPECT_EQ(std::string("hello"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 268 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 269 | xml::Element* el; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 270 | el = applicationEl->FindChild({}, "activity"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 271 | ASSERT_NE(nullptr, el); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 272 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 273 | attr = el->FindAttribute(xml::kSchemaAndroid, "name"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 274 | ASSERT_NE(nullptr, el); |
| 275 | EXPECT_EQ(std::string("android.activity.Start"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 276 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 277 | el = applicationEl->FindChild({}, "receiver"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 278 | ASSERT_NE(nullptr, el); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 279 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 280 | attr = el->FindAttribute(xml::kSchemaAndroid, "name"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 281 | ASSERT_NE(nullptr, el); |
| 282 | EXPECT_EQ(std::string("com.google.android.Receiver"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 283 | } |
| 284 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 285 | TEST_F(ManifestFixerTest, |
| 286 | RenameManifestInstrumentationPackageAndFullyQualifyTarget) { |
| 287 | ManifestFixerOptions options; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 288 | options.rename_instrumentation_target_package = std::string("com.android"); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 289 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 290 | std::unique_ptr<xml::XmlResource> doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 291 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
| 292 | package="android"> |
| 293 | <instrumentation android:targetPackage="android" /> |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 294 | </manifest>)EOF", |
| 295 | options); |
| 296 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 297 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 298 | xml::Element* manifest_el = xml::FindRootElement(doc.get()); |
| 299 | ASSERT_NE(nullptr, manifest_el); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 300 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 301 | xml::Element* instrumentation_el = |
| 302 | manifest_el->FindChild({}, "instrumentation"); |
| 303 | ASSERT_NE(nullptr, instrumentation_el); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 304 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 305 | xml::Attribute* attr = |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 306 | instrumentation_el->FindAttribute(xml::kSchemaAndroid, "targetPackage"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 307 | ASSERT_NE(nullptr, attr); |
| 308 | EXPECT_EQ(std::string("com.android"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 309 | } |
| 310 | |
| 311 | TEST_F(ManifestFixerTest, UseDefaultVersionNameAndCode) { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 312 | ManifestFixerOptions options; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 313 | options.version_name_default = std::string("Beta"); |
| 314 | options.version_code_default = std::string("0x10000000"); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 315 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 316 | std::unique_ptr<xml::XmlResource> doc = VerifyWithOptions(R"EOF( |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 317 | <manifest xmlns:android="http://schemas.android.com/apk/res/android" |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 318 | package="android" />)EOF", |
| 319 | options); |
| 320 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 321 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 322 | xml::Element* manifest_el = xml::FindRootElement(doc.get()); |
| 323 | ASSERT_NE(nullptr, manifest_el); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 324 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 325 | xml::Attribute* attr = |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 326 | manifest_el->FindAttribute(xml::kSchemaAndroid, "versionName"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 327 | ASSERT_NE(nullptr, attr); |
| 328 | EXPECT_EQ(std::string("Beta"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 329 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 330 | attr = manifest_el->FindAttribute(xml::kSchemaAndroid, "versionCode"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 331 | ASSERT_NE(nullptr, attr); |
| 332 | EXPECT_EQ(std::string("0x10000000"), attr->value); |
Adam Lesinski | 52364f7 | 2016-01-11 13:10:24 -0800 | [diff] [blame] | 333 | } |
| 334 | |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 335 | TEST_F(ManifestFixerTest, EnsureManifestAttributesAreTyped) { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 336 | EXPECT_EQ(nullptr, |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 337 | Verify("<manifest package=\"android\" coreApp=\"hello\" />")); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 338 | EXPECT_EQ(nullptr, |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 339 | Verify("<manifest package=\"android\" coreApp=\"1dp\" />")); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 340 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 341 | std::unique_ptr<xml::XmlResource> doc = |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 342 | Verify("<manifest package=\"android\" coreApp=\"true\" />"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 343 | ASSERT_NE(nullptr, doc); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 344 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 345 | xml::Element* el = xml::FindRootElement(doc.get()); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 346 | ASSERT_NE(nullptr, el); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 347 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 348 | EXPECT_EQ("manifest", el->name); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 349 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 350 | xml::Attribute* attr = el->FindAttribute("", "coreApp"); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 351 | ASSERT_NE(nullptr, attr); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 352 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 353 | EXPECT_NE(nullptr, attr->compiled_value); |
| 354 | EXPECT_NE(nullptr, ValueCast<BinaryPrimitive>(attr->compiled_value.get())); |
Adam Lesinski | 6b17d2c | 2016-08-10 11:37:06 -0700 | [diff] [blame] | 355 | } |
| 356 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 357 | } // namespace aapt |