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

Side by Side Diff: chrome/test/live_sync/two_client_live_extensions_sync_test.cc

Issue 5864001: Annotating a couple of failing extensions sync integration tests as FAILS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 "chrome/test/live_sync/live_extensions_sync_test.h" 5 #include "chrome/test/live_sync/live_extensions_sync_test.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "chrome/common/extensions/extension.h" 9 #include "chrome/common/extensions/extension.h"
10 10
(...skipping 26 matching lines...) Expand all
37 InstallExtension(verifier(), GetExtension(i)); 37 InstallExtension(verifier(), GetExtension(i));
38 } 38 }
39 39
40 ASSERT_TRUE(SetupSync()); 40 ASSERT_TRUE(SetupSync());
41 41
42 ASSERT_TRUE(AwaitQuiescence()); 42 ASSERT_TRUE(AwaitQuiescence());
43 43
44 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 44 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
45 } 45 }
46 46
47 // TODO(rsimha): Remove FAILS_ prefix after http://crbug.com/66925 is fixed.
47 IN_PROC_BROWSER_TEST_F(TwoClientLiveExtensionsSyncTest, 48 IN_PROC_BROWSER_TEST_F(TwoClientLiveExtensionsSyncTest,
48 StartWithDifferentExtensions) { 49 FAILS_StartWithDifferentExtensions) {
49 ASSERT_TRUE(SetupClients()); 50 ASSERT_TRUE(SetupClients());
50 51
51 int i = 0; 52 int i = 0;
52 53
53 const int kNumCommonExtensions = 5; 54 const int kNumCommonExtensions = 5;
54 for (int j = 0; j < kNumCommonExtensions; ++i, ++j) { 55 for (int j = 0; j < kNumCommonExtensions; ++i, ++j) {
55 InstallExtension(GetProfile(0), GetExtension(i)); 56 InstallExtension(GetProfile(0), GetExtension(i));
56 InstallExtension(GetProfile(1), GetExtension(i)); 57 InstallExtension(GetProfile(1), GetExtension(i));
57 InstallExtension(verifier(), GetExtension(i)); 58 InstallExtension(verifier(), GetExtension(i));
58 } 59 }
(...skipping 13 matching lines...) Expand all
72 ASSERT_TRUE(SetupSync()); 73 ASSERT_TRUE(SetupSync());
73 74
74 ASSERT_TRUE(AwaitQuiescence()); 75 ASSERT_TRUE(AwaitQuiescence());
75 76
76 InstallAllPendingExtensions(GetProfile(0)); 77 InstallAllPendingExtensions(GetProfile(0));
77 InstallAllPendingExtensions(GetProfile(1)); 78 InstallAllPendingExtensions(GetProfile(1));
78 79
79 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 80 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
80 } 81 }
81 82
83 // TODO(rsimha): Remove FAILS_ prefix after http://crbug.com/66925 is fixed.
82 IN_PROC_BROWSER_TEST_F(TwoClientLiveExtensionsSyncTest, 84 IN_PROC_BROWSER_TEST_F(TwoClientLiveExtensionsSyncTest,
83 InstallDifferentExtensions) { 85 FAILS_InstallDifferentExtensions) {
84 ASSERT_TRUE(SetupClients()); 86 ASSERT_TRUE(SetupClients());
85 87
86 int i = 0; 88 int i = 0;
87 89
88 const int kNumCommonExtensions = 5; 90 const int kNumCommonExtensions = 5;
89 for (int j = 0; j < kNumCommonExtensions; ++i, ++j) { 91 for (int j = 0; j < kNumCommonExtensions; ++i, ++j) {
90 InstallExtension(GetProfile(0), GetExtension(i)); 92 InstallExtension(GetProfile(0), GetExtension(i));
91 InstallExtension(GetProfile(1), GetExtension(i)); 93 InstallExtension(GetProfile(1), GetExtension(i));
92 InstallExtension(verifier(), GetExtension(i)); 94 InstallExtension(verifier(), GetExtension(i));
93 } 95 }
(...skipping 19 matching lines...) Expand all
113 InstallAllPendingExtensions(GetProfile(0)); 115 InstallAllPendingExtensions(GetProfile(0));
114 InstallAllPendingExtensions(GetProfile(1)); 116 InstallAllPendingExtensions(GetProfile(1));
115 117
116 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 118 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
117 } 119 }
118 120
119 // TODO(akalin): Add tests exercising: 121 // TODO(akalin): Add tests exercising:
120 // - Extensions enabled/disabled state sync behavior 122 // - Extensions enabled/disabled state sync behavior
121 // - Extension uninstallation 123 // - Extension uninstallation
122 // - Offline installation/uninstallation behavior 124 // - Offline installation/uninstallation behavior
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698