[Sync] Add tests for invalid specifics field number handling

sync/engine/syncer_proto_util_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.
 
 #include "sync/engine/syncer_proto_util.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
@@ skipping 30 matching lines @@
 
   MOCK_METHOD0(IsSyncingCurrentlySilenced, bool());
   MOCK_METHOD1(OnReceivedShortPollIntervalUpdate, void(const base::TimeDelta&));
   MOCK_METHOD1(OnReceivedLongPollIntervalUpdate ,void(const base::TimeDelta&));
   MOCK_METHOD1(OnReceivedSessionsCommitDelay, void(const base::TimeDelta&));
   MOCK_METHOD1(OnSyncProtocolError, void(const sessions::SyncSessionSnapshot&));
   MOCK_METHOD0(OnShouldStopSyncingPermanently, void());
   MOCK_METHOD1(OnSilencedUntil, void(const base::TimeTicks&));
 };
 
+// Builds a ClientToServerResponse with some data type ids, including
+// invalid ones.  GetTypesToMigrate() should return only the valid
+// model types.
+TEST(SyncerProtoUtil, GetTypesToMigrate) {
+  sync_pb::ClientToServerResponse response;
+  response.add_migrated_data_type_id(
+      GetSpecificsFieldNumberFromModelType(BOOKMARKS));
+  response.add_migrated_data_type_id(
+      GetSpecificsFieldNumberFromModelType(HISTORY_DELETE_DIRECTIVES));
+  response.add_migrated_data_type_id(-1);
+  EXPECT_TRUE(
+      GetTypesToMigrate(response).Equals(
+          ModelTypeSet(BOOKMARKS, HISTORY_DELETE_DIRECTIVES)));
+}
+
+// Builds a ClientToServerResponse_Error with some error data type
+// ids, including invalid ones.  ConvertErrorPBToLocalType() should
+// return a SyncProtocolError with only the valid model types.
+TEST(SyncerProtoUtil, ConvertErrorPBToLocalType) {
+  sync_pb::ClientToServerResponse_Error error_pb;
+  error_pb.set_error_type(sync_pb::SyncEnums::THROTTLED);
+  error_pb.add_error_data_type_ids(
+      GetSpecificsFieldNumberFromModelType(BOOKMARKS));
+  error_pb.add_error_data_type_ids(
+      GetSpecificsFieldNumberFromModelType(HISTORY_DELETE_DIRECTIVES));
+  error_pb.add_error_data_type_ids(-1);
+  SyncProtocolError error = ConvertErrorPBToLocalType(error_pb);
+  EXPECT_TRUE(
+      error.error_data_types.Equals(
+          ModelTypeSet(BOOKMARKS, HISTORY_DELETE_DIRECTIVES)));
+}
+
 TEST(SyncerProtoUtil, TestBlobToProtocolBufferBytesUtilityFunctions) {
   unsigned char test_data1[] = {1, 2, 3, 4, 5, 6, 7, 8, 0, 1, 4, 2, 9};
   unsigned char test_data2[] = {1, 99, 3, 4, 5, 6, 7, 8, 0, 1, 4, 2, 9};
   unsigned char test_data3[] = {99, 2, 3, 4, 5, 6, 7, 8};
 
   syncable::Blob test_blob1, test_blob2, test_blob3;
   for (size_t i = 0; i < arraysize(test_data1); ++i)
     test_blob1.push_back(test_data1[i]);
   for (size_t i = 0; i < arraysize(test_data2); ++i)
     test_blob2.push_back(test_data2[i]);
@@ skipping 107 matching lines @@
 
  protected:
   MessageLoop message_loop_;
   TestDirectorySetterUpper dir_maker_;
 };
 
 TEST_F(SyncerProtoUtilTest, VerifyResponseBirthday) {
   // Both sides empty
   EXPECT_TRUE(directory()->store_birthday().empty());
   sync_pb::ClientToServerResponse response;
-  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(directory(), &response));
+  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(response, directory()));
 
   // Remote set, local empty
   response.set_store_birthday("flan");
-  EXPECT_TRUE(SyncerProtoUtil::VerifyResponseBirthday(directory(), &response));
+  EXPECT_TRUE(SyncerProtoUtil::VerifyResponseBirthday(response, directory()));
   EXPECT_EQ(directory()->store_birthday(), "flan");
 
   // Remote empty, local set.
   response.clear_store_birthday();
-  EXPECT_TRUE(SyncerProtoUtil::VerifyResponseBirthday(directory(), &response));
+  EXPECT_TRUE(SyncerProtoUtil::VerifyResponseBirthday(response, directory()));
   EXPECT_EQ(directory()->store_birthday(), "flan");
 
   // Doesn't match
   response.set_store_birthday("meat");
-  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(directory(), &response));
+  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(response, directory()));
 
   response.set_error_code(sync_pb::SyncEnums::CLEAR_PENDING);
-  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(directory(), &response));
+  EXPECT_FALSE(SyncerProtoUtil::VerifyResponseBirthday(response, directory()));
 }
 
 TEST_F(SyncerProtoUtilTest, AddRequestBirthday) {
   EXPECT_TRUE(directory()->store_birthday().empty());
   ClientToServerMessage msg;
   SyncerProtoUtil::AddRequestBirthday(directory(), &msg);
   EXPECT_FALSE(msg.has_store_birthday());
 
   directory()->set_store_birthday("meat");
   SyncerProtoUtil::AddRequestBirthday(directory(), &msg);
@@ skipping 78 matching lines @@
   SyncProtocolError error;
   error.error_type = THROTTLED;
 
   base::TimeTicks ticks = base::TimeTicks::FromInternalValue(1);
 
   EXPECT_CALL(delegate, OnSilencedUntil(ticks));
 
   SyncerProtoUtil::HandleThrottleError(error, ticks, &tracker, &delegate);
   EXPECT_TRUE(tracker.GetThrottledTypes().Empty());
 }
+
 }  // namespace syncer