Enabled custom options.

python/descriptor.c

@@ -132,11 +132,8 @@ static PyObject* PyUpb_DescriptorBase_GetOptions(PyUpb_DescriptorBase* self,
     (void)ok;
     assert(ok);
 
-    // Disabled until python/message.c is reviewed and checked in.
-    // self->options = PyUpb_CMessage_Get(opts2, m, py_arena);
+    self->options = PyUpb_CMessage_Get(opts2, m, py_arena);
     Py_DECREF(py_arena);
-    PyErr_Format(PyExc_NotImplementedError, "Not yet implemented");
-    return NULL;
   }
 
   Py_INCREF(self->options);

python/pb_unit_tests/descriptor_pool_test_wrapper.py

@@ -32,7 +32,6 @@ print(unittest)
 descriptor_pool_test.AddDescriptorTest.testAddTypeError.__unittest_expecting_failure__ = True
 descriptor_pool_test.AddDescriptorTest.testEnum.__unittest_expecting_failure__ = True
 descriptor_pool_test.AddDescriptorTest.testFile.__unittest_expecting_failure__ = True
-descriptor_pool_test.AddDescriptorTest.testFileDescriptorOptionsWithCustomDescriptorPool.__unittest_expecting_failure__ = True
 descriptor_pool_test.AddDescriptorTest.testMessage.__unittest_expecting_failure__ = True
 descriptor_pool_test.AddDescriptorTest.testService.__unittest_expecting_failure__ = True
 descriptor_pool_test.CreateDescriptorPoolTest.testFindFieldByName.__unittest_expecting_failure__ = True

python/pb_unit_tests/descriptor_test_wrapper.py

@@ -30,7 +30,6 @@ descriptor_test.DescriptorCopyToProtoTest.testCopyToProto_TypeError.__unittest_e
 descriptor_test.GeneratedDescriptorTest.testDescriptor.__unittest_expecting_failure__ = True
 descriptor_test.MakeDescriptorTest.testCamelcaseName.__unittest_expecting_failure__ = True
 descriptor_test.MakeDescriptorTest.testJsonName.__unittest_expecting_failure__ = True
-descriptor_test.MakeDescriptorTest.testMakeDescriptorWithOptions.__unittest_expecting_failure__ = True
 descriptor_test.NewDescriptorTest.testAggregateOptions.__unittest_expecting_failure__ = True
 descriptor_test.NewDescriptorTest.testComplexExtensionOptions.__unittest_expecting_failure__ = True
 descriptor_test.NewDescriptorTest.testContainingServiceFixups.__unittest_expecting_failure__ = True

python/pb_unit_tests/generator_test_wrapper.py

@@ -27,9 +27,7 @@ from google.protobuf.internal import generator_test
 import unittest
 
 generator_test.GeneratorTest.testExtensionScope.__unittest_expecting_failure__ = True
-generator_test.GeneratorTest.testMessageWithCustomOptions.__unittest_expecting_failure__ = True
 generator_test.GeneratorTest.testOneof.__unittest_expecting_failure__ = True
-generator_test.GeneratorTest.testOptions.__unittest_expecting_failure__ = True
 
 if __name__ == '__main__':
   unittest.main(module=generator_test, verbosity=2)

python/pb_unit_tests/json_format_test_wrapper.py

@@ -26,12 +26,5 @@
 from google.protobuf.internal import json_format_test
 import unittest
 
-json_format_test.JsonFormatTest.testExtensionSerializationDictMatchesProto3Spec.__unittest_expecting_failure__ = True
-json_format_test.JsonFormatTest.testExtensionSerializationJsonMatchesProto3Spec.__unittest_expecting_failure__ = True
-json_format_test.JsonFormatTest.testExtensionToDictAndBack.__unittest_expecting_failure__ = True
-json_format_test.JsonFormatTest.testExtensionToJsonAndBack.__unittest_expecting_failure__ = True
-json_format_test.JsonFormatTest.testInvalidMap.__unittest_expecting_failure__ = True
-json_format_test.JsonFormatTest.testMapFields.__unittest_expecting_failure__ = True
-
 if __name__ == '__main__':
   unittest.main(module=json_format_test, verbosity=2)

python/pb_unit_tests/reflection_test_wrapper.py

@@ -29,8 +29,6 @@ import unittest
 reflection_test.ByteSizeTest.testRepeatedCompositesDelete.__unittest_expecting_failure__ = True
 reflection_test.ByteSizeTest.testRepeatedScalarsRemove.__unittest_expecting_failure__ = True
 reflection_test.ClassAPITest.testMakeClassWithNestedDescriptor.__unittest_expecting_failure__ = True
-reflection_test.OptionsTest.testMessageOptions.__unittest_expecting_failure__ = True
-reflection_test.OptionsTest.testPackedOptions.__unittest_expecting_failure__ = True
 reflection_test.Proto2ReflectionTest.testExtensionContainsError.__unittest_expecting_failure__ = True
 reflection_test.Proto2ReflectionTest.testExtensionDelete.__unittest_expecting_failure__ = True
 reflection_test.Proto2ReflectionTest.testExtensionFailureModes.__unittest_expecting_failure__ = True
@@ -83,7 +81,6 @@ reflection_test.SerializationTest.testInitKwargs.__unittest_expecting_failure__
 reflection_test.SerializationTest.testInitRepeatedKwargs.__unittest_expecting_failure__ = True
 reflection_test.SerializationTest.testInitRequiredForeignKwargs.__unittest_expecting_failure__ = True
 reflection_test.SerializationTest.testInitRequiredKwargs.__unittest_expecting_failure__ = True
-reflection_test.SerializationTest.testMessageSetWireFormat.__unittest_expecting_failure__ = True
 reflection_test.SerializationTest.testSerializeUninitialized.__unittest_expecting_failure__ = True
 reflection_test.SerializationTest.testSerializeUninitializedSubMessage.__unittest_expecting_failure__ = True
 

python/pb_unit_tests/text_format_test_wrapper.py

@@ -29,29 +29,17 @@ from google.protobuf.internal import _parameterized
 
 sep = _parameterized._SEPARATOR
 
-text_format_test.OnlyWorksWithProto2RightNowTests.testDuplicateMapKey.__unittest_expecting_failure__ = True
-text_format_test.OnlyWorksWithProto2RightNowTests.testMapOrderEnforcement.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testMergeLinesGolden.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testParseGolden.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testParseLinesGolden.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testPrintAllFields.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testPrintAllFieldsPointy.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testPrintInIndexOrder.__unittest_expecting_failure__ = True
-text_format_test.OnlyWorksWithProto2RightNowTests.testPrintMap.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testPrintMapUsingCppImplementation.__unittest_expecting_failure__ = True
 text_format_test.OnlyWorksWithProto2RightNowTests.testPrintUnknownFields.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testExtensionInsideAnyMessage.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testParseAllExtensions.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testParseAllowedUnknownExtension.__unittest_expecting_failure__ = True
 text_format_test.Proto2Tests.testParseGoldenExtensions.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testParseMap.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testParseMessageByFieldNumber.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testParseMessageSet.__unittest_expecting_failure__ = True
 text_format_test.Proto2Tests.testPrintAllExtensions.__unittest_expecting_failure__ = True
 text_format_test.Proto2Tests.testPrintAllExtensionsPointy.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testPrintMessageSet.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testPrintMessageSetAsOneLine.__unittest_expecting_failure__ = True
-text_format_test.Proto2Tests.testPrintMessageSetByFieldNumber.__unittest_expecting_failure__ = True
 getattr(text_format_test.TextFormatMessageToStringTests, "testCustomOptions" + sep + "0").__unittest_expecting_failure__ = True
 getattr(text_format_test.TextFormatMessageToStringTests, "testCustomOptions" + sep + "1").__unittest_expecting_failure__ = True
 getattr(text_format_test.TextFormatMessageToStringTests, "testPrintUnknownFieldsEmbeddedMessageInBytes" + sep + "0").__unittest_expecting_failure__ = True

python/pb_unit_tests/well_known_types_test_wrapper.py

@@ -27,7 +27,6 @@ from google.protobuf.internal import well_known_types_test
 import unittest
 
 well_known_types_test.AnyTest.testPackDeterministic.__unittest_expecting_failure__ = True
-well_known_types_test.StructTest.testStruct.__unittest_expecting_failure__ = True
 
 if __name__ == '__main__':
   unittest.main(module=well_known_types_test, verbosity=2)