[Sync] Added some basic extension sync integration tests.

chrome/test/live_sync/live_extensions_sync_test_base.cc

+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/test/live_sync/live_extensions_sync_test_base.h"
+
+#include <string>
+
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/logging.h"
+#include "base/ref_counted.h"
+#include "base/string_number_conversions.h"
+#include "base/values.h"
+#include "chrome/browser/extensions/extensions_service.h"
+#include "chrome/browser/profile.h"
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/extension_constants.h"
+
+LiveExtensionsSyncTestBase::LiveExtensionsSyncTestBase(TestType test_type)
+    : LiveSyncTest(test_type), extension_index_(0) {}
+
+LiveExtensionsSyncTestBase::~LiveExtensionsSyncTestBase() {}
+
+namespace {
+
+// TODO(akalin): Somehow unify this with MakeExtension() in
+// extension_util_unittest.cc.
+scoped_refptr<Extension> CreateExtension(
+    const ScopedTempDir& scoped_temp_dir,
+    int index, bool is_theme) {

[Raghu Simha, 2010/11/10 21:42:41]

nit: I believe the new style guide recommendation is for parameters to be one
per line and aligned.

[akalin, 2010/11/12 01:20:44]

Done.

+  DictionaryValue source;
+  source.SetString(
+      extension_manifest_keys::kName,
+      std::string("fakeextension") + base::IntToString(index));
+  source.SetString(extension_manifest_keys::kVersion, "0.0.0.0");
+  if (is_theme) {
+    source.Set(extension_manifest_keys::kTheme, new DictionaryValue());
+  }
+  FilePath extension_dir;
+  if (!file_util::CreateTemporaryDirInDir(
+          scoped_temp_dir.path(), FILE_PATH_LITERAL("fakeextension"),
+          &extension_dir)) {
+    return NULL;
+  }
+  std::string error;
+  scoped_refptr<Extension> extension =
+      Extension::Create(extension_dir,
+                        Extension::INTERNAL, source, false, &error);
+  if (!error.empty()) {
+    LOG(WARNING) << error;
+    return NULL;
+  }
+  return extension;
+}
+
+}  // namespace
+
+bool LiveExtensionsSyncTestBase::SetupClients() {
+  if (!LiveSyncTest::SetupClients())
+    return false;
+
+  for (int i = 0; i < num_clients(); ++i) {
+    GetProfile(i)->InitExtensions();
+  }
+  verifier()->InitExtensions();
+
+  if (!extension_base_dir_.CreateUniqueTempDir())
+    return false;
+
+  return true;
+}
+
+scoped_refptr<Extension> LiveExtensionsSyncTestBase::MakeTheme() {
+  scoped_refptr<Extension> theme =
+      CreateExtension(extension_base_dir_, extension_index_, true);
+  ++extension_index_;
+  if (!theme.get())
+    return false;
+  extensions_[theme->id()] = theme;
+  return theme;
+}
+
+scoped_refptr<Extension> LiveExtensionsSyncTestBase::MakeExtension() {
+  scoped_refptr<Extension> extension =
+      CreateExtension(extension_base_dir_, extension_index_, false);
+  ++extension_index_;
+  if (!extension.get())
+    return false;
+  extensions_[extension->id()] = extension;
+  return extension;
+}
+
+void LiveExtensionsSyncTestBase::InstallExtension(
+    Profile* profile, scoped_refptr<Extension> extension) {
+  CHECK(profile);
+  CHECK(extension.get());
+  profile->GetExtensionsService()->OnExtensionInstalled(extension, true);
+}
+
+void LiveExtensionsSyncTestBase::InstallAllPendingExtensions(
+    Profile* profile) {
+  // TODO(akalin): Mock out the servers that the extensions
+  // auto-update mechanism talk to so as to more closely match what
+  // actually happens.
+  const PendingExtensionMap& pending_extensions =
+      profile->GetExtensionsService()->pending_extensions();
+  for (PendingExtensionMap::const_iterator it = pending_extensions.begin();
+       it != pending_extensions.end(); ++it) {
+    ExtensionMap::const_iterator it2 = extensions_.find(it->first);
+    CHECK(it2 != extensions_.end());
+    InstallExtension(profile, it2->second);
+  }
+}