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 | |
Adam Lesinski | 6a00817 | 2016-02-02 17:02:58 -0800 | [diff] [blame] | 17 | #include "filter/ConfigFilter.h" |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 18 | #include "io/FileSystem.h" |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 19 | #include "link/TableMerger.h" |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 20 | #include "test/Builders.h" |
| 21 | #include "test/Context.h" |
| 22 | |
| 23 | #include <gtest/gtest.h> |
| 24 | |
| 25 | namespace aapt { |
| 26 | |
| 27 | struct TableMergerTest : public ::testing::Test { |
| 28 | std::unique_ptr<IAaptContext> mContext; |
| 29 | |
| 30 | void SetUp() override { |
| 31 | mContext = test::ContextBuilder() |
| 32 | // We are compiling this package. |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 33 | .setCompilationPackage("com.app.a") |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 34 | |
| 35 | // Merge all packages that have this package ID. |
| 36 | .setPackageId(0x7f) |
| 37 | |
| 38 | // Mangle all packages that do not have this package name. |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 39 | .setNameManglerPolicy(NameManglerPolicy{ "com.app.a", { "com.app.b" } }) |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 40 | |
| 41 | .build(); |
| 42 | } |
| 43 | }; |
| 44 | |
| 45 | TEST_F(TableMergerTest, SimpleMerge) { |
| 46 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 47 | .setPackageId("com.app.a", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 48 | .addReference("com.app.a:id/foo", "com.app.a:id/bar") |
| 49 | .addReference("com.app.a:id/bar", "com.app.b:id/foo") |
| 50 | .addValue("com.app.a:styleable/view", test::StyleableBuilder() |
| 51 | .addItem("com.app.b:id/foo") |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 52 | .build()) |
| 53 | .build(); |
| 54 | |
| 55 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 56 | .setPackageId("com.app.b", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 57 | .addSimple("com.app.b:id/foo") |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 58 | .build(); |
| 59 | |
| 60 | ResourceTable finalTable; |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 61 | TableMerger merger(mContext.get(), &finalTable, TableMergerOptions{}); |
| 62 | io::FileCollection collection; |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 63 | |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 64 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 65 | ASSERT_TRUE(merger.mergeAndMangle({}, "com.app.b", tableB.get(), &collection)); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 66 | |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 67 | EXPECT_TRUE(merger.getMergedPackages().count("com.app.b") != 0); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 68 | |
| 69 | // Entries from com.app.a should not be mangled. |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 70 | AAPT_EXPECT_TRUE(finalTable.findResource(test::parseNameOrDie("com.app.a:id/foo"))); |
| 71 | AAPT_EXPECT_TRUE(finalTable.findResource(test::parseNameOrDie("com.app.a:id/bar"))); |
| 72 | AAPT_EXPECT_TRUE(finalTable.findResource(test::parseNameOrDie("com.app.a:styleable/view"))); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 73 | |
| 74 | // The unmangled name should not be present. |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 75 | AAPT_EXPECT_FALSE(finalTable.findResource(test::parseNameOrDie("com.app.b:id/foo"))); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 76 | |
| 77 | // Look for the mangled name. |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 78 | AAPT_EXPECT_TRUE(finalTable.findResource(test::parseNameOrDie("com.app.a:id/com.app.b$foo"))); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 79 | } |
| 80 | |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 81 | TEST_F(TableMergerTest, MergeFile) { |
| 82 | ResourceTable finalTable; |
| 83 | TableMergerOptions options; |
| 84 | options.autoAddOverlay = false; |
| 85 | TableMerger merger(mContext.get(), &finalTable, options); |
| 86 | |
| 87 | ResourceFile fileDesc; |
| 88 | fileDesc.config = test::parseConfigOrDie("hdpi-v4"); |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 89 | fileDesc.name = test::parseNameOrDie("layout/main"); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 90 | fileDesc.source = Source("res/layout-hdpi/main.xml"); |
| 91 | test::TestFile testFile("path/to/res/layout-hdpi/main.xml.flat"); |
| 92 | |
| 93 | ASSERT_TRUE(merger.mergeFile(fileDesc, &testFile)); |
| 94 | |
| 95 | FileReference* file = test::getValueForConfig<FileReference>(&finalTable, |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 96 | "com.app.a:layout/main", |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 97 | test::parseConfigOrDie("hdpi-v4")); |
| 98 | ASSERT_NE(nullptr, file); |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 99 | EXPECT_EQ(std::string("res/layout-hdpi-v4/main.xml"), *file->path); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 100 | } |
| 101 | |
| 102 | TEST_F(TableMergerTest, MergeFileOverlay) { |
| 103 | ResourceTable finalTable; |
| 104 | TableMergerOptions tableMergerOptions; |
| 105 | tableMergerOptions.autoAddOverlay = false; |
| 106 | TableMerger merger(mContext.get(), &finalTable, tableMergerOptions); |
| 107 | |
| 108 | ResourceFile fileDesc; |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 109 | fileDesc.name = test::parseNameOrDie("xml/foo"); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 110 | test::TestFile fileA("path/to/fileA.xml.flat"); |
| 111 | test::TestFile fileB("path/to/fileB.xml.flat"); |
| 112 | |
| 113 | ASSERT_TRUE(merger.mergeFile(fileDesc, &fileA)); |
| 114 | ASSERT_TRUE(merger.mergeFileOverlay(fileDesc, &fileB)); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 115 | } |
| 116 | |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 117 | TEST_F(TableMergerTest, MergeFileReferences) { |
| 118 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 119 | .setPackageId("com.app.a", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 120 | .addFileReference("com.app.a:xml/file", "res/xml/file.xml") |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 121 | .build(); |
| 122 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 123 | .setPackageId("com.app.b", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 124 | .addFileReference("com.app.b:xml/file", "res/xml/file.xml") |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 125 | .build(); |
| 126 | |
| 127 | ResourceTable finalTable; |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 128 | TableMerger merger(mContext.get(), &finalTable, TableMergerOptions{}); |
| 129 | io::FileCollection collection; |
| 130 | collection.insertFile("res/xml/file.xml"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 131 | |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 132 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 133 | ASSERT_TRUE(merger.mergeAndMangle({}, "com.app.b", tableB.get(), &collection)); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 134 | |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 135 | FileReference* f = test::getValue<FileReference>(&finalTable, "com.app.a:xml/file"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 136 | ASSERT_NE(f, nullptr); |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 137 | EXPECT_EQ(std::string("res/xml/file.xml"), *f->path); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 138 | |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 139 | f = test::getValue<FileReference>(&finalTable, "com.app.a:xml/com.app.b$file"); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 140 | ASSERT_NE(f, nullptr); |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 141 | EXPECT_EQ(std::string("res/xml/com.app.b$file.xml"), *f->path); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 142 | } |
| 143 | |
| 144 | TEST_F(TableMergerTest, OverrideResourceWithOverlay) { |
| 145 | std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 146 | .setPackageId("", 0x00) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 147 | .addValue("bool/foo", ResourceUtils::tryParseBool("true")) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 148 | .build(); |
| 149 | std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 150 | .setPackageId("", 0x00) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 151 | .addValue("bool/foo", ResourceUtils::tryParseBool("false")) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 152 | .build(); |
| 153 | |
| 154 | ResourceTable finalTable; |
| 155 | TableMergerOptions tableMergerOptions; |
| 156 | tableMergerOptions.autoAddOverlay = false; |
| 157 | TableMerger merger(mContext.get(), &finalTable, tableMergerOptions); |
| 158 | |
| 159 | ASSERT_TRUE(merger.merge({}, base.get())); |
| 160 | ASSERT_TRUE(merger.mergeOverlay({}, overlay.get())); |
| 161 | |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 162 | BinaryPrimitive* foo = test::getValue<BinaryPrimitive>(&finalTable, "com.app.a:bool/foo"); |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 163 | ASSERT_NE(nullptr, foo); |
| 164 | EXPECT_EQ(0x0u, foo->value.data); |
| 165 | } |
| 166 | |
Alexandria Cornwall | 6a1f8db | 2016-08-11 13:49:44 -0700 | [diff] [blame] | 167 | TEST_F(TableMergerTest, OverrideSameResourceIdsWithOverlay) { |
| 168 | std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder() |
| 169 | .setPackageId("", 0x7f) |
| 170 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic) |
| 171 | .build(); |
| 172 | std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder() |
| 173 | .setPackageId("", 0x7f) |
| 174 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic) |
| 175 | .build(); |
| 176 | |
| 177 | ResourceTable finalTable; |
| 178 | TableMergerOptions tableMergerOptions; |
| 179 | tableMergerOptions.autoAddOverlay = false; |
| 180 | TableMerger merger(mContext.get(), &finalTable, tableMergerOptions); |
| 181 | |
| 182 | ASSERT_TRUE(merger.merge({}, base.get())); |
| 183 | ASSERT_TRUE(merger.mergeOverlay({}, overlay.get())); |
| 184 | } |
| 185 | |
| 186 | TEST_F(TableMergerTest, FailToOverrideConflictingTypeIdsWithOverlay) { |
| 187 | std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder() |
| 188 | .setPackageId("", 0x7f) |
| 189 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic) |
| 190 | .build(); |
| 191 | std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder() |
| 192 | .setPackageId("", 0x7f) |
| 193 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x02, 0x0001), SymbolState::kPublic) |
| 194 | .build(); |
| 195 | |
| 196 | ResourceTable finalTable; |
| 197 | TableMergerOptions tableMergerOptions; |
| 198 | tableMergerOptions.autoAddOverlay = false; |
| 199 | TableMerger merger(mContext.get(), &finalTable, tableMergerOptions); |
| 200 | |
| 201 | ASSERT_TRUE(merger.merge({}, base.get())); |
| 202 | ASSERT_FALSE(merger.mergeOverlay({}, overlay.get())); |
| 203 | } |
| 204 | |
| 205 | TEST_F(TableMergerTest, FailToOverrideConflictingEntryIdsWithOverlay) { |
| 206 | std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder() |
| 207 | .setPackageId("", 0x7f) |
| 208 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic) |
| 209 | .build(); |
| 210 | std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder() |
| 211 | .setPackageId("", 0x7f) |
| 212 | .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0002), SymbolState::kPublic) |
| 213 | .build(); |
| 214 | |
| 215 | ResourceTable finalTable; |
| 216 | TableMergerOptions tableMergerOptions; |
| 217 | tableMergerOptions.autoAddOverlay = false; |
| 218 | TableMerger merger(mContext.get(), &finalTable, tableMergerOptions); |
| 219 | |
| 220 | ASSERT_TRUE(merger.merge({}, base.get())); |
| 221 | ASSERT_FALSE(merger.mergeOverlay({}, overlay.get())); |
| 222 | } |
| 223 | |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 224 | TEST_F(TableMergerTest, MergeAddResourceFromOverlay) { |
| 225 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 226 | .setPackageId("", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 227 | .setSymbolState("bool/foo", {}, SymbolState::kUndefined) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 228 | .build(); |
| 229 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 230 | .setPackageId("", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 231 | .addValue("bool/foo", ResourceUtils::tryParseBool("true")) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 232 | .build(); |
| 233 | |
| 234 | ResourceTable finalTable; |
| 235 | TableMerger merger(mContext.get(), &finalTable, TableMergerOptions{}); |
| 236 | |
| 237 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
| 238 | ASSERT_TRUE(merger.mergeOverlay({}, tableB.get())); |
| 239 | } |
| 240 | |
| 241 | TEST_F(TableMergerTest, MergeAddResourceFromOverlayWithAutoAddOverlay) { |
| 242 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 243 | .setPackageId("", 0x7f) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 244 | .build(); |
| 245 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 246 | .setPackageId("", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 247 | .addValue("bool/foo", ResourceUtils::tryParseBool("true")) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 248 | .build(); |
| 249 | |
| 250 | ResourceTable finalTable; |
| 251 | TableMergerOptions options; |
| 252 | options.autoAddOverlay = true; |
| 253 | TableMerger merger(mContext.get(), &finalTable, options); |
| 254 | |
| 255 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
| 256 | ASSERT_TRUE(merger.mergeOverlay({}, tableB.get())); |
| 257 | } |
| 258 | |
| 259 | TEST_F(TableMergerTest, FailToMergeNewResourceWithoutAutoAddOverlay) { |
| 260 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 261 | .setPackageId("", 0x7f) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 262 | .build(); |
| 263 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
Adam Lesinski | d0f116b | 2016-07-08 15:00:32 -0700 | [diff] [blame] | 264 | .setPackageId("", 0x7f) |
Adam Lesinski | 58a20a6 | 2016-07-25 17:56:58 -0700 | [diff] [blame] | 265 | .addValue("bool/foo", ResourceUtils::tryParseBool("true")) |
Adam Lesinski | a6fe345 | 2015-12-09 15:20:52 -0800 | [diff] [blame] | 266 | .build(); |
| 267 | |
| 268 | ResourceTable finalTable; |
| 269 | TableMergerOptions options; |
| 270 | options.autoAddOverlay = false; |
| 271 | TableMerger merger(mContext.get(), &finalTable, options); |
| 272 | |
| 273 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
| 274 | ASSERT_FALSE(merger.mergeOverlay({}, tableB.get())); |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 275 | } |
| 276 | |
Adam Lesinski | 5c3464c | 2016-08-24 16:03:48 -0700 | [diff] [blame^] | 277 | TEST_F(TableMergerTest, OverlaidStyleablesShouldBeMerged) { |
| 278 | std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder() |
| 279 | .setPackageId("com.app.a", 0x7f) |
| 280 | .addValue("com.app.a:styleable/Foo", test::StyleableBuilder() |
| 281 | .addItem("com.app.a:attr/bar") |
| 282 | .addItem("com.app.a:attr/foo", ResourceId(0x01010000)) |
| 283 | .build()) |
| 284 | .build(); |
| 285 | |
| 286 | std::unique_ptr<ResourceTable> tableB = test::ResourceTableBuilder() |
| 287 | .setPackageId("com.app.a", 0x7f) |
| 288 | .addValue("com.app.a:styleable/Foo", test::StyleableBuilder() |
| 289 | .addItem("com.app.a:attr/bat") |
| 290 | .addItem("com.app.a:attr/foo") |
| 291 | .build()) |
| 292 | .build(); |
| 293 | |
| 294 | ResourceTable finalTable; |
| 295 | TableMergerOptions options; |
| 296 | options.autoAddOverlay = true; |
| 297 | TableMerger merger(mContext.get(), &finalTable, options); |
| 298 | |
| 299 | ASSERT_TRUE(merger.merge({}, tableA.get())); |
| 300 | ASSERT_TRUE(merger.mergeOverlay({}, tableB.get())); |
| 301 | |
| 302 | Debug::printTable(&finalTable, {}); |
| 303 | |
| 304 | Styleable* styleable = test::getValue<Styleable>(&finalTable, "com.app.a:styleable/Foo"); |
| 305 | ASSERT_NE(nullptr, styleable); |
| 306 | |
| 307 | std::vector<Reference> expectedRefs = { |
| 308 | Reference(test::parseNameOrDie("com.app.a:attr/bar")), |
| 309 | Reference(test::parseNameOrDie("com.app.a:attr/bat")), |
| 310 | Reference(test::parseNameOrDie("com.app.a:attr/foo"), ResourceId(0x01010000)), |
| 311 | }; |
| 312 | |
| 313 | EXPECT_EQ(expectedRefs, styleable->entries); |
| 314 | } |
| 315 | |
Adam Lesinski | 1ab598f | 2015-08-14 14:26:04 -0700 | [diff] [blame] | 316 | } // namespace aapt |