Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Unified Diff: components/sync/base/enum_set_unittest.cc

Issue 2701003002: [Sync] Clean up ModelType code. (Closed)
Patch Set: Switched int to size_t to fix compile. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/sync/base/enum_set.h ('k') | components/sync/base/model_type.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/base/enum_set_unittest.cc
diff --git a/components/sync/base/enum_set_unittest.cc b/components/sync/base/enum_set_unittest.cc
index e6293f942d3a73be83d6ead2596f3b567f24e187..8f32c7f783f2fcc9a7d6f1e7c61441c07034c9fb 100644
--- a/components/sync/base/enum_set_unittest.cc
+++ b/components/sync/base/enum_set_unittest.cc
@@ -77,6 +77,11 @@ TEST_F(EnumSetTest, ThreeArgConstructor) {
EXPECT_FALSE(enums.Has(TEST_4));
}
+TEST_F(EnumSetTest, DuplicatesInConstructor) {
+ EXPECT_EQ(TestEnumSet(TEST_3, TEST_1, TEST_0, TEST_3, TEST_1, TEST_3),
+ TestEnumSet(TEST_0, TEST_1, TEST_3));
+}
+
TEST_F(EnumSetTest, All) {
const TestEnumSet enums(TestEnumSet::All());
EXPECT_FALSE(enums.Empty());
@@ -88,6 +93,13 @@ TEST_F(EnumSetTest, All) {
EXPECT_TRUE(enums.Has(TEST_4));
}
+TEST_F(EnumSetTest, FromRange) {
+ EXPECT_EQ(TestEnumSet(TEST_1, TEST_2, TEST_3),
+ TestEnumSet::FromRange(TEST_1, TEST_3));
+ EXPECT_EQ(TestEnumSet::All(), TestEnumSet::FromRange(TEST_0, TEST_4));
+ EXPECT_EQ(TestEnumSet(TEST_1), TestEnumSet::FromRange(TEST_1, TEST_1));
+}
+
TEST_F(EnumSetTest, Put) {
TestEnumSet enums(TEST_3);
enums.Put(TEST_2);
@@ -102,6 +114,12 @@ TEST_F(EnumSetTest, PutAll) {
EXPECT_EQ(TestEnumSet(TEST_2, TEST_3, TEST_4), enums);
}
+TEST_F(EnumSetTest, PutRange) {
+ TestEnumSet enums;
+ enums.PutRange(TEST_1, TEST_3);
+ EXPECT_EQ(TestEnumSet(TEST_1, TEST_2, TEST_3), enums);
+}
+
TEST_F(EnumSetTest, RetainAll) {
TestEnumSet enums(TEST_3, TEST_4);
enums.RetainAll(TestEnumSet(TEST_2, TEST_3));
« no previous file with comments | « components/sync/base/enum_set.h ('k') | components/sync/base/model_type.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698