Log the sync communication that happens between client and server.

sync/protocol/proto_enum_conversions_unittest.cc

 // Copyright (c) 2012 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.
 
 // Keep this file in sync with the .proto files in this directory.
 
 #include "sync/protocol/proto_enum_conversions.h"
 
 #include <string>
 
@@ skipping 40 matching lines @@
       sync_pb::TabNavigation::PageTransitionQualifier_MAX);
 }
 
 TEST_F(ProtoEnumConversionsTest, GetUpdatesSourceString) {
   TestEnumStringFunction(
       GetUpdatesSourceString,
       sync_pb::GetUpdatesCallerInfo::GetUpdatesSource_MIN,
       sync_pb::GetUpdatesCallerInfo::GetUpdatesSource_MAX);
 }
 
+TEST_F(ProtoEnumConversionsTest, GetResponseTypeString) {
+  TestEnumStringFunction(
+      GetResponseTypeString,
+      sync_pb::CommitResponse::ResponseType_MIN,
+      sync_pb::CommitResponse::ResponseType_MAX);
+}
+
+TEST_F(ProtoEnumConversionsTest, GetErrorTypeString) {
+  TestEnumStringFunction(
+      GetErrorTypeString,
+      sync_pb::SyncEnums::ErrorType_MIN,
+      sync_pb::SyncEnums::MIGRATION_DONE);
+
+  // ErrorType enum is sequential only upto MIGRATION_DONE.

[akalin, 2012/03/17 01:55:05]

upto -> up to

Although for consistency it would be better to use the same comment/formatting
as the GetPageTransitionString test above

[lipalani1, 2012/03/19 19:24:37]

Done.

+  TestEnumStringFunction(
+      GetErrorTypeString,
+      sync_pb::SyncEnums::UNKNOWN,
+      sync_pb::SyncEnums::UNKNOWN);

[akalin, 2012/03/17 01:55:05]

replace the last UNKNOWN with ErrorType_MAX

[lipalani1, 2012/03/19 19:24:37]

Done.

+
+}
+
+TEST_F(ProtoEnumConversionsTest, GetActionString) {
+  TestEnumStringFunction(
+      GetActionString,
+      sync_pb::ClientToServerResponse::Error::Action_MIN,
+      sync_pb::ClientToServerResponse::Error::Action_MAX);
+}
+
 }  // namespace
 }  // namespace browser_sync