Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [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/Linkers.h" |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 18 | |
Adam Lesinski | 64587af | 2016-02-18 18:33:06 -0800 | [diff] [blame] | 19 | #include "test/Test.h" |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 20 | |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 21 | using ::testing::IsNull; |
| 22 | using ::testing::NotNull; |
| 23 | |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 24 | namespace aapt { |
| 25 | |
| 26 | class XmlReferenceLinkerTest : public ::testing::Test { |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 27 | public: |
| 28 | void SetUp() override { |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 29 | context_ = test::ContextBuilder() |
| 30 | .SetCompilationPackage("com.app.test") |
| 31 | .SetNameManglerPolicy(NameManglerPolicy{"com.app.test", {"com.android.support"}}) |
| 32 | .AddSymbolSource( |
| 33 | test::StaticSymbolSourceBuilder() |
| 34 | .AddPublicSymbol("android:attr/layout_width", ResourceId(0x01010000), |
| 35 | test::AttributeBuilder() |
| 36 | .SetTypeMask(android::ResTable_map::TYPE_ENUM | |
| 37 | android::ResTable_map::TYPE_DIMENSION) |
| 38 | .AddItem("match_parent", 0xffffffff) |
| 39 | .Build()) |
| 40 | .AddPublicSymbol("android:attr/background", ResourceId(0x01010001), |
| 41 | test::AttributeBuilder() |
| 42 | .SetTypeMask(android::ResTable_map::TYPE_COLOR) |
| 43 | .Build()) |
| 44 | .AddPublicSymbol("android:attr/attr", ResourceId(0x01010002), |
| 45 | test::AttributeBuilder().Build()) |
| 46 | .AddPublicSymbol("android:attr/text", ResourceId(0x01010003), |
| 47 | test::AttributeBuilder() |
| 48 | .SetTypeMask(android::ResTable_map::TYPE_STRING) |
| 49 | .Build()) |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 50 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 51 | // Add one real symbol that was introduces in v21 |
| 52 | .AddPublicSymbol("android:attr/colorAccent", ResourceId(0x01010435), |
| 53 | test::AttributeBuilder().Build()) |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 54 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 55 | // Private symbol. |
| 56 | .AddSymbol("android:color/hidden", ResourceId(0x01020001)) |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 57 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 58 | .AddPublicSymbol("android:id/id", ResourceId(0x01030000)) |
| 59 | .AddSymbol("com.app.test:id/id", ResourceId(0x7f030000)) |
| 60 | .AddSymbol("com.app.test:color/green", ResourceId(0x7f020000)) |
| 61 | .AddSymbol("com.app.test:color/red", ResourceId(0x7f020001)) |
| 62 | .AddSymbol("com.app.test:attr/colorAccent", ResourceId(0x7f010000), |
| 63 | test::AttributeBuilder() |
| 64 | .SetTypeMask(android::ResTable_map::TYPE_COLOR) |
| 65 | .Build()) |
| 66 | .AddPublicSymbol("com.app.test:attr/com.android.support$colorAccent", |
| 67 | ResourceId(0x7f010001), |
| 68 | test::AttributeBuilder() |
| 69 | .SetTypeMask(android::ResTable_map::TYPE_COLOR) |
| 70 | .Build()) |
| 71 | .AddPublicSymbol("com.app.test:attr/attr", ResourceId(0x7f010002), |
| 72 | test::AttributeBuilder().Build()) |
| 73 | .Build()) |
| 74 | .Build(); |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 75 | } |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 76 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 77 | protected: |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 78 | std::unique_ptr<IAaptContext> context_; |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 79 | }; |
| 80 | |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 81 | TEST_F(XmlReferenceLinkerTest, LinkBasicAttributes) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 82 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 83 | <View xmlns:android="http://schemas.android.com/apk/res/android" |
| 84 | android:layout_width="match_parent" |
| 85 | android:background="@color/green" |
| 86 | android:text="hello" |
| 87 | android:attr="\?hello" |
| 88 | nonAaptAttr="1" |
| 89 | nonAaptAttrRef="@id/id" |
| 90 | class="hello" />)"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 91 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 92 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 93 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 94 | |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 95 | xml::Element* view_el = doc->root.get(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 96 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 97 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 98 | xml::Attribute* xml_attr = view_el->FindAttribute(xml::kSchemaAndroid, "layout_width"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 99 | ASSERT_THAT(xml_attr, NotNull()); |
| 100 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 101 | EXPECT_EQ(make_value(ResourceId(0x01010000)), xml_attr->compiled_attribute.value().id); |
| 102 | EXPECT_THAT(ValueCast<BinaryPrimitive>(xml_attr->compiled_value.get()), NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 103 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 104 | xml_attr = view_el->FindAttribute(xml::kSchemaAndroid, "background"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 105 | ASSERT_THAT(xml_attr, NotNull()); |
| 106 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 107 | EXPECT_EQ(make_value(ResourceId(0x01010001)), xml_attr->compiled_attribute.value().id); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 108 | Reference* ref = ValueCast<Reference>(xml_attr->compiled_value.get()); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 109 | ASSERT_THAT(ref, NotNull()); |
| 110 | EXPECT_EQ(make_value(test::ParseNameOrDie("color/green")), ref->name); // Make sure the name |
| 111 | // didn't change. |
| 112 | EXPECT_EQ(make_value(ResourceId(0x7f020000)), ref->id); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 113 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 114 | xml_attr = view_el->FindAttribute(xml::kSchemaAndroid, "text"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 115 | ASSERT_THAT(xml_attr, NotNull()); |
| 116 | EXPECT_TRUE(xml_attr->compiled_attribute); |
| 117 | EXPECT_THAT(xml_attr->compiled_value, IsNull()); // Strings don't get compiled for memory sake. |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 118 | |
Adam Lesinski | 48448e8 | 2017-04-26 15:13:52 -0700 | [diff] [blame] | 119 | xml_attr = view_el->FindAttribute(xml::kSchemaAndroid, "attr"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 120 | ASSERT_THAT(xml_attr, NotNull()); |
| 121 | EXPECT_TRUE(xml_attr->compiled_attribute); |
| 122 | EXPECT_THAT(xml_attr->compiled_value, IsNull()); // Should be a plain string. |
Adam Lesinski | 48448e8 | 2017-04-26 15:13:52 -0700 | [diff] [blame] | 123 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 124 | xml_attr = view_el->FindAttribute("", "nonAaptAttr"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 125 | ASSERT_THAT(xml_attr, NotNull()); |
| 126 | EXPECT_FALSE(xml_attr->compiled_attribute); |
| 127 | EXPECT_THAT(ValueCast<BinaryPrimitive>(xml_attr->compiled_value.get()), NotNull()); |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 128 | |
| 129 | xml_attr = view_el->FindAttribute("", "nonAaptAttrRef"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 130 | ASSERT_THAT(xml_attr, NotNull()); |
| 131 | EXPECT_FALSE(xml_attr->compiled_attribute); |
| 132 | EXPECT_THAT(ValueCast<Reference>(xml_attr->compiled_value.get()), NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 133 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 134 | xml_attr = view_el->FindAttribute("", "class"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 135 | ASSERT_THAT(xml_attr, NotNull()); |
| 136 | EXPECT_FALSE(xml_attr->compiled_attribute); |
| 137 | EXPECT_THAT(xml_attr->compiled_value, IsNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 138 | } |
| 139 | |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 140 | TEST_F(XmlReferenceLinkerTest, PrivateSymbolsAreNotLinked) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 141 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 142 | <View xmlns:android="http://schemas.android.com/apk/res/android" |
| 143 | android:colorAccent="@android:color/hidden" />)"); |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 144 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 145 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 146 | ASSERT_FALSE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 147 | } |
| 148 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 149 | TEST_F(XmlReferenceLinkerTest, PrivateSymbolsAreLinkedWhenReferenceHasStarPrefix) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 150 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 151 | <View xmlns:android="http://schemas.android.com/apk/res/android" |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 152 | android:colorAccent="@*android:color/hidden" />)"); |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 153 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 154 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 155 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 467f171 | 2015-11-16 17:35:44 -0800 | [diff] [blame] | 156 | } |
| 157 | |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 158 | TEST_F(XmlReferenceLinkerTest, LinkMangledAttributes) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 159 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 160 | <View xmlns:support="http://schemas.android.com/apk/res/com.android.support" |
| 161 | support:colorAccent="#ff0000" />)"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 162 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 163 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 164 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 165 | |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 166 | xml::Element* view_el = doc->root.get(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 167 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 168 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 169 | xml::Attribute* xml_attr = |
| 170 | view_el->FindAttribute(xml::BuildPackageNamespace("com.android.support"), "colorAccent"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 171 | ASSERT_THAT(xml_attr, NotNull()); |
| 172 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 173 | EXPECT_EQ(make_value(ResourceId(0x7f010001)), xml_attr->compiled_attribute.value().id); |
| 174 | EXPECT_THAT(ValueCast<BinaryPrimitive>(xml_attr->compiled_value.get()), NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 175 | } |
| 176 | |
| 177 | TEST_F(XmlReferenceLinkerTest, LinkAutoResReference) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 178 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 179 | <View xmlns:app="http://schemas.android.com/apk/res-auto" |
| 180 | app:colorAccent="@app:color/red" />)"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 181 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 182 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 183 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 184 | |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 185 | xml::Element* view_el = doc->root.get(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 186 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 187 | |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 188 | xml::Attribute* xml_attr = view_el->FindAttribute(xml::kSchemaAuto, "colorAccent"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 189 | ASSERT_THAT(xml_attr, NotNull()); |
| 190 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 191 | EXPECT_EQ(make_value(ResourceId(0x7f010000)), xml_attr->compiled_attribute.value().id); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 192 | Reference* ref = ValueCast<Reference>(xml_attr->compiled_value.get()); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 193 | ASSERT_THAT(ref, NotNull()); |
| 194 | ASSERT_TRUE(ref->name); |
| 195 | EXPECT_EQ(make_value(ResourceId(0x7f020001)), ref->id); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 196 | } |
| 197 | |
| 198 | TEST_F(XmlReferenceLinkerTest, LinkViewWithShadowedPackageAlias) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 199 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 200 | <View xmlns:app="http://schemas.android.com/apk/res/android" app:attr="@app:id/id"> |
| 201 | <View xmlns:app="http://schemas.android.com/apk/res/com.app.test" app:attr="@app:id/id"/> |
| 202 | </View>)"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 203 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 204 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 205 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 206 | |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 207 | xml::Element* view_el = doc->root.get(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 208 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 209 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 210 | // All attributes and references in this element should be referring to |
| 211 | // "android" (0x01). |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 212 | xml::Attribute* xml_attr = view_el->FindAttribute(xml::kSchemaAndroid, "attr"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 213 | ASSERT_THAT(xml_attr, NotNull()); |
| 214 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 215 | EXPECT_EQ(make_value(ResourceId(0x01010002)), xml_attr->compiled_attribute.value().id); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 216 | Reference* ref = ValueCast<Reference>(xml_attr->compiled_value.get()); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 217 | ASSERT_THAT(ref, NotNull()); |
| 218 | EXPECT_EQ(make_value(ResourceId(0x01030000)), ref->id); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 219 | |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 220 | ASSERT_FALSE(view_el->GetChildElements().empty()); |
| 221 | view_el = view_el->GetChildElements().front(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 222 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 223 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 224 | // All attributes and references in this element should be referring to |
| 225 | // "com.app.test" (0x7f). |
Adam Lesinski | 3866554 | 2016-12-28 12:25:46 -0500 | [diff] [blame] | 226 | xml_attr = view_el->FindAttribute(xml::BuildPackageNamespace("com.app.test"), "attr"); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 227 | ASSERT_THAT(xml_attr, NotNull()); |
| 228 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 229 | EXPECT_EQ(make_value(ResourceId(0x7f010002)), xml_attr->compiled_attribute.value().id); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 230 | ref = ValueCast<Reference>(xml_attr->compiled_value.get()); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 231 | ASSERT_THAT(ref, NotNull()); |
| 232 | EXPECT_EQ(make_value(ResourceId(0x7f030000)), ref->id); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 233 | } |
| 234 | |
| 235 | TEST_F(XmlReferenceLinkerTest, LinkViewWithLocalPackageAndAliasOfTheSameName) { |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 236 | std::unique_ptr<xml::XmlResource> doc = test::BuildXmlDomForPackageName(context_.get(), R"( |
| 237 | <View xmlns:android="http://schemas.android.com/apk/res/com.app.test" |
| 238 | android:attr="@id/id"/>)"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 239 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 240 | XmlReferenceLinker linker; |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 241 | ASSERT_TRUE(linker.Consume(context_.get(), doc.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 242 | |
Adam Lesinski | 6b37299 | 2017-08-09 10:54:23 -0700 | [diff] [blame] | 243 | xml::Element* view_el = doc->root.get(); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 244 | ASSERT_THAT(view_el, NotNull()); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 245 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 246 | // All attributes and references in this element should be referring to |
| 247 | // "com.app.test" (0x7f). |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 248 | xml::Attribute* xml_attr = view_el->FindAttribute(xml::BuildPackageNamespace("com.app.test"), "attr"); |
| 249 | ASSERT_THAT(xml_attr, NotNull()); |
| 250 | ASSERT_TRUE(xml_attr->compiled_attribute); |
| 251 | EXPECT_EQ(make_value(ResourceId(0x7f010002)), xml_attr->compiled_attribute.value().id); |
Adam Lesinski | ce5e56e | 2016-10-21 17:56:45 -0700 | [diff] [blame] | 252 | Reference* ref = ValueCast<Reference>(xml_attr->compiled_value.get()); |
Adam Lesinski | a45893a | 2017-05-30 15:19:02 -0700 | [diff] [blame] | 253 | ASSERT_THAT(ref, NotNull()); |
| 254 | EXPECT_EQ(make_value(ResourceId(0x7f030000)), ref->id); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 255 | } |
| 256 | |
Adam Lesinski | cacb28f | 2016-10-19 12:18:14 -0700 | [diff] [blame] | 257 | } // namespace aapt |