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

Side by Side Diff: sync/internal_api/public/base/ordinal_unittest.cc

Issue 10989063: Changed DB to store node positions as Ordinals. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Ordinal default ctor now creates invalid Ordinals Created 8 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/basictypes.h" 5 #include "base/basictypes.h"
6 #include "sync/internal_api/public/base/ordinal.h" 6 #include "sync/internal_api/public/base/ordinal.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <cctype> 10 #include <cctype>
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 ASSERT_TRUE(ordinal1.IsValid()); 124 ASSERT_TRUE(ordinal1.IsValid());
125 ASSERT_TRUE(ordinal2.IsValid()); 125 ASSERT_TRUE(ordinal2.IsValid());
126 EXPECT_TRUE(ordinal1.Equals(TestOrdinal("2"))); 126 EXPECT_TRUE(ordinal1.Equals(TestOrdinal("2")));
127 EXPECT_TRUE(ordinal2.Equals(LongOrdinal("50000"))); 127 EXPECT_TRUE(ordinal2.Equals(LongOrdinal("50000")));
128 } 128 }
129 129
130 // Create an invalid and a valid Ordinal. EqualsOrBothInvalid should 130 // Create an invalid and a valid Ordinal. EqualsOrBothInvalid should
131 // return true if called reflexively and false otherwise. 131 // return true if called reflexively and false otherwise.
132 TEST(Ordinal, EqualsOrBothInvalid) { 132 TEST(Ordinal, EqualsOrBothInvalid) {
133 const TestOrdinal& valid_ordinal = TestOrdinal::CreateInitialOrdinal(); 133 const TestOrdinal& valid_ordinal = TestOrdinal::CreateInitialOrdinal();
134 const TestOrdinal invalid_ordinal; 134 const TestOrdinal invalid_ordinal("");
akalin 2012/10/05 22:40:27 revert
vishwath 2012/10/08 20:17:49 Done.
135 135
136 EXPECT_TRUE(valid_ordinal.EqualsOrBothInvalid(valid_ordinal)); 136 EXPECT_TRUE(valid_ordinal.EqualsOrBothInvalid(valid_ordinal));
137 EXPECT_TRUE(invalid_ordinal.EqualsOrBothInvalid(invalid_ordinal)); 137 EXPECT_TRUE(invalid_ordinal.EqualsOrBothInvalid(invalid_ordinal));
138 EXPECT_FALSE(invalid_ordinal.EqualsOrBothInvalid(valid_ordinal)); 138 EXPECT_FALSE(invalid_ordinal.EqualsOrBothInvalid(valid_ordinal));
139 EXPECT_FALSE(valid_ordinal.EqualsOrBothInvalid(invalid_ordinal)); 139 EXPECT_FALSE(valid_ordinal.EqualsOrBothInvalid(invalid_ordinal));
140 } 140 }
141 141
142 // Create three Ordinals in order. LessThan should return values 142 // Create three Ordinals in order. LessThan should return values
143 // consistent with that order. 143 // consistent with that order.
144 TEST(Ordinal, LessThan) { 144 TEST(Ordinal, LessThan) {
(...skipping 222 matching lines...) Expand 10 before | Expand all | Expand 10 after
367 std::vector<LongOrdinal> ordinals = sorted_ordinals; 367 std::vector<LongOrdinal> ordinals = sorted_ordinals;
368 std::random_shuffle(ordinals.begin(), ordinals.end()); 368 std::random_shuffle(ordinals.begin(), ordinals.end());
369 std::sort(ordinals.begin(), ordinals.end(), LongOrdinal::LessThanFn()); 369 std::sort(ordinals.begin(), ordinals.end(), LongOrdinal::LessThanFn());
370 EXPECT_TRUE(std::equal(ordinals.begin(), ordinals.end(), 370 EXPECT_TRUE(std::equal(ordinals.begin(), ordinals.end(),
371 sorted_ordinals.begin(), LongOrdinal::EqualsFn())); 371 sorted_ordinals.begin(), LongOrdinal::EqualsFn()));
372 } 372 }
373 373
374 } // namespace 374 } // namespace
375 375
376 } // namespace syncer 376 } // namespace syncer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698