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

Side by Side Diff: chrome/browser/extensions/extension_sync_data_unittest.cc

Issue 6902054: [Sync] Rip out overly-complicated ExtensionData class (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed asargent's comments Created 9 years, 8 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/extensions/extension_sync_data.h"
6
7 #include "base/memory/scoped_ptr.h"
8 #include "base/version.h"
9 #include "googleurl/src/gurl.h"
10 #include "testing/gtest/include/gtest/gtest.h"
11
12 namespace {
13
14 class ExtensionSyncDataTest : public testing::Test {
15 };
16
17 bool ExtensionSyncDataEqual(const ExtensionSyncData& a,
18 const ExtensionSyncData& b) {
19 return
20 (a.id == b.id) &&
21 (a.enabled == b.enabled) &&
22 (a.incognito_enabled == b.incognito_enabled) &&
23 (a.version.Equals(b.version)) &&
24 (a.update_url == b.update_url) &&
25 (a.name == b.name);
26 }
27
28 TEST_F(ExtensionSyncDataTest, MergeOlder) {
29 ExtensionSyncData data;
30 data.id = "id";
31 data.enabled = true;
32 data.incognito_enabled = false;
33 {
34 scoped_ptr<Version> version(Version::GetVersionFromString("1.2.0.0"));
35 data.version = *version;
36 }
37 data.update_url = GURL("http://www.old.com");
38 data.name = "data";
39
40 ExtensionSyncData new_data;
41 new_data.id = "id";
42 new_data.enabled = false;
43 new_data.incognito_enabled = true;
44 {
45 scoped_ptr<Version> version(Version::GetVersionFromString("1.1.0.0"));
46 new_data.version = *version;
47 }
48 new_data.update_url = GURL("http://www.new.com");
49 new_data.name = "new_data";
50
51 ExtensionSyncData expected_data = data;
52 expected_data.enabled = new_data.enabled;
53 expected_data.incognito_enabled = new_data.incognito_enabled;
54
55 data.Merge(new_data);
56 EXPECT_TRUE(ExtensionSyncDataEqual(data, expected_data));
57 }
58
59 TEST_F(ExtensionSyncDataTest, MergeNewer) {
60 ExtensionSyncData data;
61 data.id = "id";
62 data.enabled = true;
63 data.incognito_enabled = false;
64 {
65 scoped_ptr<Version> version(Version::GetVersionFromString("1.2.0.0"));
66 data.version = *version;
67 }
68 data.update_url = GURL("http://www.old.com");
69 data.name = "data";
70
71 ExtensionSyncData new_data;
72 new_data.id = "id";
73 new_data.enabled = false;
74 new_data.incognito_enabled = true;
75 {
76 scoped_ptr<Version> version(Version::GetVersionFromString("1.3.0.0"));
77 new_data.version = *version;
78 }
79 new_data.update_url = GURL("http://www.new.com");
80 new_data.name = "new_data";
81
82 data.Merge(new_data);
83 EXPECT_TRUE(ExtensionSyncDataEqual(data, new_data));
84 }
85
86 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_sync_data.cc ('k') | chrome/browser/sync/glue/extension_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698