diff options
Diffstat (limited to 'src/google/protobuf/extension_set_unittest.cc')
-rw-r--r-- | src/google/protobuf/extension_set_unittest.cc | 66 |
1 files changed, 35 insertions, 31 deletions
diff --git a/src/google/protobuf/extension_set_unittest.cc b/src/google/protobuf/extension_set_unittest.cc index bc65d295..af398546 100644 --- a/src/google/protobuf/extension_set_unittest.cc +++ b/src/google/protobuf/extension_set_unittest.cc @@ -53,13 +53,13 @@ #include <gtest/gtest.h> #include <google/protobuf/stubs/stl_util.h> -namespace google { +namespace google { namespace protobuf { namespace internal { namespace { -// This test closely mirrors google/protobuf/compiler/cpp/unittest.cc +// This test closely mirrors net/proto2/compiler/cpp/internal/unittest.cc // except that it uses extensions rather than regular fields. TEST(ExtensionSetTest, Defaults) { @@ -208,9 +208,9 @@ TEST(ExtensionSetTest, ReleaseExtension) { } TEST(ExtensionSetTest, ArenaUnsafeArenaSetAllocatedAndRelease) { - ::google::protobuf::Arena arena; + Arena arena; unittest::TestAllExtensions* message = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena); unittest::ForeignMessage extension; message->UnsafeArenaSetAllocatedExtension( unittest::optional_foreign_message_extension, @@ -258,9 +258,9 @@ TEST(ExtensionSetTest, UnsafeArenaSetAllocatedAndRelease) { } TEST(ExtensionSetTest, ArenaUnsafeArenaReleaseOfHeapAlloc) { - ::google::protobuf::Arena arena; + Arena arena; unittest::TestAllExtensions* message = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena); unittest::ForeignMessage* extension = new unittest::ForeignMessage; message->SetAllocatedExtension( unittest::optional_foreign_message_extension, @@ -392,17 +392,17 @@ TEST(ExtensionSetTest, SwapExtensionBothFull) { } TEST(ExtensionSetTest, ArenaSetAllExtension) { - ::google::protobuf::Arena arena1; + Arena arena1; unittest::TestAllExtensions* message1 = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); TestUtil::SetAllExtensions(message1); TestUtil::ExpectAllExtensionsSet(*message1); } TEST(ExtensionSetTest, ArenaCopyConstructor) { - ::google::protobuf::Arena arena1; + Arena arena1; unittest::TestAllExtensions* message1 = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); TestUtil::SetAllExtensions(message1); unittest::TestAllExtensions message2(*message1); arena1.Reset(); @@ -410,9 +410,9 @@ TEST(ExtensionSetTest, ArenaCopyConstructor) { } TEST(ExtensionSetTest, ArenaMergeFrom) { - ::google::protobuf::Arena arena1; + Arena arena1; unittest::TestAllExtensions* message1 = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); TestUtil::SetAllExtensions(message1); unittest::TestAllExtensions message2; message2.MergeFrom(*message1); @@ -421,9 +421,9 @@ TEST(ExtensionSetTest, ArenaMergeFrom) { } TEST(ExtensionSetTest, ArenaSetAllocatedMessageAndRelease) { - ::google::protobuf::Arena arena; + Arena arena; unittest::TestAllExtensions* message = - ::google::protobuf::Arena::CreateMessage<unittest::TestAllExtensions>(&arena); + Arena::CreateMessage<unittest::TestAllExtensions>(&arena); EXPECT_FALSE(message->HasExtension( unittest::optional_foreign_message_extension)); // Add a extension using SetAllocatedExtension @@ -444,8 +444,8 @@ TEST(ExtensionSetTest, ArenaSetAllocatedMessageAndRelease) { } TEST(ExtensionSetTest, SwapExtensionBothFullWithArena) { - ::google::protobuf::Arena arena1; - std::unique_ptr<google::protobuf::Arena> arena2(new ::google::protobuf::Arena()); + Arena arena1; + std::unique_ptr<Arena> arena2(new Arena()); unittest::TestAllExtensions* message1 = Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); @@ -467,7 +467,7 @@ TEST(ExtensionSetTest, SwapExtensionBothFullWithArena) { arena2.reset(NULL); TestUtil::ExpectAllExtensionsSet(*message1); // Test corner cases, when one is empty and other is not. - ::google::protobuf::Arena arena3, arena4; + Arena arena3, arena4; unittest::TestAllExtensions* message3 = Arena::CreateMessage<unittest::TestAllExtensions>(&arena3); @@ -480,8 +480,8 @@ TEST(ExtensionSetTest, SwapExtensionBothFullWithArena) { } TEST(ExtensionSetTest, SwapFieldsOfExtensionBothFullWithArena) { - google::protobuf::Arena arena1; - google::protobuf::Arena* arena2 = new ::google::protobuf::Arena(); + Arena arena1; + Arena* arena2 = new Arena(); unittest::TestAllExtensions* message1 = Arena::CreateMessage<unittest::TestAllExtensions>(&arena1); @@ -527,7 +527,7 @@ TEST(ExtensionSetTest, SerializationToArray) { int size = source.ByteSize(); string data; data.resize(size); - uint8* target = reinterpret_cast<uint8*>(string_as_array(&data)); + uint8* target = reinterpret_cast<uint8*>(::google::protobuf::string_as_array(&data)); uint8* end = source.SerializeWithCachedSizesToArray(target); EXPECT_EQ(size, end - target); EXPECT_TRUE(destination.ParseFromString(data)); @@ -549,7 +549,7 @@ TEST(ExtensionSetTest, SerializationToStream) { string data; data.resize(size); { - io::ArrayOutputStream array_stream(string_as_array(&data), size, 1); + io::ArrayOutputStream array_stream(::google::protobuf::string_as_array(&data), size, 1); io::CodedOutputStream output_stream(&array_stream); source.SerializeWithCachedSizes(&output_stream); ASSERT_FALSE(output_stream.HadError()); @@ -571,7 +571,7 @@ TEST(ExtensionSetTest, PackedSerializationToArray) { int size = source.ByteSize(); string data; data.resize(size); - uint8* target = reinterpret_cast<uint8*>(string_as_array(&data)); + uint8* target = reinterpret_cast<uint8*>(::google::protobuf::string_as_array(&data)); uint8* end = source.SerializeWithCachedSizesToArray(target); EXPECT_EQ(size, end - target); EXPECT_TRUE(destination.ParseFromString(data)); @@ -593,7 +593,7 @@ TEST(ExtensionSetTest, PackedSerializationToStream) { string data; data.resize(size); { - io::ArrayOutputStream array_stream(string_as_array(&data), size, 1); + io::ArrayOutputStream array_stream(::google::protobuf::string_as_array(&data), size, 1); io::CodedOutputStream output_stream(&array_stream); source.SerializeWithCachedSizes(&output_stream); ASSERT_FALSE(output_stream.HadError()); @@ -1051,11 +1051,15 @@ TEST(ExtensionSetTest, RepeatedFields) { enum_const_iter; RepeatedField<unittest::TestAllTypes_NestedEnum>::const_iterator enum_const_end; - for (enum_const_iter = message.GetRepeatedExtension( - unittest::repeated_nested_enum_extension).begin(), - enum_const_end = message.GetRepeatedExtension( - unittest::repeated_nested_enum_extension).end(); - enum_iter != enum_end; ++enum_iter) { + for (enum_const_iter = + message + .GetRepeatedExtension(unittest::repeated_nested_enum_extension) + .begin(), + enum_const_end = + message + .GetRepeatedExtension(unittest::repeated_nested_enum_extension) + .end(); + enum_const_iter != enum_const_end; ++enum_const_iter) { ASSERT_EQ(*enum_const_iter, unittest::TestAllTypes::NestedEnum_MAX); } @@ -1271,10 +1275,10 @@ TEST(ExtensionSetTest, DynamicExtensions) { const Message& sub_message = message.GetReflection()->GetMessage(message, message_extension); const unittest::ForeignMessage* typed_sub_message = -#ifdef GOOGLE_PROTOBUF_NO_RTTI - static_cast<const unittest::ForeignMessage*>(&sub_message); -#else +#if PROTOBUF_RTTI dynamic_cast<const unittest::ForeignMessage*>(&sub_message); +#else + static_cast<const unittest::ForeignMessage*>(&sub_message); #endif ASSERT_TRUE(typed_sub_message != NULL); EXPECT_EQ(456, typed_sub_message->c()); |