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