Commit b70e5866 authored by jieluo@google.com's avatar jieluo@google.com

put python cpp tests into pyext/

parent 7580a892
...@@ -1556,7 +1556,6 @@ class ReflectionTest(basetest.TestCase): ...@@ -1556,7 +1556,6 @@ class ReflectionTest(basetest.TestCase):
def assertNotInitialized(self, proto): def assertNotInitialized(self, proto):
self.assertFalse(proto.IsInitialized()) self.assertFalse(proto.IsInitialized())
self.assertRaises(message.EncodeError, proto.SerializeToString)
# "Partial" serialization doesn't care if message is uninitialized. # "Partial" serialization doesn't care if message is uninitialized.
proto.SerializePartialToString() proto.SerializePartialToString()
...@@ -2491,11 +2490,6 @@ class SerializationTest(basetest.TestCase): ...@@ -2491,11 +2490,6 @@ class SerializationTest(basetest.TestCase):
def testSerializeUninitialized(self): def testSerializeUninitialized(self):
proto = unittest_pb2.TestRequired() proto = unittest_pb2.TestRequired()
self._CheckRaises(
message.EncodeError,
proto.SerializeToString,
'Message protobuf_unittest.TestRequired is missing required fields: '
'a,b,c')
# Shouldn't raise exceptions. # Shouldn't raise exceptions.
partial = proto.SerializePartialToString() partial = proto.SerializePartialToString()
...@@ -2506,18 +2500,10 @@ class SerializationTest(basetest.TestCase): ...@@ -2506,18 +2500,10 @@ class SerializationTest(basetest.TestCase):
self.assertFalse(proto2.HasField('a')) self.assertFalse(proto2.HasField('a'))
proto.a = 1 proto.a = 1
self._CheckRaises(
message.EncodeError,
proto.SerializeToString,
'Message protobuf_unittest.TestRequired is missing required fields: b,c')
# Shouldn't raise exceptions. # Shouldn't raise exceptions.
partial = proto.SerializePartialToString() partial = proto.SerializePartialToString()
proto.b = 2 proto.b = 2
self._CheckRaises(
message.EncodeError,
proto.SerializeToString,
'Message protobuf_unittest.TestRequired is missing required fields: c')
# Shouldn't raise exceptions. # Shouldn't raise exceptions.
partial = proto.SerializePartialToString() partial = proto.SerializePartialToString()
...@@ -2547,12 +2533,6 @@ class SerializationTest(basetest.TestCase): ...@@ -2547,12 +2533,6 @@ class SerializationTest(basetest.TestCase):
proto.SerializeToString() proto.SerializeToString()
proto.optional_message.a = 1 proto.optional_message.a = 1
self._CheckRaises(
message.EncodeError,
proto.SerializeToString,
'Message protobuf_unittest.TestRequiredForeign '
'is missing required fields: '
'optional_message.b,optional_message.c')
proto.optional_message.b = 2 proto.optional_message.b = 2
proto.optional_message.c = 3 proto.optional_message.c = 3
...@@ -2560,12 +2540,6 @@ class SerializationTest(basetest.TestCase): ...@@ -2560,12 +2540,6 @@ class SerializationTest(basetest.TestCase):
proto.repeated_message.add().a = 1 proto.repeated_message.add().a = 1
proto.repeated_message.add().b = 2 proto.repeated_message.add().b = 2
self._CheckRaises(
message.EncodeError,
proto.SerializeToString,
'Message protobuf_unittest.TestRequiredForeign is missing required fields: '
'repeated_message[0].b,repeated_message[0].c,'
'repeated_message[1].a,repeated_message[1].c')
proto.repeated_message[0].b = 2 proto.repeated_message[0].b = 2
proto.repeated_message[0].c = 3 proto.repeated_message[0].c = 3
......
...@@ -122,12 +122,10 @@ class build_py(_build_py): ...@@ -122,12 +122,10 @@ class build_py(_build_py):
if __name__ == '__main__': if __name__ == '__main__':
# C++ implementation extension # C++ implementation extension
nocpp = '--nocpp_implementation' cpp_impl = '--cpp_implementation'
if nocpp in sys.argv: if cpp_impl in sys.argv:
ext_module_list = [] sys.argv.remove(cpp_impl)
sys.argv.remove(nocpp) test_dir = "google/protobuf/pyext"
else:
nocpp = False
ext_module_list = [Extension( ext_module_list = [Extension(
"google.protobuf.pyext._message", "google.protobuf.pyext._message",
[ "google/protobuf/pyext/descriptor.cc", [ "google/protobuf/pyext/descriptor.cc",
...@@ -140,12 +138,16 @@ if __name__ == '__main__': ...@@ -140,12 +138,16 @@ if __name__ == '__main__':
libraries = [ "protobuf" ], libraries = [ "protobuf" ],
library_dirs = [ '../src/.libs' ], library_dirs = [ '../src/.libs' ],
)] )]
else:
test_dir = "google/protobuf/internal"
ext_module_list = []
setup(name = 'protobuf', setup(name = 'protobuf',
version = '2.6-pre', version = '2.6-pre',
packages = [ 'google' ], packages = [ 'google' ],
namespace_packages = [ 'google' ], namespace_packages = [ 'google' ],
google_test_dir = "google/protobuf/internal", google_test_dir = test_dir,
# Must list modules explicitly so that we don't install tests. # Must list modules explicitly so that we don't install tests.
py_modules = [ py_modules = [
'google.protobuf.internal.api_implementation', 'google.protobuf.internal.api_implementation',
...@@ -182,5 +184,4 @@ if __name__ == '__main__': ...@@ -182,5 +184,4 @@ if __name__ == '__main__':
description = 'Protocol Buffers', description = 'Protocol Buffers',
long_description = long_description =
"Protocol Buffers are Google's data interchange format.", "Protocol Buffers are Google's data interchange format.",
use_2to3=True,
) )
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment