diff options
author | Adam Lesinski <adamlesinski@google.com> | 2015-03-10 17:35:38 -0700 |
---|---|---|
committer | Adam Lesinski <adamlesinski@google.com> | 2015-03-10 17:37:16 -0700 |
commit | 1c2dadddf78c3aa49129425d5f6ee480cfb54444 (patch) | |
tree | bb05ca90c766cee9cbb3ec698c6c60b16df00584 /tools/split-select/Grouper_test.cpp | |
parent | 488caeb70293b7c70e9ce128fc002a0666340fb0 (diff) |
split-select: Languages should not be grouped exclusively
If a series of languages match one configuration, this is fine. They
should all be installed. Example: target = fr-rCA, splits = [fr, fr-rCA].
Change-Id: I652348ae85073c7984a92a802f8c9f84f5d436c5
Diffstat (limited to 'tools/split-select/Grouper_test.cpp')
-rw-r--r-- | tools/split-select/Grouper_test.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tools/split-select/Grouper_test.cpp b/tools/split-select/Grouper_test.cpp index a5f9c5acfbbf..7294a86fa862 100644 --- a/tools/split-select/Grouper_test.cpp +++ b/tools/split-select/Grouper_test.cpp @@ -37,6 +37,8 @@ protected: addSplit(splits, "en-rUS-sw300dp-xhdpi"); addSplit(splits, "large"); addSplit(splits, "pl-rPL"); + addSplit(splits, "fr-rCA"); + addSplit(splits, "fr"); addSplit(splits, "xlarge"); addSplit(splits, "en-rUS-sw600dp-xhdpi"); addSplit(splits, "en-rUS-sw300dp-hdpi"); @@ -64,7 +66,7 @@ protected: }; TEST_F(GrouperTest, shouldHaveCorrectNumberOfGroups) { - EXPECT_EQ(12u, mGroups.size()); + EXPECT_EQ(15u, mGroups.size()); } TEST_F(GrouperTest, shouldGroupDensities) { @@ -79,7 +81,10 @@ TEST_F(GrouperTest, shouldGroupAbi) { } TEST_F(GrouperTest, shouldGroupLocale) { - expectHasGroupWithSplits("pl-rPL", "de-rDE"); + expectHasGroupWithSplits("pl-rPL"); + expectHasGroupWithSplits("de-rDE"); + expectHasGroupWithSplits("fr"); + expectHasGroupWithSplits("fr-rCA"); } TEST_F(GrouperTest, shouldGroupEachSplitIntoItsOwnGroup) { |