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

Unified Diff: chrome/test/live_sync/single_client_apps_sync_test.cc

Issue 7599019: Allow sync integration tests to operate on multiple datatypes: Apps, Extensions, Themes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CR feedback Created 9 years, 4 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
Index: chrome/test/live_sync/single_client_apps_sync_test.cc
diff --git a/chrome/test/live_sync/single_client_live_apps_sync_test.cc b/chrome/test/live_sync/single_client_apps_sync_test.cc
similarity index 60%
rename from chrome/test/live_sync/single_client_live_apps_sync_test.cc
rename to chrome/test/live_sync/single_client_apps_sync_test.cc
index ca12b53734a499f6a27fdc99bdcbba93d9e38ad3..da84bfdb1cf67c874f16aa0435738e5ba16db8d3 100644
--- a/chrome/test/live_sync/single_client_live_apps_sync_test.cc
+++ b/chrome/test/live_sync/single_client_apps_sync_test.cc
@@ -4,28 +4,29 @@
#include "base/basictypes.h"
#include "chrome/browser/sync/profile_sync_service_harness.h"
-#include "chrome/test/live_sync/live_apps_sync_test.h"
+#include "chrome/test/live_sync/apps_helper.h"
+#include "chrome/test/live_sync/live_sync_test.h"
-class SingleClientLiveAppsSyncTest : public LiveAppsSyncTest {
+using apps_helper::AllProfilesHaveSameAppsAsVerifier;
+using apps_helper::InstallApp;
+
+class SingleClientAppsSyncTest : public LiveSyncTest {
public:
- SingleClientLiveAppsSyncTest()
- : LiveAppsSyncTest(SINGLE_CLIENT) {}
+ SingleClientAppsSyncTest() : LiveSyncTest(SINGLE_CLIENT) {}
- virtual ~SingleClientLiveAppsSyncTest() {}
+ virtual ~SingleClientAppsSyncTest() {}
private:
- DISALLOW_COPY_AND_ASSIGN(SingleClientLiveAppsSyncTest);
+ DISALLOW_COPY_AND_ASSIGN(SingleClientAppsSyncTest);
};
-IN_PROC_BROWSER_TEST_F(SingleClientLiveAppsSyncTest,
- StartWithNoApps) {
+IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, StartWithNoApps) {
ASSERT_TRUE(SetupSync());
ASSERT_TRUE(AllProfilesHaveSameAppsAsVerifier());
}
-IN_PROC_BROWSER_TEST_F(SingleClientLiveAppsSyncTest,
- StartWithSomeApps) {
+IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, StartWithSomeApps) {
ASSERT_TRUE(SetupClients());
const int kNumApps = 5;
@@ -39,8 +40,7 @@ IN_PROC_BROWSER_TEST_F(SingleClientLiveAppsSyncTest,
ASSERT_TRUE(AllProfilesHaveSameAppsAsVerifier());
}
-IN_PROC_BROWSER_TEST_F(SingleClientLiveAppsSyncTest,
- InstallSomeApps) {
+IN_PROC_BROWSER_TEST_F(SingleClientAppsSyncTest, InstallSomeApps) {
ASSERT_TRUE(SetupSync());
const int kNumApps = 5;

Powered by Google App Engine
This is Rietveld 408576698