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

Unified Diff: chrome/browser/sync/util/protobuf_unittest.cc

Issue 3384024: sync_unit_tests: add test for unknown field retention. (Closed)
Patch Set: Created 10 years, 3 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
« no previous file with comments | « chrome/browser/sync/protocol/test.proto ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/util/protobuf_unittest.cc
diff --git a/chrome/browser/sync/util/protobuf_unittest.cc b/chrome/browser/sync/util/protobuf_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bb8be8b70bb69973c9ff6320a87e26c37718ac5e
--- /dev/null
+++ b/chrome/browser/sync/util/protobuf_unittest.cc
@@ -0,0 +1,35 @@
+// 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 <string>
+#include <vector>
+
+#include "chrome/browser/sync/protocol/test.pb.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+TEST(SyncProtobufTest, TestUnknownFields) {
+ // This tests ensures that we retain unknown fields in protocol buffers by
+ // serialising UnknownFieldsTestB, which is a superset of UnknownFieldsTestA,
+ // and checking we get back to the same message after parsing/serialising via
+ // UnknownFieldsTestA.
+ sync_pb::UnknownFieldsTestA a;
+ sync_pb::UnknownFieldsTestB b;
+ sync_pb::UnknownFieldsTestB b2;
+
+ b.set_foo(true);
+ b.set_bar(true);
+ std::string serialized;
+ ASSERT_TRUE(b.SerializeToString(&serialized));
+ ASSERT_TRUE(a.ParseFromString(serialized));
+ ASSERT_TRUE(a.foo());
+ std::string serialized2;
+ ASSERT_TRUE(a.SerializeToString(&serialized2));
+ ASSERT_TRUE(b2.ParseFromString(serialized2));
+ ASSERT_TRUE(b2.foo());
+ ASSERT_TRUE(b2.bar());
+}
+
+} // namespace
« no previous file with comments | « chrome/browser/sync/protocol/test.proto ('k') | chrome/chrome_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698