commit 1c2dadddf78c3aa49129425d5f6ee480cfb54444
author Adam Lesinski <adamlesinski@google.com> Tue Mar 10 17:35:38 2015 -0700
committer Adam Lesinski <adamlesinski@google.com> Tue Mar 10 17:37:16 2015 -0700
tree bb05ca90c766cee9cbb3ec698c6c60b16df00584
parent 488caeb70293b7c70e9ce128fc002a0666340fb0

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

tools/split-select/Grouper_test.cpp

diff --git a/tools/split-select/Grouper_test.cpp b/tools/split-select/Grouper_test.cpp
index a5f9c5a..7294a86 100644
--- a/tools/split-select/Grouper_test.cpp
+++ b/tools/split-select/Grouper_test.cpp
@@ -37,6 +37,8 @@
         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 @@
 };
 
 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, shouldGroupLocale) {
-    expectHasGroupWithSplits("pl-rPL", "de-rDE");
+    expectHasGroupWithSplits("pl-rPL");
+    expectHasGroupWithSplits("de-rDE");
+    expectHasGroupWithSplits("fr");
+    expectHasGroupWithSplits("fr-rCA");
 }
 
 TEST_F(GrouperTest, shouldGroupEachSplitIntoItsOwnGroup) {